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

Nick Guenther posted 1 patch 1 year, 1 month ago
Failed in applying to current master (apply log)
src/qemu/qemu_command.c | 72 ++++++++++++++++++++++++++++++++---------
1 file changed, 57 insertions(+), 15 deletions(-)
[PATCH v2 0/1] qemu: Allow sockets in long or deep paths.
Posted by Nick Guenther 1 year, 1 month ago
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