>> The array pageflag_names[] is doing the conversion from page flag
>> into the corresponding names so that the meaingful string again
>> the corresponding page flag can be printed. The mechniasm is used
>> while dumping the specified page frame. However, the array missed
>> PG_compound_lock. So PG_compound_lock page flag would be printed
>> as ditigal number instead of meaningful string.
>>
>> The patch fixes that and print "compound_lock" for PG_compound_lock
>> page flag.
>>
>> Signed-off-by: Gavin Shan <shangw@xxxxxxxxxxxxxxxxxx>
>
>Acked-by: Johannes Weiner <hannes@xxxxxxxxxxx>
>
>This on top?
>
>---
Thanks for your time, Johannes. The check at compliling time looks good.
Thanks,
Gavin
>From: Johannes Weiner <hannes@xxxxxxxxxxx>
>Subject: [patch] mm: page_alloc: catch out-of-date list of page flag names
>
>String tables with names of enum items are always prone to go out of
>sync with the enums themselves. Ensure during compile time that the
>name table of page flags has the same size as the page flags enum.
>
>Signed-off-by: Johannes Weiner <hannes@xxxxxxxxxxx>
>---
> mm/page_alloc.c | 2 ++
> 1 file changed, 2 insertions(+)
>
>diff --git a/mm/page_alloc.c b/mm/page_alloc.c
>index 9325913..65ae58d 100644
>--- a/mm/page_alloc.c
>+++ b/mm/page_alloc.c
>@@ -5986,6 +5986,8 @@ static void dump_page_flags(unsigned long flags)
> unsigned long mask;
> int i;
>
>+ BUILD_BUG_ON(ARRAY_SIZE(pageflag_names) - 1 != __NR_PAGEFLAGS);
>+
> printk(KERN_ALERT "page flags: %#lx(", flags);
>
> /* remove zone id */
>--
>1.7.10.1
>
>--
>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/ .
>Fight unfair telecom internet charges in Canada: sign http://stopthemeter.ca/
>Don't email: <a href=mailto:"dont@xxxxxxxxx";> email@xxxxxxxxx </a>
>
--
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/ .
Fight unfair telecom internet charges in Canada: sign http://stopthemeter.ca/
Don't email: <a href=mailto:"dont@xxxxxxxxx";> email@xxxxxxxxx </a>