lkml.org 
[lkml]   [2012]   [Dec]   [20]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
/
Date
From
SubjectRe: [PATCH] mm: compaction: count compaction events only if compaction is enabled
On Thu, Dec 20, 2012 at 11:21:34AM +0100, Jiri Kosina wrote:
> On configs which have CONFIG_CMA but no CONFIG_COMPACTION,
> isolate_migratepages_range() and isolate_freepages_block() must not
> account for COMPACTFREE_SCANNED and COMPACTISOLATED events (those
> constants are even undefined in such case, causing a build error).
>
> Signed-off-by: Jiri Kosina <jkosina@suse.cz>

Minchan has a similar patch in the works that defines count_compact_events()
similar to count_vm_numa_events(). It just needs a small correction. The
fixed version would avoid having an #ifdef in the middle of the function
which is cosmetically a bit nicer.

--
Mel Gorman
SUSE Labs


\
 
 \ /
  Last update: 2012-12-20 12:42    [W:0.032 / U:0.128 seconds]
©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site