Re: [PATCHSET 0/4] perf stat: Add --multiply-cgroup option

2020-09-10 Thread Namhyung Kim
Hi Ian, On Fri, Sep 11, 2020 at 2:11 AM Ian Rogers wrote: > Just to throw in some 2c worth. A nice thing about Namhyung's approach > is that it can apply for metrics, events and pfm-events. It would be > nice if there were a single option for specifying these, perhaps we > can figure one out.

Re: [PATCHSET 0/4] perf stat: Add --multiply-cgroup option

2020-09-10 Thread Namhyung Kim
Hi Andi, On Fri, Sep 11, 2020 at 1:05 AM Andi Kleen wrote: > > On Tue, Sep 08, 2020 at 01:42:24PM +0900, Namhyung Kim wrote: > > When we profile cgroup events with perf stat, it's very annoying to > > specify events and cgroups on the command line as it requires the > > mapping between events

Re: [PATCHSET 0/4] perf stat: Add --multiply-cgroup option

2020-09-10 Thread Arnaldo Carvalho de Melo
Em Thu, Sep 10, 2020 at 11:15:42AM +0200, Jiri Olsa escreveu: > On Tue, Sep 08, 2020 at 01:42:24PM +0900, Namhyung Kim wrote: > > When we profile cgroup events with perf stat, it's very annoying to > > specify events and cgroups on the command line as it requires the > > mapping between events and

Re: [PATCHSET 0/4] perf stat: Add --multiply-cgroup option

2020-09-10 Thread Namhyung Kim
Hi Arnaldo, On Thu, Sep 10, 2020 at 8:10 PM Arnaldo Carvalho de Melo wrote: > > Em Thu, Sep 10, 2020 at 11:15:42AM +0200, Jiri Olsa escreveu: > > On Tue, Sep 08, 2020 at 01:42:24PM +0900, Namhyung Kim wrote: > > > When we profile cgroup events with perf stat, it's very annoying to > > > specify

Re: [PATCHSET 0/4] perf stat: Add --multiply-cgroup option

2020-09-10 Thread Ian Rogers
On Thu, Sep 10, 2020 at 8:57 AM Andi Kleen wrote: > > On Tue, Sep 08, 2020 at 01:42:24PM +0900, Namhyung Kim wrote: > > When we profile cgroup events with perf stat, it's very annoying to > > specify events and cgroups on the command line as it requires the > > mapping between events and cgroups.

Re: [PATCHSET 0/4] perf stat: Add --multiply-cgroup option

2020-09-10 Thread Andi Kleen
On Tue, Sep 08, 2020 at 01:42:24PM +0900, Namhyung Kim wrote: > When we profile cgroup events with perf stat, it's very annoying to > specify events and cgroups on the command line as it requires the > mapping between events and cgroups. (Note that perf record can use > cgroup sampling but it's

Re: [PATCHSET 0/4] perf stat: Add --multiply-cgroup option

2020-09-10 Thread Jiri Olsa
On Tue, Sep 08, 2020 at 01:42:24PM +0900, Namhyung Kim wrote: > Hello, > > When we profile cgroup events with perf stat, it's very annoying to > specify events and cgroups on the command line as it requires the > mapping between events and cgroups. (Note that perf record can use > cgroup

[PATCHSET 0/4] perf stat: Add --multiply-cgroup option

2020-09-07 Thread Namhyung Kim
Hello, When we profile cgroup events with perf stat, it's very annoying to specify events and cgroups on the command line as it requires the mapping between events and cgroups. (Note that perf record can use cgroup sampling but it's not usable for perf stat). I guess most cases we just want to