Re: [Intel-xe] [PATCH 0/2] DRM scheduler changes for upstream

[Rodrigo Vivi <rodrigo.vivi@intel.com>]

On Mon, Jul 31, 2023 at 08:27:46PM -0700, Matthew Brost wrote:
> Fix the DRM scheduler changed based on feedback from upstream RFC
> feedback [1].
> 
> First patch addresses Boris's feedback [2] to not make the scheduler
> work queue run in a infinite loop, rather have it requeue itself for
> fairness.
> 
> Second patch addresse Luben's and Boris's feedback [3] to add a job to
> the pending list before caling run_job() and kicking the TDR after. This
> also includes a bit of restructure of the DRM scheduler helper
> functions that will be broken out into a different patch but squashing
> this into an earlier patch as fixup for easier repo maintainance.
> 
> Matt  
> 
> [1] https://patchwork.freedesktop.org/series/116055/
> [2] https://patchwork.freedesktop.org/patch/530655/?series=116055&rev=1
> [3] https://patchwork.freedesktop.org/patch/530652/?series=116055&rev=1
> 
> Matthew Brost (2):
>   fixup! drm/sched: Submit job before starting TDR
>   fixup! drm/sched: Convert drm scheduler to use a work queue rather
>     than kthread

should we merge them now? or should you send them first to the dri-devel
again and only change here after getting the final ack/review there?

Well, having them here ensures the CI on it... so up to you.

Acked-by: Rodrigo Vivi <rodrigo.vivi@intel.com>

> 
>  drivers/gpu/drm/scheduler/sched_main.c | 87 +++++++++++++-------------
>  1 file changed, 43 insertions(+), 44 deletions(-)
> 
> -- 
> 2.34.1
>