From: Tristan Wibberley <tristan.wibberley@gmail.com>
To: linux-kernel@vger.kernel.org
Subject: New system call wanted: fdreopen
Date: Sun, 9 Dec 2012 15:03:30 +0000 (UTC) [thread overview]
Message-ID: <ka2982$ra3$1@ger.gmane.org> (raw)
Hello,
I'd like to propose a system call called "fdreopen":
int fdreopen(int src_fd, int dst_fd, int flags);
I am willing to try implementing this system call given some suggestions
where to start and what locking to watch out for. I have given a brief of
the behaviour below, and a description of the class of problem that it
solves at the end.
Does anybody know any reasons why this system call would be impossible/
impractical or otherwise unacceptable?
Any improvements I should consider before trying to implement it?
Behaviour
=========
This system call would be like dup3 except for these things:
- if dst_fd is -1 then the lowest available file descriptor is allocated
rather than returning EBADF as dup3 does.
- the new file descriptor points to a *new* entry in the file table much
as if the original file had been opened again via open or openat. This
means that two large independent libraries can seek and read without
synchronising even when they cannot open a file by its path.
- O_RDWR access can be reduced to O_RDONLY or O_WRONLY:
int src_fd = open("/file", O_RDWR | O_CLOEXEC);
new_fd = fdreopen(src_fd, -1, O_CLOEXEC | O_RDONLY);
- it would be async signal safe.
Why
===
A common idiom on Linux is to open a file and keep the fd open so that
the underlying file can be unlinked from its directory. But if the file
needs to be read from several different parts of the codebase then due to
the file descriptor having exactly one read pointer those different parts
must be synchronised which is a relatively difficult task.
I think that this new system call is required to achieve that neatly and
simply:
- dup does not solve this problem because it only allows the new file
descriptor to have its own flags (eg O_CLOEXEC).
- /proc/self/fd/* does not solve this problem because the file might no
longer be available at the same place in the filesystem. In some
otherwise simple message passing or ReSTful IPC a different file will
be available at that path.
I suspect that user space has been solving this problem with otherwise
unnecessary levels of either synchronisation or difficult to reproduce
occasional bugs.
--
Tristan Wibberley
next reply other threads:[~2012-12-09 15:09 UTC|newest]
Thread overview: 6+ messages / expand[flat|nested] mbox.gz Atom feed top
2012-12-09 15:03 Tristan Wibberley [this message]
2012-12-09 16:27 ` New system call wanted: fdreopen Theodore Ts'o
2012-12-09 17:18 ` Tristan Wibberley
2012-12-09 19:37 ` Chris Adams
2012-12-09 20:47 ` Al Viro
2012-12-10 10:18 ` Kevin Easton
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='ka2982$ra3$1@ger.gmane.org' \
--to=tristan.wibberley@gmail.com \
--cc=linux-kernel@vger.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