From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from mail-wm0-f43.google.com (mail-wm0-f43.google.com [74.125.82.43]) by kanga.kvack.org (Postfix) with ESMTP id 5AC4482F64 for ; Tue, 24 Nov 2015 07:37:04 -0500 (EST) Received: by wmww144 with SMTP id w144so24453484wmw.0 for ; Tue, 24 Nov 2015 04:37:04 -0800 (PST) Received: from mx2.suse.de (mx2.suse.de. [195.135.220.15]) by mx.google.com with ESMTPS id y140si26526680wmc.19.2015.11.24.04.36.44 for (version=TLS1 cipher=ECDHE-RSA-AES128-SHA bits=128/128); Tue, 24 Nov 2015 04:36:44 -0800 (PST) From: Vlastimil Babka Subject: [PATCH v2 6/9] mm, debug: introduce dump_gfpflag_names() for symbolic printing of gfp_flags Date: Tue, 24 Nov 2015 13:36:18 +0100 Message-Id: <1448368581-6923-7-git-send-email-vbabka@suse.cz> In-Reply-To: <1448368581-6923-1-git-send-email-vbabka@suse.cz> References: <1448368581-6923-1-git-send-email-vbabka@suse.cz> Sender: owner-linux-mm@kvack.org List-ID: To: linux-mm@kvack.org Cc: linux-kernel@vger.kernel.org, Andrew Morton , Joonsoo Kim , Minchan Kim , Sasha Levin , "Kirill A. Shutemov" , Mel Gorman , Michal Hocko , Vlastimil Babka It would be useful to convert gfp_flags into string representation when printing them in case of allocation failure, OOM etc. There's a script ./scripts/gfp-translate to make this simpler, but it needs the matching version of the sources to be accurate, and the flags have been undergoing some changes recently. The ftrace framework already has this translation in the form of show_gfp_flags() defined in include/trace/events/gfpflags.h which defines the translation table internally. Allow reusing the table outside ftrace by putting it behind __def_gfpflag_names definition and introduce dump_gfpflag_names() to handle the printing. While at it, also fill in the names for the flags and flag combinations that have been missing in the table. GFP_NOWAIT no longer equals to "no flags", so change the output for no flags to "none". Signed-off-by: Vlastimil Babka --- include/linux/mmdebug.h | 1 + include/trace/events/gfpflags.h | 14 +++++++++++--- mm/debug.c | 10 ++++++++++ 3 files changed, 22 insertions(+), 3 deletions(-) diff --git a/include/linux/mmdebug.h b/include/linux/mmdebug.h index c447d80..3b77fab 100644 --- a/include/linux/mmdebug.h +++ b/include/linux/mmdebug.h @@ -10,6 +10,7 @@ struct mm_struct; extern void dump_page(struct page *page, const char *reason); extern void dump_page_badflags(struct page *page, const char *reason, unsigned long badflags); +extern void dump_gfpflag_names(unsigned long gfp_flags); void dump_vma(const struct vm_area_struct *vma); void dump_mm(const struct mm_struct *mm); diff --git a/include/trace/events/gfpflags.h b/include/trace/events/gfpflags.h index dde6bf0..3d580fd 100644 --- a/include/trace/events/gfpflags.h +++ b/include/trace/events/gfpflags.h @@ -8,8 +8,8 @@ * * Thus most bits set go first. */ -#define show_gfp_flags(flags) \ - (flags) ? __print_flags(flags, "|", \ + +#define __def_gfpflag_names \ {(unsigned long)GFP_TRANSHUGE, "GFP_TRANSHUGE"}, \ {(unsigned long)GFP_HIGHUSER_MOVABLE, "GFP_HIGHUSER_MOVABLE"}, \ {(unsigned long)GFP_HIGHUSER, "GFP_HIGHUSER"}, \ @@ -19,9 +19,13 @@ {(unsigned long)GFP_NOFS, "GFP_NOFS"}, \ {(unsigned long)GFP_ATOMIC, "GFP_ATOMIC"}, \ {(unsigned long)GFP_NOIO, "GFP_NOIO"}, \ + {(unsigned long)GFP_NOWAIT, "GFP_NOWAIT"}, \ + {(unsigned long)__GFP_DMA, "GFP_DMA"}, \ + {(unsigned long)__GFP_DMA32, "GFP_DMA32"}, \ {(unsigned long)__GFP_HIGH, "GFP_HIGH"}, \ {(unsigned long)__GFP_ATOMIC, "GFP_ATOMIC"}, \ {(unsigned long)__GFP_IO, "GFP_IO"}, \ + {(unsigned long)__GFP_FS, "GFP_FS"}, \ {(unsigned long)__GFP_COLD, "GFP_COLD"}, \ {(unsigned long)__GFP_NOWARN, "GFP_NOWARN"}, \ {(unsigned long)__GFP_REPEAT, "GFP_REPEAT"}, \ @@ -36,8 +40,12 @@ {(unsigned long)__GFP_RECLAIMABLE, "GFP_RECLAIMABLE"}, \ {(unsigned long)__GFP_MOVABLE, "GFP_MOVABLE"}, \ {(unsigned long)__GFP_NOTRACK, "GFP_NOTRACK"}, \ + {(unsigned long)__GFP_WRITE, "GFP_WRITE"}, \ {(unsigned long)__GFP_DIRECT_RECLAIM, "GFP_DIRECT_RECLAIM"}, \ {(unsigned long)__GFP_KSWAPD_RECLAIM, "GFP_KSWAPD_RECLAIM"}, \ {(unsigned long)__GFP_OTHER_NODE, "GFP_OTHER_NODE"} \ - ) : "GFP_NOWAIT" +#define show_gfp_flags(flags) \ + (flags) ? __print_flags(flags, "|", \ + __def_gfpflag_names \ + ) : "none" diff --git a/mm/debug.c b/mm/debug.c index d9718fc..1a71a3b 100644 --- a/mm/debug.c +++ b/mm/debug.c @@ -9,6 +9,7 @@ #include #include #include +#include static const struct trace_print_flags pageflag_names[] = { {1UL << PG_locked, "locked" }, @@ -46,6 +47,10 @@ static const struct trace_print_flags pageflag_names[] = { #endif }; +static const struct trace_print_flags gfpflag_names[] = { + __def_gfpflag_names +}; + static void dump_flag_names(unsigned long flags, const struct trace_print_flags *names, int count) { @@ -73,6 +78,11 @@ static void dump_flag_names(unsigned long flags, pr_cont(")\n"); } +void dump_gfpflag_names(unsigned long gfp_flags) +{ + dump_flag_names(gfp_flags, gfpflag_names, ARRAY_SIZE(gfpflag_names)); +} + void dump_page_badflags(struct page *page, const char *reason, unsigned long badflags) { -- 2.6.3 -- To unsubscribe, send a message with 'unsubscribe linux-mm' in the body to majordomo@kvack.org. For more info on Linux MM, see: http://www.linux-mm.org/ . Don't email: email@kvack.org