Re: [PATCH 2/2] drm/panthor: Fix sync-only jobs

2024-07-02 Thread Boris Brezillon
On Fri, 28 Jun 2024 16:55:36 +0200 Boris Brezillon wrote: > - /* Stream size is zero, nothing to do => return a NULL fence and let > - * drm_sched signal the parent. > + /* Stream size is zero, nothing to do except making sure all previously > + * submitted jobs are done before

Re: [PATCH 2/2] drm/panthor: Fix sync-only jobs

2024-07-01 Thread Steven Price
On 28/06/2024 15:55, Boris Brezillon wrote: > A sync-only job is meant to provide a synchronization point on a > queue, so we can't return a NULL fence there, we have to add a signal > operation to the command stream which executes after all other > previously submitted jobs are done. > > Fixes: d

Re: [PATCH 2/2] drm/panthor: Fix sync-only jobs

2024-07-01 Thread Boris Brezillon
On Sat, 29 Jun 2024 13:10:35 +0100 Liviu Dudau wrote: > On Sat, Jun 29, 2024 at 10:52:56AM +0200, Boris Brezillon wrote: > > On Fri, 28 Jun 2024 22:34:57 +0100 > > Liviu Dudau wrote: > > > > > On Fri, Jun 28, 2024 at 04:55:36PM +0200, Boris Brezillon wrote: > > > > A sync-only job is meant

Re: [PATCH 2/2] drm/panthor: Fix sync-only jobs

2024-06-29 Thread Boris Brezillon
On Fri, 28 Jun 2024 22:34:57 +0100 Liviu Dudau wrote: > On Fri, Jun 28, 2024 at 04:55:36PM +0200, Boris Brezillon wrote: > > A sync-only job is meant to provide a synchronization point on a > > queue, so we can't return a NULL fence there, we have to add a signal > > operation to the command stre

Re: [PATCH 2/2] drm/panthor: Fix sync-only jobs

2024-06-29 Thread Liviu Dudau
On Sat, Jun 29, 2024 at 10:52:56AM +0200, Boris Brezillon wrote: > On Fri, 28 Jun 2024 22:34:57 +0100 > Liviu Dudau wrote: > > > On Fri, Jun 28, 2024 at 04:55:36PM +0200, Boris Brezillon wrote: > > > A sync-only job is meant to provide a synchronization point on a > > > queue, so we can't return

Re: [PATCH 2/2] drm/panthor: Fix sync-only jobs

2024-06-28 Thread Liviu Dudau
On Fri, Jun 28, 2024 at 04:55:36PM +0200, Boris Brezillon wrote: > A sync-only job is meant to provide a synchronization point on a > queue, so we can't return a NULL fence there, we have to add a signal > operation to the command stream which executes after all other > previously submitted jobs ar

[PATCH 2/2] drm/panthor: Fix sync-only jobs

2024-06-28 Thread Boris Brezillon
A sync-only job is meant to provide a synchronization point on a queue, so we can't return a NULL fence there, we have to add a signal operation to the command stream which executes after all other previously submitted jobs are done. Fixes: de8548813824 ("drm/panthor: Add the scheduler logical blo