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

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



  Daniel P. Berrangé wrote:

> 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.

Hm, as I can see in the schema, type="mount" is default. That's what I
see in virDomainFSDefParseXML() @ src/conf/domain_conf.c as well.

I also check that in the driver, and there's a test for it:

tests/bhyvexml2argvdata/bhyvexml2argv-fs-9p-unsupported-type.xml

Are you referring to something different?

> 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 :|
> 

Roman Bogorodskiy

Attachment: signature.asc
Description: PGP signature


[Index of Archives]     [Virt Tools]     [Libvirt Users]     [Lib OS Info]     [Fedora Users]     [Fedora Desktop]     [Fedora SELinux]     [Big List of Linux Books]     [Yosemite News]     [KDE Users]     [Fedora Tools]

  Powered by Linux