Re: [PATCH 0/4] perf annotate-data: small random fixes and updates

From: Ian Rogers
Date: Fri Apr 05 2024 - 20:02:48 EST


On Fri, Apr 5, 2024 at 2:18 PM Namhyung Kim <namhyung@xxxxxxxxxx> wrote:
>
> Hello,
>
> I found some problems in the data type profiling with perf annotate.
> The patch 1 should go to perf-tools and others can go to perf-tools-next.
>
> Thanks,
> Namhyung
>
>
> Namhyung Kim (4):
> perf annotate: Make sure to call symbol__annotate2() in TUI
> perf annotate-data: Fix global variable lookup
> perf annotate-data: Do not delete non-asm lines
> perf annotate: Get rid of symbol__ensure_annotate()

Reviewed-by: Ian Rogers <irogers@xxxxxxxxxx>

Thanks,
Ian

> tools/perf/ui/browsers/annotate.c | 2 +-
> tools/perf/util/annotate-data.c | 10 ++-
> tools/perf/util/annotate.c | 104 ++++++++++++++++++++----------
> 3 files changed, 80 insertions(+), 36 deletions(-)
>
>
> base-commit: b6347cb5e04e9c1d17342ab46e2ace2d448de727
> --
> 2.44.0.478.gd926399ef9-goog
>