Re: [PATCH] mm/mmu_notifier: Remove non_block_start/end() from notifier invocation

David Woodhouse <[email protected]>
Newsgroups dev.linux.lists.linux-rt-devel,org.kernel.vger.kvm,org.kernel.vger.linux-kernel,org.kvack.linux-mm
Message-ID <[email protected]>
On Wed, 2026-08-12 at 11:26 -0300, Jason Gunthorpe wrote:
> On Wed, Aug 12, 2026 at 03:05:43PM +0100, David Woodhouse wrote:
> > the non_block_{start,end} guards around the MMU notifiers, which are
> > basically never being called anyway and don't actually seem to protect
> > against any real bugs.
> 
> If you think this blocked stuff is dead code then lets remove it, but
> I'm pretty sure it is called and I remember seeing bug reports about
> it being triggered in the wild. Vetter certainly added it because
> their tests were actually triggering and they had bugs in their DRM
> stack directly connected to this.
> 
> It might not trigger for your hypervisor case but we aren't here just
> to make only kvm work now are we?

I'm sure there can be real-world use cases where it can trigger, but
when I was *trying* to exercise the code path, to validate my belief
(from source inspection) that even the plain spinlock would trigger the
splat, I ended up having to hack the kernel to do so:

https://lore.kernel.org/all/[email protected]/

Perhaps in the past it was easier to trigger 'naturally'?
smime.p7s (application/pkcs7-signature, 6 KB) - not displayed
lmpx.com only provides a reader for public news (NNTP) servers. It is not affiliated with the servers or forums shown here and is not responsible for the content of articles, which is written by their respective authors.