[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-ID: <ae511933-e191-4e25-a919-21504fa31b93@amd.com>
Date: Wed, 14 Aug 2024 00:20:12 -0400
From: James Zhu <jamesz@....com>
To: Michał Winiarski <michal.winiarski@...el.com>,
Alex Deucher <alexdeucher@...il.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
Hi Michal
I did give Tested-by before. If you need Acks, Here I can give
Acked-by: James Zhu <James.Zhu@....com> for the series
Best Regards!
James
On 2024-08-13 20:18, Michał Winiarski wrote:
> On Mon, Aug 12, 2024 at 01:38:38PM GMT, Alex Deucher wrote:
>> Are there any objections to this series? We have been running into
>> this limit as a problem for a while now on big servers.
> I don't think there were any objections, just a general lack of
> interest - so there are no R-b / Acks.
> If you're interested to have a go at it - I can resend it.
> It should still apply on latest drm-tip.
>
> -Michał
>
>> 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