Follow @Openwall on Twitter for new release announcements and other news
[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-ID: <20260604163925.GI27423@brightrain.aerifal.cx>
Date: Thu, 4 Jun 2026 12:39:25 -0400
From: Rich Felker <dalias@...c.org>
To: Luis Henriques <luis@...lia.com>
Cc: musl@...ts.openwall.com
Subject: Re: [RFC PATCH 0/6] Add new Linux mount API wrappers

On Fri, May 29, 2026 at 06:10:57PM +0100, Luis Henriques wrote:
> This is a set of patches that add the needed wrappers for the new Linux
> mount API, which has been merged into the 5.2 kernel.  Sending them as an
> RFC as these are my first patches to musl, so it's quite likely that I'm
> doing something wrong.
> 
> For reference, here's the list of kernel commits that introduced these new
> syscalls:
> 
> a07b20004793 ("vfs: syscall: Add open_tree(2) to reference or clone a mount")
> 2db154b3ea8e ("vfs: syscall: Add move_mount(2) to move mounts around")
> 24dcb3d90a1f ("vfs: syscall: Add fsopen() to prepare for superblock creation")
> ecdab150fddb ("vfs: syscall: Add fsconfig() for configuring and managing a context")
> 93766fbd2696 ("vfs: syscall: Add fsmount() to create a mount for a superblock")
> cf3cba4a429b ("vfs: syscall: Add fspick() to select a superblock for reconfiguration")

I didn't see anything particularly "wrong". Since this set of
interfaces is unlikely to be used alongside mount() though, I'd rather
not have it in mount.c and linked whenever that is, but in a new file.
Normally we maximally separate TUs, but this doesn't scale well with
lots of new thin syscall wrappers like this, so just one file with
them all in it, added as a single commit, would be fine.

If it's not a big inconvenience, sending groups of patches as
attachments to a single mail rather than a lkml/git-send-email thread
is also preferable. It makes it possible to respond to and discuss
them as a group rather than branching off sub threads for each one.
But it's not a big deal either way.

Rich

Powered by blists - more mailing lists

Confused about mailing lists and their use? Read about mailing lists on Wikipedia and check out these guidelines on proper formatting of your messages.