]> git.ipfire.org Git - thirdparty/libvirt.git/commitdiff
qemuhotplugtest: Don't plug a SCSI disk at unit 7
authorMichal Privoznik <mprivozn@redhat.com>
Thu, 11 Apr 2019 14:23:38 +0000 (16:23 +0200)
committerMichal Privoznik <mprivozn@redhat.com>
Thu, 18 Apr 2019 15:08:33 +0000 (17:08 +0200)
Unit number 7 is kind of special. It's reserved for SCSI
controller. The comment in virDomainSCSIDriveAddressIsUsed()
summarizes that pretty nicely. Libvirt would never generate
such address.

Signed-off-by: Michal Privoznik <mprivozn@redhat.com>
Tested-by: Daniel Henrique Barboza <danielhb413@gmail.com>
Reviewed-by: Jim Fehlig <jfehlig@suse.com>
tests/qemuhotplugtest.c
tests/qemuhotplugtestdevices/qemuhotplug-disk-scsi-2.xml
tests/qemuhotplugtestdomains/qemuhotplug-base-without-scsi-controller-live+disk-scsi-2.xml

index 3eb97d886a164f6c4f1a62483064823570c034d1..fbf488c54cea8783552350083ce0ca054b5579af 100644 (file)
@@ -715,7 +715,7 @@ mymain(void)
                    "device_del", QMP_OK,
                    "human-monitor-command", HMP(""));
     DO_TEST_DETACH("base-with-scsi-controller-live", "disk-scsi-2", false, false,
-                   "device_del", QMP_DEVICE_DELETED("scsi3-0-5-7") QMP_OK,
+                   "device_del", QMP_DEVICE_DELETED("scsi3-0-5-6") QMP_OK,
                    "human-monitor-command", HMP(""));
 
     DO_TEST_ATTACH("base-live", "qemu-agent", false, true,
index 3a847fbda619ed294c2d80d7d44ba6ba1c6c0928..876afb182f3073ac366dbd0f22e245df6f39a17d 100644 (file)
@@ -2,7 +2,7 @@
   <driver name='qemu' type='raw' cache='none'/>
   <source file='/dev/null'/>
   <target dev='sdf' bus='scsi'/>
-  <address type='drive' controller='3' bus='0' target='5' unit='7'/>
+  <address type='drive' controller='3' bus='0' target='5' unit='6'/>
   <readonly/>
   <shareable/>
 </disk>
index d35fea6f5f775c10414b1ddd23bcc696d708959e..72b5174825b1997d54fe79d3397e312a3bcdc574 100644 (file)
@@ -26,8 +26,8 @@
       <target dev='sdf' bus='scsi'/>
       <readonly/>
       <shareable/>
-      <alias name='scsi3-0-5-7'/>
-      <address type='drive' controller='3' bus='0' target='5' unit='7'/>
+      <alias name='scsi3-0-5-6'/>
+      <address type='drive' controller='3' bus='0' target='5' unit='6'/>
     </disk>
     <controller type='usb' index='0'>
       <alias name='usb'/>