[PATCH] qemu: Do not keep swtmp pidfile around after stopping

[Michal Prívozník]

On 8/24/22 13:19, Michal Prívozník wrote:
> On 8/23/22 16:19, Michal Prívozník wrote:
>> On 8/18/22 16:20, Martin Kletzander wrote:
>>> Just like the socket, remove the pidfile when TPM emulator is being stopped.  In
>>> order to make this a bit cleaner, try to remove it even if swtpm_ioctl does not
>>> exist.
>>>
>>> Signed-off-by: Martin Kletzander <mkletzan at redhat.com>
>>> ---
>>>  src/qemu/qemu_tpm.c | 31 ++++++++++++++-----------------
>>>  1 file changed, 14 insertions(+), 17 deletions(-)
>>>
> 
> What I forgot to mention is that qemuTPMEmulatorStop() which is called
> from qemuTPMEmulatorStart() can be removed, because if you accept my
> suggestion and go with virPidFileForceCleanupPath() there's no chance
> for the swtpm process to run at either the end of qemuTPMEmulatorStop()
> or at the time of qemuTPMEmulatorStart(). But that can be done in a
> follow up patch.

In fact, it is completely independent. Because the pid file is in form of:

  ${swtpmStateDir}/${shortName}-swtpm.pid

where ${shortName} is the result of virDomainDefGetShortName() thus it
contains domain ID and therefore, it's never ever the same across two
'virsh start's of a domain. I'm sorry I haven't realized this during
review earlier.

Michal