[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [PULL 10/12] virtiofsd: Announce sub-mount points
From: |
Max Reitz |
Subject: |
Re: [PULL 10/12] virtiofsd: Announce sub-mount points |
Date: |
Tue, 3 Nov 2020 11:24:49 +0100 |
User-agent: |
Mozilla/5.0 (X11; Linux x86_64; rv:78.0) Gecko/20100101 Thunderbird/78.3.1 |
On 02.11.20 20:56, Dr. David Alan Gilbert (git) wrote:
> From: Max Reitz <mreitz@redhat.com>
>
> Whenever we encounter a directory with an st_dev or mount ID that
> differs from that of its parent, we set the FUSE_ATTR_SUBMOUNT flag so
> the guest can create a submount for it.
>
> We only need to do so in lo_do_lookup(). The following functions return
> a fuse_attr object:
> - lo_create(), though fuse_reply_create(): Calls lo_do_lookup().
> - lo_lookup(), though fuse_reply_entry(): Calls lo_do_lookup().
> - lo_mknod_symlink(), through fuse_reply_entry(): Calls lo_do_lookup().
> - lo_link(), through fuse_reply_entry(): Creating a link cannot create a
> submount, so there is no need to check for it.
> - lo_getattr(), through fuse_reply_attr(): Announcing submounts when the
> node is first detected (at lookup) is sufficient. We do not need to
> return the submount attribute later.
> - lo_do_readdir(), through fuse_add_direntry_plus(): Calls
> lo_do_lookup().
>
> Make announcing submounts optional, so submounts are only announced to
> the guest with the announce_submounts option. Some users may prefer the
> current behavior, so that the guest learns nothing about the host mount
> structure.
>
> (announce_submounts is force-disabled when the guest does not present
> the FUSE_SUBMOUNTS capability, or when there is no statx().)
>
> Signed-off-by: Max Reitz <mreitz@redhat.com>
> Reviewed-by: Stefan Hajnoczi <stefanha@redhat.com>
> Message-Id: <20201102161859.156603-6-mreitz@redhat.com>
> Signed-off-by: Dr. David Alan Gilbert <dgilbert@redhat.com>
> ---
> tools/virtiofsd/helper.c | 1 +
> tools/virtiofsd/passthrough_ll.c | 22 ++++++++++++++++++++++
> 2 files changed, 23 insertions(+)
> diff --git a/tools/virtiofsd/passthrough_ll.c
> b/tools/virtiofsd/passthrough_ll.c
[...]
> index 34d107975f..ec1008bceb 100644
> --- a/tools/virtiofsd/passthrough_ll.c
> +++ b/tools/virtiofsd/passthrough_ll.c
[...]
> @@ -601,6 +604,20 @@ static void lo_init(void *userdata, struct
> fuse_conn_info *conn)
[...]
> +#ifndef CONFIG_STATX
> + if (lo->announce_submounts) {
> + fuse_log(FUSE_LOG_WARNING, "lo_init: Cannot announce submounts,
> there "
> + "is no statx()\n");
> + lo->announce_submounts = false;
> + }
> +#endif
When reviewing this series, Miklos noted today that this warning is
wrong (we can still announce submounts even without statx()), and we
concluded that we should probably drop it and the
“lo->announce_submounts = false” assignment (i.e. this whole block).
I don’t think that needs to stop this pull request, though, we can
easily fix it on top.
Max
- [PULL 02/12] migration: Postpone the kick of the fault thread after recover, (continued)
- [PULL 02/12] migration: Postpone the kick of the fault thread after recover, Dr. David Alan Gilbert (git), 2020/11/02
- [PULL 03/12] virtiofsd: Seccomp: Add 'send' for syslog, Dr. David Alan Gilbert (git), 2020/11/02
- [PULL 01/12] migration: Unify reset of last_rb on destination node when recover, Dr. David Alan Gilbert (git), 2020/11/02
- [PULL 05/12] virtiofsd: Fix the help message of posix lock, Dr. David Alan Gilbert (git), 2020/11/02
- [PULL 04/12] tools/virtiofsd: Check vu_init() return value (CID 1435958), Dr. David Alan Gilbert (git), 2020/11/02
- [PULL 06/12] virtiofsd: Check FUSE_SUBMOUNTS, Dr. David Alan Gilbert (git), 2020/11/02
- [PULL 07/12] virtiofsd: Add attr_flags to fuse_entry_param, Dr. David Alan Gilbert (git), 2020/11/02
- [PULL 08/12] meson.build: Check for statx(), Dr. David Alan Gilbert (git), 2020/11/02
- [PULL 09/12] virtiofsd: Add mount ID to the lo_inode key, Dr. David Alan Gilbert (git), 2020/11/02
- [PULL 10/12] virtiofsd: Announce sub-mount points, Dr. David Alan Gilbert (git), 2020/11/02
- Re: [PULL 10/12] virtiofsd: Announce sub-mount points,
Max Reitz <=
- [PULL 11/12] tests/acceptance/boot_linux: Accept SSH pubkey, Dr. David Alan Gilbert (git), 2020/11/02
- [PULL 12/12] tests/acceptance: Add virtiofs_submounts.py, Dr. David Alan Gilbert (git), 2020/11/02
- Re: [PULL 00/12] migration queue, Peter Maydell, 2020/11/03