Francois Gouget : testbot/Engine: Clean up left-over VM child deadlines.

Alexandre Julliard julliard at winehq.org
Fri Sep 13 13:02:36 CDT 2019


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

Author: Francois Gouget <fgouget at codeweavers.com>
Date:   Fri Sep 13 03:06:20 2019 +0200

testbot/Engine: Clean up left-over VM child deadlines.

Normally the ChildDeadline gets unset whenever ChildPid is unset.
But if it does remain set the scheduler will eventually see a deadline
in the past, causing it to reschedule every second, until a new child
process gets created for that VM.

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

---

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

diff --git a/testbot/lib/WineTestBot/Engine/Scheduler.pm b/testbot/lib/WineTestBot/Engine/Scheduler.pm
index 39d2342..3ec51d2 100644
--- a/testbot/lib/WineTestBot/Engine/Scheduler.pm
+++ b/testbot/lib/WineTestBot/Engine/Scheduler.pm
@@ -350,7 +350,7 @@ sub _CheckAndClassifyVMs()
     }
     else
     {
-      if (defined $VM->ChildPid or
+      if (defined $VM->ChildPid or defined $VM->ChildDeadline or
           $VM->Status =~ /^(?:running|reverting|sleeping)$/)
       {
         # The VM is missing its child process or it died unexpectedly. Mark




More information about the wine-cvs mailing list