Skip to content

Commit

Permalink
rbd: call set_snap() before snap_devs_update()
Browse files Browse the repository at this point in the history
rbd_header_set_snap() is a simple initialization routine for an rbd
device's mapping.  It has to be called after the snapshot context
for the rbd_dev has been updated, but can be done before snapshot
devices have been registered.

Change the name to rbd_dev_set_mapping() to better reflect its
purpose, and call it a little sooner, before registering snapshot
devices.

Signed-off-by: Alex Elder <elder@inktank.com>
Reviewed-by: Josh Durgin <josh.durgin@inktank.com>
  • Loading branch information
Alex Elder committed Oct 1, 2012
1 parent 304f680 commit 5ed1617
Showing 1 changed file with 4 additions and 3 deletions.
7 changes: 4 additions & 3 deletions drivers/block/rbd.c
Original file line number Diff line number Diff line change
Expand Up @@ -640,7 +640,7 @@ static int snap_by_name(struct rbd_device *rbd_dev, const char *snap_name)
return -ENOENT;
}

static int rbd_header_set_snap(struct rbd_device *rbd_dev, char *snap_name)
static int rbd_dev_set_mapping(struct rbd_device *rbd_dev, char *snap_name)
{
int ret;

Expand Down Expand Up @@ -2625,12 +2625,13 @@ static ssize_t rbd_add(struct bus_type *bus,
rc = rbd_dev_snaps_update(rbd_dev);
if (rc)
goto err_out_bus;
rc = rbd_dev_snaps_register(rbd_dev);

rc = rbd_dev_set_mapping(rbd_dev, snap_name);
if (rc)
goto err_out_bus;

down_write(&rbd_dev->header_rwsem);
rc = rbd_header_set_snap(rbd_dev, snap_name);
rc = rbd_dev_snaps_register(rbd_dev);
up_write(&rbd_dev->header_rwsem);
if (rc)
goto err_out_bus;
Expand Down

0 comments on commit 5ed1617

Please sign in to comment.