drm/amdgpu: Fix comparison in amdgpu_res_cpu_visible

Michel Dänzer · alexdeucher · commit 8d2c930735f8 · 2024-05-10T13:05:13.000-04:00
It incorrectly claimed a resource isn't CPU visible if it's located at the very end of CPU visible VRAM. Fixes: a6ff969 ("drm/amdgpu: fix visible VRAM handling during faults") Closes: https://gitlab.freedesktop.org/drm/amd/-/issues/3343 Reviewed-by: Christian König <christian.koenig@amd.com> Reviewed-by: Alex Deucher <alexander.deucher@amd.com> Reported-and-Tested-by: Jeremy Day <jsday@noreason.ca> Signed-off-by: Michel Dänzer <mdaenzer@redhat.com> Signed-off-by: Alex Deucher <alexander.deucher@amd.com> CC: stable@vger.kernel.org
diff --git a/drivers/gpu/drm/amd/amdgpu/amdgpu_ttm.c b/drivers/gpu/drm/amd/amdgpu/amdgpu_ttm.c
@@ -427,7 +427,7 @@ bool amdgpu_res_cpu_visible(struct amdgpu_device *adev,
 
 	amdgpu_res_first(res, 0, res->size, &cursor);
 	while (cursor.remaining) {
-		if ((cursor.start + cursor.size) >= adev->gmc.visible_vram_size)
+		if ((cursor.start + cursor.size) > adev->gmc.visible_vram_size)
 			return false;
 		amdgpu_res_next(&cursor, cursor.size);
 	}

Original file line number	Diff line number	Diff line change
`@@ -427,7 +427,7 @@ bool amdgpu_res_cpu_visible(struct amdgpu_device *adev,`
`427`	`427`
`428`	`428`	`amdgpu_res_first(res, 0, res->size, &cursor);`
`429`	`429`	`while (cursor.remaining) {`
`430`		`- if ((cursor.start + cursor.size) >= adev->gmc.visible_vram_size)`
	`430`	`+ if ((cursor.start + cursor.size) > adev->gmc.visible_vram_size)`
`431`	`431`	`return false;`
`432`	`432`	`amdgpu_res_next(&cursor, cursor.size);`
`433`	`433`	`}`