Greetings Peter,
> Sent: Monday, October 19, 2020 at 11:06 AM
> From: "Peter Krempa" <pkrempa@xxxxxxxxxx>
> To: "daggs" <daggs@xxxxxxx>
> Cc: libvir-list@xxxxxxxxxx
> 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.
