Messages in this thread |  | | Date | Thu, 26 Jan 2012 12:16:48 +0100 | | From | Ingo Molnar <> | | Subject | Re: [GIT PULL 0/9] perf/core improvements and fixes |
| |
* Arnaldo Carvalho de Melo <acme@infradead.org> wrote:
> Hi Ingo, > > This is a signed tag, please lemme know if everything went well. > > The --uid feature works for root, we still need to > sort out that paranoia with some threads owned by a user that > prevents 'perf --uid non-root-user' to work for > 'non-root-user'.
Just wondering what detail causes that failure - the whole point of --uid mingo would be to enable nonprivileged users to do 'session wide' profiling, *especially* if paranoia is high.
So what does --uid do which perf record --pid 1234 wouldnt already do? By all means --uid ought to be a fancy way of doing a whole bunch of perf record --pid 1234 profiling sessions, at once.
[ Btw, we should probably alias --user to --uid as well, as that might be the intuitive thing people would typically use? ]
> http://git.kernel.org/?p=linux/kernel/git/acme/linux.git;a=tag;h=ce9600c4c664ce7f97e8aa5e756b0b4ea5b017c7 > looks ok to me, need just to improve on the commit log message, I'll get > used to it :-) > > - Arnaldo > > The following changes since commit 172d1b0b73256551f100fc00c69e356d047103f5: > > perf tools: Fix compile error on x86_64 Ubuntu (2012-01-08 13:34:55 -0200) > > are available in the git repository at: > > git://git.kernel.org/pub/scm/linux/kernel/git/acme/linux perf-core-for-mingo > > for you to fetch changes up to f8f4b2872295dca88339ec0c403b2217b1197353: > > perf tools: Fix strlen() bug in perf_event__synthesize_event_type() (2012-01-24 20:31:34 -0200) > > ---------------------------------------------------------------- > perf/core improvements and fixes > > ----------------------------------------------------------------
Anyway, pulled, thanks a lot Arnaldo!
One detail: don't we want some of these fixes cherry-picked into perf/urgent as well?
Thanks,
Ingo
|  |