Hi Bert, # The title of the patch should be: drm/sched: Partial revert of "Qualify drm_sched_wakeup() by drm_sched_entity_is_ready()" On 2023-11-27 08:30, Bert Karwatzki wrote: > Commit f3123c25 (in combination with the use of work queues by the gpu Commit f3123c2590005c, in combination with the use of work queues by the GPU scheduler, leads to random lock-ups of the GUI. > scheduler) leads to random lock ups of the GUI [1,2]. > > This is not a complete revert of commit f3123c25 as drm_sched_wakeup This is a partial revert of of commit f3123c2590005c since drm_sched_wakeup() > still needs its entity argument to pass it to drm_sched_can_queue. ... drm_sched_can_queue(). # Don't forget a SoB line! Signed-off-by: Bert ... >> [1] https://gitlab.freedesktop.org/drm/amd/-/issues/2994 # Use a Link: tag instead, like this: Link: https://gitlab.freedesktop.org/drm/amd/-/issues/2994 > [2] https://lists.freedesktop.org/archives/dri-devel/2023-November/431606.html # Use a Link: tag instead, like this: Link: https://lists.freedesktop.org/archives/dri-devel/2023-November/431606.html > > This reverts commit f3123c2590005c5ff631653d31428e40cd10c618. # The line above is *not* necessary, since this is a partial commit. Instead we need # a Fixes: line, like this: Fixes: f3123c2590005c ("drm/sched: Qualify drm_sched_wakeup() by drm_sched_entity_is_ready()") #######--- Then after you do "git format-patch", post it like this: git send-email \ --in-reply-to=c5292d06-2e37-4715-96dc-699f369111fa@xxxxxxxxx \ --to=ltuikov89@xxxxxxxxx \ --cc=christian.koenig@xxxxxxx \ --cc=dakr@xxxxxxxxxx \ --cc=dri-devel@xxxxxxxxxxxxxxxxxxxxx \ --cc=matthew.brost@xxxxxxxxx \ --cc=spasswolf@xxxxxx \ --cc=tvrtko.ursulin@xxxxxxxxx \ /path/to/PATCH This follows your thread where all the information is stored. Thanks! -- Regards, Luben
Attachment:
OpenPGP_0x4C15479431A334AF.asc
Description: OpenPGP public key
Attachment:
OpenPGP_signature.asc
Description: OpenPGP digital signature