Re: [PATCH] drm/amdgpu: introduce AMDGPU_GEM_CREATE_EXPLICIT_SYNC v2

2017-10-12 Thread Andres Rodriguez
On 2017-10-12 02:27 PM, Marek Olšák wrote: On Thu, Sep 21, 2017 at 4:38 PM, Andres Rodriguez wrote: Hi Christian, The reference radv patches are on the list. The basic idea is to only set the explicit sync flag for buffers allocated for dri usage. Did you mean "only set the explicit sync f

Re: [PATCH] drm/amdgpu: introduce AMDGPU_GEM_CREATE_EXPLICIT_SYNC v2

2017-10-12 Thread Christian König
Am 12.10.2017 um 20:27 schrieb Marek Olšák: On Thu, Sep 21, 2017 at 4:38 PM, Andres Rodriguez wrote: Hi Christian, The reference radv patches are on the list. The basic idea is to only set the explicit sync flag for buffers allocated for dri usage. Did you mean "only set the explicit sync fla

Re: [PATCH] drm/amdgpu: introduce AMDGPU_GEM_CREATE_EXPLICIT_SYNC v2

2017-10-12 Thread Marek Olšák
On Thu, Sep 21, 2017 at 4:38 PM, Andres Rodriguez wrote: > Hi Christian, > > The reference radv patches are on the list. The basic idea is to only set > the explicit sync flag for buffers allocated for dri usage. Did you mean "only set the explicit sync flag for buffers NOT allocated for dri usag

Re: [PATCH] drm/amdgpu: introduce AMDGPU_GEM_CREATE_EXPLICIT_SYNC v2

2017-09-21 Thread Christian König
Patch is Reviewed-by: Christian König . Regards, Christian. Am 21.09.2017 um 16:38 schrieb Andres Rodriguez: Hi Christian, The reference radv patches are on the list. The basic idea is to only set the explicit sync flag for buffers allocated for dri usage. Regards, Andres On 2017-09-19 09:

Re: [PATCH] drm/amdgpu: introduce AMDGPU_GEM_CREATE_EXPLICIT_SYNC v2

2017-09-21 Thread Andres Rodriguez
Hi Christian, The reference radv patches are on the list. The basic idea is to only set the explicit sync flag for buffers allocated for dri usage. Regards, Andres On 2017-09-19 09:24 AM, Christian König wrote: Am 19.09.2017 um 14:59 schrieb Andres Rodriguez: Introduce a flag to signal that

Re: [PATCH] drm/amdgpu: introduce AMDGPU_GEM_CREATE_EXPLICIT_SYNC v2

2017-09-19 Thread Mao, David
Hi Andres, The explicit sync should not be used for DrI3 and DRI2 but for cross process memory sharing, right? We still have to rely on implicit sync to guarantee the. Correct order of rendering and present. Could you confirm? Thanks. Sent from my iPhone On 19 Sep 2017, at 9:57 PM, Andres Rod

Re: [PATCH] drm/amdgpu: introduce AMDGPU_GEM_CREATE_EXPLICIT_SYNC v2

2017-09-19 Thread Andres Rodriguez
Correct. The idea is to only set AMDGPU_GEM_CREATE_EXPLICIT_SYNC for buffers that are not associated with dri2/3 or PRIME. Regards, Andres On 2017-09-19 10:10 AM, Mao, David wrote: Hi Andres, The explicit sync should not be used for DrI3  and DRI2 but for cross process memory sharing, right

Re: [PATCH] drm/amdgpu: introduce AMDGPU_GEM_CREATE_EXPLICIT_SYNC v2

2017-09-19 Thread Andres Rodriguez
On 2017-09-19 09:24 AM, Christian König wrote: Am 19.09.2017 um 14:59 schrieb Andres Rodriguez: Introduce a flag to signal that access to a BO will be synchronized through an external mechanism. Currently all buffers shared between contexts are subject to implicit synchronization. However, th

Re: [PATCH] drm/amdgpu: introduce AMDGPU_GEM_CREATE_EXPLICIT_SYNC v2

2017-09-19 Thread Christian König
Am 19.09.2017 um 14:59 schrieb Andres Rodriguez: Introduce a flag to signal that access to a BO will be synchronized through an external mechanism. Currently all buffers shared between contexts are subject to implicit synchronization. However, this is only required for protocols that currently d

[PATCH] drm/amdgpu: introduce AMDGPU_GEM_CREATE_EXPLICIT_SYNC v2

2017-09-19 Thread Andres Rodriguez
Introduce a flag to signal that access to a BO will be synchronized through an external mechanism. Currently all buffers shared between contexts are subject to implicit synchronization. However, this is only required for protocols that currently don't support an explicit synchronization mechanism