[PATCH 0/6] qemu: Fix cdrom as SCSI hostdev via -blockdev

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

 



-drive detects whether a device is a cdrom automatically but we need to
use 'host_cdrom' when using blockdev explicitly. Fix the hostdev code
which was recently converted to -blockdev.

Warning:

I _don't_ have a box with a CDROM handy so this code is not tested in action!

Peter Krempa (6):
  qemuProcessCreatePretendCmd: Split up preparation and command building
  qemu: hostdev: Prepare definition bits in qemuDomainPrepareHostdev
  qemu: Prepare hostdev data which depends on the host state separately
  qemuxml2argvmock: Remove mocking of 'virSCSIDeviceGetSgName'
  qemu: Detect whether a SCSI hostdev is a cdrom
  qemu: Add test cases for 'host_cdrom' blockdev backend via <disk>

 src/qemu/qemu_command.c                       |  33 +-----
 src/qemu/qemu_domain.c                        |   5 +
 src/qemu/qemu_driver.c                        |  65 ++++++++++-
 src/qemu/qemu_hotplug.c                       |   3 +
 src/qemu/qemu_process.c                       | 103 +++++++++++++++---
 src/qemu/qemu_process.h                       |  21 ++--
 tests/qemuxml2argvdata/disk-cdrom.args        |   4 +-
 .../disk-cdrom.x86_64-2.12.0.args             |   4 +-
 .../disk-cdrom.x86_64-latest.args             |   6 +-
 tests/qemuxml2argvdata/disk-cdrom.xml         |   5 +-
 ...hostdev-scsi-virtio-scsi.x86_64-2.8.0.args |   3 +
 ...hostdev-scsi-virtio-scsi.x86_64-4.1.0.args |   3 +
 ...ostdev-scsi-virtio-scsi.x86_64-latest.args |   4 +
 .../hostdev-scsi-virtio-scsi.xml              |   8 ++
 tests/qemuxml2argvmock.c                      |  13 ---
 tests/qemuxml2argvtest.c                      |  45 +++++++-
 tests/qemuxml2xmloutdata/disk-cdrom.xml       |   5 +-
 .../hostdev-scsi-virtio-scsi.xml              |   8 ++
 18 files changed, 252 insertions(+), 86 deletions(-)

-- 
2.26.2




[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