Re: [PATCH v5 1/4] kernfs: add a WARN_ON_ONCE if ->close is set

From: Christoph Hellwig
Date: Mon Aug 12 2024 - 02:38:37 EST


On Fri, Aug 09, 2024 at 07:37:49AM +0200, Greg Kroah-Hartman wrote:
> > * It is not possible to successfully wrap close.
> > * So error if someone is trying to use close.
> > */
> > - if (vma->vm_ops && vma->vm_ops->close)
> > + if (WARN_ON_ONCE(vma->vm_ops && vma->vm_ops->close))
>
> So you just rebooted a machine that hits this, loosing data everywhere.
> Not nice :(

Huh. if you are stupid enough to set panic_on_warn you get to keep
the pieces. And our file systems are reliable to not use data on
an unclean shutdown anyway.

Pleaee stop these BS arguments.