]> git.puffer.fish Git - mirror/frr.git/commitdiff
zebra: Combine 2 debugs into 1 for NHG Detail
authorDonald Sharp <sharpd@nvidia.com>
Sun, 21 Jan 2024 01:25:59 +0000 (20:25 -0500)
committerDonald Sharp <sharpd@nvidia.com>
Wed, 24 Jan 2024 23:29:34 +0000 (18:29 -0500)
When debugging NHG detail there is a whole bunch
of lines surrounding the nexthop group.  Let's
clean these up since they are extremely chatty and
spawn several lines.

Signed-off-by: Donald Sharp <sharpd@nvidia.com>
zebra/zebra_nhg.c

index 9e3aa8bc5c8e3f3b1ebd6d9d2231d9048a6a5a28..6804dbf13a3408149e514ac2b2b0358381e805e3 100644 (file)
@@ -690,12 +690,6 @@ static bool zebra_nhe_find(struct nhg_hash_entry **nhe, /* return value */
        struct nhg_hash_entry *newnhe, *backup_nhe;
        struct nexthop *nh = NULL;
 
-       if (IS_ZEBRA_DEBUG_NHG_DETAIL)
-               zlog_debug(
-                       "%s: id %u, lookup %p, vrf %d, type %d, depends %p%s",
-                       __func__, lookup->id, lookup, lookup->vrf_id,
-                       lookup->type, nhg_depends,
-                       (from_dplane ? " (from dplane)" : ""));
 
        if (lookup->id)
                (*nhe) = zebra_nhg_lookup_id(lookup->id);
@@ -703,7 +697,10 @@ static bool zebra_nhe_find(struct nhg_hash_entry **nhe, /* return value */
                (*nhe) = hash_lookup(zrouter.nhgs, lookup);
 
        if (IS_ZEBRA_DEBUG_NHG_DETAIL)
-               zlog_debug("%s: lookup => %p (%pNG)", __func__, *nhe, *nhe);
+               zlog_debug("%s: id %u, lookup %p, vrf %d, type %d, depends %p%s => Found %p(%pNG)",
+                          __func__, lookup->id, lookup, lookup->vrf_id,
+                          lookup->type, nhg_depends,
+                          (from_dplane ? " (from dplane)" : ""), *nhe, *nhe);
 
        /* If we found an existing object, we're done */
        if (*nhe)
@@ -1537,13 +1534,11 @@ zebra_nhg_rib_find_nhe(struct nhg_hash_entry *rt_nhe, afi_t rt_afi)
                return NULL;
        }
 
-       if (IS_ZEBRA_DEBUG_NHG_DETAIL)
-               zlog_debug("%s: rt_nhe %p (%pNG)", __func__, rt_nhe, rt_nhe);
-
        zebra_nhe_find(&nhe, rt_nhe, NULL, rt_afi, false);
 
        if (IS_ZEBRA_DEBUG_NHG_DETAIL)
-               zlog_debug("%s: => nhe %p (%pNG)", __func__, nhe, nhe);
+               zlog_debug("%s: rt_nhe %p(%pNG) => nhe %p(%pNG)", __func__,
+                          rt_nhe, rt_nhe, nhe, nhe);
 
        return nhe;
 }