Minimal fix to avoid busy looping. The condition_variable varant showed

spurious failures on travis so go for the minimial fix.

diff --git a/pdns/communicator.cc b/pdns/communicator.cc
index 7db5a3e5b6737a2368059311400d8bd115e95bf5..7fd59e450e3a33a37818f1c15885cbb27dd771a7 100644 (file)
--- a/pdns/communicator.cc
+++ b/pdns/communicator.cc
@@ -136,7 +136,10 @@ void CommunicatorClass::mainloop(void)
           if (extraSlaveRefresh)
             slaveRefresh(&P);
         }
-        else { 
+        else {
+          // eat up extra posts to avoid busy looping if many posts were done
+          while (d_any_sem.tryWait() == 0) {
+          }
           break; // something happened
         }
         // this gets executed at least once every second