[AMD Public Use] No worry. I made similar mistake before. Regards, Hawking -----Original Message----- From: Zeng, Oak <Oak.Zeng@xxxxxxx> Sent: Saturday, May 8, 2021 03:13 To: Zhang, Hawking <Hawking.Zhang@xxxxxxx>; amd-gfx@xxxxxxxxxxxxxxxxxxxxx Cc: Lazar, Lijo <Lijo.Lazar@xxxxxxx>; Clements, John <John.Clements@xxxxxxx>; Joshi, Mukul <Mukul.Joshi@xxxxxxx> Subject: Re: [PATCH] drm/amdgpu: Quit RAS initialization earlier if RAS is disabled Thank you Hawking for reviewing this. I made a mistake when I pushed this in. I forgot to add " Reviewed-by: Hawking Zhang <Hawking.Zhang@xxxxxxx>". Regards, Oak On 2021-05-07, 9:05 AM, "Zhang, Hawking" <Hawking.Zhang@xxxxxxx> wrote: [AMD Public Use] Reviewed-by: Hawking Zhang <Hawking.Zhang@xxxxxxx> Regards, Hawking -----Original Message----- From: Zeng, Oak <Oak.Zeng@xxxxxxx> Sent: Friday, May 7, 2021 09:15 To: amd-gfx@xxxxxxxxxxxxxxxxxxxxx Cc: Zhang, Hawking <Hawking.Zhang@xxxxxxx>; Lazar, Lijo <Lijo.Lazar@xxxxxxx>; Clements, John <John.Clements@xxxxxxx>; Joshi, Mukul <Mukul.Joshi@xxxxxxx>; Zeng, Oak <Oak.Zeng@xxxxxxx> Subject: [PATCH] drm/amdgpu: Quit RAS initialization earlier if RAS is disabled If RAS is disabled through amdgpu_ras_enable kernel parameter, we should quit the RAS initialization eariler to avoid initialization of some RAS data structure such as sysfs etc. Signed-off-by: Oak Zeng <Oak.Zeng@xxxxxxx> --- drivers/gpu/drm/amd/amdgpu/amdgpu_ras.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/gpu/drm/amd/amdgpu/amdgpu_ras.c b/drivers/gpu/drm/amd/amdgpu/amdgpu_ras.c index ebbe2c5..7e65b35 100644 --- a/drivers/gpu/drm/amd/amdgpu/amdgpu_ras.c +++ b/drivers/gpu/drm/amd/amdgpu/amdgpu_ras.c @@ -2155,7 +2155,7 @@ int amdgpu_ras_init(struct amdgpu_device *adev) amdgpu_ras_check_supported(adev, &con->hw_supported, &con->supported); - if (!con->hw_supported || (adev->asic_type == CHIP_VEGA10)) { + if (!adev->ras_features || (adev->asic_type == CHIP_VEGA10)) { /* set gfx block ras context feature for VEGA20 Gaming * send ras disable cmd to ras ta during ras late init. */ -- 2.7.4 _______________________________________________ amd-gfx mailing list amd-gfx@xxxxxxxxxxxxxxxxxxxxx https://lists.freedesktop.org/mailman/listinfo/amd-gfx