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=a2dbe176c15c58b4c6c8fdfe53ba285129b9a963

Author: Francois Gouget <fgouget at 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 at codeweavers.com>
Signed-off-by: Alexandre Julliard <julliard at 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;
 }