[PATCH v6 0/2] Add gputop support for sysfs profiling knob

Adrián Larumbe adrian.larumbe at collabora.com
Thu Jul 18 17:47:38 UTC 2024


On 18.07.2024 19:29, Kamil Konieczny wrote:
> Hi Adrián,
> On 2024-07-18 at 14:58:08 +0100, Adrián Larumbe wrote:
> > Some GPUs like Panfrost need a sysfs file to be toggled before the HW can
> > initiate the job accounting necessary to feed fdinfo with engine and cycle
> > data. This sysfs knob has to be disabled when the profiler is done, to save
> > power.
> > 
> > Changelog:
> > v6:
> >  - Deleted spurious line
> >  - Changed license identifier comment style in header file
> >  - Added some ack tags to th commits
> 
> I already merged your patches with fixes and acks,
> sorry for not letting you know.

No worries, many thanks for the reviews !

> Regards,
> Kamil
> 
> > v5:
> >  - Deleted unnecesary blank line and added Ack tags
> > v4:
> >  - Improve error handling *igt_devices_profiled
> >  - Changed name of some symbols to better reflect their semantics
> >  - Changed header with copyright notice
> >  - Added documentation for public functions in igt_profiling.c
> > v3:
> >  - Created separate lib_igt_profiling to avoid dynamic linking
> >  of gputop with igt_lib, which also meant isolating the functions
> >  therein from the rest of igt_lib.
> >  - Make gputop check the sysfs knob state at the end of every period
> >  in case other instances of itself or other profilers might have
> >  changed it, so that the knob can be returned to its original state.
> > v2:
> >  - Added header file guards around igt_profiling.h
> >  - Modified licensing information to comply with SPDX format
> >  - Sorted included header files in alphabetic order
> >  - Added volatile qualifier to gputop stop variable
> > 
> > Adrián Larumbe (2):
> >   lib: Add DRM driver sysfs profiling knob toggling functions
> >   tools/gputop: toggle sysfs profiling knob if available for device
> > 
> >  lib/igt_profiling.c | 189 ++++++++++++++++++++++++++++++++++++++++++++
> >  lib/igt_profiling.h |  22 ++++++
> >  lib/meson.build     |   8 ++
> >  tools/gputop.c      |  32 +++++++-
> >  tools/meson.build   |   2 +-
> >  5 files changed, 251 insertions(+), 2 deletions(-)
> >  create mode 100644 lib/igt_profiling.c
> >  create mode 100644 lib/igt_profiling.h
> > 
> > 
> > base-commit: a2ab0ec12ef447c96c67dc539813462b6b39f857
> > -- 
> > 2.45.1
> > 



More information about the igt-dev mailing list