Changeset 103547 in vbox for trunk/src/VBox/Runtime

Timestamp:

Feb 23, 2024 3:23:03 PM (11 months ago)

Author:

vboxsync

Message:

Runtime/reqpool.cpp: Another place where we need to decrement the idle thread count or exiting idle threads would still have one idle reference. Also spawn a new thread if the number of pending requests is greater than the number of idling threads (each one can only handle on request at a time and depending on the processing time of each request others might get delayed too much)

File:

• trunk/src/VBox/Runtime/common/misc/reqpool.cpp (modified) (2 diffs)

trunk/src/VBox/Runtime/common/misc/reqpool.cpp

@@ -439 +439 @@
             uint64_t cNsIdle = RTTimeNanoTS() - pThread->uIdleNanoTs;
             if (cNsIdle >= pPool->cNsMinIdle)
+            {
+                ASMAtomicDecU32(&pPool->cIdleThreads); /* Was already marked as idle above. */
                 return rtReqPoolThreadExit(pPool, pThread, true /*fLocked*/);
+            }
         }
 
@@ -580 +583 @@
      * maximum number of worker threads, we're done.
      */
-    if (   pPool->cIdleThreads > 0
+    if (   pPool->cIdleThreads >= pPool->cCurPendingRequests
         || pPool->cCurThreads >= pPool->cMaxThreads)
     {