Re: [PATCH] tools/perf: Fix aggr_printout to display cpu field irrespective of core value

2022-10-06 Thread Athira Rajeev
> On 06-Oct-2022, at 7:33 PM, Arnaldo Carvalho de Melo wrote: > > Em Thu, Oct 06, 2022 at 06:16:14PM +0530, Athira Rajeev escreveu: >> >> >>> On 05-Oct-2022, at 6:05 PM, Arnaldo Carvalho de Melo >>> wrote: >>> >>> Em Wed, Oct 05, 2022 at 09:28:52AM -0300, Arnaldo Carvalho de Melo escreveu

Re: [PATCH] tools/perf: Fix aggr_printout to display cpu field irrespective of core value

2022-10-06 Thread Arnaldo Carvalho de Melo
Em Thu, Oct 06, 2022 at 06:16:14PM +0530, Athira Rajeev escreveu: > > > > On 05-Oct-2022, at 6:05 PM, Arnaldo Carvalho de Melo > > wrote: > > > > Em Wed, Oct 05, 2022 at 09:28:52AM -0300, Arnaldo Carvalho de Melo escreveu: > >> Em Wed, Oct 05, 2022 at 10:23:39AM +0530, Athira Rajeev escreveu:

Re: [PATCH] tools/perf: Fix aggr_printout to display cpu field irrespective of core value

2022-10-06 Thread Athira Rajeev
> On 05-Oct-2022, at 6:05 PM, Arnaldo Carvalho de Melo wrote: > > Em Wed, Oct 05, 2022 at 09:28:52AM -0300, Arnaldo Carvalho de Melo escreveu: >> Em Wed, Oct 05, 2022 at 10:23:39AM +0530, Athira Rajeev escreveu: >>> diff --git a/tools/perf/util/stat-display.c b/tools/perf/util/stat-display.c >

Re: [PATCH] tools/perf: Fix aggr_printout to display cpu field irrespective of core value

2022-10-05 Thread Arnaldo Carvalho de Melo
Em Wed, Oct 05, 2022 at 09:28:52AM -0300, Arnaldo Carvalho de Melo escreveu: > Em Wed, Oct 05, 2022 at 10:23:39AM +0530, Athira Rajeev escreveu: > > diff --git a/tools/perf/util/stat-display.c b/tools/perf/util/stat-display.c > > index b82844cb0ce7..cf28020798ec 100644 > > --- a/tools/perf/util/sta

Re: [PATCH] tools/perf: Fix aggr_printout to display cpu field irrespective of core value

2022-10-05 Thread Arnaldo Carvalho de Melo
Em Wed, Oct 05, 2022 at 10:23:39AM +0530, Athira Rajeev escreveu: > > > > On 04-Oct-2022, at 11:44 PM, Arnaldo Carvalho de Melo > > wrote: > > > > Em Tue, Oct 04, 2022 at 03:14:27PM -0300, Arnaldo Carvalho de Melo escreveu: > >> Em Tue, Oct 04, 2022 at 07:49:21AM -0700, Ian Rogers escreveu: >

Re: [PATCH] tools/perf: Fix aggr_printout to display cpu field irrespective of core value

2022-10-05 Thread Arnaldo Carvalho de Melo
Em Wed, Oct 05, 2022 at 10:23:39AM +0530, Athira Rajeev escreveu: > > > > On 04-Oct-2022, at 11:44 PM, Arnaldo Carvalho de Melo > > wrote: > > > > Em Tue, Oct 04, 2022 at 03:14:27PM -0300, Arnaldo Carvalho de Melo escreveu: > >> Em Tue, Oct 04, 2022 at 07:49:21AM -0700, Ian Rogers escreveu: >

Re: [PATCH] tools/perf: Fix aggr_printout to display cpu field irrespective of core value

2022-10-04 Thread Athira Rajeev
> On 04-Oct-2022, at 11:44 PM, Arnaldo Carvalho de Melo wrote: > > Em Tue, Oct 04, 2022 at 03:14:27PM -0300, Arnaldo Carvalho de Melo escreveu: >> Em Tue, Oct 04, 2022 at 07:49:21AM -0700, Ian Rogers escreveu: >>> On Tue, Oct 4, 2022, 12:06 AM Athira Rajeev Thanks for helping with testin

Re: [PATCH] tools/perf: Fix aggr_printout to display cpu field irrespective of core value

2022-10-04 Thread Ian Rogers
On Tue, Oct 4, 2022, 12:06 AM Athira Rajeev wrote: > > > > On 04-Oct-2022, at 12:21 AM, Ian Rogers wrote: > > > > On Mon, Oct 3, 2022 at 7:03 AM atrajeev > wrote: > >> > >> On 2022-10-02 05:17, Ian Rogers wrote: > >>> On Thu, Sep 29, 2022 at 5:56 AM James Clark > >>> wrote: > > > >>>

Re: [PATCH] tools/perf: Fix aggr_printout to display cpu field irrespective of core value

