Re: [PATCH 0/6] make ctx mgr global

[zhoucm1 <david1.zhou-5C7GfCeVMHo@public.gmane.org>]

On 2016年08月18日 16:15, Christian König wrote:
> NAK to the whole approach.
>
> If we want to share dependencies in the form of fences between devices 
> and especially processes we must use android fences and the sync file 
> framework.
Then if we want to share semaphore between devices and  processes, we 
must re-implement semaphore and move it to kernel side from libdrm, right?
then bind unused fd to semaphore object, and then export/import fd.
What do you think of it?

Regards,
David Zhou
>
> Sharing numbers in the form of the IDR is a security nightmare we 
> already ran into with the GEM flink design.
>
> Regards,
> Christian.
>
> Am 18.08.2016 um 09:50 schrieb Chunming Zhou:
>> If we want to share semaphore/dependency across process across 
>> device, we
>> must make ctx id be global, so that we can index it everywhere.
>>
>> Chunming Zhou (6):
>>    drm/amdgpu: use global ctx mgr instead of vm specified
>>    drm/amdgpu: clean up for amdgpu ctx
>>    drm/amdgpu: allocate progressively higher ids for ctx until idr
>>      counter wraps
>>    drm/amdgpu: ctx id should be removed when ctx is freed
>>    drm/amdgpu: use fence-array for ctx release
>>    drm/amdgpu: dependency is already signaled if ctx has been freed
>>
>>   drivers/gpu/drm/amd/amdgpu/amdgpu.h     |  17 ++--
>>   drivers/gpu/drm/amd/amdgpu/amdgpu_cs.c  |   9 +-
>>   drivers/gpu/drm/amd/amdgpu/amdgpu_ctx.c | 142 
>> ++++++++++++++++++--------------
>>   drivers/gpu/drm/amd/amdgpu/amdgpu_drv.c |   3 +
>>   drivers/gpu/drm/amd/amdgpu/amdgpu_kms.c |   4 -
>>   5 files changed, 92 insertions(+), 83 deletions(-)
>>
>

_______________________________________________
amd-gfx mailing list
amd-gfx@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/amd-gfx