[AMD Official Use Only - AMD Internal Distribution Only] Hi Stanley, Sure, will do, thanks.
Regards, Liu, Xiang ________________________________ From: Yang, Stanley <[email protected]> Sent: Wednesday, August 20, 2025 3:28 PM To: Liu, Xiang(Dean) <[email protected]>; Chai, Thomas <[email protected]>; [email protected] <[email protected]> Cc: Zhang, Hawking <[email protected]>; Zhou1, Tao <[email protected]> Subject: RE: [PATCH 2/2] drm/amdgpu: Check VF critical region before RAS poison injection [AMD Official Use Only - AMD Internal Distribution Only] All checking work should be done in host side, not only check current vf critical region but also other vf critical region. For example, in extreme cases, current row memory addresses may be in other vfs critical region, please handle this case in host side. Regards, Stanley From: Liu, Xiang(Dean) <[email protected]> Sent: Wednesday, August 20, 2025 2:02 PM To: Chai, Thomas <[email protected]>; [email protected] Cc: Zhang, Hawking <[email protected]>; Zhou1, Tao <[email protected]>; Yang, Stanley <[email protected]> Subject: Re: [PATCH 2/2] drm/amdgpu: Check VF critical region before RAS poison injection [AMD Official Use Only - AMD Internal Distribution Only] Hi, As discuss offline with Thomas, will check the VF critical address regions of guest in another patch as a new feature. Regards, Liu, Xiang ________________________________ From: amd-gfx <[email protected]<mailto:[email protected]>> on behalf of Liu, Xiang(Dean) <[email protected]<mailto:[email protected]>> Sent: Wednesday, August 20, 2025 1:33 PM To: Chai, Thomas <[email protected]<mailto:[email protected]>>; [email protected]<mailto:[email protected]> <[email protected]<mailto:[email protected]>> Cc: Zhang, Hawking <[email protected]<mailto:[email protected]>>; Zhou1, Tao <[email protected]<mailto:[email protected]>>; Yang, Stanley <[email protected]<mailto:[email protected]>> Subject: Re: [PATCH 2/2] drm/amdgpu: Check VF critical region before RAS poison injection [AMD Official Use Only - AMD Internal Distribution Only] Hi Thomas, Sure, will do, thanks. Regards, Liu, Xiang ________________________________ From: Chai, Thomas <[email protected]<mailto:[email protected]>> Sent: Wednesday, August 20, 2025 9:39 AM To: Liu, Xiang(Dean) <[email protected]<mailto:[email protected]>>; [email protected]<mailto:[email protected]> <[email protected]<mailto:[email protected]>> Cc: Zhang, Hawking <[email protected]<mailto:[email protected]>>; Zhou1, Tao <[email protected]<mailto:[email protected]>>; Yang, Stanley <[email protected]<mailto:[email protected]>> Subject: RE: [PATCH 2/2] drm/amdgpu: Check VF critical region before RAS poison injection [AMD Official Use Only - AMD Internal Distribution Only] -----Original Message----- From: Liu, Xiang(Dean) <[email protected]<mailto:[email protected]>> Sent: Tuesday, August 19, 2025 1:26 PM To: [email protected]<mailto:[email protected]> Cc: Zhang, Hawking <[email protected]<mailto:[email protected]>>; Zhou1, Tao <[email protected]<mailto:[email protected]>>; Yang, Stanley <[email protected]<mailto:[email protected]>>; Chai, Thomas <[email protected]<mailto:[email protected]>>; Liu, Xiang(Dean) <[email protected]<mailto:[email protected]>> Subject: [PATCH 2/2] drm/amdgpu: Check VF critical region before RAS poison injection Check VF critical region before RAS poison injection to ensure that the poison injection will not hit the VF critical region. Signed-off-by: Xiang Liu <[email protected]<mailto:[email protected]>> --- drivers/gpu/drm/amd/amdgpu/amdgpu_ras.c | 7 +++++++ 1 file changed, 7 insertions(+) diff --git a/drivers/gpu/drm/amd/amdgpu/amdgpu_ras.c b/drivers/gpu/drm/amd/amdgpu/amdgpu_ras.c index b00cbb927ca8..6730de574fdc 100644 --- a/drivers/gpu/drm/amd/amdgpu/amdgpu_ras.c +++ b/drivers/gpu/drm/amd/amdgpu/amdgpu_ras.c @@ -219,10 +219,17 @@ static int amdgpu_check_address_validity(struct amdgpu_device *adev, struct amdgpu_vram_block_info blk_info; uint64_t page_pfns[32] = {0}; int i, ret, count; + bool hit = false; if (amdgpu_ip_version(adev, UMC_HWIP, 0) < IP_VERSION(12, 0, 0)) return 0; + if (amdgpu_sriov_vf(adev)) { + if (amdgpu_virt_check_vf_critical_region(adev, address, &hit)) + return -EPERM; + return hit ? -EACCES : 0; [Thomas]Only checking if the address is within the PF critical address regions may be not sufficient, The VF critical address regions of guest may be also need to be checked. + } + if ((address >= adev->gmc.mc_vram_size) || (address >= RAS_UMC_INJECT_ADDR_LIMIT)) return -EFAULT; -- 2.34.1
