[PATCH 1/2] drm/amdgpu: update df_v3_6 for xgmi perfmons

Kuehling, Felix Felix.Kuehling at amd.com
Thu Jun 20 15:23:59 UTC 2019


On 2019-06-20 9:49, Alex Deucher wrote:
> You never addressed my comments about df init from before.  df_init
> gets called from hw_init, so you shouldn't create files.  Move df init
> to being called from soc15 sw_init/fini and probably rename the df
> callbacks to sw_init and sw_fini so we know not to add hw state to
> those functions.  As is now, we'll end up needlessly creating and
> destorying files on suspend/resume.

Sorry I also missed that comment and the implications, or I would have 
pointed it out in my code review.

Jon's last patch looks mostly good to me and I think it addresses this 
with minimal fuss.

Regards,
   Felix

>
> Alex


More information about the amd-gfx mailing list