PERFORCE change 43335 for review
    Peter Wemm 
    peter at FreeBSD.org
       
    Tue Dec  2 17:33:00 PST 2003
    
    
  
http://perforce.freebsd.org/chv.cgi?CH=43335
Change 43335 by peter at peter_overcee on 2003/12/02 17:32:38
	actually, since we've already checked for zero in the loop,
	directly call bsfq() rather than "ffsl() - 1".  In theory,
	gcc -O should optimize out the mask == 0 case (already tested)
	and the "- 1 + 1", but dont depend on it.  This is why we have
	RQB_FFS() exposed like this anyway.
Affected files ...
.. //depot/projects/hammer/sys/amd64/include/runq.h#6 edit
Differences ...
==== //depot/projects/hammer/sys/amd64/include/runq.h#6 (text+ko) ====
@@ -36,7 +36,7 @@
 #define	RQB_BIT(pri)	(1 << ((pri) & (RQB_BPW - 1)))
 #define	RQB_WORD(pri)	((pri) >> RQB_L2BPW)
 
-#define	RQB_FFS(word)	(ffsl(word) - 1)
+#define	RQB_FFS(word)	(bsfq(word))
 
 /*
  * Type of run queue status word.
    
    
More information about the p4-projects
mailing list