From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from mail-wi0-f179.google.com (mail-wi0-f179.google.com [209.85.212.179]) by kanga.kvack.org (Postfix) with ESMTP id E964B6B0256 for ; Mon, 31 Aug 2015 11:02:41 -0400 (EDT) Received: by wibz8 with SMTP id z8so3206579wib.1 for ; Mon, 31 Aug 2015 08:02:41 -0700 (PDT) Received: from mx2.suse.de (mx2.suse.de. [195.135.220.15]) by mx.google.com with ESMTPS id g6si12125864wje.56.2015.08.31.08.02.39 for (version=TLSv1 cipher=ECDHE-RSA-RC4-SHA bits=128/128); Mon, 31 Aug 2015 08:02:40 -0700 (PDT) Subject: Re: [PATCH 2/3] mm, compaction: export tracepoints zone names to userspace References: <1440689044-2922-1-git-send-email-vbabka@suse.cz> <1440689044-2922-2-git-send-email-vbabka@suse.cz> <20150831105834.34a5e69e@gandalf.local.home> From: Vlastimil Babka Message-ID: <55E46C8E.8070906@suse.cz> Date: Mon, 31 Aug 2015 17:02:38 +0200 MIME-Version: 1.0 In-Reply-To: <20150831105834.34a5e69e@gandalf.local.home> Content-Type: text/plain; charset=windows-1252; format=flowed Content-Transfer-Encoding: 7bit Sender: owner-linux-mm@kvack.org List-ID: To: Steven Rostedt Cc: Andrew Morton , linux-kernel@vger.kernel.org, linux-mm@kvack.org, Joonsoo Kim , Ingo Molnar , Mel Gorman , David Rientjes On 08/31/2015 04:58 PM, Steven Rostedt wrote: > On Thu, 27 Aug 2015 17:24:03 +0200 > Vlastimil Babka wrote: > >> Some compaction tracepoints use zone->name to print which zone is being >> compacted. This works for in-kernel printing, but not userspace trace printing >> of raw captured trace such as via trace-cmd report. >> >> This patch uses zone_idx() instead of zone->name as the raw value, and when >> printing, converts the zone_type to string using the appropriate EM() macros >> and some ugly tricks to overcome the problem that half the values depend on >> CONFIG_ options and one does not simply use #ifdef inside of #define. >> >> trace-cmd output before: >> transhuge-stres-4235 [000] 453.149280: mm_compaction_finished: node=0 >> zone=ffffffff81815d7a order=9 ret=partial >> >> after: >> transhuge-stres-4235 [000] 453.149280: mm_compaction_finished: node=0 >> zone=Normal order=9 ret=partial >> >> Signed-off-by: Vlastimil Babka >> Cc: Joonsoo Kim >> Cc: Steven Rostedt >> Cc: Ingo Molnar >> Cc: Mel Gorman >> Cc: David Rientjes >> --- >> include/trace/events/compaction.h | 38 ++++++++++++++++++++++++++++++++------ >> 1 file changed, 32 insertions(+), 6 deletions(-) >> >> diff --git a/include/trace/events/compaction.h b/include/trace/events/compaction.h >> index 1275a55..8daa8fa 100644 >> --- a/include/trace/events/compaction.h >> +++ b/include/trace/events/compaction.h >> @@ -18,6 +18,31 @@ >> EM( COMPACT_NO_SUITABLE_PAGE, "no_suitable_page") \ >> EMe(COMPACT_NOT_SUITABLE_ZONE, "not_suitable_zone") >> >> +#ifdef CONFIG_ZONE_DMA >> +#define IFDEF_ZONE_DMA(X) X >> +#else >> +#define IFDEF_ZONE_DMA(X) >> +#endif >> + >> +#ifdef CONFIG_ZONE_DMA32 >> +#define IFDEF_ZONE_DMA32(X) X >> +#else >> +#define IFDEF_ZONE_DMA32(X) >> +#endif >> + >> +#ifdef CONFIG_ZONE_HIGHMEM_ >> +#define IFDEF_ZONE_HIGHMEM(X) X >> +#else >> +#define IFDEF_ZONE_HIGHMEM(X) >> +#endif >> + >> +#define ZONE_TYPE \ >> + IFDEF_ZONE_DMA( EM (ZONE_DMA, "DMA")) \ >> + IFDEF_ZONE_DMA32( EM (ZONE_DMA32, "DMA32")) \ >> + EM (ZONE_NORMAL, "Normal") \ >> + IFDEF_ZONE_HIGHMEM( EM (ZONE_HIGHMEM,"HighMem")) \ >> + EMe(ZONE_MOVABLE,"Movable") >> + > > Hmm, have you tried to compile this with CONFIG_ZONE_HIGHMEM disabled, > and CONFIG_ZONE_DMA and/or CONFIG_ZONE_DMA32 enabled? Yep, that's standard x86_64 situation (highmem disabled, dma+dma32 enabled). > The EMe() macro must come last, as it doesn't have the ending comma and > the __print_symbolic() can fail to compile due to it. Thanks to ZONE_MOVABLE being unconditional, EMe(ZONE_MOVABLE...) is always last. Otherwise the macros would get even more ugly... > -- Steve > > >> /* >> * First define the enums in the above macros to be exported to userspace >> * via TRACE_DEFINE_ENUM(). >> @@ -28,6 +53,7 @@ -- 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