VPP-142 Follow up fix for shared_count of indirect adjacencies 21/2121/2
authorJohn Lo <loj@cisco.com>
Thu, 28 Jul 2016 22:41:45 +0000 (18:41 -0400)
committerDave Barach <openvpp@barachs.net>
Fri, 29 Jul 2016 00:25:55 +0000 (00:25 +0000)
Clean up dead code path added in previous attempt.

Change-Id: Ib95b07bfd7699496f6808da97200993e2104d796
Signed-off-by: John Lo <loj@cisco.com>
vnet/vnet/ip/ip4_forward.c
vnet/vnet/ip/ip6_forward.c

index 767cc8f..8b71d2d 100644 (file)
@@ -444,7 +444,6 @@ ip4_add_del_route_next_hop (ip4_main_t * im,
     {
       /* create / delete additional mapping of existing adjacency */
       ip4_add_del_route_args_t a;
-      ip_adjacency_t * nh_adj = ip_get_adjacency (lm, nh_adj_index);
 
       a.table_index_or_table_id = fib_index;
       a.flags = ((is_del ? IP4_ROUTE_FLAG_DEL : IP4_ROUTE_FLAG_ADD)
@@ -459,13 +458,6 @@ ip4_add_del_route_next_hop (ip4_main_t * im,
       a.n_add_adj = 0;
 
       ip4_add_del_route (im, &a);
-
-      /* adjust share count. This cannot be the only use of the adjacency 
-         unless next hop is an indiect adj where share count is already
-         incremented */
-      if (next_hop_sw_if_index != ~0) 
-       nh_adj->share_count += is_del ? -1 : 1;
-        
       goto done;
     }
 
index 57057a7..1e4afa8 100644 (file)
@@ -501,7 +501,6 @@ ip6_add_del_route_next_hop (ip6_main_t * im,
   {
     /* create / delete additional mapping of existing adjacency */
     ip6_add_del_route_args_t a;
-    ip_adjacency_t * nh_adj = ip_get_adjacency (lm, nh_adj_index);
 
     a.table_index_or_table_id = fib_index;
     a.flags = ((is_del ? IP6_ROUTE_FLAG_DEL : IP6_ROUTE_FLAG_ADD)
@@ -516,13 +515,6 @@ ip6_add_del_route_next_hop (ip6_main_t * im,
     a.n_add_adj = 0;
 
     ip6_add_del_route (im, &a);
-
-    /* adjust share count. This cannot be the only use of the adjacency 
-       unless next hop is an indiect adj where share count is already
-       incremented */
-    if (next_hop_sw_if_index != ~0) 
-      nh_adj->share_count += is_del ? -1 : 1;
-
     goto done;
   }