* [PATCH] docs: add virtiofsd(1) man page
@ 2020-01-27 16:25 Stefan Hajnoczi
2020-01-27 16:55 ` Liam Merwick
` (2 more replies)
0 siblings, 3 replies; 6+ messages in thread
From: Stefan Hajnoczi @ 2020-01-27 16:25 UTC (permalink / raw)
To: qemu-devel; +Cc: Peter Maydell, Dr. David Alan Gilbert, Stefan Hajnoczi
Document the virtiofsd(1) program and its command-line options. This
man page is a rST conversion of the original texi documentation that I
wrote.
Signed-off-by: Stefan Hajnoczi <stefanha@redhat.com>
---
Based-on: 20200124162606.8787-1-peter.maydell@linaro.org
("[PATCH v2 0/8] qemu-img, qemu-trace-stap, virtfs-proxy-helper: convert to rST")
---
MAINTAINERS | 1 +
Makefile | 8 ++-
docs/interop/conf.py | 5 +-
docs/interop/index.rst | 1 +
docs/interop/virtiofsd.rst | 123 +++++++++++++++++++++++++++++++++++++
5 files changed, 136 insertions(+), 2 deletions(-)
create mode 100644 docs/interop/virtiofsd.rst
diff --git a/MAINTAINERS b/MAINTAINERS
index 83fb32b860..1da0709129 100644
--- a/MAINTAINERS
+++ b/MAINTAINERS
@@ -1603,6 +1603,7 @@ S: Supported
F: tools/virtiofsd/*
F: hw/virtio/vhost-user-fs*
F: include/hw/virtio/vhost-user-fs.h
+F: docs/interop/virtiofsd.rst
virtio-input
M: Gerd Hoffmann <kraxel@redhat.com>
diff --git a/Makefile b/Makefile
index 539f9ef079..ecd26044bc 100644
--- a/Makefile
+++ b/Makefile
@@ -348,6 +348,9 @@ DOCS=qemu-doc.html qemu-doc.txt qemu.1
DOCS+=$(MANUAL_BUILDDIR)/interop/qemu-img.1
DOCS+=$(MANUAL_BUILDDIR)/interop/qemu-nbd.8
DOCS+=$(MANUAL_BUILDDIR)/interop/qemu-ga.8
+ifeq ($(CONFIG_LINUX)$(CONFIG_SECCOMP)$(CONFIG_LIBCAP_NG),yyy)
+DOCS+=$(MANUAL_BUILDDIR)/interop/virtiofsd.1
+endif
DOCS+=$(MANUAL_BUILDDIR)/system/qemu-block-drivers.7
DOCS+=docs/interop/qemu-qmp-ref.html docs/interop/qemu-qmp-ref.txt docs/interop/qemu-qmp-ref.7
DOCS+=docs/interop/qemu-ga-ref.html docs/interop/qemu-ga-ref.txt docs/interop/qemu-ga-ref.7
@@ -861,6 +864,9 @@ ifdef CONFIG_VIRTFS
$(INSTALL_DIR) "$(DESTDIR)$(mandir)/man1"
$(INSTALL_DATA) $(MANUAL_BUILDDIR)/interop/virtfs-proxy-helper.1 "$(DESTDIR)$(mandir)/man1"
endif
+ifeq ($(CONFIG_LINUX)$(CONFIG_SECCOMP)$(CONFIG_LIBCAP_NG),yyy)
+ $(INSTALL_DATA) docs/interop/virtiofsd.1 "$(DESTDIR)$(mandir)/man1"
+endif
install-datadir:
$(INSTALL_DIR) "$(DESTDIR)$(qemu_datadir)"
@@ -1052,7 +1058,7 @@ $(MANUAL_BUILDDIR)/system/index.html: $(call manual-deps,system)
$(call define-manpage-rule,interop,\
qemu-ga.8 qemu-img.1 qemu-nbd.8 qemu-trace-stap.1\
- virtfs-proxy-helper.1,\
+ virtiofsd.1 virtfs-proxy-helper.1,\
$(SRC_PATH/qemu-img-cmds.hx))
$(call define-manpage-rule,system,qemu-block-drivers.7)
diff --git a/docs/interop/conf.py b/docs/interop/conf.py
index b0f322207c..b3cda17042 100644
--- a/docs/interop/conf.py
+++ b/docs/interop/conf.py
@@ -27,5 +27,8 @@ man_pages = [
[], 1),
('virtfs-proxy-helper', 'virtfs-proxy-helper',
u'QEMU 9p virtfs proxy filesystem helper',
- ['M. Mohan Kumar'], 1)
+ ['M. Mohan Kumar'], 1),
+ ('virtiofsd', 'virtiofsd', u'QEMU virtio-fs shared file system daemon',
+ ['Stefan Hajnoczi <stefanha@redhat.com>',
+ 'Masayoshi Mizuma <m.mizuma@jp.fujitsu.com>'], 1),
]
diff --git a/docs/interop/index.rst b/docs/interop/index.rst
index 3b763b1eeb..e8455b4270 100644
--- a/docs/interop/index.rst
+++ b/docs/interop/index.rst
@@ -24,3 +24,4 @@ Contents:
vhost-user
vhost-user-gpu
virtfs-proxy-helper
+ virtiofsd
diff --git a/docs/interop/virtiofsd.rst b/docs/interop/virtiofsd.rst
new file mode 100644
index 0000000000..51a657ac0c
--- /dev/null
+++ b/docs/interop/virtiofsd.rst
@@ -0,0 +1,123 @@
+QEMU virtio-fs shared file system daemon
+========================================
+
+Synopsis
+--------
+
+**virtiofsd** [*OPTIONS*]
+
+Description
+-----------
+
+Share a host directory tree with a guest through a virtio-fs device. This
+program is a vhost-user backend that implements the virtio-fs device. Each
+virtio-fs device instance requires its own virtiofsd process.
+
+This program is designed to work with QEMU's ``--device vhost-user-fs-pci``
+but should work with any virtual machine monitor (VMM) that supports
+vhost-user. See the Examples section below.
+
+This program must be run as the root user. Upon startup the program will
+switch into a new file system namespace with the shared directory tree as its
+root. This prevents "file system escapes" due to symlinks and other file
+system objects that might lead to files outside the shared directory. The
+program also sandboxes itself using seccomp(2) to prevent ptrace(2) and other
+vectors that could allow an attacker to compromise the system after gaining
+control of the virtiofsd process.
+
+Options
+-------
+
+.. program:: virtiofsd
+
+.. option:: -h, --help
+
+ Print help.
+
+.. option:: -V, --version
+
+ Print version.
+
+.. option:: -d
+
+ Enable debug output.
+
+.. option:: --syslog
+
+ Print log messages to syslog instead of stderr.
+
+.. option:: -o OPTION
+
+ * debug -
+ Enable debug output.
+
+ * flock|no_flock -
+ Enable/disable flock. The default is ``no_flock``.
+
+ * log_level=LEVEL -
+ Print only log messages matching LEVEL or more severe. LEVEL is one of
+ ``err``, ``warn``, ``info``, or ``debug``. The default is ``info``.
+
+ * norace -
+ Disable racy fallback. The default is false.
+
+ * posix_lock|no_posix_lock -
+ Enable/disable remote POSIX locks. The default is ``posix_lock``.
+
+ * readdirplus|no_readdirplus -
+ Enable/disable readdirplus. The default is ``readdirplus``.
+
+ * source=PATH -
+ Share host directory tree located at PATH. This option is required.
+
+ * timeout=TIMEOUT -
+ I/O timeout in seconds. The default depends on cache= option.
+
+ * vhost_user_socket=PATH -
+ Listen on vhost-user UNIX domain socket at PATH.
+
+ * writeback|no_writeback -
+ Enable/disable writeback cache. The cache alows the FUSE client to buffer
+ and merge write requests. The default is ``no_writeback``.
+
+ * xattr|no_xattr -
+ Enable/disable extended attributes (xattr) on files and directories. The
+ default is ``no_xattr``.
+
+.. option:: --socket-path=PATH
+
+ Listen on vhost-user UNIX domain socket at PATH.
+
+.. option:: --fd=FDNUM
+
+ Accept connections from vhost-user UNIX domain socket file descriptor FDNUM.
+ The file descriptor must already be listening for connections.
+
+.. option:: --thread-pool-size=NUM
+
+ Restrict the number of worker threads per request queue to NUM. The default
+ is 64.
+
+.. option:: --cache=none|auto|always
+
+ Select the desired trade-off between coherency and performance. ``none``
+ forbids the FUSE client from caching to achieve best coherency at the cost of
+ performance. ``auto`` acts similar to NFS with a 1 second metadata cache
+ timeout. ``always`` sets a long cache lifetime at the expense of coherency.
+
+Examples
+--------
+
+Export ``/var/lib/fs/vm001/`` on vhost-user UNIX domain socket
+``/var/run/vm001-vhost-fs.sock``:
+
+::
+
+ host# virtiofsd --socket-path=/var/run/vm001-vhost-fs.sock -o source=/var/lib/fs/vm001
+ host# qemu-system-x86_64 \
+ -chardev socket,id=char0,path=/var/run/vm001-vhost-fs.sock \
+ -device vhost-user-fs-pci,chardev=char0,tag=myfs \
+ -object memory-backend-file,id=mem,size=4G,mem-path=/dev/shm,share=on \
+ -numa node,memdev=mem \
+ ...
+ guest# mount -t virtio_fs myfs /mnt
--
2.24.1
^ permalink raw reply related [flat|nested] 6+ messages in thread
* Re: [PATCH] docs: add virtiofsd(1) man page
2020-01-27 16:25 [PATCH] docs: add virtiofsd(1) man page Stefan Hajnoczi
@ 2020-01-27 16:55 ` Liam Merwick
2020-01-29 10:43 ` Stefan Hajnoczi
2020-01-29 15:39 ` Christophe de Dinechin
2020-01-30 14:29 ` Peter Maydell
2 siblings, 1 reply; 6+ messages in thread
From: Liam Merwick @ 2020-01-27 16:55 UTC (permalink / raw)
To: Stefan Hajnoczi, qemu-devel; +Cc: Peter Maydell, Dr. David Alan Gilbert
On 27/01/2020 16:25, Stefan Hajnoczi wrote:
> Document the virtiofsd(1) program and its command-line options. This
> man page is a rST conversion of the original texi documentation that I
> wrote.
>
> Signed-off-by: Stefan Hajnoczi <stefanha@redhat.com>
> ---
> Based-on: 20200124162606.8787-1-peter.maydell@linaro.org
> ("[PATCH v2 0/8] qemu-img, qemu-trace-stap, virtfs-proxy-helper: convert to rST")
> ---
> MAINTAINERS | 1 +
> Makefile | 8 ++-
> docs/interop/conf.py | 5 +-
> docs/interop/index.rst | 1 +
> docs/interop/virtiofsd.rst | 123 +++++++++++++++++++++++++++++++++++++
> 5 files changed, 136 insertions(+), 2 deletions(-)
> create mode 100644 docs/interop/virtiofsd.rst
>
[ ... deleted ...]
> +
> +Examples
> +--------
> +
> +Export ``/var/lib/fs/vm001/`` on vhost-user UNIX domain socket
> +``/var/run/vm001-vhost-fs.sock``:
> +
> +::
> +
> + host# virtiofsd --socket-path=/var/run/vm001-vhost-fs.sock -o source=/var/lib/fs/vm001
> + host# qemu-system-x86_64 \
> + -chardev socket,id=char0,path=/var/run/vm001-vhost-fs.sock \
> + -device vhost-user-fs-pci,chardev=char0,tag=myfs \
> + -object memory-backend-file,id=mem,size=4G,mem-path=/dev/shm,share=on \
> + -numa node,memdev=mem \
> + ...
> + guest# mount -t virtio_fs myfs /mnt
>
This should be 'mount -t virtiofs myfs /mnt' like on
https://virtio-fs.gitlab.io/howto-qemu.html
otherwise
Reviewed-by: Liam Merwick <liam.merwick@oracle.com>
^ permalink raw reply [flat|nested] 6+ messages in thread
* Re: [PATCH] docs: add virtiofsd(1) man page
2020-01-27 16:55 ` Liam Merwick
@ 2020-01-29 10:43 ` Stefan Hajnoczi
0 siblings, 0 replies; 6+ messages in thread
From: Stefan Hajnoczi @ 2020-01-29 10:43 UTC (permalink / raw)
To: Liam Merwick; +Cc: Peter Maydell, qemu-devel, Dr. David Alan Gilbert
[-- Attachment #1: Type: text/plain, Size: 1753 bytes --]
On Mon, Jan 27, 2020 at 04:55:12PM +0000, Liam Merwick wrote:
> On 27/01/2020 16:25, Stefan Hajnoczi wrote:
> > Document the virtiofsd(1) program and its command-line options. This
> > man page is a rST conversion of the original texi documentation that I
> > wrote.
> >
> > Signed-off-by: Stefan Hajnoczi <stefanha@redhat.com>
> > ---
> > Based-on: 20200124162606.8787-1-peter.maydell@linaro.org
> > ("[PATCH v2 0/8] qemu-img, qemu-trace-stap, virtfs-proxy-helper: convert to rST")
> > ---
> > MAINTAINERS | 1 +
> > Makefile | 8 ++-
> > docs/interop/conf.py | 5 +-
> > docs/interop/index.rst | 1 +
> > docs/interop/virtiofsd.rst | 123 +++++++++++++++++++++++++++++++++++++
> > 5 files changed, 136 insertions(+), 2 deletions(-)
> > create mode 100644 docs/interop/virtiofsd.rst
> >
>
> [ ... deleted ...]
>
> > +
> > +Examples
> > +--------
> > +
> > +Export ``/var/lib/fs/vm001/`` on vhost-user UNIX domain socket
> > +``/var/run/vm001-vhost-fs.sock``:
> > +
> > +::
> > +
> > + host# virtiofsd --socket-path=/var/run/vm001-vhost-fs.sock -o source=/var/lib/fs/vm001
> > + host# qemu-system-x86_64 \
> > + -chardev socket,id=char0,path=/var/run/vm001-vhost-fs.sock \
> > + -device vhost-user-fs-pci,chardev=char0,tag=myfs \
> > + -object memory-backend-file,id=mem,size=4G,mem-path=/dev/shm,share=on \
> > + -numa node,memdev=mem \
> > + ...
> > + guest# mount -t virtio_fs myfs /mnt
> >
>
> This should be 'mount -t virtiofs myfs /mnt' like on
> https://virtio-fs.gitlab.io/howto-qemu.html
>
> otherwise
>
> Reviewed-by: Liam Merwick <liam.merwick@oracle.com>
Will fix, thanks!
Stefan
[-- Attachment #2: signature.asc --]
[-- Type: application/pgp-signature, Size: 488 bytes --]
^ permalink raw reply [flat|nested] 6+ messages in thread
* Re: [PATCH] docs: add virtiofsd(1) man page
2020-01-27 16:25 [PATCH] docs: add virtiofsd(1) man page Stefan Hajnoczi
2020-01-27 16:55 ` Liam Merwick
@ 2020-01-29 15:39 ` Christophe de Dinechin
2020-01-29 15:54 ` Marc-André Lureau
2020-01-30 14:29 ` Peter Maydell
2 siblings, 1 reply; 6+ messages in thread
From: Christophe de Dinechin @ 2020-01-29 15:39 UTC (permalink / raw)
To: qemu-devel; +Cc: Peter Maydell, Dr. David Alan Gilbert, Stefan Hajnoczi
Stefan Hajnoczi writes:
> Document the virtiofsd(1) program and its command-line options. This
> man page is a rST conversion of the original texi documentation that I
> wrote.
>
> Signed-off-by: Stefan Hajnoczi <stefanha@redhat.com>
> ---
> Based-on: 20200124162606.8787-1-peter.maydell@linaro.org
> ("[PATCH v2 0/8] qemu-img, qemu-trace-stap, virtfs-proxy-helper: convert to rST")
> ---
> MAINTAINERS | 1 +
> Makefile | 8 ++-
> docs/interop/conf.py | 5 +-
> docs/interop/index.rst | 1 +
> docs/interop/virtiofsd.rst | 123 +++++++++++++++++++++++++++++++++++++
> 5 files changed, 136 insertions(+), 2 deletions(-)
> create mode 100644 docs/interop/virtiofsd.rst
>
> diff --git a/MAINTAINERS b/MAINTAINERS
> index 83fb32b860..1da0709129 100644
> --- a/MAINTAINERS
> +++ b/MAINTAINERS
> @@ -1603,6 +1603,7 @@ S: Supported
> F: tools/virtiofsd/*
> F: hw/virtio/vhost-user-fs*
> F: include/hw/virtio/vhost-user-fs.h
> +F: docs/interop/virtiofsd.rst
>
> virtio-input
> M: Gerd Hoffmann <kraxel@redhat.com>
> diff --git a/Makefile b/Makefile
> index 539f9ef079..ecd26044bc 100644
> --- a/Makefile
> +++ b/Makefile
> @@ -348,6 +348,9 @@ DOCS=qemu-doc.html qemu-doc.txt qemu.1
> DOCS+=$(MANUAL_BUILDDIR)/interop/qemu-img.1
> DOCS+=$(MANUAL_BUILDDIR)/interop/qemu-nbd.8
> DOCS+=$(MANUAL_BUILDDIR)/interop/qemu-ga.8
> +ifeq ($(CONFIG_LINUX)$(CONFIG_SECCOMP)$(CONFIG_LIBCAP_NG),yyy)
> +DOCS+=$(MANUAL_BUILDDIR)/interop/virtiofsd.1
> +endif
> DOCS+=$(MANUAL_BUILDDIR)/system/qemu-block-drivers.7
> DOCS+=docs/interop/qemu-qmp-ref.html docs/interop/qemu-qmp-ref.txt docs/interop/qemu-qmp-ref.7
> DOCS+=docs/interop/qemu-ga-ref.html docs/interop/qemu-ga-ref.txt docs/interop/qemu-ga-ref.7
> @@ -861,6 +864,9 @@ ifdef CONFIG_VIRTFS
> $(INSTALL_DIR) "$(DESTDIR)$(mandir)/man1"
> $(INSTALL_DATA) $(MANUAL_BUILDDIR)/interop/virtfs-proxy-helper.1 "$(DESTDIR)$(mandir)/man1"
> endif
> +ifeq ($(CONFIG_LINUX)$(CONFIG_SECCOMP)$(CONFIG_LIBCAP_NG),yyy)
> + $(INSTALL_DATA) docs/interop/virtiofsd.1 "$(DESTDIR)$(mandir)/man1"
> +endif
>
> install-datadir:
> $(INSTALL_DIR) "$(DESTDIR)$(qemu_datadir)"
> @@ -1052,7 +1058,7 @@ $(MANUAL_BUILDDIR)/system/index.html: $(call manual-deps,system)
>
> $(call define-manpage-rule,interop,\
> qemu-ga.8 qemu-img.1 qemu-nbd.8 qemu-trace-stap.1\
> - virtfs-proxy-helper.1,\
> + virtiofsd.1 virtfs-proxy-helper.1,\
> $(SRC_PATH/qemu-img-cmds.hx))
>
> $(call define-manpage-rule,system,qemu-block-drivers.7)
> diff --git a/docs/interop/conf.py b/docs/interop/conf.py
> index b0f322207c..b3cda17042 100644
> --- a/docs/interop/conf.py
> +++ b/docs/interop/conf.py
> @@ -27,5 +27,8 @@ man_pages = [
> [], 1),
> ('virtfs-proxy-helper', 'virtfs-proxy-helper',
> u'QEMU 9p virtfs proxy filesystem helper',
> - ['M. Mohan Kumar'], 1)
> + ['M. Mohan Kumar'], 1),
> + ('virtiofsd', 'virtiofsd', u'QEMU virtio-fs shared file system daemon',
> + ['Stefan Hajnoczi <stefanha@redhat.com>',
> + 'Masayoshi Mizuma <m.mizuma@jp.fujitsu.com>'], 1),
> ]
> diff --git a/docs/interop/index.rst b/docs/interop/index.rst
> index 3b763b1eeb..e8455b4270 100644
> --- a/docs/interop/index.rst
> +++ b/docs/interop/index.rst
> @@ -24,3 +24,4 @@ Contents:
> vhost-user
> vhost-user-gpu
> virtfs-proxy-helper
> + virtiofsd
> diff --git a/docs/interop/virtiofsd.rst b/docs/interop/virtiofsd.rst
> new file mode 100644
> index 0000000000..51a657ac0c
> --- /dev/null
> +++ b/docs/interop/virtiofsd.rst
> @@ -0,0 +1,123 @@
> +QEMU virtio-fs shared file system daemon
> +========================================
> +
> +Synopsis
> +--------
> +
> +**virtiofsd** [*OPTIONS*]
> +
> +Description
> +-----------
> +
> +Share a host directory tree with a guest through a virtio-fs device. This
> +program is a vhost-user backend that implements the virtio-fs device. Each
> +virtio-fs device instance requires its own virtiofsd process.
> +
> +This program is designed to work with QEMU's ``--device vhost-user-fs-pci``
> +but should work with any virtual machine monitor (VMM) that supports
> +vhost-user. See the Examples section below.
> +
> +This program must be run as the root user. Upon startup the program will
> +switch into a new file system namespace with the shared directory tree as its
> +root. This prevents "file system escapes" due to symlinks and other file
> +system objects that might lead to files outside the shared directory. The
> +program also sandboxes itself using seccomp(2) to prevent ptrace(2) and other
> +vectors that could allow an attacker to compromise the system after gaining
> +control of the virtiofsd process.
> +
> +Options
> +-------
> +
> +.. program:: virtiofsd
> +
> +.. option:: -h, --help
> +
> + Print help.
> +
> +.. option:: -V, --version
> +
> + Print version.
> +
> +.. option:: -d
> +
> + Enable debug output.
> +
> +.. option:: --syslog
> +
> + Print log messages to syslog instead of stderr.
> +
> +.. option:: -o OPTION
> +
> + * debug -
> + Enable debug output.
> +
> + * flock|no_flock -
> + Enable/disable flock. The default is ``no_flock``.
> +
> + * log_level=LEVEL -
> + Print only log messages matching LEVEL or more severe. LEVEL is one of
> + ``err``, ``warn``, ``info``, or ``debug``. The default is ``info``.
> +
> + * norace -
> + Disable racy fallback. The default is false.
It's not a problem with the doc, but seeing it exposes that options are
not overly consistent in their spelling. "norace" has no _ but no_flock
does (as does no_writeback).
> +
> + * posix_lock|no_posix_lock -
> + Enable/disable remote POSIX locks. The default is ``posix_lock``.
> +
> + * readdirplus|no_readdirplus -
> + Enable/disable readdirplus. The default is ``readdirplus``.
> +
> + * source=PATH -
> + Share host directory tree located at PATH. This option is required.
> +
> + * timeout=TIMEOUT -
> + I/O timeout in seconds. The default depends on cache= option.
> +
> + * vhost_user_socket=PATH -
> + Listen on vhost-user UNIX domain socket at PATH.
I believe that this option has been removed.
> +
> + * writeback|no_writeback -
> + Enable/disable writeback cache. The cache alows the FUSE client to buffer
> + and merge write requests. The default is ``no_writeback``.
> +
> + * xattr|no_xattr -
> + Enable/disable extended attributes (xattr) on files and directories. The
> + default is ``no_xattr``.
> +
> +.. option:: --socket-path=PATH
> +
> + Listen on vhost-user UNIX domain socket at PATH.
Not related to the doc, but this looks like an option that might deserve
a short form.
> +
> +.. option:: --fd=FDNUM
> +
> + Accept connections from vhost-user UNIX domain socket file descriptor FDNUM.
> + The file descriptor must already be listening for connections.
Not directly related to the manual itself, but any reason why it's
"socket-path" and "fd" (as opposed e.g. to "socket-path" and
"socket-fd"?
> +
> +.. option:: --thread-pool-size=NUM
> +
> + Restrict the number of worker threads per request queue to NUM. The default
> + is 64.
> +
> +.. option:: --cache=none|auto|always
> +
> + Select the desired trade-off between coherency and performance. ``none``
> + forbids the FUSE client from caching to achieve best coherency at the cost of
> + performance. ``auto`` acts similar to NFS with a 1 second metadata cache
> + timeout. ``always`` sets a long cache lifetime at the expense of coherency.
> +
> +Examples
> +--------
> +
> +Export ``/var/lib/fs/vm001/`` on vhost-user UNIX domain socket
> +``/var/run/vm001-vhost-fs.sock``:
> +
> +::
> +
> + host# virtiofsd --socket-path=/var/run/vm001-vhost-fs.sock -o source=/var/lib/fs/vm001
> + host# qemu-system-x86_64 \
> + -chardev socket,id=char0,path=/var/run/vm001-vhost-fs.sock \
> + -device vhost-user-fs-pci,chardev=char0,tag=myfs \
> + -object memory-backend-file,id=mem,size=4G,mem-path=/dev/shm,share=on \
> + -numa node,memdev=mem \
> + ...
> + guest# mount -t virtio_fs myfs /mnt
--
Cheers,
Christophe de Dinechin (IRC c3d)
^ permalink raw reply [flat|nested] 6+ messages in thread
* Re: [PATCH] docs: add virtiofsd(1) man page
2020-01-29 15:39 ` Christophe de Dinechin
@ 2020-01-29 15:54 ` Marc-André Lureau
0 siblings, 0 replies; 6+ messages in thread
From: Marc-André Lureau @ 2020-01-29 15:54 UTC (permalink / raw)
To: Christophe de Dinechin
Cc: Peter Maydell, QEMU, Stefan Hajnoczi, Dr. David Alan Gilbert
Hi
On Wed, Jan 29, 2020 at 4:40 PM Christophe de Dinechin
<dinechin@redhat.com> wrote:
>
>
> Stefan Hajnoczi writes:
>
> > Document the virtiofsd(1) program and its command-line options. This
> > man page is a rST conversion of the original texi documentation that I
> > wrote.
> >
> > Signed-off-by: Stefan Hajnoczi <stefanha@redhat.com>
> > ---
> > Based-on: 20200124162606.8787-1-peter.maydell@linaro.org
> > ("[PATCH v2 0/8] qemu-img, qemu-trace-stap, virtfs-proxy-helper: convert to rST")
> > ---
> > MAINTAINERS | 1 +
> > Makefile | 8 ++-
> > docs/interop/conf.py | 5 +-
> > docs/interop/index.rst | 1 +
> > docs/interop/virtiofsd.rst | 123 +++++++++++++++++++++++++++++++++++++
> > 5 files changed, 136 insertions(+), 2 deletions(-)
> > create mode 100644 docs/interop/virtiofsd.rst
> >
> > diff --git a/MAINTAINERS b/MAINTAINERS
> > index 83fb32b860..1da0709129 100644
> > --- a/MAINTAINERS
> > +++ b/MAINTAINERS
> > @@ -1603,6 +1603,7 @@ S: Supported
> > F: tools/virtiofsd/*
> > F: hw/virtio/vhost-user-fs*
> > F: include/hw/virtio/vhost-user-fs.h
> > +F: docs/interop/virtiofsd.rst
> >
> > virtio-input
> > M: Gerd Hoffmann <kraxel@redhat.com>
> > diff --git a/Makefile b/Makefile
> > index 539f9ef079..ecd26044bc 100644
> > --- a/Makefile
> > +++ b/Makefile
> > @@ -348,6 +348,9 @@ DOCS=qemu-doc.html qemu-doc.txt qemu.1
> > DOCS+=$(MANUAL_BUILDDIR)/interop/qemu-img.1
> > DOCS+=$(MANUAL_BUILDDIR)/interop/qemu-nbd.8
> > DOCS+=$(MANUAL_BUILDDIR)/interop/qemu-ga.8
> > +ifeq ($(CONFIG_LINUX)$(CONFIG_SECCOMP)$(CONFIG_LIBCAP_NG),yyy)
> > +DOCS+=$(MANUAL_BUILDDIR)/interop/virtiofsd.1
> > +endif
> > DOCS+=$(MANUAL_BUILDDIR)/system/qemu-block-drivers.7
> > DOCS+=docs/interop/qemu-qmp-ref.html docs/interop/qemu-qmp-ref.txt docs/interop/qemu-qmp-ref.7
> > DOCS+=docs/interop/qemu-ga-ref.html docs/interop/qemu-ga-ref.txt docs/interop/qemu-ga-ref.7
> > @@ -861,6 +864,9 @@ ifdef CONFIG_VIRTFS
> > $(INSTALL_DIR) "$(DESTDIR)$(mandir)/man1"
> > $(INSTALL_DATA) $(MANUAL_BUILDDIR)/interop/virtfs-proxy-helper.1 "$(DESTDIR)$(mandir)/man1"
> > endif
> > +ifeq ($(CONFIG_LINUX)$(CONFIG_SECCOMP)$(CONFIG_LIBCAP_NG),yyy)
> > + $(INSTALL_DATA) docs/interop/virtiofsd.1 "$(DESTDIR)$(mandir)/man1"
> > +endif
> >
> > install-datadir:
> > $(INSTALL_DIR) "$(DESTDIR)$(qemu_datadir)"
> > @@ -1052,7 +1058,7 @@ $(MANUAL_BUILDDIR)/system/index.html: $(call manual-deps,system)
> >
> > $(call define-manpage-rule,interop,\
> > qemu-ga.8 qemu-img.1 qemu-nbd.8 qemu-trace-stap.1\
> > - virtfs-proxy-helper.1,\
> > + virtiofsd.1 virtfs-proxy-helper.1,\
> > $(SRC_PATH/qemu-img-cmds.hx))
> >
> > $(call define-manpage-rule,system,qemu-block-drivers.7)
> > diff --git a/docs/interop/conf.py b/docs/interop/conf.py
> > index b0f322207c..b3cda17042 100644
> > --- a/docs/interop/conf.py
> > +++ b/docs/interop/conf.py
> > @@ -27,5 +27,8 @@ man_pages = [
> > [], 1),
> > ('virtfs-proxy-helper', 'virtfs-proxy-helper',
> > u'QEMU 9p virtfs proxy filesystem helper',
> > - ['M. Mohan Kumar'], 1)
> > + ['M. Mohan Kumar'], 1),
> > + ('virtiofsd', 'virtiofsd', u'QEMU virtio-fs shared file system daemon',
> > + ['Stefan Hajnoczi <stefanha@redhat.com>',
> > + 'Masayoshi Mizuma <m.mizuma@jp.fujitsu.com>'], 1),
> > ]
> > diff --git a/docs/interop/index.rst b/docs/interop/index.rst
> > index 3b763b1eeb..e8455b4270 100644
> > --- a/docs/interop/index.rst
> > +++ b/docs/interop/index.rst
> > @@ -24,3 +24,4 @@ Contents:
> > vhost-user
> > vhost-user-gpu
> > virtfs-proxy-helper
> > + virtiofsd
> > diff --git a/docs/interop/virtiofsd.rst b/docs/interop/virtiofsd.rst
> > new file mode 100644
> > index 0000000000..51a657ac0c
> > --- /dev/null
> > +++ b/docs/interop/virtiofsd.rst
> > @@ -0,0 +1,123 @@
> > +QEMU virtio-fs shared file system daemon
> > +========================================
> > +
> > +Synopsis
> > +--------
> > +
> > +**virtiofsd** [*OPTIONS*]
> > +
> > +Description
> > +-----------
> > +
> > +Share a host directory tree with a guest through a virtio-fs device. This
> > +program is a vhost-user backend that implements the virtio-fs device. Each
> > +virtio-fs device instance requires its own virtiofsd process.
> > +
> > +This program is designed to work with QEMU's ``--device vhost-user-fs-pci``
> > +but should work with any virtual machine monitor (VMM) that supports
> > +vhost-user. See the Examples section below.
> > +
> > +This program must be run as the root user. Upon startup the program will
> > +switch into a new file system namespace with the shared directory tree as its
> > +root. This prevents "file system escapes" due to symlinks and other file
> > +system objects that might lead to files outside the shared directory. The
> > +program also sandboxes itself using seccomp(2) to prevent ptrace(2) and other
> > +vectors that could allow an attacker to compromise the system after gaining
> > +control of the virtiofsd process.
> > +
> > +Options
> > +-------
> > +
> > +.. program:: virtiofsd
> > +
> > +.. option:: -h, --help
> > +
> > + Print help.
> > +
> > +.. option:: -V, --version
> > +
> > + Print version.
> > +
> > +.. option:: -d
> > +
> > + Enable debug output.
> > +
> > +.. option:: --syslog
> > +
> > + Print log messages to syslog instead of stderr.
> > +
> > +.. option:: -o OPTION
> > +
> > + * debug -
> > + Enable debug output.
> > +
> > + * flock|no_flock -
> > + Enable/disable flock. The default is ``no_flock``.
> > +
> > + * log_level=LEVEL -
> > + Print only log messages matching LEVEL or more severe. LEVEL is one of
> > + ``err``, ``warn``, ``info``, or ``debug``. The default is ``info``.
> > +
> > + * norace -
> > + Disable racy fallback. The default is false.
>
> It's not a problem with the doc, but seeing it exposes that options are
> not overly consistent in their spelling. "norace" has no _ but no_flock
> does (as does no_writeback).
>
> > +
> > + * posix_lock|no_posix_lock -
> > + Enable/disable remote POSIX locks. The default is ``posix_lock``.
> > +
> > + * readdirplus|no_readdirplus -
> > + Enable/disable readdirplus. The default is ``readdirplus``.
> > +
> > + * source=PATH -
> > + Share host directory tree located at PATH. This option is required.
> > +
> > + * timeout=TIMEOUT -
> > + I/O timeout in seconds. The default depends on cache= option.
> > +
> > + * vhost_user_socket=PATH -
> > + Listen on vhost-user UNIX domain socket at PATH.
>
> I believe that this option has been removed.
>
> > +
> > + * writeback|no_writeback -
> > + Enable/disable writeback cache. The cache alows the FUSE client to buffer
> > + and merge write requests. The default is ``no_writeback``.
> > +
> > + * xattr|no_xattr -
> > + Enable/disable extended attributes (xattr) on files and directories. The
> > + default is ``no_xattr``.
> > +
> > +.. option:: --socket-path=PATH
> > +
> > + Listen on vhost-user UNIX domain socket at PATH.
>
> Not related to the doc, but this looks like an option that might deserve
> a short form.
>
> > +
> > +.. option:: --fd=FDNUM
> > +
> > + Accept connections from vhost-user UNIX domain socket file descriptor FDNUM.
> > + The file descriptor must already be listening for connections.
>
> Not directly related to the manual itself, but any reason why it's
> "socket-path" and "fd" (as opposed e.g. to "socket-path" and
> "socket-fd"?
This dates back from commit 482580a658df58f5d9f91a87d957660637d59432
("vhost-user: define conventions for vhost-user backends"). I agree it
lacks some consistency. It's a bit late to change it though
>
>
> > +
> > +.. option:: --thread-pool-size=NUM
> > +
> > + Restrict the number of worker threads per request queue to NUM. The default
> > + is 64.
> > +
> > +.. option:: --cache=none|auto|always
> > +
> > + Select the desired trade-off between coherency and performance. ``none``
> > + forbids the FUSE client from caching to achieve best coherency at the cost of
> > + performance. ``auto`` acts similar to NFS with a 1 second metadata cache
> > + timeout. ``always`` sets a long cache lifetime at the expense of coherency.
> > +
> > +Examples
> > +--------
> > +
> > +Export ``/var/lib/fs/vm001/`` on vhost-user UNIX domain socket
> > +``/var/run/vm001-vhost-fs.sock``:
> > +
> > +::
> > +
> > + host# virtiofsd --socket-path=/var/run/vm001-vhost-fs.sock -o source=/var/lib/fs/vm001
> > + host# qemu-system-x86_64 \
> > + -chardev socket,id=char0,path=/var/run/vm001-vhost-fs.sock \
> > + -device vhost-user-fs-pci,chardev=char0,tag=myfs \
> > + -object memory-backend-file,id=mem,size=4G,mem-path=/dev/shm,share=on \
> > + -numa node,memdev=mem \
> > + ...
> > + guest# mount -t virtio_fs myfs /mnt
>
>
> --
> Cheers,
> Christophe de Dinechin (IRC c3d)
>
>
--
Marc-André Lureau
^ permalink raw reply [flat|nested] 6+ messages in thread
* Re: [PATCH] docs: add virtiofsd(1) man page
2020-01-27 16:25 [PATCH] docs: add virtiofsd(1) man page Stefan Hajnoczi
2020-01-27 16:55 ` Liam Merwick
2020-01-29 15:39 ` Christophe de Dinechin
@ 2020-01-30 14:29 ` Peter Maydell
2 siblings, 0 replies; 6+ messages in thread
From: Peter Maydell @ 2020-01-30 14:29 UTC (permalink / raw)
To: Stefan Hajnoczi; +Cc: QEMU Developers, Dr. David Alan Gilbert
On Mon, 27 Jan 2020 at 16:25, Stefan Hajnoczi <stefanha@redhat.com> wrote:
>
> Document the virtiofsd(1) program and its command-line options. This
> man page is a rST conversion of the original texi documentation that I
> wrote.
>
> Signed-off-by: Stefan Hajnoczi <stefanha@redhat.com>
> ---
> Based-on: 20200124162606.8787-1-peter.maydell@linaro.org
> ("[PATCH v2 0/8] qemu-img, qemu-trace-stap, virtfs-proxy-helper: convert to rST")
> ---
> MAINTAINERS | 1 +
> Makefile | 8 ++-
> docs/interop/conf.py | 5 +-
> docs/interop/index.rst | 1 +
> docs/interop/virtiofsd.rst | 123 +++++++++++++++++++++++++++++++++++++
> 5 files changed, 136 insertions(+), 2 deletions(-)
> create mode 100644 docs/interop/virtiofsd.rst
>
> diff --git a/MAINTAINERS b/MAINTAINERS
> index 83fb32b860..1da0709129 100644
> --- a/MAINTAINERS
> +++ b/MAINTAINERS
> @@ -1603,6 +1603,7 @@ S: Supported
> F: tools/virtiofsd/*
> F: hw/virtio/vhost-user-fs*
> F: include/hw/virtio/vhost-user-fs.h
> +F: docs/interop/virtiofsd.rst
>
> virtio-input
> M: Gerd Hoffmann <kraxel@redhat.com>
> diff --git a/Makefile b/Makefile
> index 539f9ef079..ecd26044bc 100644
> --- a/Makefile
> +++ b/Makefile
> @@ -348,6 +348,9 @@ DOCS=qemu-doc.html qemu-doc.txt qemu.1
> DOCS+=$(MANUAL_BUILDDIR)/interop/qemu-img.1
> DOCS+=$(MANUAL_BUILDDIR)/interop/qemu-nbd.8
> DOCS+=$(MANUAL_BUILDDIR)/interop/qemu-ga.8
> +ifeq ($(CONFIG_LINUX)$(CONFIG_SECCOMP)$(CONFIG_LIBCAP_NG),yyy)
> +DOCS+=$(MANUAL_BUILDDIR)/interop/virtiofsd.1
> +endif
> DOCS+=$(MANUAL_BUILDDIR)/system/qemu-block-drivers.7
> DOCS+=docs/interop/qemu-qmp-ref.html docs/interop/qemu-qmp-ref.txt docs/interop/qemu-qmp-ref.7
> DOCS+=docs/interop/qemu-ga-ref.html docs/interop/qemu-ga-ref.txt docs/interop/qemu-ga-ref.7
> @@ -861,6 +864,9 @@ ifdef CONFIG_VIRTFS
> $(INSTALL_DIR) "$(DESTDIR)$(mandir)/man1"
> $(INSTALL_DATA) $(MANUAL_BUILDDIR)/interop/virtfs-proxy-helper.1 "$(DESTDIR)$(mandir)/man1"
> endif
> +ifeq ($(CONFIG_LINUX)$(CONFIG_SECCOMP)$(CONFIG_LIBCAP_NG),yyy)
> + $(INSTALL_DATA) docs/interop/virtiofsd.1 "$(DESTDIR)$(mandir)/man1"
Thie first argument should be "$(MANUAL_BUILDDIR)/docs/interop/virtiofsd.1",
otherwise build-and-install for an in-source-directory build
won't work.
> +endif
>
> install-datadir:
> $(INSTALL_DIR) "$(DESTDIR)$(qemu_datadir)"
> --- /dev/null
> +++ b/docs/interop/virtiofsd.rst
> @@ -0,0 +1,123 @@
> +QEMU virtio-fs shared file system daemon
> +========================================
> +
> +Synopsis
> +--------
> +
> +**virtiofsd** [*OPTIONS*]
> +
> +Description
> +-----------
> +
> +Share a host directory tree with a guest through a virtio-fs device. This
> +program is a vhost-user backend that implements the virtio-fs device. Each
> +virtio-fs device instance requires its own virtiofsd process.
> +
> +This program is designed to work with QEMU's ``--device vhost-user-fs-pci``
> +but should work with any virtual machine monitor (VMM) that supports
> +vhost-user. See the Examples section below.
> +
> +This program must be run as the root user. Upon startup the program will
> +switch into a new file system namespace with the shared directory tree as its
> +root. This prevents "file system escapes" due to symlinks and other file
> +system objects that might lead to files outside the shared directory. The
> +program also sandboxes itself using seccomp(2) to prevent ptrace(2) and other
You can mark these manpage references up with :manpage:`seccomp(2)` if
you like, though it only affects the formatting (bold in manpage, italics
in HTML), so I haven't been very consistent in using it in the manpages
I converted from texinfo.
Thanks for doing the conversion to rST format!
PS: do you (ie redhat downstream) want a macro to make it easy to
replace instances of 'qemu-system-x86_64' in the example command
lines, the way qemu-block-drivers.rst does, or do you not care
for this particular manpage? (If you do we should probably move
that to conf.py rather than proliferating it across rst files.)
thanks
-- PMM
^ permalink raw reply [flat|nested] 6+ messages in thread
end of thread, other threads:[~2020-01-30 14:30 UTC | newest]
Thread overview: 6+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2020-01-27 16:25 [PATCH] docs: add virtiofsd(1) man page Stefan Hajnoczi
2020-01-27 16:55 ` Liam Merwick
2020-01-29 10:43 ` Stefan Hajnoczi
2020-01-29 15:39 ` Christophe de Dinechin
2020-01-29 15:54 ` Marc-André Lureau
2020-01-30 14:29 ` Peter Maydell
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).