On Tue, Jan 04, 2022 at 10:33:14AM +0000, Daniel P. Berrangé wrote:
On Thu, Dec 23, 2021 at 11:36:58AM +0100, Laszlo Ersek wrote:
> Bugzilla:
https://bugzilla.redhat.com/show_bug.cgi?id=2034160
>
> The first patch extends our current <qemu:commandline> hack, moving the
> virtio-net-pci device to slot 0x1E, where it is very unlikely to
> conflict with any libvirt-assigned PCI address.
Remind me why we still need to use <qemu:commandline> ? A need
for this obviously reflects a failure of libvirt to address the
libguestfs requirements. Can we fix the root cause here with proper
XML support for whatever feature is missing.
Laszlo actually fixed this properly in the case we care about (modern
libvirt), if you look at the 3rd patch which went upstream:
https://github.com/libguestfs/libguestfs/commit/5858c2cf6c24b3776e3867eaf...
Ignoring the old libvirt case, the only reason we still need
<qemu:commandline> is to allow us to set TMPDIR accurately.
(Otherwise if you recall we can get a TMPDIR from an unrelated run of
libguestfs that happens to share the same session libvirtd).
The final code should make this clear:
https://github.com/libguestfs/libguestfs/blob/d1e7e1a323619d8f1e913a7833d...
Rich.
--
Richard Jones, Virtualization Group, Red Hat
http://people.redhat.com/~rjones
Read my programming and virtualization blog:
http://rwmj.wordpress.com
Fedora Windows cross-compiler. Compile Windows programs, test, and
build Windows installers. Over 100 libraries supported.
http://fedoraproject.org/wiki/MinGW