Re: [PATCH] drm/syncobj: Deal with signalled fences in transfer.

2021-12-07 Thread Christian König
Am 07.12.21 um 12:35 schrieb Bas Nieuwenhuizen: On Tue, Dec 7, 2021 at 12:28 PM Lionel Landwerlin wrote: On 07/12/2021 13:00, Christian König wrote: Am 07.12.21 um 11:40 schrieb Bas Nieuwenhuizen: On Tue, Dec 7, 2021 at 8:21 AM Christian König wrote: Am 07.12.21 um 08:10 schrieb Lionel Land

Re: [PATCH] drm/syncobj: Deal with signalled fences in transfer.

2021-12-07 Thread Bas Nieuwenhuizen
On Tue, Dec 7, 2021 at 12:28 PM Lionel Landwerlin wrote: > > On 07/12/2021 13:00, Christian König wrote: > > Am 07.12.21 um 11:40 schrieb Bas Nieuwenhuizen: > >> On Tue, Dec 7, 2021 at 8:21 AM Christian König > >> wrote: > >>> Am 07.12.21 um 08:10 schrieb Lionel Landwerlin: > On 07/12/2021 0

Re: [PATCH] drm/syncobj: Deal with signalled fences in transfer.

2021-12-07 Thread Lionel Landwerlin
On 07/12/2021 13:00, Christian König wrote: Am 07.12.21 um 11:40 schrieb Bas Nieuwenhuizen: On Tue, Dec 7, 2021 at 8:21 AM Christian König wrote: Am 07.12.21 um 08:10 schrieb Lionel Landwerlin: On 07/12/2021 03:32, Bas Nieuwenhuizen wrote: See the comments in the code. Basically if the seqno

Re: [PATCH] drm/syncobj: Deal with signalled fences in transfer.

2021-12-07 Thread Christian König
Am 07.12.21 um 11:40 schrieb Bas Nieuwenhuizen: On Tue, Dec 7, 2021 at 8:21 AM Christian König wrote: Am 07.12.21 um 08:10 schrieb Lionel Landwerlin: On 07/12/2021 03:32, Bas Nieuwenhuizen wrote: See the comments in the code. Basically if the seqno is already signalled then we get a NULL fenc

Re: [PATCH] drm/syncobj: Deal with signalled fences in transfer.

2021-12-07 Thread Bas Nieuwenhuizen
On Tue, Dec 7, 2021 at 8:21 AM Christian König wrote: > > Am 07.12.21 um 08:10 schrieb Lionel Landwerlin: > > On 07/12/2021 03:32, Bas Nieuwenhuizen wrote: > >> See the comments in the code. Basically if the seqno is already > >> signalled then we get a NULL fence. If we then put the NULL fence >

Re: [PATCH] drm/syncobj: Deal with signalled fences in transfer.

2021-12-06 Thread Christian König
Am 07.12.21 um 08:09 schrieb Christian König: Am 07.12.21 um 02:32 schrieb Bas Nieuwenhuizen: See the comments in the code. Basically if the seqno is already signalled then we get a NULL fence. If we then put the NULL fence in a binary syncobj it counts as unsignalled, making that syncobj pre

Re: [PATCH] drm/syncobj: Deal with signalled fences in transfer.

2021-12-06 Thread Christian König
Am 07.12.21 um 08:10 schrieb Lionel Landwerlin: On 07/12/2021 03:32, Bas Nieuwenhuizen wrote: See the comments in the code. Basically if the seqno is already signalled then we get a NULL fence. If we then put the NULL fence in a binary syncobj it counts as unsignalled, making that syncobj pretty

Re: [PATCH] drm/syncobj: Deal with signalled fences in transfer.

2021-12-06 Thread Lionel Landwerlin
On 07/12/2021 03:32, Bas Nieuwenhuizen wrote: See the comments in the code. Basically if the seqno is already signalled then we get a NULL fence. If we then put the NULL fence in a binary syncobj it counts as unsignalled, making that syncobj pretty much useless for all expected uses. Not 100% su

Re: [PATCH] drm/syncobj: Deal with signalled fences in transfer.

2021-12-06 Thread Christian König
Am 07.12.21 um 02:32 schrieb Bas Nieuwenhuizen: See the comments in the code. Basically if the seqno is already signalled then we get a NULL fence. If we then put the NULL fence in a binary syncobj it counts as unsignalled, making that syncobj pretty much useless for all expected uses. Not 100%

[PATCH] drm/syncobj: Deal with signalled fences in transfer.

2021-12-06 Thread Bas Nieuwenhuizen
See the comments in the code. Basically if the seqno is already signalled then we get a NULL fence. If we then put the NULL fence in a binary syncobj it counts as unsignalled, making that syncobj pretty much useless for all expected uses. Not 100% sure about the transfer to a timeline syncobj but