diff options
author | Jens Axboe <axboe@fb.com> | 2015-03-06 16:37:46 +0100 |
---|---|---|
committer | Jens Axboe <axboe@fb.com> | 2015-07-11 17:57:32 +0200 |
commit | 77b5a08427e87514c33730afc18cd02c9475e2c3 (patch) | |
tree | b0d325533280f47253871044e875c0f71a369896 /drivers/md/bcache/io.c | |
parent | blkcg: fix blkcg_policy_data allocation bug (diff) | |
download | linux-77b5a08427e87514c33730afc18cd02c9475e2c3.tar.xz linux-77b5a08427e87514c33730afc18cd02c9475e2c3.zip |
bcache: don't embed 'return' statements in closure macros
This is horribly confusing, it breaks the flow of the code without
it being apparent in the caller.
Signed-off-by: Jens Axboe <axboe@fb.com>
Acked-by: Christoph Hellwig <hch@lst.de>
Diffstat (limited to 'drivers/md/bcache/io.c')
-rw-r--r-- | drivers/md/bcache/io.c | 1 |
1 files changed, 1 insertions, 0 deletions
diff --git a/drivers/md/bcache/io.c b/drivers/md/bcache/io.c index cb64e64a4789..bf6a9ca18403 100644 --- a/drivers/md/bcache/io.c +++ b/drivers/md/bcache/io.c @@ -105,6 +105,7 @@ void bch_generic_make_request(struct bio *bio, struct bio_split_pool *p) } while (n != bio); continue_at(&s->cl, bch_bio_submit_split_done, NULL); + return; submit: generic_make_request(bio); } |