From cada9eb7894117db898f7c4def92cba5511baa4f Mon Sep 17 00:00:00 2001 From: Damjan Marion Date: Wed, 18 May 2022 22:16:11 +0200 Subject: api: refactor api data storage single struct to hold all api handler, flags, etc. Provide functions to toggle flags instead of writing directly to internal data. Type: refactor Change-Id: I4730d7290e57489de8eda34a72211527e015b721 Signed-off-by: Damjan Marion --- src/plugins/arping/arping_api.c | 2 +- src/plugins/avf/avf_api.c | 2 +- src/plugins/geneve/geneve_api.c | 4 ++-- src/plugins/lacp/lacp_api.c | 3 ++- src/plugins/tracedump/graph_api.c | 2 +- src/plugins/tracedump/tracedump.c | 2 +- 6 files changed, 8 insertions(+), 7 deletions(-) (limited to 'src/plugins') diff --git a/src/plugins/arping/arping_api.c b/src/plugins/arping/arping_api.c index e0c47c7cf33..50fbb04115b 100644 --- a/src/plugins/arping/arping_api.c +++ b/src/plugins/arping/arping_api.c @@ -74,7 +74,7 @@ arping_plugin_api_hookup (vlib_main_t *vm) am->msg_id_base = setup_message_id_table (); /* Mark API as mp safe */ - vam->is_mp_safe[am->msg_id_base + VL_API_ARPING] = 1; + vl_api_set_msg_thread_safe (vam, am->msg_id_base + VL_API_ARPING, 1); return 0; } diff --git a/src/plugins/avf/avf_api.c b/src/plugins/avf/avf_api.c index 366a983f373..05b4ead90fb 100644 --- a/src/plugins/avf/avf_api.c +++ b/src/plugins/avf/avf_api.c @@ -95,7 +95,7 @@ avf_plugin_api_hookup (vlib_main_t * vm) /* ask for a correctly-sized block of API message decode slots */ avm->msg_id_base = setup_message_id_table (); - am->is_mp_safe[avm->msg_id_base + VL_API_AVF_DELETE] = 1; + vl_api_set_msg_thread_safe (am, avm->msg_id_base + VL_API_AVF_DELETE, 1); return 0; } diff --git a/src/plugins/geneve/geneve_api.c b/src/plugins/geneve/geneve_api.c index d35a1bf5dbc..5a983948c26 100644 --- a/src/plugins/geneve/geneve_api.c +++ b/src/plugins/geneve/geneve_api.c @@ -238,8 +238,8 @@ geneve_api_hookup (vlib_main_t * vm) */ geneve_base_msg_id = setup_message_id_table (); - am->api_trace_cfg[VL_API_GENEVE_ADD_DEL_TUNNEL + REPLY_MSG_ID_BASE].size += - 16 * sizeof (u32); + vl_api_increase_msg_trace_size ( + am, VL_API_GENEVE_ADD_DEL_TUNNEL + REPLY_MSG_ID_BASE, 16 * sizeof (u32)); return 0; } diff --git a/src/plugins/lacp/lacp_api.c b/src/plugins/lacp/lacp_api.c index fce2492be8c..fab28cb6a13 100644 --- a/src/plugins/lacp/lacp_api.c +++ b/src/plugins/lacp/lacp_api.c @@ -125,7 +125,8 @@ lacp_plugin_api_hookup (vlib_main_t * vm) lm->msg_id_base = setup_message_id_table (); /* Mark these APIs as mp safe */ - am->is_mp_safe[lm->msg_id_base + VL_API_SW_INTERFACE_LACP_DUMP] = 1; + vl_api_set_msg_thread_safe ( + am, lm->msg_id_base + VL_API_SW_INTERFACE_LACP_DUMP, 1); return 0; } diff --git a/src/plugins/tracedump/graph_api.c b/src/plugins/tracedump/graph_api.c index 0626f7d6b42..20eb1b920fc 100644 --- a/src/plugins/tracedump/graph_api.c +++ b/src/plugins/tracedump/graph_api.c @@ -252,7 +252,7 @@ graph_api_hookup (vlib_main_t * vm) gmp->msg_id_base = setup_message_id_table (); - am->is_mp_safe[gmp->msg_id_base + VL_API_GRAPH_NODE_GET] = 1; + vl_api_set_msg_thread_safe (am, gmp->msg_id_base + VL_API_GRAPH_NODE_GET, 1); return 0; } diff --git a/src/plugins/tracedump/tracedump.c b/src/plugins/tracedump/tracedump.c index f1073fe247d..17b9a423b53 100644 --- a/src/plugins/tracedump/tracedump.c +++ b/src/plugins/tracedump/tracedump.c @@ -349,7 +349,7 @@ tracedump_init (vlib_main_t * vm) /* Add our API messages to the global name_crc hash table */ tdmp->msg_id_base = setup_message_id_table (); - am->is_mp_safe[tdmp->msg_id_base + VL_API_TRACE_DUMP] = 1; + vl_api_set_msg_thread_safe (am, tdmp->msg_id_base + VL_API_TRACE_DUMP, 1); return error; } -- cgit 1.2.3-korg