[Bug][Regression][Bisected] pp_table writes began to fail for Navi10 on amd-staging-drm-next

[Matt Coffin]

Hi Evan,

Thanks for the work. It would have taken me probably forever to find that.

I replied to your series with a Tested-by, and I tested locally. With
edad8312cbbf9a33c86873fc4093664f150dd5c1 reverted to fix the sysfs
interfaces, and your series, I once again have a 100% working OD system!

Thanks for the snappy work,
Matt

On 8/2/20 10:50 PM, Quan, Evan wrote:
> [AMD Official Use Only - Internal Distribution Only]
>
> Hi Matt,
>
> I just sent out two patches to fix the issue reported here. It worked
on my local env.
> Please confirm on your platform.
> https://lists.freedesktop.org/archives/amd-gfx/2020-August/052245.html
> https://lists.freedesktop.org/archives/amd-gfx/2020-August/052246.html
>
> BR,
> Evan
> -----Original Message-----
> From: Quan, Evan
> Sent: Friday, July 31, 2020 9:20 AM
> To: Matt Coffin <mcoffin13 at gmail.com>; amd-gfx at lists.freedesktop.org
> Cc: Alex Deucher <alexdeucher at gmail.com>
> Subject: RE: [Bug][Regression][Bisected] pp_table writes began to fail
for Navi10 on amd-staging-drm-next
>
> Thanks for reporting this. I will check it.
>
> BR
> Evan
> -----Original Message-----
> From: Matt Coffin <mcoffin13 at gmail.com>
> Sent: Thursday, July 30, 2020 10:25 PM
> To: Quan, Evan <Evan.Quan at amd.com>; amd-gfx at lists.freedesktop.org
> Cc: Alex Deucher <alexdeucher at gmail.com>
> Subject: [Bug][Regression][Bisected] pp_table writes began to fail for
Navi10 on amd-staging-drm-next
>
> Hey Evan,
> 
> I've been having an issue with uploading `pp_table`s on recent `amd-staging-drm-next` kernels. I bisected the issue, and it came back to a commit of yours - ec8ee23f610578c71885a36ddfcf58d35cccab67.
> 
> I didn't have your gitlab handle to CC you on the issue, so I thought I'd at least alert you to it.
> 
> Here's a link to the issue on GitLab:
> https://gitlab.freedesktop.org/drm/amd/-/issues/1243
> 
> I'd appreciate any help or insight you could offer here as I work on a fix.
> 
> First bad commit header:
> 
> commit ec8ee23f610578c71885a36ddfcf58d35cccab67 (refs/bisect/bad)
> Author: Evan Quan <evan.quan at amd.com>
> Date:   Wed Jun 10 16:52:32 2020 +0800
> 
>     drm/amd/powerplay: update Navi10 default dpm table setup
> 
>     Cache all clocks levels for every dpm table. They are needed
>     by other APIs.
> 
>     Change-Id: I8114cf31e6ec8c9af4578d51749eb213befdcc71
>     Signed-off-by: Evan Quan <evan.quan at amd.com>
>     Reviewed-by: Alex Deucher <alexander.deucher at amd.com>
> 
> Thanks everyone, and cheers,
> Matt Coffin
>

-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 833 bytes
Desc: OpenPGP digital signature
URL: <https://lists.freedesktop.org/archives/amd-gfx/attachments/20200803/a5b26210/attachment-0001.sig>