On 07/11/14 at 10:26am, Vivek Goyal wrote: > On Fri, Jul 11, 2014 at 03:16:04PM +0800, Baoquan He wrote: > Hi Bao, > > How about priting number of pages in decimal numbers. Humans read decimal > numbers much better. That's fine too. I can change it to decimal number. Just think that maybe it's mainly used by script. > > > > -------------------------------------------------- > > TYPE PAGES EXCLUDABLE DESCRIPTION > > pfn_zero 0x0000000000000000 yes Pages filled with zero > > I am wondering if we display TYPE of page as follows. Will it be more > redable OK, will change. > > ZERO > CACHE > CACHE_PRIVATE > .. > .. > > > pfn_cache 0x000000000002b19f yes Cache pages > > pfn_cache_private 0x0000000000042526 yes Cache pages + private > > pfn_user 0x0000000000026bc3 yes User process data pages > > How about calling them just "User process pages" and not "user process > data pages" I am assuming that they could contain text too. This function is based on the print_report() which is in makedumpfile. It has been defined like these already. But I will change this according reviewer's comments. After all, they are different. > > > pfn_free 0x0000000000133f79 yes Free pages > > pfn_hwpoison 0x0000000000000000 yes Hwpoison pages > > pfn_kernel_data 0x000000000001dd56 no Dumpable kernel data > > > > Total pages on system: 0x00000000001e5f57 > > Excludable pages: 0x00000000001c8201 > > Memory Hole: 0x00000000000386a9 > > What is "Memory Hole" ? That areas which are between "System Ram". > > > -------------------------------------------------- > > Total pages: 0x000000000021e600 > > How this total pages is different from "Total pages on system"? "Total pages on system" is all physical pages without hole. "Total pages" is equal to the max_pfn in current system. This is got from the print_report() too. It could be removed if people think it's useless. > > Thanks > Vivek