drm/xe: Use ordered wq for preempt fence waiting
authorMatthew Brost <matthew.brost@intel.com>
Mon, 1 Apr 2024 22:19:11 +0000 (15:19 -0700)
committerLucas De Marchi <lucas.demarchi@intel.com>
Wed, 3 Apr 2024 14:11:00 +0000 (07:11 -0700)
commit37c15c4aae1fe3f67efd2641db8d8c25c2d524ab
tree222ddb9241237b0e8400a4854f292e0988981f26
parent9f18b55b6d3f77b9e778257efdec385d2d5dfa8e
drm/xe: Use ordered wq for preempt fence waiting

Preempt fences can sleep waiting for an exec queue suspend operation to
complete. If the system_unbound_wq is used for waiting and the number of
waiters exceeds max_active this will result in other users of the
system_unbound_wq getting starved. Use a device private work queue for
preempt fences to avoid starvation of the system_unbound_wq.

Even though suspend operations can complete out-of-order, all suspend
operations within a VM need to complete before the preempt rebind worker
can start. With that, use a device private ordered wq for preempt fence
waiting.

v2:
 - Add comment about cleanup on failure (Matt R)
 - Update commit message (Lucas)

Fixes: dd08ebf6c352 ("drm/xe: Introduce a new DRM driver for Intel GPUs")
Signed-off-by: Matthew Brost <matthew.brost@intel.com>
Reviewed-by: Lucas De Marchi <lucas.demarchi@intel.com>
Link: https://patchwork.freedesktop.org/patch/msgid/20240401221913.139672-2-matthew.brost@intel.com
Signed-off-by: Lucas De Marchi <lucas.demarchi@intel.com>
drivers/gpu/drm/xe/xe_device.c
drivers/gpu/drm/xe/xe_device_types.h
drivers/gpu/drm/xe/xe_preempt_fence.c