From: Henrik Nordstrom <henrik@henriknordstrom.net>
Date: Thu, 20 Mar 2008 05:15:04 +0000 (+0100)
Subject: At the sprint the issue about our current 10ms main loop timeout came
X-Git-Tag: BASIC_TPROXY4~20
X-Git-Url: http://git.ipfire.org/?a=commitdiff_plain;h=b9619b85f01e13faa9879baa9dbe461ad73fb60a;p=thirdparty%2Fsquid.git

At the sprint the issue about our current 10ms main loop timeout came
up, and it was suggested the problem most likely have been fixed in
HEAD. And even if it hasn't been fixed it's something which should be
fixed rather than plastered over by spinning around on a short timeout
when there is no activity..

This backs out the change to revert the change..

Alex commit reversing the change said:

revno: 8332
committer: rousskov
branch nick: HEAD
timestamp: Tue 2007-07-24 01:55:21 +0000
message:
  Reversed bug #2011 fix because it may slow down ICAP, BodyPipe, and other code
  using zero-delay events to implement "asynchronous" calls.

  The code should probably be rewritten (a) to avoid any waiting/blocking when
  there are ready events and (b) to allow waiting longer when there are no ready
  events.
---

diff --git a/src/EventLoop.cc b/src/EventLoop.cc
index cc2af54310..dc45f68d8d 100644
--- a/src/EventLoop.cc
+++ b/src/EventLoop.cc
@@ -103,7 +103,7 @@ EventLoop::runOnce()
     bool sawActivity = false;
     runOnceResult = true;
     error = false;
-    loop_delay = 10; /* 10 ms default delay */
+    loop_delay = 1000; /* 1s default delay */
 
     AsyncEngine *waitingEngine = primaryEngine;
     if (!waitingEngine && !engines.empty())