diff options
| author |   Ilya Dryomov <idryomov@gmail.com> | 2018-01-30 17:52:11 +0100 |
|---|---|---|
| committer | Ilya Dryomov <idryomov@gmail.com> | 2018-04-02 10:12:41 +0200 |
| commit | efbd1a1106f15f9260c7cb9a67f5c380a39b4fcc (patch) | |
| tree | bd9f8cd708dfff3f66bad2f04aca1d7727464a61 /drivers/block | |
| parent | rbd: add img_req->op_type field (diff) | |
| download | linux-dev-efbd1a1106f15f9260c7cb9a67f5c380a39b4fcc.tar.xz linux-dev-efbd1a1106f15f9260c7cb9a67f5c380a39b4fcc.zip | |
rbd: update rbd_img_request_submit() signature
It should be void now. Also, object requests are unlinked only in
image request destructor, which can't run before rbd_img_request_put(),
so no need for _safe.
Signed-off-by: Ilya Dryomov <idryomov@gmail.com>
Diffstat (limited to 'drivers/block')
| -rw-r--r-- | drivers/block/rbd.c | 13 |
1 files changed, 3 insertions, 10 deletions
diff --git a/drivers/block/rbd.c b/drivers/block/rbd.c index e542fda18395..015bd5303f0e 100644 --- a/drivers/block/rbd.c +++ b/drivers/block/rbd.c @@ -2223,21 +2223,17 @@ out_unwind: return -ENOMEM; } -static int rbd_img_request_submit(struct rbd_img_request *img_request) +static void rbd_img_request_submit(struct rbd_img_request *img_request) { struct rbd_obj_request *obj_request; - struct rbd_obj_request *next_obj_request; - int ret = 0; dout("%s: img %p\n", __func__, img_request); rbd_img_request_get(img_request); - for_each_obj_request_safe(img_request, obj_request, next_obj_request) { + for_each_obj_request(img_request, obj_request) rbd_obj_request_submit(obj_request); - } rbd_img_request_put(img_request); - return ret; } static void rbd_img_end_child_request(struct rbd_img_request *img_req); @@ -3668,10 +3664,7 @@ static void rbd_queue_workfn(struct work_struct *work) if (result) goto err_img_request; - result = rbd_img_request_submit(img_request); - if (result) - goto err_img_request; - + rbd_img_request_submit(img_request); if (must_be_locked) up_read(&rbd_dev->lock_rwsem); return; |