Re: [PATCH] drm/amd: fix gfx hang on renoir in IGT reload test

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

 



[AMD Official Use Only]


Thank you very much Alex!

Regards,
Rico

From: Alex Deucher <alexdeucher@xxxxxxxxx>
Sent: Friday, March 11, 2022 0:52
To: Yin, Tianci (Rico) <Tianci.Yin@xxxxxxx>
Cc: amd-gfx list <amd-gfx@xxxxxxxxxxxxxxxxxxxxx>; Chen, Guchun <Guchun.Chen@xxxxxxx>; Zhuo, Qingqing (Lillian) <Qingqing.Zhuo@xxxxxxx>; Liu, Wenjing <Wenjing.Liu@xxxxxxx>; Wang, Yu (Charlie) <Yu.Wang4@xxxxxxx>; Deucher, Alexander <Alexander.Deucher@xxxxxxx>; Quan, Evan <Evan.Quan@xxxxxxx>
Subject: Re: [PATCH] drm/amd: fix gfx hang on renoir in IGT reload test
 
On Thu, Mar 10, 2022 at 9:04 AM Tianci Yin <tianci.yin@xxxxxxx> wrote:
>
> From: "Tianci.Yin" <tianci.yin@xxxxxxx>
>
> [why]
> CP hangs in igt reloading test on renoir, more precisely, hangs on the
> second time insmod.
>
> [how]
> mode2 reset can make it recover, and mode2 reset only effects gfx core,
> dcn and the screen will not be impacted.
>
> Signed-off-by: Tianci.Yin <tianci.yin@xxxxxxx>
> ---
>  drivers/gpu/drm/amd/amdgpu/soc15.c | 5 +++++
>  1 file changed, 5 insertions(+)
>
> diff --git a/drivers/gpu/drm/amd/amdgpu/soc15.c b/drivers/gpu/drm/amd/amdgpu/soc15.c
> index 496c4a6e23ac..f0713c027ed5 100644
> --- a/drivers/gpu/drm/amd/amdgpu/soc15.c
> +++ b/drivers/gpu/drm/amd/amdgpu/soc15.c
> @@ -853,6 +853,11 @@ static bool soc15_need_reset_on_init(struct amdgpu_device *adev)
>  {
>         u32 sol_reg;
>
> +       /* CP hangs in IGT reloading test on RN, reset to WA */
> +       if(adev->asic_type == CHIP_RENOIR) {

Space between if and (.  Also, you can drop the { }.  With that fixed:
Acked-by: Alex Deucher <alexander.deucher@xxxxxxx>

Alex


> +               return true;
> +       }
> +
>         /* Just return false for soc15 GPUs.  Reset does not seem to
>          * be necessary.
>          */
> --
> 2.25.1
>

[Index of Archives]     [Linux USB Devel]     [Linux Audio Users]     [Yosemite News]     [Linux Kernel]     [Linux SCSI]

  Powered by Linux