On Wed, Oct 22, 2025 at 06:00:08PM -0700, Bobby Eshleman wrote:
From: Bobby Eshleman bobbyeshleman@meta.com
Change QEMU to use generated pidfile names instead of just a single globally-defined pidfile. This allows multiple QEMU instances to co-exist with different pidfiles. This is required for future tests that use multiple VMs to check for CID collissions.
Additionally, this also places the burden of killing the QEMU process and cleaning up the pidfile on the caller of vm_start(). To help with this, a function terminate_pidfiles() is introduced that callers use to perform the cleanup. The terminate_pidfiles() function supports multiple pidfile removals because future patches will need to process two pidfiles at a time.
It seems that this will no longer cleanup, via a trap, if there is an early exit. Is that intentional?
This patch also changes the handling of QEMU_OPTS. I think that should be mentioned in the commit message too.
Signed-off-by: Bobby Eshleman bobbyeshleman@meta.com
...