[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-ID: <CADnq5_NwDn5DXPadzZtegUJ=y=LfVHykO7kG3edmiqRTTCxMNQ@mail.gmail.com>
Date: Mon, 12 Aug 2024 13:38:38 -0400
From: Alex Deucher <alexdeucher@...il.com>
To: Michał Winiarski <michal.winiarski@...el.com>
Cc: dri-devel@...ts.freedesktop.org, linux-kernel@...r.kernel.org,
intel-gfx@...ts.freedesktop.org,
Pekka Paalanen <pekka.paalanen@...labora.com>, David Airlie <airlied@...ux.ie>,
Oded Gabbay <ogabbay@...nel.org>, Maxime Ripard <mripard@...nel.org>,
Emil Velikov <emil.l.velikov@...il.com>, Matthew Wilcox <willy@...radead.org>,
Thomas Zimmermann <tzimmermann@...e.de>, James Zhu <James.Zhu@....com>,
Christian König <christian.koenig@....com>
Subject: Re: [PATCH v6 0/4] drm: Use full allocated minor range for DRM
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@...el.com> 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
>
Powered by blists - more mailing lists