Messages in this thread | | | Date | Wed, 21 Dec 2011 16:32:40 -0800 | From | Andrew Morton <> | Subject | Re: [PATCH 2/2] mempool: fix first round failure behavior |
| |
On Wed, 21 Dec 2011 16:19:39 -0800 Tejun Heo <tj@kernel.org> wrote:
> For the initial allocation, mempool passes modified gfp mask to the > backing allocator so that it doesn't try too hard when there are > reserved elements waiting in the pool; however, when that allocation > fails and pool is empty too, it either waits for the pool to be > replenished before retrying or fails if !__GFP_WAIT. > > * If the caller was calling in with GFP_ATOMIC, it never gets to try > emergency reserve. Allocations which would have succeeded without > mempool may fail, which is just wrong. > > * Allocation which could have succeeded after a bit of reclaim now has > to wait on the reserved items and it's not like mempool doesn't > retry with the original gfp mask. It just does that *after* someone > returns an element, pointlessly delaying things.
This is a significant change in behaviour. Previously the mempool code would preserve emergency pools while waiting for someone to return an item. Now, it will permit many more items to be allocated, chewing into the emergency pools.
We *know* that items will soon become available, so why not wait for that to happen rather than consuming memory which less robust callers could have utilised?
IOW, this change appears to make the kernel more vulnerable to memory exhaustion failures?
| |