From: Horst Birthelmer <horst@birthelmer.de>
To: Miklos Szeredi <miklos@szeredi.hu>
Cc: Horst Birthelmer <horst@birthelmer.com>,
Bernd Schubert <bschubert@ddn.com>,
Joanne Koong <joannelkoong@gmail.com>,
Luis Henriques <luis@igalia.com>,
linux-kernel@vger.kernel.org, linux-fsdevel@vger.kernel.org,
Horst Birthelmer <hbirthelmer@ddn.com>
Subject: Re: Re: Re: [PATCH v6 1/3] fuse: add compound command to combine multiple requests
Date: Fri, 27 Feb 2026 12:37:52 +0100 [thread overview]
Message-ID: <aaGA2YnQnlB27xAu@fedora.fritz.box> (raw)
In-Reply-To: <CAJfpegun=NNM099f6GC2_E2TbG0s936V_sW5SExt6mOEC0_WMQ@mail.gmail.com>
On Fri, Feb 27, 2026 at 12:29:00PM +0100, Miklos Szeredi wrote:
> On Fri, 27 Feb 2026 at 11:48, Horst Birthelmer <horst@birthelmer.de> wrote:
>
> > > FUSE_SUB_IS_ENTRY - this sub request will return a new entry on
> > > success (nodeid, filehandle)
> > > FUSE_SUB_DEP_ENTRY - this sub request depends on the result of a previous lookup
> > >
> >
> > we don't need this if we use my converters from above.
>
> Dependencies need to be handled by the kernel and libfuse as well.
> Makes no sense to have two separate mechanisms for handling
> dependencies, so the kernel should use the same flags.
>
OK, got it.
> > Could you maybe provide some examples of usecases, that I should try to drill the
> > new logic?
>
> - LOOKUP + GETATTR[L]
> - MKOBJ + (SETXATTR[L] (only for posix_acl inheritance)) + GETATTR[L]
> + (OPEN[L] (optional)
> - SETATTR + SETXATTR (setting posix_acl that modifies mode or setting
> mode on file with posix_acl)
> - INIT + LOOKUP_ROOT + GETATTR[L]
> - OPEN + IOCTL[O] + RELEASE[O] (fileattr_get/set)
>
> Only two dependencies here: lookup or open. Both are simple in terms
> of just needing to copy a field from a previous request to the current
> one with fixed positions in all of the above cases.
>
> The LOOKUP + MKNOD one *is* more complicated, because it makes
> execution of the MKNOD dependent on the result of the LOOKUP, so the
> dependency handler needs to look inside the result and decide how to
> proceed based on that. Some pros and cons of both approaches, so I'm
> curious to see how yours looks like.
>
I really am greateful for this list. Helps me a lot, since I was looking at this from the
perspective of the fuse server, which truns out to be different.
> > I have used compounds to send groups of semantically linked requests to the fuse server
> > signalling to it if the kernel expects it to be one atomic operation or a preferred
> > 'group' of requests (like open+getattr, nothing happens if those are not processed atomic
> > in a distributed file system)
>
> Which is the case where the kernel expects them to be atomic?
>
I naively thought that fuse_atomic_open() was actually there to do an atomic open ... ;-)
> Thanks,
> Miklos
>
Thanks,
Horst
next prev parent reply other threads:[~2026-02-27 11:37 UTC|newest]
Thread overview: 31+ messages / expand[flat|nested] mbox.gz Atom feed top
2026-02-26 16:43 [PATCH v6 0/3] fuse: compound commands Horst Birthelmer
2026-02-26 16:43 ` [PATCH v6 1/3] fuse: add compound command to combine multiple requests Horst Birthelmer
2026-02-26 23:05 ` Joanne Koong
2026-02-27 9:45 ` Miklos Szeredi
2026-02-27 10:48 ` Horst Birthelmer
2026-02-27 11:29 ` Miklos Szeredi
2026-02-27 11:37 ` Horst Birthelmer [this message]
2026-02-27 11:58 ` Miklos Szeredi
2026-03-02 9:56 ` Horst Birthelmer
2026-03-02 11:03 ` Miklos Szeredi
2026-03-02 13:19 ` Horst Birthelmer
2026-03-02 13:30 ` Miklos Szeredi
2026-03-06 14:27 ` Horst Birthelmer
2026-02-26 16:43 ` [PATCH v6 2/3] fuse: create helper functions for filling in fuse args for open and getattr Horst Birthelmer
2026-02-26 16:43 ` [PATCH v6 3/3] fuse: add an implementation of open+getattr Horst Birthelmer
2026-02-26 19:12 ` Joanne Koong
2026-02-27 7:48 ` Horst Birthelmer
2026-02-27 17:51 ` Joanne Koong
2026-02-27 18:07 ` Joanne Koong
2026-02-28 8:14 ` Horst Birthelmer
2026-03-02 18:56 ` Joanne Koong
2026-03-02 20:03 ` Bernd Schubert
2026-03-03 5:06 ` Darrick J. Wong
2026-03-03 7:26 ` Horst Birthelmer
2026-03-03 10:03 ` Miklos Szeredi
2026-03-03 10:38 ` Horst Birthelmer
2026-03-03 21:19 ` Joanne Koong
2026-03-04 9:11 ` Horst Birthelmer
2026-03-04 21:42 ` Joanne Koong
2026-03-03 23:13 ` Joanne Koong
2026-03-04 9:37 ` Miklos Szeredi
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=aaGA2YnQnlB27xAu@fedora.fritz.box \
--to=horst@birthelmer.de \
--cc=bschubert@ddn.com \
--cc=hbirthelmer@ddn.com \
--cc=horst@birthelmer.com \
--cc=joannelkoong@gmail.com \
--cc=linux-fsdevel@vger.kernel.org \
--cc=linux-kernel@vger.kernel.org \
--cc=luis@igalia.com \
--cc=miklos@szeredi.hu \
/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®