Re: [PATCH 1/3] mm/compaction: enhance trace output to know more about compaction internals

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On Tue, Jan 06, 2015 at 10:05:39AM +0100, Vlastimil Babka wrote:
> On 12/03/2014 08:52 AM, Joonsoo Kim wrote:
> > It'd be useful to know where the both scanner is start. And, it also be
> > useful to know current range where compaction work. It will help to find
> > odd behaviour or problem on compaction.
> 
> Overall it looks good, just two questions:
> 1) Why change the pfn output to hexadecimal with different printf layout and
> change the variable names and? Is it that better to warrant people having to
> potentially modify their scripts parsing the old output?

Deciaml output has really bad readability since we manage all pages by order
of 2 which is well represented by hexadecimal. With hex output, we can
easily notice whether we move out from one pageblock to another one.

> 2) Would it be useful to also print in the mm_compaction_isolate_template based
> tracepoints, pfn of where the particular scanner left off a block prematurely?
> It doesn't always match start_pfn + nr_scanned.

With start_pfn and end_pfn, detailed analysis is possible. We can know pageblock
where we actually scan and isolate and how much pages we try in that
pageblock and can guess why it doesn't become freepage with pageblock
order roughly.

nr_scanned is just different metric. end_pfn don't need to match with
start_pfn + nr_scanned.

Thanks.

--
To unsubscribe, send a message with 'unsubscribe linux-mm' in
the body to majordomo@xxxxxxxxx.  For more info on Linux MM,
see: http://www.linux-mm.org/ .
Don't email: <a href=mailto:"dont@xxxxxxxxx";> email@xxxxxxxxx </a>



[Index of Archives]     [Linux ARM Kernel]     [Linux ARM]     [Linux Omap]     [Fedora ARM]     [IETF Annouce]     [Bugtraq]     [Linux]     [Linux OMAP]     [Linux MIPS]     [ECOS]     [Asterisk Internet PBX]     [Linux API]