Am 07.08.2018 um 11:52 schrieb Zhang, Jerry (Junwei):
> On 08/07/2018 05:33 PM, Christian König wrote:
>> Am 07.08.2018 um 10:28 schrieb Zhang, Jerry (Junwei):
>>> On 08/07/2018 04:20 PM, Christian König wrote:
>>>> Well NAK, that wasn't the intention of putting all BOs into the
>>>> handle table.
>>>>
>>>> You should still use the kernel implementation.
>>>
>>> I thought we have discussed that in below mail thread. any gap?
>>>
>>> [PATCH 1/2] drm/amdgpu: return bo itself if userptr is cpu addr of
>>> bo (v3)
>>> {{{
>>> >> Well we could just completely drop the kernel implementation and use
>>> >> an userspace implementation.
>>> >
>>> > Do you mean to implement finding bo by cpu address in libdrm
>>> completely?
>>>
>>> Yes, exactly.
>>> }}}
>>
>> Ok, well that is a misunderstanding. I was just speculating about that.
>>
>> On the other hand if searching all BOs for the right one is not to
>> much overhead for that workaround I'm fine with it.
>>
>> Just drop patch #2 in this series. All unused entries should be
>> initialized to zero and walking the extra 512 isn't that much overhead.
>
> Yes, reserve it locally for now.
> May I get your RB?
I've got a few minor comments on the last patch as well.
Give me a moment to figure out how to push to libdrm master repostory
now (they migrated to github and my old account doesn't work any more).
After that I can comment on your patches.
Thanks,
Christian.
>
> Regards,
> Jerry
>
>>
>> Regards,
>> Christian.
>>
>>>
>>> Regards,
>>> Jerry
>>>
>>>>
>>>> Christian.
>>>>
>>>> Am 07.08.2018 um 09:26 schrieb Junwei Zhang:
>>>>> Userspace needs to know if the user memory is from BO or malloc.
>>>>>
>>>>> Signed-off-by: Junwei Zhang <Jerry.Zhang at amd.com>
>>>>> ---
>>>>>  amdgpu/amdgpu.h   | 23 +++++++++++++++++++++++
>>>>> Â amdgpu/amdgpu_bo.c | 34 ++++++++++++++++++++++++++++++++++
>>>>> Â 2 files changed, 57 insertions(+)
>>>>>
>>>>> diff --git a/amdgpu/amdgpu.h b/amdgpu/amdgpu.h
>>>>> index be83b45..a8c353c 100644
>>>>> --- a/amdgpu/amdgpu.h
>>>>> +++ b/amdgpu/amdgpu.h
>>>>> @@ -678,6 +678,29 @@ int
>>>>> amdgpu_create_bo_from_user_mem(amdgpu_device_handle dev,
>>>>> Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â amdgpu_bo_handle *buf_handle);
>>>>> Â /**
>>>>> + * Validate if the user memory comes from BO
>>>>> + *
>>>>> + * \param dev - [in] Device handle. See #amdgpu_device_initialize()
>>>>> + * \param cpu - [in] CPU address of user allocated memory which we
>>>>> + * want to map to GPU address space (make GPU accessible)
>>>>> + * (This address must be correctly aligned).
>>>>> + * \param size - [in] Size of allocation (must be correctly aligned)
>>>>> + * \param buf_handle - [out] Buffer handle for the userptr memory
>>>>> + * if the user memory is not from BO, the buf_handle will be NULL.
>>>>> + * \param offset_in_bo - [out] offset in this BO for this user
>>>>> memory
>>>>> + *
>>>>> + *
>>>>> + * \return  0 on success\n
>>>>> + *Â Â Â Â Â Â Â Â Â <0 - Negative POSIX Error code
>>>>> + *
>>>>> +*/
>>>>> +int amdgpu_find_bo_by_cpu_mapping(amdgpu_device_handle dev,
>>>>> +Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â void *cpu,
>>>>> +Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â uint64_t size,
>>>>> +Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â amdgpu_bo_handle *buf_handle,
>>>>> +Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â uint64_t *offset_in_bo);
>>>>> +
>>>>> +/**
>>>>> Â Â * Free previosuly allocated memory
>>>>> Â Â *
>>>>>   * \param  dev          - \c [in] Device handle. See
>>>>> #amdgpu_device_initialize()
>>>>> diff --git a/amdgpu/amdgpu_bo.c b/amdgpu/amdgpu_bo.c
>>>>> index b24e698..a631050 100644
>>>>> --- a/amdgpu/amdgpu_bo.c
>>>>> +++ b/amdgpu/amdgpu_bo.c
>>>>> @@ -529,6 +529,40 @@ int amdgpu_bo_wait_for_idle(amdgpu_bo_handle bo,
>>>>> Â Â Â Â Â }
>>>>> Â }
>>>>> +int amdgpu_find_bo_by_cpu_mapping(amdgpu_device_handle dev,
>>>>> +Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â void *cpu,
>>>>> +Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â uint64_t size,
>>>>> +Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â amdgpu_bo_handle *buf_handle,
>>>>> +Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â uint64_t *offset_in_bo)
>>>>> +{
>>>>> +Â Â Â int i;
>>>>> +Â Â Â struct amdgpu_bo *bo;
>>>>> +
>>>>> +Â Â Â if (cpu == NULL || size == 0)
>>>>> +Â Â Â Â Â Â Â return -EINVAL;
>>>>> +
>>>>> +Â Â Â pthread_mutex_lock(&dev->bo_table_mutex);
>>>>> +Â Â Â for (i = 0; i <= dev->bo_handles.max_key; i++) {
>>>>> +Â Â Â Â Â Â Â 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))
>>>>> +Â Â Â Â Â Â Â Â Â Â Â break;
>>>>> +Â Â Â }
>>>>> +Â Â Â pthread_mutex_unlock(&dev->bo_table_mutex);
>>>>> +
>>>>> +Â Â Â if (i <= dev->bo_handles.max_key) {
>>>>> +Â Â Â Â Â Â Â atomic_inc(&bo->refcount);
>>>>> +Â Â Â Â Â Â Â *buf_handle = bo;
>>>>> +Â Â Â Â Â Â Â *offset_in_bo = cpu - bo->cpu_ptr;
>>>>> +Â Â Â } else {
>>>>> +Â Â Â Â Â Â Â *buf_handle = NULL;
>>>>> +Â Â Â Â Â Â Â *offset_in_bo = 0;
>>>>> +Â Â Â }
>>>>> +
>>>>> +Â Â Â return 0;
>>>>> +}
>>>>> +
>>>>> Â int amdgpu_create_bo_from_user_mem(amdgpu_device_handle dev,
>>>>> Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â void *cpu,
>>>>> Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â uint64_t size,
>>>>
>>>> _______________________________________________
>>>> amd-gfx mailing list
>>>> amd-gfx at lists.freedesktop.org
>>>> https://lists.freedesktop.org/mailman/listinfo/amd-gfx
>>> _______________________________________________
>>> amd-gfx mailing list
>>> amd-gfx at lists.freedesktop.org
>>> https://lists.freedesktop.org/mailman/listinfo/amd-gfx
>>
