On 13/12/19 16:10, Daniel P. Berrangé wrote:
I don't think it really matters. QEMU is deprecating it with no
seemless direct replacement, so I don't think libvirt needs to
be concerned. The feature simply becomes unsupported.
At the very most we need to check if it exists before using it,
but even that's just a nice to have which results in a slightly
prettier error message.
Libvirt right now assumes that a QEMU without the scsi property defaults
to "on", but if the oldest supported QEMU version is 1.5.3 then it
already has the property. So you can just give an error if there is no
scsi property and device='lun'.
Paolo