commit 13e6fce7c5b3a16a6af0b27fc259ef3f65d8c861
author Ahmed Abdelsalam <ahabdels@cisco.com> Sun Dec 08 12:58:27 2019 +0100
committer Neale Ranns <nranns@cisco.com> Thu Jan 16 20:31:08 2020 +0000
tree 6e35602be83b56adf71e1a96b4aa742c9418f25e
parent 1bf83b9d9bd9013cd07435c5bf0c5a584906857a

sr: some fixes for SRv6 CLI/API

Return FIB table_id instead of vrf_index to clients

Type: fix
Signed-off-by: Ahmed Abdelsalam <ahabdels@cisco.com>
Change-Id: I76a97bad3ecd3ac8eb045efb1657eaa90c2a57b6

src/vnet/srv6/sr_api.c

diff --git a/src/vnet/srv6/sr_api.c b/src/vnet/srv6/sr_api.c
index ffd0c4d..606eaa6 100644
--- a/src/vnet/srv6/sr_api.c
+++ b/src/vnet/srv6/sr_api.c
@@ -24,6 +24,7 @@
 #include <vnet/interface.h>
 #include <vnet/api_errno.h>
 #include <vnet/feature/feature.h>
+#include <vnet/fib/fib_table.h>
 
 #include <vnet/vnet_msg_enum.h>
 
@@ -239,7 +240,16 @@
   else
     clib_memcpy (rmp->xconnect_nh_addr6, &t->next_hop.ip6,
 		 sizeof (ip6_address_t));
-  rmp->xconnect_iface_or_vrf_table = htonl (t->sw_if_index);
+
+  if (t->behavior == SR_BEHAVIOR_T || t->behavior == SR_BEHAVIOR_DT6)
+    rmp->xconnect_iface_or_vrf_table =
+      htonl (fib_table_get_table_id (t->sw_if_index, FIB_PROTOCOL_IP6));
+  else if (t->behavior == SR_BEHAVIOR_DT4)
+    rmp->xconnect_iface_or_vrf_table =
+      htonl (fib_table_get_table_id (t->sw_if_index, FIB_PROTOCOL_IP4));
+  else
+    rmp->xconnect_iface_or_vrf_table = htonl (t->sw_if_index);
+
   rmp->context = context;
 
   vl_api_send_msg (reg, (u8 *) rmp);

src/vnet/srv6/sr_localsid.c

diff --git a/src/vnet/srv6/sr_localsid.c b/src/vnet/srv6/sr_localsid.c
index 05651bb..79e4c2a 100755
--- a/src/vnet/srv6/sr_localsid.c
+++ b/src/vnet/srv6/sr_localsid.c
@@ -504,7 +504,9 @@
 	  vlib_cli_output (vm,
 			   "\tAddress: \t%U\n\tBehavior: \tT (Endpoint with specific IPv6 table lookup)"
 			   "\n\tTable:  \t%u",
-			   format_ip6_address, &ls->localsid, ls->vrf_index);
+			   format_ip6_address, &ls->localsid,
+			   fib_table_get_table_id (ls->vrf_index,
+						   FIB_PROTOCOL_IP6));
 	  break;
 	case SR_BEHAVIOR_DX4:
 	  vlib_cli_output (vm,
@@ -537,13 +539,16 @@
 	  vlib_cli_output (vm,
 			   "\tAddress: \t%U\n\tBehavior: \tDT6 (Endpoint with decapsulation and specific IPv6 table lookup)"
 			   "\n\tTable: %u", format_ip6_address, &ls->localsid,
-			   ls->vrf_index);
+			   fib_table_get_table_id (ls->vrf_index,
+						   FIB_PROTOCOL_IP6));
 	  break;
 	case SR_BEHAVIOR_DT4:
 	  vlib_cli_output (vm,
 			   "\tAddress: \t%U\n\tBehavior: \tDT4 (Endpoint with decapsulation and specific IPv4 table lookup)"
 			   "\n\tTable: \t%u", format_ip6_address,
-			   &ls->localsid, ls->vrf_index);
+			   &ls->localsid,
+			   fib_table_get_table_id (ls->vrf_index,
+						   FIB_PROTOCOL_IP4));
 	  break;
 	default:
 	  if (ls->behavior >= SR_BEHAVIOR_LAST)