[<prev] [next>] [<thread-prev] [day] [month] [year] [list]
Message-ID: <CAO9wTFgR6HPe28eVhXm7_CPPdiQqD-E3qarFu5xV3wg5E+Eh2g@mail.gmail.com>
Date: Tue, 22 Jul 2025 21:39:23 +0530
From: Suchit K <suchitkarunakaran@...il.com>
To: Namhyung Kim <namhyung@...nel.org>
Cc: peterz@...radead.org, mingo@...hat.com, acme@...nel.org,
mark.rutland@....com, alexander.shishkin@...ux.intel.com,
linux-perf-users@...r.kernel.org, linux-kernel-mentees@...ts.linux.dev,
linux-kernel@...r.kernel.org, skhan@...uxfoundation.org
Subject: Re: [PATCH v2] perf stat: Fix JSON output formatting in iostat_prefix()
On Fri, 18 Jul 2025 at 23:33, Namhyung Kim <namhyung@...nel.org> wrote:
>
> Hello,
>
> On Sun, Jul 13, 2025 at 08:42:24PM +0530, Suchit Karunakaran wrote:
> > The iostat_prefix() function previously included a TODO noting that its output
> > format was incorrect in JSON mode. This patch corrects that by conditionally
> > formatting the prefix string based on the output mode specified in
> > perf_stat_config. Note, I didn't test it since my system doesn't support
> > it.
>
> Can any Intel folks verify this with combination of various options
> like -x/-j, -I and --metric-only?
>
> Thanks,
> Namhyung
>
Just following up on this patch to see if anyone had a chance to test it. Thanks
Powered by blists - more mailing lists