vlog: Optimize logging disabled, rate-limited messages.
authorBen Pfaff <blp@nicira.com>
Wed, 17 Dec 2008 00:32:27 +0000 (16:32 -0800)
committerBen Pfaff <blp@nicira.com>
Wed, 17 Dec 2008 00:41:25 +0000 (16:41 -0800)
Previously the arguments of rate-limited messages were always evaluated
and passed to vlog_rate_limit().  This commit short-circuits the case
where the vlog levels would prevent any messages from being logged even
without considering rate limiting.

lib/vlog.h

index 407177d748c69ba59de0bf315027e521f7a9ecb1..a01309bfcb8cec29d6ea9d50a7d882b13243400c 100644 (file)
@@ -163,12 +163,9 @@ void vlog_rate_limit(enum vlog_module, enum vlog_level,
  *      #define THIS_MODULE VLM_netlink
  * Guaranteed to preserve errno.
  */
-#define VLOG_ERR_RL(RL, ...) \
-        vlog_rate_limit(THIS_MODULE, VLL_ERR, RL, __VA_ARGS__)
-#define VLOG_WARN_RL(RL, ...) \
-        vlog_rate_limit(THIS_MODULE, VLL_WARN, RL, __VA_ARGS__)
-#define VLOG_DBG_RL(RL, ...) \
-        vlog_rate_limit(THIS_MODULE, VLL_DBG, RL, __VA_ARGS__)
+#define VLOG_ERR_RL(RL, ...) VLOG_RL(RL, VLL_ERR, __VA_ARGS__)
+#define VLOG_WARN_RL(RL, ...) VLOG_RL(RL, VLL_WARN, __VA_ARGS__)
+#define VLOG_DBG_RL(RL, ...) VLOG_RL(RL, VLL_DBG, __VA_ARGS__)
 
 /* Command line processing. */
 #define VLOG_OPTION_ENUMS OPT_LOG_FILE
@@ -191,6 +188,12 @@ void vlog_usage(void);
             vlog(THIS_MODULE, LEVEL, __VA_ARGS__);      \
         }                                               \
     } while (0)
+#define VLOG_RL(RL, LEVEL, ...)                                     \
+    do {                                                            \
+        if (min_vlog_levels[THIS_MODULE] >= LEVEL) {                \
+            vlog_rate_limit(THIS_MODULE, LEVEL, RL, __VA_ARGS__);   \
+        }                                                           \
+    } while (0)
 extern enum vlog_level min_vlog_levels[VLM_N_MODULES];