drm/ttm: Reorder sys manager cleanup step
authorKarolina Stolarek <karolina.stolarek@intel.com>
Mon, 16 Oct 2023 12:15:25 +0000 (14:15 +0200)
committerChristian König <christian.koenig@amd.com>
Mon, 16 Oct 2023 12:27:27 +0000 (14:27 +0200)
commit3b401e30c249849d803de6c332dad2a595a58658
treed5ee31c0e565b2656793624d52e6926226980e27
parent4366faf43308bd91c59a20c43a9f853a9c3bb6e4
drm/ttm: Reorder sys manager cleanup step

With the current cleanup flow, we could trigger a NULL pointer
dereference if there is a delayed destruction of a BO with a
system resource that gets executed on drain_workqueue() call,
as we attempt to free a resource using an already released
resource manager.

Remove the device from the device list and drain its workqueue
before releasing the system domain manager in ttm_device_fini().

Signed-off-by: Karolina Stolarek <karolina.stolarek@intel.com>
Reviewed-by: Christian König <christian.koenig@amd.com>
Link: https://patchwork.freedesktop.org/patch/msgid/20231016121525.2237838-1-karolina.stolarek@intel.com
Signed-off-by: Christian König <christian.koenig@amd.com>
drivers/gpu/drm/ttm/ttm_device.c