Greetings Peter, > Sent: Monday, October 19, 2020 at 11:06 AM > From: "Peter Krempa" <[email protected]> > To: "daggs" <[email protected]> > Cc: [email protected] > 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.
