Francois Gouget : testbot/Engine: Kill the child before forking a new one for sacrifices.

[Alexandre Julliard]

Module: tools
Branch: master
Commit: a2dbe176c15c58b4c6c8fdfe53ba285129b9a963
URL:    https://source.winehq.org/git/tools.git/?a=commit;h=a2dbe176c15c58b4c6c8fdfe...

Author: Francois Gouget <fgouget(a)codeweavers.com>
Date:   Sat Sep 21 20:33:32 2019 +0200

testbot/Engine: Kill the child before forking a new one for sacrifices.

VM::Run() now checks that no process is actively using the VM before
starting the new one. Killing the current process makes more sense
anyway and could help ensure it won't interfere with the new one by
sending TestAgent commands.

Signed-off-by: Francois Gouget <fgouget(a)codeweavers.com>
Signed-off-by: Alexandre Julliard <julliard(a)winehq.org>

---

 testbot/lib/WineTestBot/Engine/Scheduler.pm | 1 +
 1 file changed, 1 insertion(+)

diff --git a/testbot/lib/WineTestBot/Engine/Scheduler.pm b/testbot/lib/WineTestBot/Engine/Scheduler.pm
index e9a3ff9..7375337 100644
--- a/testbot/lib/WineTestBot/Engine/Scheduler.pm
+++ b/testbot/lib/WineTestBot/Engine/Scheduler.pm
@@ -765,6 +765,7 @@ sub _SacrificeVM($$$)
   $Host->{$Victim->Status}--;
   $Host->{dirty}++;
   $Victim->RecordStatus($Sched->{records}, $Victim->Status eq "dirty" ? "dirty poweroff" : "dirty sacrifice");
+  $Victim->KillChild();
   $Victim->RunPowerOff();
   return 1;
 }