Messages in this thread | | | Date | Mon, 25 Nov 2013 17:08:04 -0500 | From | Rik van Riel <> | Subject | Re: [PATCH 1/5] mm: compaction: encapsulate defer reset logic |
| |
On 11/25/2013 09:26 AM, Vlastimil Babka wrote: > Currently there are several functions to manipulate the deferred compaction > state variables. The remaining case where the variables are touched directly > is when a successful allocation occurs in direct compaction, or is expected > to be successful in the future by kswapd. Here, the lowest order that is > expected to fail is updated, and in the case of direct compaction, the deferred > status is reset completely. > > Create a new function compaction_defer_reset() to encapsulate this > functionality and make it easier to understand the code. No functional change. > > Cc: Mel Gorman <mgorman@suse.de> > Cc: Rik van Riel <riel@redhat.com> > Signed-off-by: Vlastimil Babka <vbabka@suse.cz>
Reviewed-by: Rik van Riel <riel@redhat.com>
| |