Re: [PATCH 01/13] drm/msm: Remove obsolete perf infrastructure
From: Dmitry Baryshkov
Date: Tue Apr 21 2026 - 11:44:43 EST
On Tue, Apr 21, 2026 at 06:07:20AM -0700, Rob Clark wrote:
> On Mon, Apr 20, 2026 at 4:49 PM Dmitry Baryshkov
> <dmitry.baryshkov@xxxxxxxxxxxxxxxx> wrote:
> >
> > On Mon, Apr 20, 2026 at 03:25:23PM -0700, Rob Clark wrote:
> > > Outside of a3xx, this was never really used. And it low-key gets in the
> > > way of the new perfcntr support (or at least it is confusing to have two
> > > things called "perf"). So lets remove it.
> > >
> > > This drops the "perf" debugfs file. But these days, nvtop is a better
> > > option. (Plus perfetto for newer gens.)
> >
> > Would it be possible to resurrect "perf" later? It was really useful.
>
> Nothing is impossible.. but I was having trouble naming things with
> both old and new perf stuff in parallel.
That why I wrote about resurrecting it later, once the new stuff is in.
BTW: do you plan to get perf counters for a3xx-a5xx back? I remember
that there was some issue with a2xx ABI.
> Is it really adding value
> over nvtop?
--
With best wishes
Dmitry