Re: [PATCH 1/1] perf symbols: Add 'intel_idle_ibrs' to the list of idle symbols

From: Namhyung Kim
Date: Thu Oct 12 2023 - 13:21:28 EST


On Thu, Oct 5, 2023 at 10:16 PM Namhyung Kim <namhyung@xxxxxxxxxx> wrote:
>
> Hi Arnaldo,
>
> On Thu, Oct 5, 2023 at 6:29 AM Arnaldo Carvalho de Melo <acme@xxxxxxxxxx> wrote:
> >
> > From: Arnaldo Carvalho de Melo <acme@xxxxxxxxxx>
> >
> > This is a longstanding to do list entry: we need a way to see that a
> > sample took place while in idle state, as the current way to do it is
> > to infer that by the name of the functions that in such state have
> > more samples, IOW: a hack.
> >
> > Maybe we can do flip a bit in samples that take place inside the
> > enter/exit idle section in do_idle()?
> >
> > But till then, add one more :-\
> >
> > Cc: Adrian Hunter <adrian.hunter@xxxxxxxxx>
> > cc: Frédéric Weisbecker <fweisbec@xxxxxxxxx>
> > Cc: Ian Rogers <irogers@xxxxxxxxxx>
> > Cc: Ingo Molnar <mingo@xxxxxxxxxx>,
> > Cc: Jiri Olsa <jolsa@xxxxxxxxxx>
> > Cc: Namhyung Kim <namhyung@xxxxxxxxxx>
> > Cc: Peter Zijlstra <peterz@xxxxxxxxxxxxx>
> > Signed-off-by: Arnaldo Carvalho de Melo <acme@xxxxxxxxxx>
>
> Acked-by: Namhyung Kim <namhyung@xxxxxxxxxx>

Applied to perf-tools-next, thanks!