From: Steven Luong <sluong@cisco.com>
Date: Fri, 16 Aug 2019 03:30:23 +0000 (-0700)
Subject: lacp: mark is_mp_safe for show and dump binary API
X-Git-Tag: v19.08.2~357
X-Git-Url: https://gerrit.fd.io/r/gitweb?a=commitdiff_plain;h=refs%2Fchanges%2F44%2F22244%2F2;p=vpp.git

lacp: mark is_mp_safe for show and dump binary API

show and dump binary APIs for lacp neighbors are running in the same thread
as the create and delete interface. There is no need for barrier lock.

Type: fix

Signed-off-by: Steven Luong <sluong@cisco.com>
Change-Id: Id584bd7408210fcc23b464ef2084f11f88bca58b
(cherry picked from commit cda35b38d7b690a63a17967268e8c1339a16dfd2)
---

diff --git a/src/plugins/lacp/cli.c b/src/plugins/lacp/cli.c
index 92a890d2e37..36a6f9bfbdf 100644
--- a/src/plugins/lacp/cli.c
+++ b/src/plugins/lacp/cli.c
@@ -315,6 +315,7 @@ VLIB_CLI_COMMAND (show_lacp_command, static) = {
   .path = "show lacp",
   .short_help = "show lacp [<interface>] [details]",
   .function = show_lacp_fn,
+  .is_mp_safe = 1,
 };
 /* *INDENT-ON* */
 
diff --git a/src/plugins/lacp/lacp_api.c b/src/plugins/lacp/lacp_api.c
index ed3ead626c9..f28f06cb2ca 100644
--- a/src/plugins/lacp/lacp_api.c
+++ b/src/plugins/lacp/lacp_api.c
@@ -190,6 +190,10 @@ lacp_plugin_api_hookup (vlib_main_t * vm)
   foreach_lacp_plugin_api_msg;
 #undef _
 
+  /* Mark these APIs as mp safe */
+  am->is_mp_safe[VL_API_SW_INTERFACE_LACP_DUMP] = 1;
+  am->is_mp_safe[VL_API_SW_INTERFACE_LACP_DETAILS] = 1;
+
   /*
    * Set up the (msg_name, crc, message-id) table
    */