On 7/4/25 14:36, Andrea Bolognani via Devel wrote: > I was working on this last year, then sort of lost track. Jim's > recent patch[1] caused me to remember about this work and look into > picking it up again. > > This is only half of the original series, which itself was reduced in > scope compared to the first revision. I'll try to get around to > everything, but addressing one area at the time is most likely to > succeed. > > Of course this will no longer apply cleanly once Jim's patch has been > pushed. The conflict will be trivial to solve though. > > Changes from [v2]: > > * several patches have been pushed; > > * address review comments. > > Changes from [v1]: > > * several patches have been pushed; > > * of the remaining changes, only the ones related to SCSI and USB > controllers have been retained. I still intend to pursue the > rest, but those two are where the real nasty stuff happens, so > I'm focusing on them only for now; > > * improve the handling of USB controllers on s390x; > > * make all the code dealing with the legacy USB controller obsolete > and get rid of it; > > * use out arguments to return models, making the new helpers fall > in line with the usual libvirt API conventions. > > [1] > https://lists.libvirt.org/archives/list/devel@lists.libvirt.org/thread/B45SLRWLS2TFVLLNFNPF3GAN2UZZ5YAA/ > [v2] > https://lists.libvirt.org/archives/list/devel@lists.libvirt.org/thread/FZ6BTIXG26SLVIX6AYSBZH3NCLWJKIYM/ > [v1] > https://lists.libvirt.org/archives/list/devel@lists.libvirt.org/thread/G53MRO2IEJOS3M53OKON2S5RM2MPH55K/ > > Andrea Bolognani (3): > qemu: Improve qemuDomainDefaultSCSIControllerModel() > qemu: Clean up qemuDomainDefaultSCSIControllerModel() > qemu: Use virtio-scsi by default on RISC-V > > src/qemu/qemu_domain.c | 48 +++++++++++++------ > src/qemu/qemu_domain.h | 5 +- > src/qemu/qemu_hotplug.c | 16 ++++--- > src/qemu/qemu_postparse.c | 11 +++-- > ...ault-models.riscv64-latest.abi-update.args | 5 +- > ...fault-models.riscv64-latest.abi-update.xml | 11 +++-- > ...64-virt-default-models.riscv64-latest.args | 5 +- > ...v64-virt-default-models.riscv64-latest.xml | 11 +++-- > 8 files changed, 73 insertions(+), 39 deletions(-) >
Reviewed-by: Michal Privoznik <mpriv...@redhat.com> Michal
