On 2018-08-15 03:07 AM, Zhang, Jerry (Junwei) wrote:
> On 08/14/2018 05:58 PM, Michel Dänzer wrote:
>> From: Michel Dänzer <michel.daenzer at amd.com>
>>
>> Arithmetic using void* pointers isn't defined by the C standard, only as
>> a GCC extension. Avoids compiler warnings:
>>
>> ../../amdgpu/amdgpu_bo.c: In function â??amdgpu_find_bo_by_cpu_mappingâ??:
>> ../../amdgpu/amdgpu_bo.c:554:48: warning: pointer of type â??void *â??
>> used in arithmetic [-Wpointer-arith]
>> Â Â Â if (cpu >= bo->cpu_ptr && cpu < (bo->cpu_ptr + bo->alloc_size))
>> Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â ^
>> ../../amdgpu/amdgpu_bo.c:561:23: warning: pointer of type â??void *â??
>> used in subtraction [-Wpointer-arith]
>> Â Â Â *offset_in_bo = cpu - bo->cpu_ptr;
>> Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â ^
>>
>> Fixes: 4d454424e1f2 ("amdgpu: add a function to find bo by cpu mapping
>> Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â (v2)")
>> Signed-off-by: Michel Dänzer <michel.daenzer at amd.com>
>> ---
>>  amdgpu/amdgpu.h   | 2 +-
>> Â amdgpu/amdgpu_bo.c | 7 ++++---
>> Â 2 files changed, 5 insertions(+), 4 deletions(-)
>>
>> diff --git a/amdgpu/amdgpu.h b/amdgpu/amdgpu.h
>> index a8c353c6..f2bdeb95 100644
>> --- a/amdgpu/amdgpu.h
>> +++ b/amdgpu/amdgpu.h
>> @@ -695,7 +695,7 @@ int
>> amdgpu_create_bo_from_user_mem(amdgpu_device_handle dev,
>> Â Â *
>> Â */
>> Â int amdgpu_find_bo_by_cpu_mapping(amdgpu_device_handle dev,
>> -Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â void *cpu,
>> +Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â char *cpu,
>
> Shall we cast the cpu pointer when do arithmetic and keep the arguments
> as other cpu ponter?
>
> e.g.
> @@ -565,14 +565,15 @@ int
> amdgpu_find_bo_by_cpu_mapping(amdgpu_device_handle dev,
> Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â bo = handle_table_lookup(&dev->bo_handles, i);
> Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â if (!bo || !bo->cpu_ptr || size > bo->alloc_size)
> Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â continue;
> -Â Â Â Â Â Â Â Â Â Â Â Â Â Â if (cpu >= bo->cpu_ptr && cpu < (bo->cpu_ptr +
> bo->alloc_size))
> +Â Â Â Â Â Â Â Â Â Â Â Â Â Â if (cpu >= bo->cpu_ptr &&
> +Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â (uint64_t)cpu < ((uint64_t)bo->cpu_ptr +
> bo->alloc_size))
> Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â break;
> Â Â Â Â Â Â Â }
>
> Â Â Â Â Â Â Â if (i < dev->bo_handles.max_key) {
> Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â atomic_inc(&bo->refcount);
> Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â *buf_handle = bo;
> -Â Â Â Â Â Â Â Â Â Â Â Â Â Â *offset_in_bo = cpu - bo->cpu_ptr;
> +Â Â Â Â Â Â Â Â Â Â Â Â Â Â *offset_in_bo = (uint64_t)cpu - (uint64_t)bo->cpu_ptr;
> Â Â Â Â Â Â Â } else {
> Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â *buf_handle = NULL;
> Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â *offset_in_bo = 0;
Thanks for the suggestion, I sent out a v2 patch along those lines
(though slightly different).
--
Earthling Michel Dänzer | http://www.amd.com
Libre software enthusiast | Mesa and X developer
