Context conversion patches need a lot of scrutiny (and passing through the automated test suites), while the bo list changes are perhaps a bit of a bike shed, although they do result in less lines of code and hopefully easier to read.
BO list sorting is really independent, as is the IDR MES cleanup (which could as well have the warnings removed). Just say the work and I can easily split them out. Series has been smoke tested, including lockdep and game + desktop workloads so at least shouldn't be too obviously broken. v2: * Re-order to pull reviewed to the head. * Added more patches, some suggested by Christian, some I spotted new issues or cleanup opportunities. * No need for kref_get_unless_zero when under the xa_lock. * RCU is no longer needed for bo list free. Tvrtko Ursulin (12): drm/amdgpu/mes: Remove idr leftovers drm/amdgpu: Simplify sorting of the bo list drm/amdgpu: Remove unneccessary memset in bo list creation drm/amdgpu: Remove the bo list mutex drm/amdgpu: Remove a few holes from struct amdgpu_ctx drm/amdgpu: Remove duplicate struct member drm/amdgpu: Remove arbitrary number of contexts limitation drm/amdgpu: Consolidate ctx put drm/amdgpu: Remove live context error log and skip drm/amdgpu: Replace idr with xarray in amdgpu_bo_list drm/amdgpu: Remove output parameter in bo list handling drm/amdgpu: Convert context manager to xarray drivers/gpu/drm/amd/amdgpu/amdgpu.h | 5 +- drivers/gpu/drm/amd/amdgpu/amdgpu_bo_list.c | 153 ++++++------- drivers/gpu/drm/amd/amdgpu/amdgpu_bo_list.h | 22 +- drivers/gpu/drm/amd/amdgpu/amdgpu_cs.c | 67 +++--- drivers/gpu/drm/amd/amdgpu/amdgpu_ctx.c | 225 ++++++-------------- drivers/gpu/drm/amd/amdgpu/amdgpu_ctx.h | 27 ++- drivers/gpu/drm/amd/amdgpu/amdgpu_kms.c | 11 +- drivers/gpu/drm/amd/amdgpu/amdgpu_mes.c | 9 - drivers/gpu/drm/amd/amdgpu/amdgpu_mes.h | 3 - drivers/gpu/drm/amd/amdgpu/amdgpu_sched.c | 8 +- drivers/gpu/drm/amd/amdgpu/sdma_v5_0.c | 18 +- drivers/gpu/drm/amd/amdgpu/sdma_v5_2.c | 18 +- 12 files changed, 205 insertions(+), 361 deletions(-) -- 2.51.1
