[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [PATCH 3/3] scripts/performance: Add perf_top_25.py script
From: |
Ahmed Karaman |
Subject: |
Re: [PATCH 3/3] scripts/performance: Add perf_top_25.py script |
Date: |
Wed, 17 Jun 2020 20:21:50 +0200 |
On Wed, Jun 17, 2020 at 7:35 PM Alex Bennée <alex.bennee@linaro.org> wrote:
>
> Right - which I do as a developer. It would be rude to sudo things if
> you don't need to because then you end up running your potentially
> un-trusted application with root privileges.
>
> Could we either probe for the requirement or require an explicit sudo
> flag which we can prompt for if it fails?
>
To make sure I got it right. You mean I should specify in the script
comment on the top that the user should modify the
kernel.perf_event_paranoid setting in order to run the script, otherwise,
they should add a --sudo flag when running the Python script to invoke
perf as sudo?
> >
> >> Also redirecting just stderr? why?
> >
> > Perf, as well as Valgrind, print their output on stderr not stdout.
>
> Right so I think a bit of splitting apart and use of subprocess can make
> this cleaner and not involve quite so much being done with shell
> redirection in one invocation.
>
Noted!
- Re: [PATCH 1/3] MAINTAINERS: Add 'Miscellaneous' section, (continued)
Re: [PATCH 0/3] Add Scripts for Finding Top 25 Executed Functions, no-reply, 2020/06/16
Re: [PATCH 0/3] Add Scripts for Finding Top 25 Executed Functions, Eric Blake, 2020/06/17
Re: [PATCH 0/3] Add Scripts for Finding Top 25 Executed Functions, Ahmed Karaman, 2020/06/17