[<prev] [next>] [<thread-prev] [day] [month] [year] [list]
Message-ID: <f164cb3a-5285-46e9-9ba1-062cfcb2f920@collabora.com>
Date: Wed, 5 Mar 2025 14:26:52 +0300
From: Dmitry Osipenko <dmitry.osipenko@...labora.com>
To: Thomas Zimmermann <tzimmermann@...e.de>, David Airlie
<airlied@...il.com>, Gerd Hoffmann <kraxel@...hat.com>,
Gurchetan Singh <gurchetansingh@...omium.org>, Chia-I Wu
<olvaffe@...il.com>, Daniel Vetter <daniel@...ll.ch>,
Maarten Lankhorst <maarten.lankhorst@...ux.intel.com>,
Maxime Ripard <mripard@...nel.org>,
Christian König <christian.koenig@....com>,
Qiang Yu <yuq825@...il.com>, Steven Price <steven.price@....com>,
Boris Brezillon <boris.brezillon@...labora.com>,
Emma Anholt <emma@...olt.net>, Melissa Wen <mwen@...lia.com>
Cc: kernel@...labora.com, linux-kernel@...r.kernel.org,
dri-devel@...ts.freedesktop.org, virtualization@...ts.linux-foundation.org
Subject: Re: [PATCH v19 22/30] drm/shmem-helper: Add common memory shrinker
On 3/4/25 14:43, Thomas Zimmermann wrote:
> Hi
>
> Am 04.03.25 um 11:59 schrieb Dmitry Osipenko:
>> On 3/4/25 13:29, Thomas Zimmermann wrote:
>>> Hi
>>>
>>> Am 05.01.24 um 19:46 schrieb Dmitry Osipenko:
>>>> Introduce common drm-shmem shrinker for DRM drivers.
>>> What's the status of this patch?
>> It was de-prioritized on my list a year ago as I had to move to a higher
>> priority problems. Rebasing and re-testing these patches takes much
>> time, it either has to be split up in a smaller parts or applied in
>> one go.
>
> The first 11 patches seem uncontroversial to me; maybe except for patch
> 9. Could they be merged any time soon?
>
> The next batch could then be patches up to #20, which seem to be
> reviewed as well.
>
>>
>> The current status is that I started to work on v20 rather long time ago
>> and the patches need to be rebased and re-tested again. Heard that
>> Panthor driver may want shmem shrinker more than others today.
>
> I've been receiving reports about low performance of ast plus gem-shmem
> recently. I suspect that the vmap/vunmap overhead could be related and
> this series would likely lessen the problem.
Alright, will refresh the patches. See if the first ones sill can be
merged right away and then split the remainder into batches.
--
Best regards,
Dmitry
Powered by blists - more mailing lists