Commit 62054da6 authored by Ilya Dryomov's avatar Ilya Dryomov Committed by Yan, Zheng

rbd: remove out_partial label in rbd_img_request_fill()

Commit 03507db6 ("rbd: fix buffer size for writes to images with
snapshots") moved the call to rbd_img_obj_request_add() up, making the
out_partial label bogus.  Remove it.
Signed-off-by: default avatarIlya Dryomov <ilya.dryomov@inktank.com>
Reviewed-by: default avatarAlex Elder <elder@linaro.org>
parent 9d521470
...@@ -2190,6 +2190,7 @@ static int rbd_img_request_fill(struct rbd_img_request *img_request, ...@@ -2190,6 +2190,7 @@ static int rbd_img_request_fill(struct rbd_img_request *img_request,
rbd_segment_name_free(object_name); rbd_segment_name_free(object_name);
if (!obj_request) if (!obj_request)
goto out_unwind; goto out_unwind;
/* /*
* set obj_request->img_request before creating the * set obj_request->img_request before creating the
* osd_request so that it gets the right snapc * osd_request so that it gets the right snapc
...@@ -2207,7 +2208,7 @@ static int rbd_img_request_fill(struct rbd_img_request *img_request, ...@@ -2207,7 +2208,7 @@ static int rbd_img_request_fill(struct rbd_img_request *img_request,
clone_size, clone_size,
GFP_ATOMIC); GFP_ATOMIC);
if (!obj_request->bio_list) if (!obj_request->bio_list)
goto out_partial; goto out_unwind;
} else { } else {
unsigned int page_count; unsigned int page_count;
...@@ -2222,7 +2223,7 @@ static int rbd_img_request_fill(struct rbd_img_request *img_request, ...@@ -2222,7 +2223,7 @@ static int rbd_img_request_fill(struct rbd_img_request *img_request,
osd_req = rbd_osd_req_create(rbd_dev, write_request, osd_req = rbd_osd_req_create(rbd_dev, write_request,
obj_request); obj_request);
if (!osd_req) if (!osd_req)
goto out_partial; goto out_unwind;
obj_request->osd_req = osd_req; obj_request->osd_req = osd_req;
obj_request->callback = rbd_img_obj_callback; obj_request->callback = rbd_img_obj_callback;
...@@ -2249,8 +2250,6 @@ static int rbd_img_request_fill(struct rbd_img_request *img_request, ...@@ -2249,8 +2250,6 @@ static int rbd_img_request_fill(struct rbd_img_request *img_request,
return 0; return 0;
out_partial:
rbd_obj_request_put(obj_request);
out_unwind: out_unwind:
for_each_obj_request_safe(img_request, obj_request, next_obj_request) for_each_obj_request_safe(img_request, obj_request, next_obj_request)
rbd_obj_request_put(obj_request); rbd_obj_request_put(obj_request);
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment