Re: [PATCH v2 4/4] qemu: add support for 'multidevs' option

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

 



On Freitag, 27. März 2020 18:16:37 CEST Ján Tomko wrote:
> On a Friday in 2020, Christian Schoenebeck wrote:
> >This option prevents misbehaviours on guest if a qemu 9pfs export
> >contains multiple devices, due to the potential file ID collisions
> >this otherwise may cause.
> >
> >Signed-off-by: Christian Schoenebeck <qemu_oss@xxxxxxxxxxxxx>
> >---
> >
> > src/qemu/qemu_command.c |  7 +++++++
> > src/qemu/qemu_domain.c  | 12 ++++++++++++
> > 2 files changed, 19 insertions(+)
> 
> A change to qemuxml2argvtest is needed.
> 
> If you already added the XML file to qemuxml2argvdata in the previous
> commit, all you need to to to generate the output file is:
> * add a new DO_TEST_CAPS_LATEST line to qemuxml2argvtest.c

Looking at the existing test cases, it probably makes sense to extend 
qemuxml2argvdata/virtio-options.xml instead of adding a separate XML file, 
that is:

diff --git a/tests/qemuxml2argvdata/virtio-options.xml b/tests/
qemuxml2argvdata/virtio-options.xml
index dd9a4f4a01..7ea624dfad 100644
--- a/tests/qemuxml2argvdata/virtio-options.xml
+++ b/tests/qemuxml2argvdata/virtio-options.xml
@@ -47,6 +47,21 @@
       <target dir='fs2'/>
       <address type='pci' domain='0x0000' bus='0x00' slot='0x04' 
function='0x0'/>
     </filesystem>
+    <filesystem type='mount' accessmode='mapped' multidevs='remap'>
+      <source dir='/export/fs3'/>
+      <target dir='fs3'/>
+      <address type='pci' domain='0x0000' bus='0x00' slot='0x05' 
function='0x0'/>
+    </filesystem>
+    <filesystem type='mount' accessmode='mapped' multidevs='forbid'>
+      <source dir='/export/fs4'/>
+      <target dir='fs4'/>
+      <address type='pci' domain='0x0000' bus='0x00' slot='0x07' 
function='0x0'/>
+    </filesystem>
+    <filesystem type='mount' accessmode='mapped' multidevs='warn'>
+      <source dir='/export/fs5'/>
+      <target dir='fs5'/>
+      <address type='pci' domain='0x0000' bus='0x00' slot='0x13' 
function='0x0'/>
+    </filesystem>
     <interface type='user'>
       <mac address='52:54:56:58:5a:5c'/>
       <model type='virtio'/>

> * run the test with VIR_TEST_REGENERATE_OUTPUT=1

Ah, that's the one! :) I was already wondering whether other people are auto 
generating the files I manually changed so far.

I'll post a rebased v3 with the things discussed so far. Thanks Ján!

Best regards,
Christian Schoenebeck







[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