]> git.puffer.fish Git - matthieu/frr.git/commitdiff
pbr: Add tableid to some debugs
authorDonald Sharp <sharpd@nvidia.com>
Thu, 13 Feb 2025 15:40:17 +0000 (10:40 -0500)
committerDonald Sharp <sharpd@nvidia.com>
Thu, 13 Feb 2025 15:40:17 +0000 (10:40 -0500)
Add prefix and tableid to one debug instead of several.
Consolidate a bit.

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

index dd15beaff4402506b634653470f6d5b45224cf4c..277271c6b6a05af68086e613b1a628ff011341db 100644 (file)
@@ -135,9 +135,11 @@ static int route_notify_owner(ZAPI_CALLBACK_ARGS)
        enum zapi_route_notify_owner note;
        uint32_t table_id;
 
-       if (!zapi_route_notify_decode(zclient->ibuf, &p, &table_id, &note,
-                                     NULL, NULL))
+       if (!zapi_route_notify_decode(zclient->ibuf, &p, &table_id, &note, NULL, NULL)) {
+               DEBUGD(&pbr_dbg_zebra, "%s: Received Notification for which PBR could not decode",
+                      __func__);
                return -1;
+       }
 
        switch (note) {
        case ZAPI_ROUTE_FAIL_INSTALL:
@@ -236,8 +238,6 @@ static void route_add_helper(struct zapi_route *api, struct nexthop_group nhg,
 
        api->prefix.family = install_afi;
 
-       DEBUGD(&pbr_dbg_zebra, "    Encoding %pFX", &api->prefix);
-
        i = 0;
        for (ALL_NEXTHOPS(nhg, nhop)) {
                api_nh = &api->nexthops[i];
@@ -284,10 +284,10 @@ void route_add(struct pbr_nexthop_group_cache *pnhgc, struct nexthop_group nhg,
 {
        struct zapi_route api;
 
-       DEBUGD(&pbr_dbg_zebra, "%s for Table: %d", __func__, pnhgc->table_id);
-
        memset(&api, 0, sizeof(api));
 
+       DEBUGD(&pbr_dbg_zebra, "%s %pFX for Table: %d", __func__, &api.prefix, pnhgc->table_id);
+
        api.vrf_id = VRF_DEFAULT;
        api.type = ZEBRA_ROUTE_PBR;
        api.safi = SAFI_UNICAST;