diff options
| author |   Damjan Marion <damarion@cisco.com> | 2022-05-18 22:16:11 +0200 |
|---|---|---|
| committer | Damjan Marion <damarion@cisco.com> | 2022-05-19 18:32:23 +0200 |
| commit | cada9eb7894117db898f7c4def92cba5511baa4f (patch) | |
| tree | 4cf552e2ad9e67b8bbc144eb5c6cb9aad0a8424b /src/vnet/interface_api.c | |
| parent | bf95e3efde3402cf2f7beaf6d70433646cc68280 (diff) | |
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 <damarion@cisco.com>
Diffstat (limited to 'src/vnet/interface_api.c')
| -rw-r--r-- | src/vnet/interface_api.c | 25 |
1 files changed, 14 insertions, 11 deletions
diff --git a/src/vnet/interface_api.c b/src/vnet/interface_api.c index 29f98f7851f..5766f2ca21f 100644 --- a/src/vnet/interface_api.c +++ b/src/vnet/interface_api.c @@ -1630,21 +1630,24 @@ interface_api_hookup (vlib_main_t * vm) REPLY_MSG_ID_BASE = setup_message_id_table (); /* Mark these APIs as mp safe */ - am->is_mp_safe[REPLY_MSG_ID_BASE + VL_API_SW_INTERFACE_DUMP] = 1; - am->is_mp_safe[REPLY_MSG_ID_BASE + VL_API_SW_INTERFACE_DETAILS] = 1; - am->is_mp_safe[REPLY_MSG_ID_BASE + VL_API_SW_INTERFACE_TAG_ADD_DEL] = 1; - am->is_mp_safe[REPLY_MSG_ID_BASE + VL_API_SW_INTERFACE_SET_INTERFACE_NAME] = - 1; + vl_api_set_msg_thread_safe (am, REPLY_MSG_ID_BASE + VL_API_SW_INTERFACE_DUMP, + 1); + vl_api_set_msg_thread_safe ( + am, REPLY_MSG_ID_BASE + VL_API_SW_INTERFACE_DETAILS, 1); + vl_api_set_msg_thread_safe ( + am, REPLY_MSG_ID_BASE + VL_API_SW_INTERFACE_TAG_ADD_DEL, 1); + vl_api_set_msg_thread_safe ( + am, REPLY_MSG_ID_BASE + VL_API_SW_INTERFACE_SET_INTERFACE_NAME, 1); /* Do not replay VL_API_SW_INTERFACE_DUMP messages */ - am->api_trace_cfg[REPLY_MSG_ID_BASE + VL_API_SW_INTERFACE_DUMP] - .replay_enable = 0; + vl_api_allow_msg_replay (am, REPLY_MSG_ID_BASE + VL_API_SW_INTERFACE_DUMP, + 0); /* Mark these APIs as autoendian */ - am->is_autoendian[REPLY_MSG_ID_BASE + VL_API_SW_INTERFACE_SET_TX_PLACEMENT] = - 1; - am->is_autoendian[REPLY_MSG_ID_BASE + VL_API_SW_INTERFACE_TX_PLACEMENT_GET] = - 1; + vl_api_set_msg_autoendian ( + am, REPLY_MSG_ID_BASE + VL_API_SW_INTERFACE_SET_TX_PLACEMENT, 1); + vl_api_set_msg_autoendian ( + am, REPLY_MSG_ID_BASE + VL_API_SW_INTERFACE_TX_PLACEMENT_GET, 1); return 0; } |