[PATCH v2 0/1] qemu: Allow sockets in long or deep paths.

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



v1: https://listman.redhat.com/archives/libvir-list/2023-April/239441.html

Address v1's cleanup bugs by using g_autofree and the concurrency bugs by
forking a subprocess before chdir(). It works!

I have been working on the other solution of putting paths in /var/run, from
https://listman.redhat.com/archives/libvir-list/2023-April/239536.html, but
so my attempts have ballooned out of control. I would like to offer this
improved version of my original patch in the meantime. Plus there's the
unclear impact of the other strategy on upgrades: https://listman.redhat.com/archives/libvir-list/2023-April/239470.html,
which means even once that's implemented it will take a long time to vet.
The chdir() approach cuts right to the point.

I'm hoping you'll accept both approaches as complementary and worth adopting.

Original issue: https://gitlab.com/libvirt/libvirt/-/issues/466

Cheers!

Nick Guenther (1):
  qemu: Allow sockets in long or deep paths.

 src/qemu/qemu_command.c | 72 ++++++++++++++++++++++++++++++++---------
 1 file changed, 57 insertions(+), 15 deletions(-)

-- 
2.34.1




[Index of Archives]     [Virt Tools]     [Libvirt Users]     [Lib OS Info]     [Fedora Users]     [Fedora Desktop]     [Fedora SELinux]     [Big List of Linux Books]     [Yosemite News]     [KDE Users]     [Fedora Tools]

  Powered by Linux