Re: [PATCH 2/2] mm: Abort reclaim/compaction if compaction can proceed

Rik van Riel
2011-10-07T20:07:21Z

On 10/07/2011 11:17 AM, Mel Gorman wrote:
> If compaction can proceed, shrink_zones() stops doing any work but
> the callers still shrink_slab(), raises the priority and potentially
> sleeps. This patch aborts direct reclaim/compaction entirely if
> compaction can proceed.
>
> Signed-off-by: Mel Gorman<mgorman@suse.de>

This patch makes sense to me, but I have not tested it like
the first one.

Mel, have you tested this patch? Did you see any changed
behaviour vs. just the first patch?

Having said that, I'm pretty sure the patch is ok :)

--
All rights reversed