On 32-bit:

    ../amdgpu/amdgpu_bo.c: In function ‘amdgpu_find_bo_by_cpu_mapping’:
    ../amdgpu/amdgpu_bo.c:554:13: warning: cast to pointer from integer of 
different size [-Wint-to-pointer-cast]
           cpu < (void*)((uintptr_t)bo->cpu_ptr + bo->alloc_size))
                 ^

Indeed, as amdgpu_bo_info.alloc_size is "uint64_t", the sum is
always 64-bit, while "void *" can be 32-bit or 64-bit.

Fix this by casting bo->alloc_size to "size_t", which is either
32-bit or 64-bit, just like "void *".

Fixes: c6493f360e7529c2 ("amdgpu: Eliminate void* arithmetic in 
amdgpu_find_bo_by_cpu_mapping")
Signed-off-by: Geert Uytterhoeven <[email protected]>
---
 amdgpu/amdgpu_bo.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/amdgpu/amdgpu_bo.c b/amdgpu/amdgpu_bo.c
index f4e0435254f6aa9f..672f000d64801012 100644
--- a/amdgpu/amdgpu_bo.c
+++ b/amdgpu/amdgpu_bo.c
@@ -551,7 +551,7 @@ drm_public int 
amdgpu_find_bo_by_cpu_mapping(amdgpu_device_handle dev,
                if (!bo || !bo->cpu_ptr || size > bo->alloc_size)
                        continue;
                if (cpu >= bo->cpu_ptr &&
-                   cpu < (void*)((uintptr_t)bo->cpu_ptr + bo->alloc_size))
+                   cpu < (void*)((uintptr_t)bo->cpu_ptr + 
(size_t)bo->alloc_size))
                        break;
        }
 
-- 
2.34.1

Reply via email to