commit 10db26f7bfed97022734fb808bd56532fdda48c5
author Klement Sekera <ksekera@cisco.com> Wed Jan 11 08:16:53 2017 +0100
committer Damjan Marion <dmarion.lists@gmail.com> Wed Jan 11 19:49:31 2017 +0000
tree 7a9bd58b90229ebc0838bb1563dc40a7bd8ffc76
parent cab65ec86b6c04d7a4674312989b5be0c0e394a4

BFD: fix bfd_udp_add API

Fix reporting of bs_index in the return message. Enhance test suite
to cover this case.

Change-Id: I37d35b850818bc1a05abe67ca919c22aeac978b6
Signed-off-by: Klement Sekera <ksekera@cisco.com>

src/vnet/bfd/bfd_api.c

diff --git a/src/vnet/bfd/bfd_api.c b/src/vnet/bfd/bfd_api.c
index 126cf29..2e63fe9 100644
--- a/src/vnet/bfd/bfd_api.c
+++ b/src/vnet/bfd/bfd_api.c
@@ -43,12 +43,12 @@
 
 #include <vlibapi/api_helper_macros.h>
 
-#define foreach_vpe_api_msg                             \
-_(BFD_UDP_ADD, bfd_udp_add)                             \
-_(BFD_UDP_DEL, bfd_udp_del)                             \
-_(BFD_UDP_SESSION_DUMP, bfd_udp_session_dump)           \
-_(BFD_SESSION_SET_FLAGS, bfd_session_set_flags)         \
-_(WANT_BFD_EVENTS, want_bfd_events)
+#define foreach_vpe_api_msg                        \
+  _ (BFD_UDP_ADD, bfd_udp_add)                     \
+  _ (BFD_UDP_DEL, bfd_udp_del)                     \
+  _ (BFD_UDP_SESSION_DUMP, bfd_udp_session_dump)   \
+  _ (BFD_SESSION_SET_FLAGS, bfd_session_set_flags) \
+  _ (WANT_BFD_EVENTS, want_bfd_events)
 
 pub_sub_handler (bfd_events, BFD_EVENTS);
 
@@ -75,13 +75,16 @@
       clib_memcpy (&peer_addr.ip4, mp->peer_addr, sizeof (peer_addr.ip4));
     }
 
+  u32 bs_index = 0;
   rv = bfd_udp_add_session (clib_net_to_host_u32 (mp->sw_if_index),
 			    clib_net_to_host_u32 (mp->desired_min_tx),
 			    clib_net_to_host_u32 (mp->required_min_rx),
-			    mp->detect_mult, &local_addr, &peer_addr);
+			    mp->detect_mult, &local_addr, &peer_addr,
+			    &bs_index);
 
   BAD_SW_IF_INDEX_LABEL;
-  REPLY_MACRO (VL_API_BFD_UDP_ADD_REPLY);
+  REPLY_MACRO2 (VL_API_BFD_UDP_ADD_REPLY,
+		rmp->bs_index = clib_host_to_net_u32 (bs_index));
 }
 
 static void
@@ -107,9 +110,8 @@
       clib_memcpy (&peer_addr.ip4, mp->peer_addr, sizeof (peer_addr.ip4));
     }
 
-  rv =
-    bfd_udp_del_session (clib_net_to_host_u32 (mp->sw_if_index), &local_addr,
-			 &peer_addr);
+  rv = bfd_udp_del_session (clib_net_to_host_u32 (mp->sw_if_index),
+			    &local_addr, &peer_addr);
 
   BAD_SW_IF_INDEX_LABEL;
   REPLY_MACRO (VL_API_BFD_UDP_DEL_REPLY);
@@ -201,14 +203,12 @@
   vl_api_bfd_session_set_flags_reply_t *rmp;
   int rv;
 
-  rv =
-    bfd_session_set_flags (clib_net_to_host_u32 (mp->bs_index),
-			   mp->admin_up_down);
+  rv = bfd_session_set_flags (clib_net_to_host_u32 (mp->bs_index),
+			      mp->admin_up_down);
 
   REPLY_MACRO (VL_API_BFD_SESSION_SET_FLAGS_REPLY);
 }
 
-
 /*
  * bfd_api_hookup
  * Add vpe's API message handlers to the table.
@@ -223,7 +223,7 @@
 static void
 setup_message_id_table (api_main_t * am)
 {
-#define _(id,n,crc) vl_msg_api_add_msg_name_crc (am, #n "_" #crc, id);
+#define _(id, n, crc) vl_msg_api_add_msg_name_crc (am, #n "_" #crc, id);
   foreach_vl_msg_name_crc_bfd;
 #undef _
 }
@@ -233,13 +233,10 @@
 {
   api_main_t *am = &api_main;
 
-#define _(N,n)                                                  \
-    vl_msg_api_set_handlers(VL_API_##N, #n,                     \
-                           vl_api_##n##_t_handler,              \
-                           vl_noop_handler,                     \
-                           vl_api_##n##_t_endian,               \
-                           vl_api_##n##_t_print,                \
-                           sizeof(vl_api_##n##_t), 1);
+#define _(N, n)                                                    \
+  vl_msg_api_set_handlers (VL_API_##N, #n, vl_api_##n##_t_handler, \
+                           vl_noop_handler, vl_api_##n##_t_endian, \
+                           vl_api_##n##_t_print, sizeof (vl_api_##n##_t), 1);
   foreach_vpe_api_msg;
 #undef _