aboutsummaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorAlex Elder <elder@inktank.com>2012-08-29 18:11:06 -0400
committerAlex Elder <elder@inktank.com>2012-10-01 15:30:51 -0400
commit1fcdb8aa1f58af72eb8206ba97fab2df77df2b14 (patch)
treea9490606d69803a08a1739044c569257e9e10fa2
parent2ac4e75d89e9df8eea6390a759eac2b6df0ebff6 (diff)
rbd: simplify rbd_init_disk() a bit
This just simplifies a few things in rbd_init_disk(), now that the previous patch has moved a bunch of initialization code out if it. Done separately to facilitate review. Signed-off-by: Alex Elder <elder@inktank.com> Reviewed-by: Josh Durgin <josh.durgin@inktank.com>
-rw-r--r--drivers/block/rbd.c10
1 files changed, 3 insertions, 7 deletions
diff --git a/drivers/block/rbd.c b/drivers/block/rbd.c
index 6e735a754b5f..634a16c40291 100644
--- a/drivers/block/rbd.c
+++ b/drivers/block/rbd.c
@@ -1870,14 +1870,12 @@ static int rbd_init_disk(struct rbd_device *rbd_dev)
1870{ 1870{
1871 struct gendisk *disk; 1871 struct gendisk *disk;
1872 struct request_queue *q; 1872 struct request_queue *q;
1873 int rc;
1874 u64 segment_size; 1873 u64 segment_size;
1875 1874
1876 /* create gendisk info */ 1875 /* create gendisk info */
1877 rc = -ENOMEM;
1878 disk = alloc_disk(RBD_MINORS_PER_MAJOR); 1876 disk = alloc_disk(RBD_MINORS_PER_MAJOR);
1879 if (!disk) 1877 if (!disk)
1880 goto out; 1878 return -ENOMEM;
1881 1879
1882 snprintf(disk->disk_name, sizeof(disk->disk_name), RBD_DRV_NAME "%d", 1880 snprintf(disk->disk_name, sizeof(disk->disk_name), RBD_DRV_NAME "%d",
1883 rbd_dev->dev_id); 1881 rbd_dev->dev_id);
@@ -1887,7 +1885,6 @@ static int rbd_init_disk(struct rbd_device *rbd_dev)
1887 disk->private_data = rbd_dev; 1885 disk->private_data = rbd_dev;
1888 1886
1889 /* init rq */ 1887 /* init rq */
1890 rc = -ENOMEM;
1891 q = blk_init_queue(rbd_rq_fn, &rbd_dev->lock); 1888 q = blk_init_queue(rbd_rq_fn, &rbd_dev->lock);
1892 if (!q) 1889 if (!q)
1893 goto out_disk; 1890 goto out_disk;
@@ -1910,11 +1907,10 @@ static int rbd_init_disk(struct rbd_device *rbd_dev)
1910 rbd_dev->disk = disk; 1907 rbd_dev->disk = disk;
1911 1908
1912 return 0; 1909 return 0;
1913
1914out_disk: 1910out_disk:
1915 put_disk(disk); 1911 put_disk(disk);
1916out: 1912
1917 return rc; 1913 return -ENOMEM;
1918} 1914}
1919 1915
1920/* 1916/*