aboutsummaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorJosh Durgin <josh.durgin@inktank.com>2013-08-29 22:16:42 -0400
committerGreg Kroah-Hartman <gregkh@linuxfoundation.org>2014-01-09 15:24:26 -0500
commit6fe77759c31c0f5fc36837549c5848470113509b (patch)
treec4dc676e2766974df7913c39c9e3069400e5910c
parent5b213542db631f8b0bf7b257e8ae2d37b134895c (diff)
rbd: ignore unmapped snapshots that no longer exist
commit efadc98aab674153709cc357ba565f04e3164fcd upstream. This prevents erroring out while adding a device when a snapshot unrelated to the current mapping is deleted between reading the snapshot context and reading the snapshot names. If the mapped snapshot name is not found an error still occurs as usual. Signed-off-by: Josh Durgin <josh.durgin@inktank.com> Reviewed-by: Alex Elder <elder@linaro.org> Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
-rw-r--r--drivers/block/rbd.c9
1 files changed, 7 insertions, 2 deletions
diff --git a/drivers/block/rbd.c b/drivers/block/rbd.c
index 58a4e03b7311..e917d4f7b76e 100644
--- a/drivers/block/rbd.c
+++ b/drivers/block/rbd.c
@@ -4055,8 +4055,13 @@ static u64 rbd_v2_snap_id_by_name(struct rbd_device *rbd_dev, const char *name)
4055 4055
4056 snap_id = snapc->snaps[which]; 4056 snap_id = snapc->snaps[which];
4057 snap_name = rbd_dev_v2_snap_name(rbd_dev, snap_id); 4057 snap_name = rbd_dev_v2_snap_name(rbd_dev, snap_id);
4058 if (IS_ERR(snap_name)) 4058 if (IS_ERR(snap_name)) {
4059 break; 4059 /* ignore no-longer existing snapshots */
4060 if (PTR_ERR(snap_name) == -ENOENT)
4061 continue;
4062 else
4063 break;
4064 }
4060 found = !strcmp(name, snap_name); 4065 found = !strcmp(name, snap_name);
4061 kfree(snap_name); 4066 kfree(snap_name);
4062 } 4067 }