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