diff options
| author |   Dave Barach <dave@barachs.net> | 2019-07-09 10:04:18 -0400 |
|---|---|---|
| committer |   Florin Coras <florin.coras@gmail.com> | 2019-07-09 15:24:41 +0000 |
| commit | 21a438465ca0dee88d13d9dd9621929116c95614 (patch) | |
| tree | 863a47f5c77e24b7fdee96db22080ff0db52e397 /src/plugins/vmxnet3 | |
| parent | 3f5ebed6ea3a024b7396afe81f895719a8286681 (diff) | |
vat: unload unused vat plugins
If the corresponding vpp plugin is absent, return a non-zero
clib_error_t * from vat_plugin_register ("xxx plugin not loaded"). The
vat plugin calls dlclose on the vat plugin, and it disappears.
Depending on the plugin configuration, this can reduce the vpp virtual
size by several gigabytes.
Added a VAT_PLUGIN(<plugin-name>) macro to vat_helper_macros, clean up
boilerplate vat_plugin_register() implementations. Fixed a number of
non-standard vat_plugin_register methods.
Type: refactor
Change-Id: Iac908e5af7d5497c78d6aa9c3c51cdae08374045
Signed-off-by: Dave Barach <dave@barachs.net>
Diffstat (limited to 'src/plugins/vmxnet3')
| -rw-r--r-- | src/plugins/vmxnet3/vmxnet3_test.c | 9 |
1 files changed, 5 insertions, 4 deletions
diff --git a/src/plugins/vmxnet3/vmxnet3_test.c b/src/plugins/vmxnet3/vmxnet3_test.c index d32bceefe5b..7a70f619a55 100644 --- a/src/plugins/vmxnet3/vmxnet3_test.c +++ b/src/plugins/vmxnet3/vmxnet3_test.c @@ -330,6 +330,10 @@ vat_plugin_register (vat_main_t * vam) name = format (0, "vmxnet3_%08x%c", api_version, 0); vxm->msg_id_base = vl_client_get_first_plugin_msg_id ((char *) name); + vec_free (name); + + if (vxm->msg_id_base == (u16) ~ 0) + return clib_error_return (0, "vmxnet3 plugin not loaded..."); /* Get the control ping ID */ #define _(id,n,crc) \ @@ -338,10 +342,7 @@ vat_plugin_register (vat_main_t * vam) #undef _ vxm->ping_id = vl_msg_api_get_msg_index ((u8 *) (VL_API_CONTROL_PING_CRC)); - if (vxm->msg_id_base != (u16) ~ 0) - vmxnet3_vat_api_hookup (vam); - - vec_free (name); + vmxnet3_vat_api_hookup (vam); return 0; } |