The perfc_valuea macro was effectively a no-op. This trivial patch
makes it return the value it reads.

Make perfc_valuea actually return the value it reads.
Signed-off-by: David Lively <dlively@xxxxxxxxxxxxxxx>
diff -r ac4a961f7e64 xen/include/xen/perfc.h
--- a/xen/include/xen/perfc.h Thu May 25 22:57:44 2006 +0100
+++ b/xen/include/xen/perfc.h Fri May 26 11:10:33 2006 -0400
@@ -56,10 +56,8 @@ extern struct perfcounter perfcounters;
 #define perfc_value(x) atomic_read(&perfcounters.x[0])
 #define perfc_valuec(x) atomic_read(&perfcounters.x[smp_processor_id()])
 #define perfc_valuea(x,y) \
- do { \
- if ( (y) < (sizeof(perfcounters.x) / sizeof(*perfcounters.x)) ) \
- atomic_read(&perfcounters.x[y]); \
- } while ( 0 )
+ ( (y) < (sizeof(perfcounters.x) / sizeof(*perfcounters.x)) ? \
+ atomic_read(&perfcounters.x[y]) : 0 )
 #define perfc_set(x,v) atomic_set(&perfcounters.x[0], v)
 #define perfc_setc(x,v) atomic_set(&perfcounters.x[smp_processor_id()], v)
 #define perfc_seta(x,y,v) \

_______________________________________________
Xen-devel mailing list
Xen-devel@xxxxxxxxxxxxxxxxxxx
http://lists.xensource.com/xen-devel