[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-ID: <CAJfpeguwqEsPLtph73AG7bhm1Dp4ahyJtyW=Ud7L-OFwyEmwWg@mail.gmail.com>
Date: Wed, 11 Mar 2020 15:19:18 +0100
From: Miklos Szeredi <miklos@...redi.hu>
To: Amir Goldstein <amir73il@...il.com>
Cc: Vivek Goyal <vgoyal@...hat.com>,
linux-fsdevel <linux-fsdevel@...r.kernel.org>,
linux-kernel <linux-kernel@...r.kernel.org>,
linux-nvdimm <linux-nvdimm@...ts.01.org>, virtio-fs@...hat.com,
Stefan Hajnoczi <stefanha@...hat.com>,
"Dr. David Alan Gilbert" <dgilbert@...hat.com>,
"Michael S. Tsirkin" <mst@...hat.com>,
Peng Tao <tao.peng@...ux.alibaba.com>
Subject: Re: [PATCH 12/20] fuse: Introduce setupmapping/removemapping commands
On Wed, Mar 11, 2020 at 8:03 AM Amir Goldstein <amir73il@...il.com> wrote:
>
> On Tue, Mar 10, 2020 at 10:34 PM Vivek Goyal <vgoyal@...hat.com> wrote:
> >
> > On Tue, Mar 10, 2020 at 08:49:49PM +0100, Miklos Szeredi wrote:
> > > On Wed, Mar 4, 2020 at 5:59 PM Vivek Goyal <vgoyal@...hat.com> wrote:
> > > >
> > > > Introduce two new fuse commands to setup/remove memory mappings. This
> > > > will be used to setup/tear down file mapping in dax window.
> > > >
> > > > Signed-off-by: Vivek Goyal <vgoyal@...hat.com>
> > > > Signed-off-by: Peng Tao <tao.peng@...ux.alibaba.com>
> > > > ---
> > > > include/uapi/linux/fuse.h | 37 +++++++++++++++++++++++++++++++++++++
> > > > 1 file changed, 37 insertions(+)
> > > >
> > > > diff --git a/include/uapi/linux/fuse.h b/include/uapi/linux/fuse.h
> > > > index 5b85819e045f..62633555d547 100644
> > > > --- a/include/uapi/linux/fuse.h
> > > > +++ b/include/uapi/linux/fuse.h
> > > > @@ -894,4 +894,41 @@ struct fuse_copy_file_range_in {
> > > > uint64_t flags;
> > > > };
> > > >
> > > > +#define FUSE_SETUPMAPPING_ENTRIES 8
> > > > +#define FUSE_SETUPMAPPING_FLAG_WRITE (1ull << 0)
> > > > +struct fuse_setupmapping_in {
> > > > + /* An already open handle */
> > > > + uint64_t fh;
> > > > + /* Offset into the file to start the mapping */
> > > > + uint64_t foffset;
> > > > + /* Length of mapping required */
> > > > + uint64_t len;
> > > > + /* Flags, FUSE_SETUPMAPPING_FLAG_* */
> > > > + uint64_t flags;
> > > > + /* Offset in Memory Window */
> > > > + uint64_t moffset;
> > > > +};
> > > > +
> > > > +struct fuse_setupmapping_out {
> > > > + /* Offsets into the cache of mappings */
> > > > + uint64_t coffset[FUSE_SETUPMAPPING_ENTRIES];
> > > > + /* Lengths of each mapping */
> > > > + uint64_t len[FUSE_SETUPMAPPING_ENTRIES];
> > > > +};
> > >
> > > fuse_setupmapping_out together with FUSE_SETUPMAPPING_ENTRIES seem to be unused.
> >
> > This looks like leftover from the old code. I will get rid of it. Thanks.
> >
>
> Hmm. I wonder if we should keep some out args for future extensions.
> Maybe return the mapped size even though it is all or nothing at this
> point?
>
> I have interest in a similar FUSE mapping functionality that was prototyped
> by Miklos and published here:
> https://lore.kernel.org/linux-fsdevel/CAJfpegtjEoE7H8tayLaQHG9fRSBiVuaspnmPr2oQiOZXVB1+7g@mail.gmail.com/
>
> In this prototype, a FUSE_MAP command is used by the server to map a
> range of file to the kernel for io. The command in args are quite similar to
> those in fuse_setupmapping_in, but since the server is on the same host,
> the mapping response is {mapfd, offset, size}.
Right. So the difference is in which entity allocates the mapping.
IOW whether the {fd, offset, size} is input or output in the protocol.
I don't remember the reasons for going with the mapping being
allocated by the client, not the other way round. Vivek?
If the allocation were to be by the server, we could share the request
type and possibly some code between the two, although the I/O
mechanism would still be different.
Thanks,
Miklos
Powered by blists - more mailing lists