Re: [PATCH 11/62] drm/pagemap: Unlock cache->lock before freeing it

[Matthew Brost <matthew.brost@intel.com>]

On Mon, Feb 23, 2026 at 02:20:20PM -0800, Bart Van Assche wrote:
> On 2/23/26 2:14 PM, Matthew Brost wrote:
> > On Mon, Feb 23, 2026 at 02:00:11PM -0800, Bart Van Assche wrote:
> > > From: Bart Van Assche <bvanassche@acm.org>
> > > 
> > > Although freeing a spinlock without unlocking it is fine, this confuses
> > 
> > Fine, but pretty bad practice.
> > 
> > > static analyzers. Hence this patch.
> > > 
> > > Cc: Maarten Lankhorst <maarten.lankhorst@linux.intel.com>
> > > Cc: Maxime Ripard <mripard@kernel.org>
> > > Cc: Thomas Zimmermann <tzimmermann@suse.de>
> > > Cc: Thomas Hellström <thomas.hellstrom@linux.intel.com>
> > > Cc: Matthew Brost <matthew.brost@intel.com>
> > 
> > With that:
> > Reviewed-by: Matthew Brost <matthew.brost@intel.com>
> > 
> > We can apply this to the DRM branches and propagate this up to Linus
> > during 7.0 cycle if that works for you? Or if you have other plans to
> > merge, let us know.
> Peter asked me to split this series into one series per maintainer and
> to send the patches directly to the respective maintainers. Feel free to
> queue this patch in a DRM branch.
> 

I've reposted the patch, retaining you as the author, to our lists [1],
since we pull patches from the lists when merging and to run our CI.

Let me know if anything looks wrong there, but I’ll merge once CI comes
back (might take a few days, as CI is currently broken after we rebased
on 7.0‑rc1). After that, it will be in the PR up to Linus during the
fixes phase.

Matt

[1] https://patchwork.freedesktop.org/series/162019/

> Thanks,
> 
> Bart.