diff options
| author | Srinivasan Shanmugam <srinivasan.shanmugam@amd.com> | 2023-08-09 10:59:40 +0530 | 
|---|---|---|
| committer | Alex Deucher <alexander.deucher@amd.com> | 2023-08-15 17:40:44 -0400 | 
| commit | 4c452b5c7d73dadd8161a55a7a3f30599e4318aa (patch) | |
| tree | ec569c9112db670dddd4bf9fee9ec504063febf8 /drivers/gpu/drm/amd/amdgpu/amdgpu_device.c | |
| parent | c84f512387fcda784b907c4754b8c4088432fa5d (diff) | |
drm/amdgpu: Fix missing comment for mb() in 'amdgpu_device_aper_access'
This patch adds the missing code comment for memory barrier
WARNING: memory barrier without comment
+                       mb();
WARNING: memory barrier without comment
+                       mb();
Cc: Guchun Chen <guchun.chen@amd.com>
Cc: Christian König <christian.koenig@amd.com>
Cc: Alex Deucher <alexander.deucher@amd.com>
Cc: "Pan, Xinhui" <Xinhui.Pan@amd.com>
Signed-off-by: Srinivasan Shanmugam <srinivasan.shanmugam@amd.com>
Reviewed-by: Guchun Chen <guchun.chen@amd.com>
Signed-off-by: Alex Deucher <alexander.deucher@amd.com>
Diffstat (limited to 'drivers/gpu/drm/amd/amdgpu/amdgpu_device.c')
| -rw-r--r-- | drivers/gpu/drm/amd/amdgpu/amdgpu_device.c | 6 | 
1 files changed, 6 insertions, 0 deletions
| diff --git a/drivers/gpu/drm/amd/amdgpu/amdgpu_device.c b/drivers/gpu/drm/amd/amdgpu/amdgpu_device.c index 91d7f9e4fefc..97d202db1ce5 100644 --- a/drivers/gpu/drm/amd/amdgpu/amdgpu_device.c +++ b/drivers/gpu/drm/amd/amdgpu/amdgpu_device.c @@ -305,10 +305,16 @@ size_t amdgpu_device_aper_access(struct amdgpu_device *adev, loff_t pos,  		if (write) {  			memcpy_toio(addr, buf, count); +			/* Make sure HDP write cache flush happens without any reordering +			 * after the system memory contents are sent over PCIe device +			 */  			mb();  			amdgpu_device_flush_hdp(adev, NULL);  		} else {  			amdgpu_device_invalidate_hdp(adev, NULL); +			/* Make sure HDP read cache is invalidated before issuing a read +			 * to the PCIe device +			 */  			mb();  			memcpy_fromio(buf, addr, count);  		} | 
