igmp: fix debug macro 10/11810/2
authorJakub Grajciar <jgrajcia@cisco.com>
Mon, 16 Apr 2018 08:39:24 +0000 (10:39 +0200)
committerJakub Grajciar <jgrajcia@cisco.com>
Tue, 17 Apr 2018 07:58:14 +0000 (07:58 +0000)
Change-Id: I071d4cfcf6ea9763dd4842a3594c486a8e400e8c
Signed-off-by: Jakub Grajciar <jgrajcia@cisco.com>
src/plugins/igmp/igmp.c
src/plugins/igmp/igmp.h
src/plugins/igmp/input.c

index e98e976..55a5d0e 100644 (file)
@@ -567,7 +567,7 @@ igmp_timer_process (vlib_main_t * vm, vlib_node_runtime_t * rt,
       if (event_type == IGMP_PROCESS_EVENT_UPDATE_TIMER)
        goto next_timer;
 
-      DBG ("time: %f", vlib_time_now (vm));
+      IGMP_DBG ("time: %f", vlib_time_now (vm));
 
       /* timer expired */
       if (NULL != timer)
index 0dcf11f..f5fb958 100644 (file)
 #define IGMP_SG_TIMER                          (3 * IGMP_QUERY_TIMER)
 #define IGMP_DEFAULT_ROBUSTNESS_VARIABLE       (2)
 
-#define IGMP_DBG 1
+#define ENABLE_IGMP_DBG 0
 
-#if IGMP_DBG
-#define DBG(...) clib_warning(__VA_ARGS__)
+#if ENABLE_IGMP_DBG == 1
+#define IGMP_DBG(...) clib_warning(__VA_ARGS__)
 #else
-#define DBG(...)
-#endif /* IGMP_DBG */
+#define IGMP_DBG(...)
+#endif /* ENABLE_IGMP_DBG */
 
 #define group_ptr(p, l) ((igmp_membership_group_v3_t *)((char*)p + l))
 
index 9e46e60..ec0ae36 100644 (file)
@@ -118,7 +118,7 @@ uword
 igmp_input (vlib_main_t * vm, vlib_node_runtime_t * node,
            vlib_frame_t * frame)
 {
-  DBG ("IGMP_INPUT");
+  IGMP_DBG ("IGMP_INPUT");
   u32 n_left_from, *from, *to_next;
   igmp_parse_query_next_t next_index;
   vlib_node_runtime_t *error_node =
@@ -243,7 +243,7 @@ uword
 igmp_parse_query (vlib_main_t * vm, vlib_node_runtime_t * node,
                  vlib_frame_t * frame)
 {
-  DBG ("IGMP_PARSE_QUERY");
+  IGMP_DBG ("IGMP_PARSE_QUERY");
 
   u32 n_left_from, *from, *to_next;
   igmp_parse_query_next_t next_index;
@@ -285,7 +285,7 @@ igmp_parse_query (vlib_main_t * vm, vlib_node_runtime_t * node,
              config = igmp_config_lookup (im, sw_if_index);
              if (!config)
                {
-                 DBG ("No config on interface %u", sw_if_index);
+                 IGMP_DBG ("No config on interface %u", sw_if_index);
                }
              else
                {
@@ -348,7 +348,7 @@ uword
 igmp_parse_report (vlib_main_t * vm, vlib_node_runtime_t * node,
                   vlib_frame_t * frame)
 {
-  DBG ("IGMP_PARSE_REPORT");
+  IGMP_DBG ("IGMP_PARSE_REPORT");
 
   igmp_main_t *im = &igmp_main;
   u32 n_left_from, *from, *to_next;
@@ -409,14 +409,14 @@ igmp_parse_report (vlib_main_t * vm, vlib_node_runtime_t * node,
              config->flags |= IGMP_CONFIG_FLAG_QUERY_RESP_RECVED;
              if (config->cli_api_configured)
                {
-                 DBG ("Interface %u has (S,G)s configured by CLI/API",
-                      sw_if_index);
+                 IGMP_DBG ("Interface %u has (S,G)s configured by CLI/API",
+                           sw_if_index);
                  error = IGMP_ERROR_CLI_API_CONFIG;
                  b->error = error_node->errors[error];
                  goto next_frame;
                }
            }
-         DBG ("interface %u", sw_if_index);
+         IGMP_DBG ("interface %u", sw_if_index);
          int i, j = 0;
          for (i = 0; i < clib_net_to_host_u16 (igmp->n_groups); i++)
            {