diff options
author | Markus Pargmann <mpa@pengutronix.de> | 2015-04-02 04:11:41 -0400 |
---|---|---|
committer | Jens Axboe <axboe@fb.com> | 2015-04-02 14:39:28 -0400 |
commit | de9ad6d4edb63e0ba5d5aae365fb3565064fc00d (patch) | |
tree | db4a3b3b4a54c20b57d83fee0be5f80fbe9c9c62 /drivers/block | |
parent | dab5313aa4e668d87253dd4289c816cb08f63e52 (diff) |
nbd: Return error pointer directly
Signed-off-by: Markus Pargmann <mpa@pengutronix.de>
Acked-by: Pavel Machek <pavel@ucw.cz>
Signed-off-by: Jens Axboe <axboe@fb.com>
Diffstat (limited to 'drivers/block')
-rw-r--r-- | drivers/block/nbd.c | 7 |
1 files changed, 2 insertions, 5 deletions
diff --git a/drivers/block/nbd.c b/drivers/block/nbd.c index 54bf633c9013..39e5f7fae3ef 100644 --- a/drivers/block/nbd.c +++ b/drivers/block/nbd.c | |||
@@ -285,7 +285,7 @@ static struct request *nbd_find_request(struct nbd_device *nbd, | |||
285 | 285 | ||
286 | err = wait_event_interruptible(nbd->active_wq, nbd->active_req != xreq); | 286 | err = wait_event_interruptible(nbd->active_wq, nbd->active_req != xreq); |
287 | if (unlikely(err)) | 287 | if (unlikely(err)) |
288 | goto out; | 288 | return ERR_PTR(err); |
289 | 289 | ||
290 | spin_lock(&nbd->queue_lock); | 290 | spin_lock(&nbd->queue_lock); |
291 | list_for_each_entry_safe(req, tmp, &nbd->queue_head, queuelist) { | 291 | list_for_each_entry_safe(req, tmp, &nbd->queue_head, queuelist) { |
@@ -297,10 +297,7 @@ static struct request *nbd_find_request(struct nbd_device *nbd, | |||
297 | } | 297 | } |
298 | spin_unlock(&nbd->queue_lock); | 298 | spin_unlock(&nbd->queue_lock); |
299 | 299 | ||
300 | err = -ENOENT; | 300 | return ERR_PTR(-ENOENT); |
301 | |||
302 | out: | ||
303 | return ERR_PTR(err); | ||
304 | } | 301 | } |
305 | 302 | ||
306 | static inline int sock_recv_bvec(struct nbd_device *nbd, struct bio_vec *bvec) | 303 | static inline int sock_recv_bvec(struct nbd_device *nbd, struct bio_vec *bvec) |