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

["Christian König" <deathsimple-ANTagKRnAhcb1SvskN2V4Q@public.gmane.org>]

Am 24.08.2016 um 11:33 schrieb zhoucm1:
>
>
> 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?

That is basically what sync_file does. It just doesn't call it semaphore 
and doesn't use the signal/wait semantic.

Instead fences can be added to a sync_file and waited for completion 
before a command submission is made.

Regards,
Christian.

>
> 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