[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-ID: <CAHv-k_9zS5yEF2cxf3gVEEX7C9qjdWUbZBebC=UFmPM6OUo8dA@mail.gmail.com>
Date: Tue, 30 Aug 2016 15:11:26 +0530
From: Binoy Jayan <binoy.jayan@...aro.org>
To: Daniel Wagner <daniel.wagner@...-carit.de>
Cc: "Steven Rostedt (Red Hat)" <rostedt@...dmis.org>,
Ingo Molnar <mingo@...hat.com>, Arnd Bergmann <arnd@...db.de>,
linaro-kernel@...ts.linaro.org, Carsten Emde <C.Emde@...dl.org>,
Linux kernel mailing list <linux-kernel@...r.kernel.org>,
Masami <masami.hiramatsu@...aro.org>
Subject: Re: [PATCH v3 2/3] tracing: Add trace_irqsoff tracepoints
On 29 August 2016 at 15:13, Daniel Wagner <daniel.wagner@...-carit.de> wrote:
> Hi Binoy,
>
> Some minor nitpicking.
>
> As Steven already pointed out, the cpu field is available in all traces.
> It's one of the predefined fields (see
> kernel/trace/trace_events.c/trace_define_generic_fields()). Just drop it.
>
Hi Daniel,
Thank you for looking at the patch again. When I do not use cpu as a field,
I am not able to make it part of the key or value fields. I can only
use cpu in the
filter predicate.
For example, the following trigger does not seem to work:
'hist:key=ltype,cpu:val=latency:sort=ltype,cpu if ltype==0'
But the following works:
'hist:key=ltype,latency:val=hitcount:sort=latency if cpu==1'
-Binoy
Powered by blists - more mailing lists