diff options
author | Alex Elder <elder@inktank.com> | 2012-08-29 18:11:06 -0400 |
---|---|---|
committer | Alex Elder <elder@inktank.com> | 2012-10-01 15:30:51 -0400 |
commit | 1fcdb8aa1f58af72eb8206ba97fab2df77df2b14 (patch) | |
tree | a9490606d69803a08a1739044c569257e9e10fa2 | |
parent | 2ac4e75d89e9df8eea6390a759eac2b6df0ebff6 (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.c | 10 |
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 | |||
1914 | out_disk: | 1910 | out_disk: |
1915 | put_disk(disk); | 1911 | put_disk(disk); |
1916 | out: | 1912 | |
1917 | return rc; | 1913 | return -ENOMEM; |
1918 | } | 1914 | } |
1919 | 1915 | ||
1920 | /* | 1916 | /* |