
Greetings Peter,
Sent: Monday, October 19, 2020 at 11:06 AM From: "Peter Krempa" <pkrempa@redhat.com> To: "daggs" <daggs@gmx.com> Cc: libvir-list@redhat.com Subject: Re: [PATCH 0/6] qemu: Fix cdrom as SCSI hostdev via -blockdev
Well, you used an XML from our tests, with following config:
<hostdev mode='subsystem' type='scsi' managed='yes'> <source> <adapter name='cdrom'/>
If you look into patch 5/6 for the change to tests/qemuxml2argvtest.c you'll see that using 'cdrom' as an adapter name is a hack made for just tests. That will _not_ work anywhere else.
You should use the XML that you've used before with the adapter name corresponding to the adapter the cdrom is on.
The command line you've posted:
-blockdev '{"driver":"host_device","filename":"/dev/sg0","node-name":"libvirt-2-backend","read-only":true}' \ -device scsi-generic,bus=scsi0.0,channel=0,scsi-id=0,lun=0,drive=libvirt-2-backend,id=hostdev0 \
Shows that nothing has changed, but unfortunately doesn't show enough for me to debug it. Unfortunately the cdrom detection code doesn't log anything so debug logs won't help either.
I'll need to try it in a nested virtualization environment since I don't have a physical CD drive handy.
if you wish, you can send my a patch with the debug code you want to add, I'll apply it, run and send you the log. Dagg.