[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-ID: <4b0b46f29955956916765d8d615f96849c8ce3f7.camel@linaro.org>
Date: Wed, 18 Oct 2023 12:46:13 +0100
From: André Draszik <andre.draszik@...aro.org>
To: Bernd Schubert <bschubert@....com>
Cc: Miklos Szeredi <mszeredi@...hat.com>,
"stable@...r.kernel.org" <stable@...r.kernel.org>,
Greg Kroah-Hartman <gregkh@...uxfoundation.org>,
"linux-kernel@...r.kernel.org" <linux-kernel@...r.kernel.org>
Subject: Re: [PATCH v2] Revert "fuse: Apply flags2 only when userspace set
the FUSE_INIT_EXT"
On Wed, 2023-10-18 at 11:39 +0000, Bernd Schubert wrote:
> On 10/18/23 13:15, André Draszik wrote:
> > From: André Draszik <andre.draszik@...aro.org>
> >
> > This reverts commit 3066ff93476c35679cb07a97cce37d9bb07632ff.
> >
> > This patch breaks all existing userspace by requiring updates as
> > mentioned in the commit message, which is not allowed.
> >
> > Revert to restore compatibility with existing userspace
> > implementations.
>
> Which fuse file system does it exactly break? In fact there haven't
> been
> added too many flags after - what exactly is broken?
The original patch broke the existing kernel <-> user ABI by now
requiring user space applications to pass in an extra flag.
There are various side-effects of this, like unbootable systems, just
because the kernel was updated.
Breaking the ABI is the one thing that is not allowed. This is not
specific to any particular fuse file system.
Kind Regards,
Andre
Powered by blists - more mailing lists