Hi Aric,

I've been looking into an issue on my RX 7900 XTX where
`dm_handle_vmin_vmax_update` runs every refresh, even with the desktop idle
and VRR disabled on a FreeSync-capable monitor.

This started happening at idle after “Restore 5s vbl offdelay for NV3x+
DGPUs”. Reverting that commit stopped the idle activity, though I was
already seeing similar worker activity on older kernels while moving the
mouse, browsing, or playing video.

Your patch looks like it should help here too. If I'm reading it correctly,
though, the driver would still allocate and queue the work and take
`dc_lock` every refresh, even when the timings haven't changed. Would it
be worth looking at skipping those duplicate submissions as a follow-up?
I realize that would need to account for deferred updates and cases where
the hardware needs to be programmed again.

My report has more details:
https://gitlab.freedesktop.org/drm/amd/-/work_items/5418

Thanks,
Bruno

Reply via email to