On 05/21/2015 07:35 AM, Ilya Dryomov wrote: > As part of unmap sequence, kernel client has to talk to the OSDs to > teardown watch on the header object. If none of the OSDs are available > it would hang forever, until interrupted by a signal - when that > happens we follow through with the rest of unmap procedure (i.e. > unregister the device and put all the data structures) and the unmap is > still considired successful (rbd cli tool exits with 0). The watch on > the userspace side should eventually timeout so that's fine. > > This isn't very nice, because various userspace tools (pacemaker rbd > resource agent, for example) then have to worry about setting up their > own timeouts. Timeout it with mount_timeout (60 seconds by default). > > Signed-off-by: Ilya Dryomov <[email protected]> > Reviewed-by: Sage Weil <[email protected]>
You can now add: Reviewed-by: Alex Elder <[email protected]> > --- > drivers/block/rbd.c | 38 ++++++++++++++++++++++++++++---------- > 1 file changed, 28 insertions(+), 10 deletions(-) > > diff --git a/drivers/block/rbd.c b/drivers/block/rbd.c > index 992683b6b299..89fe8a4bc02e 100644 > --- a/drivers/block/rbd.c > +++ b/drivers/block/rbd.c > @@ -1563,22 +1563,39 @@ static void rbd_obj_request_end(struct > rbd_obj_request *obj_request) > /* > * Wait for an object request to complete. If interrupted, cancel the > * underlying osd request. > + * > + * @timeout: in jiffies, 0 means "wait forever" > */ > -static int rbd_obj_request_wait(struct rbd_obj_request *obj_request) > +static int __rbd_obj_request_wait(struct rbd_obj_request *obj_request, > + unsigned long timeout) > { > - int ret; > + long ret; > > dout("%s %p\n", __func__, obj_request); > - > - ret = wait_for_completion_interruptible(&obj_request->completion); > - if (ret < 0) { > - dout("%s %p interrupted\n", __func__, obj_request); > + ret = wait_for_completion_interruptible_timeout( > + &obj_request->completion, > + ceph_timeout_jiffies(timeout)); > + if (ret <= 0) { > + if (ret == 0) > + ret = -ETIMEDOUT; > rbd_obj_request_end(obj_request); > - return ret; > + } else { > + ret = 0; > } > > - dout("%s %p done\n", __func__, obj_request); > - return 0; > + dout("%s %p ret %d\n", __func__, obj_request, (int)ret); > + return ret; > +} > + > +static int rbd_obj_request_wait(struct rbd_obj_request *obj_request) > +{ > + return __rbd_obj_request_wait(obj_request, 0); > +} > + > +static int rbd_obj_request_wait_timeout(struct rbd_obj_request *obj_request, > + unsigned long timeout) > +{ > + return __rbd_obj_request_wait(obj_request, timeout); > } > > static void rbd_img_request_complete(struct rbd_img_request *img_request) > @@ -3122,6 +3139,7 @@ static struct rbd_obj_request > *rbd_obj_watch_request_helper( > bool watch) > { > struct ceph_osd_client *osdc = &rbd_dev->rbd_client->client->osdc; > + struct ceph_options *opts = osdc->client->options; > struct rbd_obj_request *obj_request; > int ret; > > @@ -3148,7 +3166,7 @@ static struct rbd_obj_request > *rbd_obj_watch_request_helper( > if (ret) > goto out; > > - ret = rbd_obj_request_wait(obj_request); > + ret = rbd_obj_request_wait_timeout(obj_request, opts->mount_timeout); > if (ret) > goto out; > > -- To unsubscribe from this list: send the line "unsubscribe ceph-devel" in the body of a message to [email protected] More majordomo info at http://vger.kernel.org/majordomo-info.html
