RE: [PATCH] drm/amd/pm: parse pp_handle under appropriate conditions

2023-05-05 Thread Chen, Guchun
> Cc: Limonciello, Mario > Subject: RE: [PATCH] drm/amd/pm: parse pp_handle under appropriate > conditions > > [AMD Official Use Only - General] > > +Mario > > I believe the comment below from Mario points the right direction to > suppress the "UBSAN: invalid-load...&q

RE: [PATCH] drm/amd/pm: parse pp_handle under appropriate conditions

2023-05-05 Thread Quan, Evan
[AMD Official Use Only - General] +Mario I believe the comment below from Mario points the right direction to suppress the "UBSAN: invalid-load..." warnings. https://gitlab.freedesktop.org/drm/amd/-/issues/2541#note_1898035 However, it might be not right to simply drop the line below since the

RE: [PATCH] drm/amd/pm: parse pp_handle under appropriate conditions

2023-05-05 Thread Chen, Guchun
> -Original Message- > From: Lazar, Lijo > Sent: Friday, May 5, 2023 5:00 PM > To: Chen, Guchun ; amd- > g...@lists.freedesktop.org; Deucher, Alexander > ; Zhang, Hawking > ; Quan, Evan ; Koenig, > Christian ; Pan, Xinhui > Subject: Re: [PATCH] drm/am

Re: [PATCH] drm/amd/pm: parse pp_handle under appropriate conditions

2023-05-05 Thread Lazar, Lijo
On 5/5/2023 1:54 PM, Guchun Chen wrote: amdgpu_dpm_is_overdrive_supported is a common API across all asics, so we should cast pp_handle into correct structure under different power frameworks. Link: https://gitlab.freedesktop.org/drm/amd/-/issues/2541 Fixes: ebfc253335af("drm/amd/pm: do not e