[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-ID: <20140926113134.GA14072@gmail.com>
Date: Fri, 26 Sep 2014 13:31:34 +0200
From: Ingo Molnar <mingo@...nel.org>
To: Pawel Moll <pawel.moll@....com>
Cc: Ingo Molnar <mingo@...hat.com>,
Arnaldo Carvalho de Melo <acme@...nel.org>,
Richard Cochran <richardcochran@...il.com>,
Steven Rostedt <rostedt@...dmis.org>,
Peter Zijlstra <peterz@...radead.org>,
Paul Mackerras <paulus@...ba.org>,
John Stultz <john.stultz@...aro.org>,
"linux-kernel@...r.kernel.org" <linux-kernel@...r.kernel.org>,
"linux-api@...r.kernel.org" <linux-api@...r.kernel.org>
Subject: Re: [PATCH 2/2] perf: Userspace software event and ioctl
* Pawel Moll <pawel.moll@....com> wrote:
> On Fri, 2014-09-26 at 12:23 +0100, Ingo Molnar wrote:
> > > As in: allows *all* tasks to inject the data? Are you sure we
> > > don't want more fine-grained control, in particular per task?
> >
> > Yeah. If the profiler allows it, then any task that is being
> > traced can inject data.
>
> The "that is being traced" fragment was the key here. I missed
> the fact that perf trace already takes a list of pids, so we're
> not talking about all tasks in the system. That should work.
Yeah, when we generate a user trace event, we should look at the
currently active perf context's (percpu ones plus task ones), and
inject into those only.
This way we limit event generation to those buffers that are
actively interested in this task.
Thanks,
Ingo
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@...r.kernel.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/
Powered by blists - more mailing lists