On Mon, Feb 04, 2019 at 04:47:04PM +0100, Peter Krempa wrote:
QEMU accidentally exposed the id of -drive (or same value as disk
serial, if provided) in one of the identifiers visible from the guest.
To avoid regression in case when -blockdev will be used we need to
always specify it ourselves.
Signed-off-by: Peter Krempa <pkrempa(a)redhat.com>
---
src/qemu/qemu_command.c | 22 +++++++++++++++++++
.../controller-virtio-scsi.x86_64-latest.args | 20 ++++++++---------
.../disk-cache.x86_64-latest.args | 4 ++--
.../disk-scsi-device-auto.x86_64-latest.args | 3 ++-
.../disk-scsi.x86_64-latest.args | 16 ++++++++------
.../disk-shared.x86_64-latest.args | 5 +++--
...threads-virtio-scsi-pci.x86_64-latest.args | 4 ++--
7 files changed, 50 insertions(+), 24 deletions(-)
@@ -2021,6 +2037,12 @@ qemuBuildDiskDeviceStr(const virDomainDef *def,
disk->info.addr.drive.target,
disk->info.addr.drive.unit);
}
+
+ if (scsiVPDDeviceId) {
+ virBufferAddLit(&opt, ",device_id=");
+ virBufferEscape(&opt, '\\', " ", "%s",
scsiVPDDeviceId);
commas should be escaped, not spaces
+ }
+
break;
case VIR_DOMAIN_DISK_BUS_SATA:
Reviewed-by: Ján Tomko <jtomko(a)redhat.com>
Jano