[ros-diffs] [cfinck] 29287: uniata patch by encoded: - Make building with QUEUE_STATISTICS possible - Fix the best_c check. As best_c is a ULONG variable, you cannot check for -1

cfinck at svn.reactos.org cfinck at svn.reactos.org
Sat Sep 29 00:00:53 CEST 2007


Author: cfinck
Date: Sat Sep 29 02:00:52 2007
New Revision: 29287

URL: http://svn.reactos.org/svn/reactos?rev=29287&view=rev
Log:
uniata patch by encoded:
- Make building with QUEUE_STATISTICS possible
- Fix the best_c check. As best_c is a ULONG variable, you cannot check for -1

Modified:
    trunk/reactos/drivers/storage/ide/uniata/id_queue.cpp

Modified: trunk/reactos/drivers/storage/ide/uniata/id_queue.cpp
URL: http://svn.reactos.org/svn/reactos/trunk/reactos/drivers/storage/ide/uniata/id_queue.cpp?rev=29287&r1=29286&r2=29287&view=diff
==============================================================================
--- trunk/reactos/drivers/storage/ide/uniata/id_queue.cpp (original)
+++ trunk/reactos/drivers/storage/ide/uniata/id_queue.cpp Sat Sep 29 02:00:52 2007
@@ -133,9 +133,11 @@
                 new_cost1 = UniataGetCost(LunExt, AtaReq1, AtaReq);
                 new_cost2 = UniataGetCost(LunExt, AtaReq, AtaReq2);
 
+#ifdef QUEUE_STATISTICS
                 if(new_cost1 == REORDER_COST_INTERSECT ||
                    new_cost2 == REORDER_COST_INTERSECT)
                     chan->IntersectCount++;
+#endif //QUEUE_STATISTICS
 
                 if(new_cost2 > REORDER_COST_RESELECT)
                     break;
@@ -348,7 +350,7 @@
             cost_c = chan->queue_depth * (chan->ChannelSelectWaitCount+1);
         }
     }
-    if(best_c == -1) {
+    if(best_c == 0xFFFFFFFF) {
         KdPrint2((PRINT_PREFIX "  empty queues\n"));
         return NULL;
     }




More information about the Ros-diffs mailing list