Are there any objections to this series? We have been running into this limit as a problem for a while now on big servers. Alex On Mon, Jul 24, 2023 at 5:15 PM Michał Winiarski <michal.winiarski@xxxxxxxxx> wrote: > > 64 DRM device nodes is not enough for everyone. > Upgrade it to ~512K (which definitely is more than enough). > > To allow testing userspace support for >64 devices, add additional DRM > modparam (force_extended_minors) which causes DRM to skip allocating minors > in 0-192 range. > Additionally - convert minors to use XArray instead of IDR to simplify the > locking. > > v1 -> v2: > Don't touch DRM_MINOR_CONTROL and its range (Simon Ser) > > v2 -> v3: > Don't use legacy scheme for >=192 minor range (Dave Airlie) > Add modparam for testing (Dave Airlie) > Add lockdep annotation for IDR (Daniel Vetter) > > v3 -> v4: > Convert from IDR to XArray (Matthew Wilcox) > > v4 -> v5: > Fixup IDR to XArray conversion (Matthew Wilcox) > > v5 -> v6: > Also convert Accel to XArray > Rename skip_legacy_minors to force_extended_minors > > Michał Winiarski (4): > drm: Use XArray instead of IDR for minors > accel: Use XArray instead of IDR for minors > drm: Expand max DRM device number to full MINORBITS > drm: Introduce force_extended_minors modparam > > drivers/accel/drm_accel.c | 110 +++------------------------------ > drivers/gpu/drm/drm_drv.c | 105 ++++++++++++++++--------------- > drivers/gpu/drm/drm_file.c | 2 +- > drivers/gpu/drm/drm_internal.h | 4 -- > include/drm/drm_accel.h | 18 +----- > include/drm/drm_file.h | 5 ++ > 6 files changed, 69 insertions(+), 175 deletions(-) > > -- > 2.41.0 >
