[PATCH Review V2 2/2] drm/amdgpu: Disable RAS by default on APU flatform
Felix Kuehling
felix.kuehling at amd.com
Thu Jul 13 14:58:18 UTC 2023
On 2023-07-13 10:50, Stanley.Yang wrote:
> Disable RAS feature by default for aqua vanjaram on APU platform.
>
> Changed from V1:
> Splite Disable RAS by default on APU platform into a
> separated patch.
>
> Signed-off-by: Stanley.Yang <Stanley.Yang at amd.com>
> Reviewed-by: Hawking Zhang <Hawking.Zhang at amd.com>
> ---
> drivers/gpu/drm/amd/amdgpu/amdgpu_ras.c | 9 +++++++++
> 1 file changed, 9 insertions(+)
>
> diff --git a/drivers/gpu/drm/amd/amdgpu/amdgpu_ras.c b/drivers/gpu/drm/amd/amdgpu/amdgpu_ras.c
> index 8673d9790bb0..ec5f60b64346 100644
> --- a/drivers/gpu/drm/amd/amdgpu/amdgpu_ras.c
> +++ b/drivers/gpu/drm/amd/amdgpu/amdgpu_ras.c
> @@ -2517,6 +2517,15 @@ static void amdgpu_ras_check_supported(struct amdgpu_device *adev)
> adev->ras_hw_enabled |= (1 << AMDGPU_RAS_BLOCK__GFX |
> 1 << AMDGPU_RAS_BLOCK__SDMA |
> 1 << AMDGPU_RAS_BLOCK__MMHUB);
> +
> + if (adev->ip_versions[MP0_HWIP][0] == IP_VERSION(13, 0, 6)) {
> + /*
> + * Disable ras feature for aqua vanjaram
> + * by default on apu platform.
> + */
> + if (-1 == amdgpu_ras_enable)
> + amdgpu_ras_enable = 0;
Changing a global variable here is probably not appropriate. The
condition above looks like this should affect a device-specific variable
only.
Regards,
Felix
> + }
> }
>
> amdgpu_ras_get_quirks(adev);
More information about the amd-gfx
mailing list