diff options
author | Benjamin LaHaise <bcrl@kvack.org> | 2013-07-30 12:06:37 -0400 |
---|---|---|
committer | Benjamin LaHaise <bcrl@kvack.org> | 2013-07-30 12:06:37 -0400 |
commit | 4cd81c3dfc4a34e4a0b6fa577860077c8e5b13af (patch) | |
tree | aba6e085ad0193bde59f599b0ad7b3ddb484b744 | |
parent | d29c445b635b3a03cf683cafcbae58a4ec1e1125 (diff) |
aio: double aio_max_nr in calculations
With the changes to use percpu counters for aio event ring size calculation,
existing increases to aio_max_nr are now insufficient to allow for the
allocation of enough events. Double the value used for aio_max_nr to account
for the doubling introduced by the percpu slack.
Signed-off-by: Benjamin LaHaise <bcrl@kvack.org>
-rw-r--r-- | fs/aio.c | 4 |
1 files changed, 2 insertions, 2 deletions
@@ -490,7 +490,7 @@ static struct kioctx *ioctx_alloc(unsigned nr_events) | |||
490 | return ERR_PTR(-EINVAL); | 490 | return ERR_PTR(-EINVAL); |
491 | } | 491 | } |
492 | 492 | ||
493 | if (!nr_events || (unsigned long)nr_events > aio_max_nr) | 493 | if (!nr_events || (unsigned long)nr_events > (aio_max_nr * 2UL)) |
494 | return ERR_PTR(-EAGAIN); | 494 | return ERR_PTR(-EAGAIN); |
495 | 495 | ||
496 | ctx = kmem_cache_zalloc(kioctx_cachep, GFP_KERNEL); | 496 | ctx = kmem_cache_zalloc(kioctx_cachep, GFP_KERNEL); |
@@ -522,7 +522,7 @@ static struct kioctx *ioctx_alloc(unsigned nr_events) | |||
522 | 522 | ||
523 | /* limit the number of system wide aios */ | 523 | /* limit the number of system wide aios */ |
524 | spin_lock(&aio_nr_lock); | 524 | spin_lock(&aio_nr_lock); |
525 | if (aio_nr + nr_events > aio_max_nr || | 525 | if (aio_nr + nr_events > (aio_max_nr * 2UL) || |
526 | aio_nr + nr_events < aio_nr) { | 526 | aio_nr + nr_events < aio_nr) { |
527 | spin_unlock(&aio_nr_lock); | 527 | spin_unlock(&aio_nr_lock); |
528 | goto out_cleanup; | 528 | goto out_cleanup; |