Commit 8785b1d4 authored by Alex Elder's avatar Alex Elder

rbd: don't release write request until necessary

Previously when a layered write was going to involve a copyup
request, the original osd request was released before submitting the
parent full-object read.  The osd request for the copyup would then
be allocated in rbd_img_obj_parent_read_full_callback().

Shortly we will be handling the event of mapped layered images
getting flattened, and when that occurs we need to resubmit the
original request.  We therefore don't want to release the osd
request until we really konw we're going to replace it--in the
callback function.
Signed-off-by: default avatarAlex Elder <elder@inktank.com>
Reviewed-by: default avatarJosh Durgin <josh.durgin@inktank.com>
parent 642a2537
...@@ -2194,13 +2194,17 @@ rbd_img_obj_parent_read_full_callback(struct rbd_img_request *img_request) ...@@ -2194,13 +2194,17 @@ rbd_img_obj_parent_read_full_callback(struct rbd_img_request *img_request)
if (result) if (result)
goto out_err; goto out_err;
/* Allocate the new copyup osd request for the original request */ /*
* The original osd request is of no use to use any more.
* We need a new one that can hold the two ops in a copyup
* request. Allocate the new copyup osd request for the
* original request, and release the old one.
*/
result = -ENOMEM; result = -ENOMEM;
rbd_assert(!orig_request->osd_req);
osd_req = rbd_osd_req_create_copyup(orig_request); osd_req = rbd_osd_req_create_copyup(orig_request);
if (!osd_req) if (!osd_req)
goto out_err; goto out_err;
rbd_osd_req_destroy(orig_request->osd_req);
orig_request->osd_req = osd_req; orig_request->osd_req = osd_req;
orig_request->copyup_pages = pages; orig_request->copyup_pages = pages;
orig_request->copyup_page_count = page_count; orig_request->copyup_page_count = page_count;
...@@ -2276,15 +2280,6 @@ static int rbd_img_obj_parent_read_full(struct rbd_obj_request *obj_request) ...@@ -2276,15 +2280,6 @@ static int rbd_img_obj_parent_read_full(struct rbd_obj_request *obj_request)
rbd_dev = img_request->rbd_dev; rbd_dev = img_request->rbd_dev;
rbd_assert(rbd_dev->parent != NULL); rbd_assert(rbd_dev->parent != NULL);
/*
* First things first. The original osd request is of no
* use to use any more, we'll need a new one that can hold
* the two ops in a copyup request. We'll get that later,
* but for now we can release the old one.
*/
rbd_osd_req_destroy(obj_request->osd_req);
obj_request->osd_req = NULL;
/* /*
* Determine the byte range covered by the object in the * Determine the byte range covered by the object in the
* child image to which the original request was to be sent. * child image to which the original request was to be sent.
......
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