From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: X-Spam-Checker-Version: SpamAssassin 3.4.0 (2014-02-07) on aws-us-west-2-korg-lkml-1.web.codeaurora.org X-Spam-Level: X-Spam-Status: No, score=-8.6 required=3.0 tests=DKIMWL_WL_HIGH,DKIM_SIGNED, DKIM_VALID,DKIM_VALID_AU,INCLUDES_PATCH,MAILING_LIST_MULTI,SIGNED_OFF_BY, SPF_HELO_NONE,SPF_PASS,USER_AGENT_SANE_2 autolearn=ham autolearn_force=no version=3.4.0 Received: from mail.kernel.org (mail.kernel.org [198.145.29.99]) by smtp.lore.kernel.org (Postfix) with ESMTP id 1D4F6C742CA for ; Fri, 12 Jul 2019 16:05:11 +0000 (UTC) Received: from vger.kernel.org (vger.kernel.org [209.132.180.67]) by mail.kernel.org (Postfix) with ESMTP id E72A321019 for ; Fri, 12 Jul 2019 16:05:10 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=kernel.org; s=default; t=1562947511; bh=630KwTu7whk9xQreq+knRFfY8HS/p1vIvfAA5HYv2bQ=; h=Subject:From:To:Cc:Date:In-Reply-To:References:List-ID:From; b=hYCyKH3M8V6IlopLUKkNAyqmEEF+nrzALVGS3awiclQoFa8heLdb6w5DHtxNW/ceH ehFr/5ijXPoVmi625NBo+Nvh2MebE7ZTjr3ORrLFAlc8BtQ2YHnxCvIP4i+1ZIUOBy 63ZZ72EmBXh174X9zflb3fsqZMTz8yEhCZbuJvls= Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1727376AbfGLQFJ (ORCPT ); Fri, 12 Jul 2019 12:05:09 -0400 Received: from mail.kernel.org ([198.145.29.99]:55092 "EHLO mail.kernel.org" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1726811AbfGLQFJ (ORCPT ); Fri, 12 Jul 2019 12:05:09 -0400 Received: from tzanussi-mobl (c-98-220-238-81.hsd1.il.comcast.net [98.220.238.81]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by mail.kernel.org (Postfix) with ESMTPSA id E911D2080A; Fri, 12 Jul 2019 16:05:07 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=kernel.org; s=default; t=1562947508; bh=630KwTu7whk9xQreq+knRFfY8HS/p1vIvfAA5HYv2bQ=; h=Subject:From:To:Cc:Date:In-Reply-To:References:From; b=SBQv8BgGvlsplpPqa3vXR86qoo3OvJIQBWuAHCfxpNKKemMmDPjGrLS0TI2gtJfoh ZmzvWPzzwJ6ZXyNlM3BNEDoQWXfj/sltAF/SWj2EiLkP/tV1uMf1ZkAOkk1MBxvaAL Mn9biD9MHtPx5XxQS6ljnc69ZwP3wgSGQZn+Qxuo= Message-ID: <1562947506.12920.0.camel@kernel.org> Subject: Re: [PATCH v3] trace:Add "gfp_t" support in synthetic_events From: Tom Zanussi To: Zhengjun Xing , rostedt@goodmis.org, mingo@redhat.com, tom.zanussi@linux.intel.com Cc: linux-kernel@vger.kernel.org Date: Fri, 12 Jul 2019 11:05:06 -0500 In-Reply-To: <20190712015308.9908-1-zhengjun.xing@linux.intel.com> References: <20190712015308.9908-1-zhengjun.xing@linux.intel.com> Content-Type: text/plain; charset="UTF-8" X-Mailer: Evolution 3.26.1-1 Mime-Version: 1.0 Content-Transfer-Encoding: 7bit Sender: linux-kernel-owner@vger.kernel.org Precedence: bulk List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Hi Zhengjun, On Fri, 2019-07-12 at 09:53 +0800, Zhengjun Xing wrote: > Add "gfp_t" support in synthetic_events, then the "gfp_t" type > parameter in some functions can be traced. > > Prints the gfp flags as hex in addition to the human-readable flag > string. Example output: > > whoopsie-630 [000] ...1 78.969452: testevent: bar=b20 > (GFP_ATOMIC|__GFP_ZERO) > rcuc/0-11 [000] ...1 81.097555: testevent: bar=a20 (GFP_ATOMIC) > rcuc/0-11 [000] ...1 81.583123: testevent: bar=a20 (GFP_ATOMIC) > > Signed-off-by: Tom Zanussi > Signed-off-by: Zhengjun Xing Looks good to me, thanks! Tom > --- > kernel/trace/trace_events_hist.c | 19 +++++++++++++++++++ > 1 file changed, 19 insertions(+) > > diff --git a/kernel/trace/trace_events_hist.c > b/kernel/trace/trace_events_hist.c > index ca6b0dff60c5..30f0f32aca62 100644 > --- a/kernel/trace/trace_events_hist.c > +++ b/kernel/trace/trace_events_hist.c > @@ -13,6 +13,10 @@ > #include > #include > > +/* for gfp flag names */ > +#include > +#include > + > #include "tracing_map.h" > #include "trace.h" > #include "trace_dynevent.h" > @@ -752,6 +756,8 @@ static int synth_field_size(char *type) > size = sizeof(unsigned long); > else if (strcmp(type, "pid_t") == 0) > size = sizeof(pid_t); > + else if (strcmp(type, "gfp_t") == 0) > + size = sizeof(gfp_t); > else if (synth_field_is_string(type)) > size = synth_field_string_size(type); > > @@ -792,6 +798,8 @@ static const char *synth_field_fmt(char *type) > fmt = "%lu"; > else if (strcmp(type, "pid_t") == 0) > fmt = "%d"; > + else if (strcmp(type, "gfp_t") == 0) > + fmt = "%x"; > else if (synth_field_is_string(type)) > fmt = "%s"; > > @@ -834,9 +842,20 @@ static enum print_line_t > print_synth_event(struct trace_iterator *iter, > i == se->n_fields - 1 ? "" > : " "); > n_u64 += STR_VAR_LEN_MAX / sizeof(u64); > } else { > + struct trace_print_flags __flags[] = { > + __def_gfpflag_names, {-1, NULL} }; > + > trace_seq_printf(s, print_fmt, se- > >fields[i]->name, > entry->fields[n_u64], > i == se->n_fields - 1 ? "" > : " "); > + > + if (strcmp(se->fields[i]->type, "gfp_t") == > 0) { > + trace_seq_puts(s, " ("); > + trace_print_flags_seq(s, "|", > + entry- > >fields[n_u64], > + __flags); > + trace_seq_putc(s, ')'); > + } > n_u64++; > } > }