Remove client_index field from replies in API

- replies should not need to contain client_index since
  it is used to identify sender of requests to VPP

Change-Id: Iece3853b3f020054ee1652b149d0cf8d9580db4e
Signed-off-by: Ondrej Fabry <ofabry@cisco.com>

diff --git a/src/plugins/nat/nat.api b/src/plugins/nat/nat.api
index 26d0fba..6a9472b 100644 (file)
--- a/src/plugins/nat/nat.api
+++ b/src/plugins/nat/nat.api
@@ -937,7 +937,6 @@ define nat_det_session_dump {
     @param expire - session expiration timestamp
 */
 define nat_det_session_details {
-  u32 client_index;
   u32 context;
   u16 in_port;
   u8 ext_addr[4];

diff --git a/src/vlibmemory/memclnt.api b/src/vlibmemory/memclnt.api
index cc37c7c..f88e5bd 100644 (file)
--- a/src/vlibmemory/memclnt.api
+++ b/src/vlibmemory/memclnt.api
@@ -105,7 +105,6 @@ define get_first_msg_id {
 };
 
 define get_first_msg_id_reply {
-    u32 client_index;
     u32 context;
     i32 retval;
     u16 first_msg_id;
@@ -125,7 +124,6 @@ define api_versions {
   u32 context;
 };
 define api_versions_reply {
-  u32 client_index;
   u32 context;
   i32 retval;
   u32 count;

diff --git a/src/vnet/bier/bier.api b/src/vnet/bier/bier.api
index d07379e..fb6923b 100644 (file)
--- a/src/vnet/bier/bier.api
+++ b/src/vnet/bier/bier.api
@@ -96,7 +96,6 @@ define bier_route_dump
 
 define bier_route_details
 {
-  u32 client_index;
   u32 context;
   u16 br_bp;
   vl_api_bier_table_id_t br_tbl_id;
@@ -157,7 +156,6 @@ define bier_imp_dump
 
 define bier_imp_details
 {
-  u32 client_index;
   u32 context;
   vl_api_bier_table_id_t bi_tbl_id;
   u16 bi_src;

diff --git a/src/vnet/ip/ip.api b/src/vnet/ip/ip.api
index 923249b..5f664f3 100644 (file)
--- a/src/vnet/ip/ip.api
+++ b/src/vnet/ip/ip.api
@@ -287,7 +287,6 @@ autoreply define ip6nd_proxy_add_del
 */
 define ip6nd_proxy_details
 {
-  u32 client_index;
   u32 context;
   u32 sw_if_index;
   u8 address[16];
@@ -510,7 +509,6 @@ manual_endian manual_print define ip6_mfib_details
 
 define ip_address_details
 {
-  u32 client_index;
   u32 context;
   u8 ip[16];
   u8 prefix_length;
@@ -532,7 +530,6 @@ define ip_address_dump
 */
 define ip_unnumbered_details
 {
-  u32 client_index;
   u32 context;
   u32 sw_if_index;
   u32 ip_sw_if_index;
@@ -570,7 +567,6 @@ define mfib_signal_dump
 
 define mfib_signal_details
 {
-  u32 client_index;
   u32 context;
   u32 sw_if_index;
   u32 table_id;
@@ -1014,7 +1010,6 @@ define ip_reassembly_get
 
 define ip_reassembly_get_reply
 {
-  u32 client_index;
   u32 context;
   i32 retval;
   u32 timeout_ms;

diff --git a/src/vnet/lisp-cp/lisp.api b/src/vnet/lisp-cp/lisp.api
index 0e47585..b1bea09 100644 (file)
--- a/src/vnet/lisp-cp/lisp.api
+++ b/src/vnet/lisp-cp/lisp.api
@@ -626,13 +626,11 @@ define lisp_eid_table_vni_dump
 };
 
 /** \brief reply to lisp_eid_table_vni_dump
-    @param client_index - opaque cookie to identify the sender
     @param context - sender context, to match reply w/ request
     @param vni - virtual network instance
  */
 define lisp_eid_table_vni_details
 {
-  u32 client_index;
   u32 context;
   u32 vni;
 };

diff --git a/src/vnet/lisp-cp/one.api b/src/vnet/lisp-cp/one.api
index 32ac702..7f0f5a9 100644 (file)
--- a/src/vnet/lisp-cp/one.api
+++ b/src/vnet/lisp-cp/one.api
@@ -835,13 +835,11 @@ define one_eid_table_vni_dump
 };
 
 /** \brief reply to one_eid_table_vni_dump
-    @param client_index - opaque cookie to identify the sender
     @param context - sender context, to match reply w/ request
     @param vni - virtual network instance
  */
 define one_eid_table_vni_details
 {
-  u32 client_index;
   u32 context;
   u32 vni;
 };

diff --git a/src/vnet/lisp-gpe/lisp_gpe.api b/src/vnet/lisp-gpe/lisp_gpe.api
index 140d1be..fbfc5e3 100644 (file)
--- a/src/vnet/lisp-gpe/lisp_gpe.api
+++ b/src/vnet/lisp-gpe/lisp_gpe.api
@@ -147,7 +147,6 @@ define gpe_fwd_entry_path_dump
 
 manual_endian manual_print define gpe_fwd_entry_path_details
 {
-  u32 client_index;
   u32 context;
   vl_api_gpe_locator_t lcl_loc;
   vl_api_gpe_locator_t rmt_loc;

diff --git a/src/vnet/session/session.api b/src/vnet/session/session.api
index 58054ae..c60e99e 100644 (file)
--- a/src/vnet/session/session.api
+++ b/src/vnet/session/session.api
@@ -258,14 +258,11 @@ define reset_session {
 };
 
 /** \brief client->vpp reset session reply
-    @param client_index - opaque cookie to identify the sender
-                          client to vpp direction only
     @param context - sender context, to match reply w/ request
     @param retval - return code for the request
     @param handle - session handle obtained via accept/connect
 */
 define reset_session_reply {
-  u32 client_index;
   u32 context;
   i32 retval;
   u64 handle;

diff --git a/src/vnet/session/session_api.c b/src/vnet/session/session_api.c
index d1cbc64..dddc1f5 100755 (executable)
--- a/src/vnet/session/session_api.c
+++ b/src/vnet/session/session_api.c
@@ -955,7 +955,7 @@ vl_api_reset_session_reply_t_handler (vl_api_reset_session_reply_t * mp)
   stream_session_t *s;
   u32 index, thread_index;
 
-  app = application_lookup (mp->client_index);
+  app = application_lookup (mp->context);
   if (!app)
     return;