Re: [PATCH v12 0/8] perf tools: Add progress diagnostics and a false-sharing workload

[Arnaldo Carvalho de Melo]

On Tue, Oct 06, 2026 at 05:07:29PM -0700, Namhyung Kim wrote:
> On Tue, Oct 06, 2026 at 04:57:21PM +0200, Arnaldo Carvalho de Melo wrote:
> >   - move perf_config__set_variable() to util/config.c and serialize config
> >     parser and read-modify-write state, so non-builtin perf code can persist
> >     configuration changes safely;
> >   - add 'perf report --progress' for stdio users, showing the current phase,
> >     percentage, and counts while a session is processed;
> >   - wire up 'perf report --no-progress', the counterpart of the option
> >     above, for the TUI and GTK browsers, whose progress there is no other
> >     way to turn off;
> >   - add 'perf test -w false_sharing', a synthetic TCP-shaped workload with
> >     identity and packet counters sharing a cacheline, and include it in the
> >     data type profiling shell test.
> > Follow up work: the DO_ONCE() one-time init primitive added here mirrors
> > what the eight pre-existing raw pthread_once() users in util/ (annotate.c,
> > callchain.c, comm.c, dso.c, fncache.c, intel-tpebs.c, libbfd.c, pmus.c)
> > need, conversions that will also exercise the DEFINE_MUTEX() static
> > initializer; converting them is left for after this series.
> > Best regards,
 
> Reviewed-by: Namhyung Kim <namhyung@xxxxxxxxxx>

Thanks, applied to perf-tools-next, for v7.4.

- Arnaldo