lib: Expose SAT_MUT as OVS_SAT_MUL in <openvswitch/util.h>
authorThomas Graf <tgraf@noironetworks.com>
Mon, 15 Dec 2014 13:10:38 +0000 (14:10 +0100)
committerThomas Graf <tgraf@noironetworks.com>
Mon, 15 Dec 2014 13:15:08 +0000 (14:15 +0100)
Insted of exposing the full sat-math.h API, only the macros
used in headers is exposed through <openvswitch/util.h>

Signed-off-by: Thomas Graf <tgraf@noironetworks.com>
Acked-by: Ben Pfaff <blp@nicira.com>
include/openvswitch/util.h
lib/sat-math.h
lib/vlog.h

index b0b9935..683f76b 100644 (file)
@@ -40,6 +40,12 @@ const char *ovs_get_program_version(void);
 #define OVS_STRINGIZE(ARG) OVS_STRINGIZE2(ARG)
 #define OVS_STRINGIZE2(ARG) #ARG
 
+/* Saturating multiplication of "unsigned int"s: overflow yields UINT_MAX. */
+#define OVS_SAT_MUL(X, Y)                                               \
+    ((Y) == 0 ? 0                                                       \
+     : (X) <= UINT_MAX / (Y) ? (unsigned int) (X) * (unsigned int) (Y)  \
+     : UINT_MAX)
+
 #ifdef __cplusplus
 }
 #endif
index 3d1c50b..beeff8b 100644 (file)
@@ -18,6 +18,7 @@
 #define SAT_MATH_H 1
 
 #include <limits.h>
+#include "openvswitch/util.h"
 
 /* Saturating addition: overflow yields UINT_MAX. */
 static inline unsigned int
@@ -33,15 +34,10 @@ sat_sub(unsigned int x, unsigned int y)
     return x >= y ? x - y : 0;
 }
 
-/* Saturating multiplication of "unsigned int"s: overflow yields UINT_MAX. */
-#define SAT_MUL(X, Y)                                                   \
-    ((Y) == 0 ? 0                                                       \
-     : (X) <= UINT_MAX / (Y) ? (unsigned int) (X) * (unsigned int) (Y)  \
-     : UINT_MAX)
 static inline unsigned int
 sat_mul(unsigned int x, unsigned int y)
 {
-    return SAT_MUL(x, y);
+    return OVS_SAT_MUL(x, y);
 }
 
 #endif /* sat-math.h */
index 67d37eb..8190a44 100644 (file)
@@ -116,7 +116,7 @@ struct vlog_rate_limit {
  * messages per minute and a maximum burst size of BURST messages. */
 #define VLOG_RATE_LIMIT_INIT(RATE, BURST)                               \
         {                                                               \
-            TOKEN_BUCKET_INIT(RATE, SAT_MUL(BURST, VLOG_MSG_TOKENS)),   \
+            TOKEN_BUCKET_INIT(RATE, OVS_SAT_MUL(BURST, VLOG_MSG_TOKENS)),\
             0,                              /* first_dropped */         \
             0,                              /* last_dropped */          \
             0,                              /* n_dropped */             \