Re: [PATCH v2 38/49] perf python: Port arm-cs-trace-disasm to perf module

[Ian Rogers]

On Wed, Sep 23, 2026 at 2:08 AM Leo Yan <leo.yan@xxxxxxx> wrote:
>
> On 9/23/26 10:00, Leo Yan wrote:
> > On Sun, Sep 20, 2026 at 10:06:56PM -0700, Ian Rogers wrote:
> >
> > [...]
> >
> >> ---
> >>  tools/perf/python/arm-cs-trace-disasm.py      | 356 ++++++++++++++++++
> >
> > A new script is added under tools/perf/python/, but I don't see the
> > legacy script under tools/perf/scripts/python/ being removed.
> >
> > Should we maintain two copies of the script for the same purpose?
>
> Please ignore this question. I saw the patch 46 removes all legacy scripts.

Thanks, Leo. Yes, this is intentional so that if needed, we can land
the ported scripts without removing the previous scripts. We could
always add and remove them together; I was trying to think of ways to
reduce friction for some or all of these changes to land.

Ian