Francois Gouget : testbot: Ensure we prepare the right future VM on startup.

[Alexandre Julliard]

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

Author: Francois Gouget 
Date:   Thu Feb 22 01:34:17 2018 +0100

testbot: Ensure we prepare the right future VM on startup.

Signed-off-by: Francois Gouget 
Signed-off-by: Alexandre Julliard

---

testbot/lib/WineTestBot/Jobs.pm | 19 ++++++++++++-------
 1 file changed, 12 insertions(+), 7 deletions(-)

diff --git a/testbot/lib/WineTestBot/Jobs.pm b/testbot/lib/WineTestBot/Jobs.pm
index 56c903d..ae1e4ec 100644
--- a/testbot/lib/WineTestBot/Jobs.pm
+++ b/testbot/lib/WineTestBot/Jobs.pm
@@ -471,6 +471,7 @@ sub _GetSchedHost($$)
       sleeping => 0,
       running => 0,
       dirty => 0,
+      dirtychild => 0,
       MaxRevertingVMs => $MaxRevertingVMs,
       MaxRevertsWhileRunningVMs => $MaxRevertsWhileRunningVMs,
       MaxActiveVMs => $MaxActiveVMs,
@@ -604,6 +605,7 @@ sub _CheckAndClassifyVMs()
         $Sched->{busyvms}->{$VMKey} = 1;
         $Host->{$VM->Status}++;
         $Host->{active}++;
+        $Host->{dirtychild}++ if ($VM->Status eq "dirty");
       }
       elsif ($VM->Status eq "sleeping")
       {
@@ -1103,13 +1105,16 @@ sub _RevertVMs($$)
     {
       # Only start preparing VMs for future jobs on a host which is idle, i.e.
       # which no longer has queued tasks (ignoring blocked ones).
-      # Note that we could also check that the host only has idle VMs. This
-      # would help ensure that we are not prevented from preparing the best VM
-      # (e.g. build) on startup just because it is still being checked (i.e.
-      # marked dirty). But during regular operation this would force the host
-      # to go through an extra poweroff during which we lose track of which
-      # VM is 'hot'
-      if ($Host->{queued} != 0 or $Host->{MaxVMsWhenIdle} == 0)
+      # Note that during regular operation we get dirty VMs before they are
+      # assigned a process to shut them down. This makes it possible to pick
+      # the best future VM while we still know which VM is hot.
+      # In constrast on startup the dirty VMs all have processes checking their
+      # status, hence the dirtychild check to ensure we are not prevented from
+      # preparing the best VM (e.g. build): it delays preparing the future VMs
+      # until either there are no dirty VM or a VM got prepared for a task
+      # which means the host is not idle.
+      if ($Host->{queued} != 0 or $Host->{MaxVMsWhenIdle} == 0 or
+          ($Host->{active} and $Host->{active} == $Host->{dirtychild}))
       {
         # The TestBot is busy or does not prepare VMs when idle
         next;