Re: [PATCH v3 1/4] tests: qemuxml2argv: Use scsi controller instead of ide in pseries

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

 



On Fri, Dec 01, 2017 at 07:31:29PM +0800, Lin Ma wrote:
Adding an IDE controller for a machinetype that has no built-in IDE
controller, Libvirt will log an error. Currently the machinetype list
which returns by qemuDomainMachineHasBuiltinIDE only includes 440fx,
malta, sun4u and g3beige.

Signed-off-by: Lin Ma <lma@xxxxxxxx>
---
tests/qemuxml2argvdata/qemuxml2argv-disk-floppy-pseries.args | 2 +-
tests/qemuxml2argvdata/qemuxml2argv-disk-floppy-pseries.xml  | 4 ++--
2 files changed, 3 insertions(+), 3 deletions(-)

diff --git a/tests/qemuxml2argvdata/qemuxml2argv-disk-floppy-pseries.args b/tests/qemuxml2argvdata/qemuxml2argv-disk-floppy-pseries.args
index 72a4183..44b5751 100644
--- a/tests/qemuxml2argvdata/qemuxml2argv-disk-floppy-pseries.args
+++ b/tests/qemuxml2argvdata/qemuxml2argv-disk-floppy-pseries.args
@@ -16,7 +16,7 @@ QEMU_AUDIO_DRV=none \
-no-acpi \
-boot c \
-usb \
--drive file=/dev/HostVG/QEMUGuest1,format=raw,if=ide,bus=0,unit=0 \
+-hda /dev/HostVG/QEMUGuest1 \
-drive file=/dev/fd0,format=raw,if=floppy,unit=0 \
-drive file=/tmp/firmware.img,format=raw,if=floppy,unit=1 \
-net none \

This change is bogus, current libvirt should never generate -hda

The .args file can be dropped for this test case, since it expects
failure.

Jan

Attachment: signature.asc
Description: Digital signature

--
libvir-list mailing list
libvir-list@xxxxxxxxxx
https://www.redhat.com/mailman/listinfo/libvir-list

[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