[Date Prev][Date Next] [Thread Prev][Thread Next] [Date Index] [Thread Index]

Bug#1114806: Bisect result



Hi Niklas,

On Fri, Sep 12, 2025 at 08:02:02PM +0200, Niklas Cathor wrote:
> Hi Salvatore,
> 
> I encountered the same issue, and was able to bisect. I'm pasting the result
> below.
> Thank you for looking into this. Let me know if I should report it upstream
> instead.
> 
> cheers,
> Niklas
> 
> 
> 165a69a87d6bde85cac2c051fa6da611ca4524f6 is the first bad commit
> commit 165a69a87d6bde85cac2c051fa6da611ca4524f6 (HEAD)
> Author: Lijo Lazar <lijo.lazar@amd.com>
> Date:   Mon Jun 2 12:55:14 2025 +0530
> 
>     drm/amdgpu: Add more checks to PSP mailbox
> 
>     [ Upstream commit 8345a71fc54b28e4d13a759c45ce2664d8540d28 ]
> 
>     Instead of checking the response flag, use status mask also to check
>     against any unexpected failures like a device drop. Also, log error if
>     waiting on a psp response fails/times out.
> 
>     Signed-off-by: Lijo Lazar <lijo.lazar@amd.com>
>     Reviewed-by: Hawking Zhang <Hawking.Zhang@amd.com>
>     Signed-off-by: Alex Deucher <alexander.deucher@amd.com>
>     Signed-off-by: Sasha Levin <sashal@kernel.org>
> 
>  drivers/gpu/drm/amd/amdgpu/amdgpu_psp.c  |  4 ++++
>  drivers/gpu/drm/amd/amdgpu/amdgpu_psp.h  | 11 +++++++++++
>  drivers/gpu/drm/amd/amdgpu/psp_v10_0.c   |  4 ++--
>  drivers/gpu/drm/amd/amdgpu/psp_v11_0.c   | 31
> +++++++++++++++++++------------
>  drivers/gpu/drm/amd/amdgpu/psp_v11_0_8.c | 25 +++++++++++++++----------
>  drivers/gpu/drm/amd/amdgpu/psp_v12_0.c   | 18 +++++++++++-------
>  drivers/gpu/drm/amd/amdgpu/psp_v13_0.c   | 25 +++++++++++++++----------
>  drivers/gpu/drm/amd/amdgpu/psp_v13_0_4.c | 25 +++++++++++++++----------
>  drivers/gpu/drm/amd/amdgpu/psp_v14_0.c   | 25 +++++++++++++++----------
>  9 files changed, 107 insertions(+), 61 deletions(-)

Ok that is great you found the offending commit. Can you try if
applying 440cec4ca1c2 ("drm/amdgpu: Wait for bootloader after PSPv11
reset") fixes the issue?

Regards,
Salvatore


Reply to: