Rob Shearman : ntdll: Don' t print a fixme in RtlQueueUserWorkItem if WT_EXECUTELONGFUNCTION is specified

Alexandre Julliard julliard at wine.codeweavers.com
Thu Jan 4 15:09:27 CST 2007


Module: wine
Branch: master
Commit: 7e1fead3fbd05b4ff6b61fd119e3dfde9b3d8ea8
URL:    http://source.winehq.org/git/wine.git/?a=commit;h=7e1fead3fbd05b4ff6b61fd119e3dfde9b3d8ea8

Author: Rob Shearman <rob at codeweavers.com>
Date:   Thu Jan  4 18:23:44 2007 +0000

ntdll: Don't print a fixme in RtlQueueUserWorkItem if WT_EXECUTELONGFUNCTION is specified
since that is what the current behaviour is tuned for.

---

 dlls/ntdll/threadpool.c |    4 +++-
 1 files changed, 3 insertions(+), 1 deletions(-)

diff --git a/dlls/ntdll/threadpool.c b/dlls/ntdll/threadpool.c
index cfe4f65..3d06cb4 100644
--- a/dlls/ntdll/threadpool.c
+++ b/dlls/ntdll/threadpool.c
@@ -181,11 +181,13 @@ NTSTATUS WINAPI RtlQueueWorkItem(PRTL_WO
     work_item->function = Function;
     work_item->context = Context;
 
-    if (Flags != WT_EXECUTEDEFAULT)
+    if (Flags & ~WT_EXECUTELONGFUNCTION)
         FIXME("Flags 0x%x not supported\n", Flags);
 
     status = add_work_item_to_queue(work_item);
 
+    /* FIXME: tune this algorithm to not be as aggressive with creating threads
+     * if WT_EXECUTELONGFUNCTION isn't specified */
     if ((status == STATUS_SUCCESS) &&
         ((num_workers == 0) || (num_workers == num_busy_workers)))
     {




More information about the wine-cvs mailing list