Re: [PATCH v3 RESEND] selftests: filesystems: Add functional test for the abort file in fusectl

[Chen Linxuan]

On Thu, Jun 12, 2025 at 4:56 PM Miklos Szeredi <miklos@xxxxxxxxxx> wrote:
>
> On Tue, 10 Jun 2025 at 04:10, Chen Linxuan <chenlinxuan@xxxxxxxxxxxxx> wrote:
> >
> > This patch add a simple functional test for the "abort" file
> > in fusectlfs (/sys/fs/fuse/connections/ID/about).
> >
> > A simple fuse daemon is added for testing.
> >
> > Related discussion can be found in the link below.
> >
> > Link: https://lore.kernel.org/all/CAOQ4uxjKFXOKQxPpxtS6G_nR0tpw95w0GiO68UcWg_OBhmSY=Q@xxxxxxxxxxxxxx/
> > Signed-off-by: Chen Linxuan <chenlinxuan@xxxxxxxxxxxxx>
> > Acked-by: Shuah Khan <skhan@xxxxxxxxxxxxxxxxxxx>
> > Reviewed-by: Amir Goldstein <amir73il@xxxxxxxxx>
>
> Thanks.
>
> I suggest setting up a userns environment, see attached patch (also
> fixes a EBUSY on umount/rmdir).

The v4 patch series has been sent with your suggested changes applied:
https://lore.kernel.org/all/20250612094033.2538122-2-chenlinxuan@xxxxxxxxxxxxx/

However, I have some concerns about creating a user namespace.
Some downstream distributions (such as Ubuntu?) may disable
unprivileged user namespaces by default.
If we create the user namespace before mounting FUSE, these tests
would require privileges.

>
> Thanks,
> Miklos