2022-10-04 Thread Arnaldo Carvalho de Melo
Em Tue, Oct 04, 2022 at 03:14:27PM -0300, Arnaldo Carvalho de Melo escreveu: > Em Tue, Oct 04, 2022 at 07:49:21AM -0700, Ian Rogers escreveu: > > On Tue, Oct 4, 2022, 12:06 AM Athira Rajeev > > > Thanks for helping with testing. Can I add your Tested-by for the patch ? > > > Yep. > > > Tested-

Re: [PATCH] tools/perf: Fix aggr_printout to display cpu field irrespective of core value

2022-10-04 Thread Arnaldo Carvalho de Melo
Em Tue, Oct 04, 2022 at 07:49:21AM -0700, Ian Rogers escreveu: > On Tue, Oct 4, 2022, 12:06 AM Athira Rajeev > > Thanks for helping with testing. Can I add your Tested-by for the patch ? > Yep. > Tested-by: Ian Rogers > Thanks, > Ian

Re: [PATCH] tools/perf: Fix aggr_printout to display cpu field irrespective of core value

2022-10-04 Thread Athira Rajeev
> On 04-Oct-2022, at 12:21 AM, Ian Rogers wrote: > > On Mon, Oct 3, 2022 at 7:03 AM atrajeev wrote: >> >> On 2022-10-02 05:17, Ian Rogers wrote: >>> On Thu, Sep 29, 2022 at 5:56 AM James Clark >>> wrote: On 29/09/2022 09:49, Athira Rajeev wrote: > > >> O

Re: [PATCH] tools/perf: Fix aggr_printout to display cpu field irrespective of core value

2022-10-03 Thread Ian Rogers
On Mon, Oct 3, 2022 at 7:03 AM atrajeev wrote: > > On 2022-10-02 05:17, Ian Rogers wrote: > > On Thu, Sep 29, 2022 at 5:56 AM James Clark > > wrote: > >> > >> > >> > >> On 29/09/2022 09:49, Athira Rajeev wrote: > >> > > >> > > >> >> On 28-Sep-2022, at 9:05 PM, James Clark wrote: > >> >> > >> >>

Re: [PATCH] tools/perf: Fix aggr_printout to display cpu field irrespective of core value

2022-10-01 Thread Ian Rogers
On Thu, Sep 29, 2022 at 5:56 AM James Clark wrote: > > > > On 29/09/2022 09:49, Athira Rajeev wrote: > > > > > >> On 28-Sep-2022, at 9:05 PM, James Clark wrote: > >> > >> > >> > > > > Hi James, > > > > Thanks for looking at the patch and sharing review comments. > > > >> On 13/09/2022 12:57, Athi

Re: [PATCH] tools/perf: Fix aggr_printout to display cpu field irrespective of core value

2022-09-29 Thread James Clark
On 29/09/2022 09:49, Athira Rajeev wrote: > > >> On 28-Sep-2022, at 9:05 PM, James Clark wrote: >> >> >> > > Hi James, > > Thanks for looking at the patch and sharing review comments. > >> On 13/09/2022 12:57, Athira Rajeev wrote: >>> perf stat includes option to specify aggr_mode to displ

Re: [PATCH] tools/perf: Fix aggr_printout to display cpu field irrespective of core value

2022-09-29 Thread Athira Rajeev
> On 28-Sep-2022, at 9:05 PM, James Clark wrote: > > > Hi James, Thanks for looking at the patch and sharing review comments. > On 13/09/2022 12:57, Athira Rajeev wrote: >> perf stat includes option to specify aggr_mode to display >> per-socket, per-core, per-die, per-node counter details

Re: [PATCH] tools/perf: Fix aggr_printout to display cpu field irrespective of core value

2022-09-28 Thread James Clark
On 13/09/2022 12:57, Athira Rajeev wrote: > perf stat includes option to specify aggr_mode to display > per-socket, per-core, per-die, per-node counter details. > Also there is option -A ( AGGR_NONE, -no-aggr ), where the > counter values are displayed for each cpu along with "CPU" > value in on

Re: [PATCH] tools/perf: Fix aggr_printout to display cpu field irrespective of core value

2022-09-28 Thread Athira Rajeev
> On 16-Sep-2022, at 5:01 PM, Disha Goel wrote: > > This Message Is From an External Sender > This message came from outside your organization. > > > On 9/13/22 5:27 PM, Athira Rajeev wrote: >> perf stat includes option to specify aggr_mode to display >> per-socket, per-core, per-die, per-no

Re: [PATCH] tools/perf: Fix aggr_printout to display cpu field irrespective of core value

2022-09-16 Thread Disha Goel
On 9/13/22 5:27 PM, Athira Rajeev wrote: perf stat includes option to specify aggr_mode to display per-socket, per-core, per-die, per-node counter details. Also there is option -A ( AGGR_NONE, -no-aggr ), where the counter values are displayed for each cpu along with "CPU" value in one field of

[PATCH] tools/perf: Fix aggr_printout to display cpu field irrespective of core value

2022-09-13 Thread Athira Rajeev
perf stat includes option to specify aggr_mode to display per-socket, per-core, per-die, per-node counter details. Also there is option -A ( AGGR_NONE, -no-aggr ), where the counter values are displayed for each cpu along with "CPU" value in one field of the output. Each of the aggregate mode uses