nvtop icon indicating copy to clipboard operation
nvtop copied to clipboard

panfrost: replace debugfs profiling knob with sysfs

Open larunbe opened this issue 1 year ago • 3 comments

Kernel commit b12f3ea7c188 ("drm/panfrost: Replace fdinfo's profiling debugfs knob with sysfs") did away with the debugfs knob that lets profilers enable the hardware support for job accounting and fdinfo numbers. It was replaced with a similar interface, but this time around through sysfs.

Reflect this change in nvtop.

larunbe avatar Mar 14 '24 12:03 larunbe

@larunbe thx for this fix! It works well for me on 6.10 mainline on g52 (e.g. rk3566) and g610 (rk3588). Unfortunatelly it not fixing non-working nvtop on g31 (e.g. h616). On g31 i'm getting:

Profiling state not available in Panfrost
No GPU to monitor. 

Maybe you have idea where issue might be? btw: nvtop is working well for me on 6.9 with g31 - so issue starts by 6.9->6.10 kernel change

warpme avatar Jul 18 '24 13:07 warpme

Hi, Thanks for the patch. If I understand correctly this patch will make nvtop non working on kernels < 6.10 since it's the old interface or has your patch been back-ported to earlier kernel versions?

Syllo avatar Jul 27 '24 11:07 Syllo

Hi, Thanks for the patch. If I understand correctly this patch will make nvtop non working on kernels < 6.10 since it's the old interface or has your patch been back-ported to earlier kernel versions?

Yeah, that's right, 6.10 kernel release has the change that replaces the debugfs interface with sysfs.

larunbe avatar Sep 16 '24 13:09 larunbe