Re: [PATCH] bhyve: implement virtio-9p support

On Thu, Oct 08, 2020 at 05:06:16PM +0400, Roman Bogorodskiy wrote: > Recently virtio-9p support was added to bhyve. > > On the host side it looks this way: > > bhyve .... -s 25:0,virtio-9p,sharename=/path/to/shared/dir > > It could also have ",ro" suffix to make share read-only. > > In the Linux guest, this share is mounted with: > > mount -t 9p sharename /mnt/sharename > > In the guest user will see the same permissions and ownership > information for this directory as on the host. No uid/gid remapping is > supported, so those could resolve to wrong user or group names. > > The same applies to the other side: chowning/chmodding in the guest will > set specified ownership and permissions on the host. > > In libvirt domain XML it's modeled using the 'filesystem' element: > > <filesystem type='mount'> > <source dir='/path/to/shared/dir'/> > <target dir='sharename'/> > </filesystem> > diff --git a/tests/bhyvexml2argvdata/bhyvexml2argv-fs-9p-readonly.xml b/tests/bhyvexml2argvdata/bhyvexml2argv-fs-9p-readonly.xml > new file mode 100644 > index 0000000000..6341236654 > --- /dev/null > +++ b/tests/bhyvexml2argvdata/bhyvexml2argv-fs-9p-readonly.xml > @@ -0,0 +1,28 @@ > +<domain type='bhyve'> > + <name>bhyve</name> > + <uuid>df3be7e7-a104-11e3-aeb0-50e5492bd3dc</uuid> > + <memory>219136</memory> > + <vcpu>1</vcpu> > + <os> > + <type>hvm</type> > + </os> > + <devices> > + <disk type='file'> > + <driver name='file' type='raw'/> > + <source file='/tmp/freebsd.img'/> > + <target dev='hda' bus='sata'/> > + <address type='drive' controller='0' bus='0' target='2' unit='0'/> > + </disk> > + <interface type='bridge'> > + <mac address='52:54:00:b9:94:02'/> > + <model type='virtio'/> > + <source bridge="virbr0"/> > + <address type='pci' domain='0x0000' bus='0x00' slot='0x03' function='0x0'/> > + </interface> > + <filesystem> This is missing the type="mount" attribute which should be mandatory. It suggests we're not validating the type in the driver, before accessing the <source> element, which is dangerous. > + <source dir='/shared/dir'/> > + <target dir='shared_dir'/> > + <readonly/> > + </filesystem> > + </devices> > +</domain> The other demo XML files are the same.

Regards, Daniel -- |: https://berrange.com -o- https://www.flickr.com/photos/dberrange :| |: https://libvirt.org -o- https://fstop138.berrange.com :| |: https://entangle-photo.org -o- https://www.instagram.com/dberrange :|