On Sun, Oct 08, 2017 at 03:26:13PM +0530, Kothapally Madhu Pavan wrote:
Signed-off-by: Kothapally Madhu Pavan <[email protected]> --- src/qemu/qemu_parse_command.c | 5 +++++ 1 file changed, 5 insertions(+)diff --git a/src/qemu/qemu_parse_command.c b/src/qemu/qemu_parse_command.c index 7c409b0..d5745ce 100644 --- a/src/qemu/qemu_parse_command.c +++ b/src/qemu/qemu_parse_command.c @@ -760,6 +760,11 @@ qemuParseCommandLineDisk(virDomainXMLOptionPtr xmlopt, } else if (STREQ(values[i], "floppy")) { def->bus = VIR_DOMAIN_DISK_BUS_FDC; def->device = VIR_DOMAIN_DISK_DEVICE_FLOPPY; + if (qemuDomainIsPSeries(dom)) { + virReportError(VIR_ERR_INTERNAL_ERROR, + _("pseries systems do not support floppy devices '%s'"), val); + goto error;
Do we actually need to worry about that here? I would expect that if pseries does not support floppy devices, you will not be able to run QEMU with such command line in the first place. So this case would never happen with QEMU that is actually running. Jan
+ }
} else if (STREQ(values[i], "virtio")) {
def->bus = VIR_DOMAIN_DISK_BUS_VIRTIO;
} else if (STREQ(values[i], "xen")) {
--
1.8.3.1
--
libvir-list mailing list
[email protected]
https://www.redhat.com/mailman/listinfo/libvir-list
signature.asc
Description: Digital signature
-- libvir-list mailing list [email protected] https://www.redhat.com/mailman/listinfo/libvir-list
