On Mon, Jun 17, 2013 at 10:10:12AM +0200, Michal Hocko wrote:
>On Sun 16-06-13 09:14:47, Wanpeng Li wrote:
>> There is a blank in show_free_areas which lead to dump messages aren't
>> aligned. This patch remove blank.
>>
>> Before patch:
>>
>> [155219.720141] active_anon:50675 inactive_anon:35273 isolated_anon:0
>> [155219.720141] active_file:215421 inactive_file:344268 isolated_file:0
>> [155219.720141] unevictable:0 dirty:35 writeback:0 unstable:0
>> [155219.720141] free:1334870 slab_reclaimable:28833 slab_unreclaimable:5115
>> [155219.720141] mapped:25233 shmem:35511 pagetables:1705 bounce:0
>> [155219.720141] free_cma:0
>>
>> After patch:
>>
>> [ 73.913889] active_anon:39578 inactive_anon:32082 isolated_anon:0
>> [ 73.913889] active_file:14621 inactive_file:57993 isolated_file:0
>> [ 73.913889] unevictable:0dirty:263 writeback:0 unstable:0
>> [ 73.913889] free:1865614 slab_reclaimable:3264 slab_unreclaimable:4566
>> [ 73.913889] mapped:21192 shmem:32327 pagetables:1572 bounce:0
>> [ 73.913889] free_cma:0
>
>Not that I would care much but this format is here for ages. An
>additional space was kind of nice to visually separate this part from
>the per-zone data.
>
>Is there any special reason for this change?
>
No special reason, I'm ok if you prefer the old format. ;-)
>> Signed-off-by: Wanpeng Li <liwanp@xxxxxxxxxxxxxxxxxx>
>> ---
>> mm/page_alloc.c | 12 ++++++------
>> 1 file changed, 6 insertions(+), 6 deletions(-)
>>
>> diff --git a/mm/page_alloc.c b/mm/page_alloc.c
>> index 18102e1..e6e881a 100644
>> --- a/mm/page_alloc.c
>> +++ b/mm/page_alloc.c
>> @@ -3004,12 +3004,12 @@ void show_free_areas(unsigned int filter)
>> }
>>
>> printk("active_anon:%lu inactive_anon:%lu isolated_anon:%lu\n"
>> - " active_file:%lu inactive_file:%lu isolated_file:%lu\n"
>> - " unevictable:%lu"
>> - " dirty:%lu writeback:%lu unstable:%lu\n"
>> - " free:%lu slab_reclaimable:%lu slab_unreclaimable:%lu\n"
>> - " mapped:%lu shmem:%lu pagetables:%lu bounce:%lu\n"
>> - " free_cma:%lu\n",
>> + "active_file:%lu inactive_file:%lu isolated_file:%lu\n"
>> + "unevictable:%lu"
>> + "dirty:%lu writeback:%lu unstable:%lu\n"
>
>There is a space missing between unevictable and dirty.
Oh, I miss that. Thanks for pointing out. ;-)
Regards,
Wanpeng Li
>
>> + "free:%lu slab_reclaimable:%lu slab_unreclaimable:%lu\n"
>> + "mapped:%lu shmem:%lu pagetables:%lu bounce:%lu\n"
>> + "free_cma:%lu\n",
>> global_page_state(NR_ACTIVE_ANON),
>> global_page_state(NR_INACTIVE_ANON),
>> global_page_state(NR_ISOLATED_ANON),
>
>--
>Michal Hocko
>SUSE Labs
--
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>