diff options
author | Benjamin LaHaise <bcrl@kvack.org> | 2013-07-31 10:34:18 -0400 |
---|---|---|
committer | Benjamin LaHaise <bcrl@kvack.org> | 2013-07-31 10:34:18 -0400 |
commit | 6878ea72a5d1aa6caae86449975a50b7fe9abed5 (patch) | |
tree | 9750b6a7f37d340227399ff282a6aefb6892cd98 | |
parent | db446a08c23d5475e6b08c87acca79ebb20f283c (diff) | |
download | blackbird-op-linux-6878ea72a5d1aa6caae86449975a50b7fe9abed5.tar.gz blackbird-op-linux-6878ea72a5d1aa6caae86449975a50b7fe9abed5.zip |
aio: be defensive to ensure request batching is non-zero instead of BUG_ON()
In the event that an overflow/underflow occurs while calculating req_batch,
clamp the minimum at 1 request instead of doing a BUG_ON().
Signed-off-by: Benjamin LaHaise <bcrl@kvack.org>
-rw-r--r-- | fs/aio.c | 3 |
1 files changed, 2 insertions, 1 deletions
@@ -576,7 +576,8 @@ static struct kioctx *ioctx_alloc(unsigned nr_events) atomic_set(&ctx->reqs_available, ctx->nr_events - 1); ctx->req_batch = (ctx->nr_events - 1) / (num_possible_cpus() * 4); - BUG_ON(!ctx->req_batch); + if (ctx->req_batch < 1) + ctx->req_batch = 1; err = ioctx_add_table(ctx, mm); if (err) |