From: Christian Brauner <brauner@kernel.org>
To: Jeff Layton <jlayton@kernel.org>,
Chuck Lever <chuck.lever@oracle.com>, Jan Kara <jack@suse.cz>,
Amir Goldstein <amir73il@gmail.com>,
Simona Vetter <simona@ffwll.ch>
Cc: linux-fsdevel@vger.kernel.org, linux-nfs@vger.kernel.org,
Christian Brauner <brauner@kernel.org>,
stable@kernel.org
Subject: [PATCH v2 00/11] fhandle, pidfs: allow open_by_handle_at() purely based on file handle
Date: Tue, 24 Jun 2025 10:29:03 +0200 [thread overview]
Message-ID: <20250624-work-pidfs-fhandle-v2-0-d02a04858fe3@kernel.org> (raw)
Various filesystems such as pidfs and drm support opening file handles
without having to require a file descriptor to identify the filesystem.
The filesystem are global single instances and can be trivially
identified solely on the information encoded in the file handle.
This makes it possible to not have to keep or acquire a sentinal file
descriptor just to pass it to open_by_handle_at() to identify the
filesystem. That's especially useful when such sentinel file descriptor
cannot or should not be acquired.
For pidfs this means a file handle can function as full replacement for
storing a pid in a file. Instead a file handle can be stored and
reopened purely based on the file handle.
Such autonomous file handles can be opened with or without specifying a
a file descriptor. If no proper file descriptor is used the FD_INVALID
sentinel must be passed. This allows us to define further special
negative fd sentinels in the future.
Userspace can trivially test for support by trying to open the file
handle with an invalid file descriptor.
Signed-off-by: Christian Brauner <brauner@kernel.org>
---
Changes in v2:
- Simplify the FILEID_PIDFS enum.
- Introduce FD_INVALID.
- Require FD_INVALID for autonomous file handles.
- Link to v1: https://lore.kernel.org/20250623-work-pidfs-fhandle-v1-0-75899d67555f@kernel.org
---
Christian Brauner (11):
fhandle: raise FILEID_IS_DIR in handle_type
fhandle: hoist copy_from_user() above get_path_from_fd()
fhandle: rename to get_path_anchor()
pidfs: add pidfs_root_path() helper
fhandle: reflow get_path_anchor()
uapi/fcntl: mark range as reserved
uapi/fcntl: add FD_INVALID
exportfs: add FILEID_PIDFS
fhandle: add EXPORT_OP_AUTONOMOUS_HANDLES marker
fhandle, pidfs: support open_by_handle_at() purely based on file handle
selftests/pidfd: decode pidfd file handles withou having to specify an fd
fs/fhandle.c | 82 ++++++++++++++--------
fs/internal.h | 1 +
fs/pidfs.c | 16 ++++-
include/linux/exportfs.h | 9 ++-
include/uapi/linux/fcntl.h | 17 +++++
include/uapi/linux/pidfd.h | 15 ----
tools/testing/selftests/pidfd/Makefile | 2 +-
tools/testing/selftests/pidfd/pidfd.h | 6 +-
.../selftests/pidfd/pidfd_file_handle_test.c | 60 ++++++++++++++++
9 files changed, 158 insertions(+), 50 deletions(-)
---
base-commit: 4e3d1e6e1b2d9df9650be14380c534b3c5081ddd
change-id: 20250619-work-pidfs-fhandle-b63ff35c4924
next reply other threads:[~2025-06-24 8:29 UTC|newest]
Thread overview: 47+ messages / expand[flat|nested] mbox.gz Atom feed top
2025-06-24 8:29 Christian Brauner [this message]
2025-06-24 8:29 ` [PATCH v2 01/11] fhandle: raise FILEID_IS_DIR in handle_type Christian Brauner
2025-06-24 9:31 ` Jan Kara
2025-06-24 8:29 ` [PATCH v2 02/11] fhandle: hoist copy_from_user() above get_path_from_fd() Christian Brauner
2025-06-24 9:31 ` Jan Kara
2025-06-24 8:29 ` [PATCH v2 03/11] fhandle: rename to get_path_anchor() Christian Brauner
2025-06-24 9:31 ` Jan Kara
2025-06-24 8:29 ` [PATCH v2 04/11] pidfs: add pidfs_root_path() helper Christian Brauner
2025-06-24 9:31 ` Jan Kara
2025-06-24 8:29 ` [PATCH v2 05/11] fhandle: reflow get_path_anchor() Christian Brauner
2025-06-24 9:16 ` Jan Kara
2025-06-24 10:16 ` Christian Brauner
2025-06-24 8:29 ` [PATCH v2 06/11] uapi/fcntl: mark range as reserved Christian Brauner
2025-06-24 9:16 ` Jan Kara
2025-06-24 10:57 ` Amir Goldstein
2025-06-24 13:47 ` Christian Brauner
2025-06-24 8:29 ` [PATCH v2 07/11] uapi/fcntl: add FD_INVALID Christian Brauner
2025-06-24 9:17 ` Jan Kara
2025-06-24 8:29 ` [PATCH v2 08/11] exportfs: add FILEID_PIDFS Christian Brauner
2025-06-24 9:17 ` Jan Kara
2025-06-24 13:15 ` Amir Goldstein
2025-06-24 13:43 ` Christian Brauner
2025-06-24 14:20 ` Amir Goldstein
2025-06-24 8:29 ` [PATCH v2 09/11] fhandle: add EXPORT_OP_AUTONOMOUS_HANDLES marker Christian Brauner
2025-06-24 9:18 ` Jan Kara
2025-06-24 9:20 ` Jan Kara
2025-06-24 10:16 ` Christian Brauner
2025-06-24 8:29 ` [PATCH v2 10/11] fhandle, pidfs: support open_by_handle_at() purely based on file handle Christian Brauner
2025-06-24 9:30 ` Jan Kara
2025-06-24 10:15 ` Christian Brauner
2025-06-24 10:53 ` Amir Goldstein
2025-06-24 14:28 ` Amir Goldstein
2025-06-24 14:51 ` Christian Brauner
2025-06-24 15:07 ` Amir Goldstein
2025-06-24 15:23 ` Christian Brauner
2025-06-24 17:45 ` Jan Kara
2025-06-24 19:23 ` Amir Goldstein
2025-06-25 7:52 ` Christian Brauner
2025-06-24 23:07 ` Al Viro
2025-06-25 7:52 ` Christian Brauner
2025-06-24 8:29 ` [PATCH v2 11/11] selftests/pidfd: decode pidfd file handles withou having to specify an fd Christian Brauner
2025-06-24 9:39 ` Jan Kara
2025-06-24 10:58 ` [PATCH v2 00/11] fhandle, pidfs: allow open_by_handle_at() purely based on file handle Amir Goldstein
2025-06-24 10:59 ` Christian Brauner
2025-06-24 14:15 ` Jan Kara
2025-06-24 14:34 ` Amir Goldstein
2025-06-24 14:39 ` Christian Brauner
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=20250624-work-pidfs-fhandle-v2-0-d02a04858fe3@kernel.org \
--to=brauner@kernel.org \
--cc=amir73il@gmail.com \
--cc=chuck.lever@oracle.com \
--cc=jack@suse.cz \
--cc=jlayton@kernel.org \
--cc=linux-fsdevel@vger.kernel.org \
--cc=linux-nfs@vger.kernel.org \
--cc=simona@ffwll.ch \
--cc=stable@kernel.org \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).