From: Chuck Lever III <chuck.lever@oracle.com>
To: Miklos Szeredi <miklos@szeredi.hu>, Steve French <smfrench@gmail.com>
Cc: lsf-pc <lsf-pc@lists.linux-foundation.org>,
linux-fsdevel <linux-fsdevel@vger.kernel.org>,
LKML <linux-kernel@vger.kernel.org>
Subject: Re: [LSF/MM/BPF TOPIC] making O_TMPFILE more atomic
Date: Mon, 9 May 2022 15:47:49 +0000 [thread overview]
Message-ID: <DAE0F6CD-F232-4305-9CCF-3F601124BFBC@oracle.com> (raw)
In-Reply-To: <CAJfpegskJzpXXhWCdw6K9r2hKORiBdXfSrgpUhKqn9VVyuVuqw@mail.gmail.com>
> On May 3, 2022, at 2:52 PM, Miklos Szeredi <miklos@szeredi.hu> wrote:
>
> On Thu, 24 Feb 2022 at 06:33, Steve French <smfrench@gmail.com> wrote:
>>
>> Currently creating tmpfiles on Linux can be problematic because the
>> tmpfile is not created and opened at the same time (vfs_tmpfile calls
>> into the fs, then later vfs_open is called to open the tmpfile). For
>> some filesystems it would be more natural to create and open the
>> tmpfile as one operation (because the action of creating the file on
>> some filesystems returns an open handle, so closing it then reopening
>> it would cause the tmpfile to be deleted).
>>
>> I would like to discuss whether the function do_tmpfile (which creates
>> and then opens the tmpfile) could have an option for a filesystem to
>> do this as one operation which would allow it to be more atomic and
>> allow it to work on a wider variety of filesystems.
>
> A related thread:
>
> https://lore.kernel.org/all/20201109100343.3958378-3-chirantan@chromium.org/#r
>
> There was no conclusion in the end. Not sure how hacky it would be to
> store the open file in the inode...
I just proposed adding a VFS API to make open/create atomic.
See 8/8 in this thread:
https://lore.kernel.org/linux-nfs/20220420192418.GB27805@fieldses.org/T/#m505a59ad4e4ed1413ffc055a088de3182fb50bb4
It adds a sibling API to dentry_open(). I didn't have
O_TMPFILE in mind when I created this API.
--
Chuck Lever
prev parent reply other threads:[~2022-05-09 15:48 UTC|newest]
Thread overview: 3+ messages / expand[flat|nested] mbox.gz Atom feed top
2022-02-24 5:30 Steve French
2022-05-03 18:52 ` Miklos Szeredi
2022-05-09 15:47 ` Chuck Lever III [this message]
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=DAE0F6CD-F232-4305-9CCF-3F601124BFBC@oracle.com \
--to=chuck.lever@oracle.com \
--cc=linux-fsdevel@vger.kernel.org \
--cc=linux-kernel@vger.kernel.org \
--cc=lsf-pc@lists.linux-foundation.org \
--cc=miklos@szeredi.hu \
--cc=smfrench@gmail.com \
/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
all inboxes | Powered by JetHome®