misc: fix init order to avoid startup warnings 18/32818/4
authorBin Zhou (bzhou2) <bzhou2@cisco.com>
Mon, 21 Jun 2021 15:09:30 +0000 (11:09 -0400)
committerDamjan Marion <dmarion@me.com>
Tue, 13 Jul 2021 17:16:54 +0000 (17:16 +0000)
Put plugin init order inside plugin instead of in vnet

Type: improvement

Signed-off-by: Bin Zhou (bzhou2) <bzhou2@cisco.com>
Change-Id: Icbacdb3f1cb4ac9d74e3f78458e8bc333793b4d6

src/plugins/acl/dataplane_node.c
src/plugins/dns/dns.c
src/vnet/dpo/dvr_dpo.c
src/vnet/ip/ip_init.c

index 25fa4e9..3a351a0 100644 (file)
@@ -945,11 +945,10 @@ VLIB_REGISTER_NODE (acl_out_fa_ip6_node) =
   }
 };
 
-VNET_FEATURE_INIT (acl_out_ip6_fa_feature, static) =
-{
+VNET_FEATURE_INIT (acl_out_ip6_fa_feature, static) = {
   .arc_name = "ip6-output",
   .node_name = "acl-plugin-out-ip6-fa",
-  .runs_before = VNET_FEATURES ("interface-output"),
+  .runs_before = VNET_FEATURES ("ip6-dvr-reinject", "interface-output"),
 };
 
 VLIB_REGISTER_NODE (acl_out_fa_ip4_node) =
@@ -968,11 +967,10 @@ VLIB_REGISTER_NODE (acl_out_fa_ip4_node) =
   }
 };
 
-VNET_FEATURE_INIT (acl_out_ip4_fa_feature, static) =
-{
+VNET_FEATURE_INIT (acl_out_ip4_fa_feature, static) = {
   .arc_name = "ip4-output",
   .node_name = "acl-plugin-out-ip4-fa",
-  .runs_before = VNET_FEATURES ("interface-output"),
+  .runs_before = VNET_FEATURES ("ip4-dvr-reinject", "interface-output"),
 };
 
 /* *INDENT-ON* */
index 0684a10..0801681 100644 (file)
@@ -3020,9 +3020,11 @@ dns_init (vlib_main_t * vm)
   return 0;
 }
 
-VLIB_INIT_FUNCTION (dns_init);
-
 /* *INDENT-OFF* */
+VLIB_INIT_FUNCTION (dns_init) = {
+  .init_order = VLIB_INITS ("flow_classify_init", "dns_init"),
+};
+
 VLIB_PLUGIN_REGISTER () =
 {
   .version = VPP_BUILD_VER,
index 533ce4b..5db9c80 100644 (file)
@@ -632,13 +632,10 @@ VNET_FEATURE_INIT (ip4_dvr_reinject_feat_node, static) =
 {
   .arc_name = "ip4-output",
   .node_name = "ip4-dvr-reinject",
-  .runs_after = VNET_FEATURES ("nat44-in2out-output",
-                               "acl-plugin-out-ip4-fa"),
 };
 VNET_FEATURE_INIT (ip6_dvr_reinject_feat_node, static) =
 {
   .arc_name = "ip6-output",
   .node_name = "ip6-dvr-reinject",
-  .runs_after = VNET_FEATURES ("acl-plugin-out-ip6-fa"),
 };
 
index b3ab90d..8894a87 100644 (file)
@@ -105,22 +105,12 @@ do {                                              \
 }
 
 /* *INDENT-OFF* */
-VLIB_INIT_FUNCTION (ip_main_init) =
-{
-  .init_order =
-  VLIB_INITS ("vnet_main_init",
-              "ip4_init",
-              "ip6_init",
-              "icmp4_init",
-              "icmp6_init",
-              "ip6_hop_by_hop_init",
-              "udp_local_init",
-              "udp_init",
-              "ip_classify_init",
-              "in_out_acl_init",
-              "policer_classify_init",
-              "flow_classify_init",
-              "dns_init"),
+VLIB_INIT_FUNCTION (ip_main_init) = {
+  .init_order = VLIB_INITS ("vnet_main_init", "ip4_init", "ip6_init",
+                           "icmp4_init", "icmp6_init", "ip6_hop_by_hop_init",
+                           "udp_local_init", "udp_init", "ip_classify_init",
+                           "in_out_acl_init", "policer_classify_init",
+                           "flow_classify_init"),
 };
 /* *INDENT-ON* */