diff --git a/apps/emqx_sn/examples/simple_example2.erl b/apps/emqx_sn/examples/simple_example2.erl
index b9ada6d22..9af9262e9 100644
--- a/apps/emqx_sn/examples/simple_example2.erl
+++ b/apps/emqx_sn/examples/simple_example2.erl
@@ -6,6 +6,7 @@
 -define(PORT, 1884).
 
 -export([start/0]).
+-export([gen_register_packet/2]).
 
 start() ->
     io:format("start to connect ~p:~p~n", [?HOST, ?PORT]),
diff --git a/apps/emqx_sn/examples/simple_example3.erl b/apps/emqx_sn/examples/simple_example3.erl
index 40f0bf572..6bc41f1e8 100644
--- a/apps/emqx_sn/examples/simple_example3.erl
+++ b/apps/emqx_sn/examples/simple_example3.erl
@@ -6,6 +6,7 @@
 -define(PORT, 1884).
 
 -export([start/0]).
+-export([gen_register_packet/2]).
 
 start() ->
     io:format("start to connect ~p:~p~n", [?HOST, ?PORT]),
diff --git a/apps/emqx_sn/examples/simple_example4.erl b/apps/emqx_sn/examples/simple_example4.erl
index 6beb5835c..1b4809626 100644
--- a/apps/emqx_sn/examples/simple_example4.erl
+++ b/apps/emqx_sn/examples/simple_example4.erl
@@ -5,7 +5,7 @@
 -define(HOST, {127,0,0,1}).
 -define(PORT, 1884).
 
--export([start/0]).
+-export([start/1]).
 
 start(LoopTimes) ->
     io:format("start to connect ~p:~p~n", [?HOST, ?PORT]),
diff --git a/src/emqx.appup.src b/src/emqx.appup.src
index c86661891..70d18ff48 100644
--- a/src/emqx.appup.src
+++ b/src/emqx.appup.src
@@ -1,13 +1,17 @@
 %% -*- mode: erlang -*-
 %% Unless you know what you are doing, DO NOT edit manually!!
 {VSN,
-  [{"4.3.18",[{load_module,emqx_app,brutal_purge,soft_purge,[]}]},
+  [
+   {"4.3.18",
+    [{load_module,emqx_app,brutal_purge,soft_purge,[]},
+     {load_module,emqx_plugins,brutal_purge,soft_purge,[]}]},
    {"4.3.17",
     [{load_module,emqx_exclusive_subscription,brutal_purge,soft_purge,[]},
      {load_module,emqx_session,brutal_purge,soft_purge,[]},
      {load_module,emqx_shared_sub,brutal_purge,soft_purge,[]},
      {update,emqx_broker_sup,supervisor},
      {load_module,emqx_app,brutal_purge,soft_purge,[]},
+     {load_module,emqx_plugins,brutal_purge,soft_purge,[]},
      {load_module,emqx_access_control,brutal_purge,soft_purge,[]}]},
    {"4.3.16",
     [{load_module,emqx_session,brutal_purge,soft_purge,[]},
@@ -678,13 +682,17 @@
      {load_module,emqx_message,brutal_purge,soft_purge,[]},
      {load_module,emqx_limiter,brutal_purge,soft_purge,[]}]},
    {<<".*">>,[]}],
-  [{"4.3.18",[{load_module,emqx_app,brutal_purge,soft_purge,[]}]},
+  [
+   {"4.3.18",
+     [{load_module,emqx_app,brutal_purge,soft_purge,[]},
+      {load_module,emqx_plugins,brutal_purge,soft_purge,[]}]},
    {"4.3.17",
     [{load_module,emqx_exclusive_subscription,brutal_purge,soft_purge,[]},
      {load_module,emqx_session,brutal_purge,soft_purge,[]},
      {load_module,emqx_shared_sub,brutal_purge,soft_purge,[]},
      {update,emqx_broker_sup,supervisor},
      {load_module,emqx_app,brutal_purge,soft_purge,[]},
+     {load_module,emqx_plugins,brutal_purge,soft_purge,[]},
      {load_module,emqx_access_control,brutal_purge,soft_purge,[]}]},
    {"4.3.16",
     [{load_module,emqx_session,brutal_purge,soft_purge,[]},
diff --git a/src/emqx_plugins.erl b/src/emqx_plugins.erl
index 3177f05e7..764778b74 100644
--- a/src/emqx_plugins.erl
+++ b/src/emqx_plugins.erl
@@ -219,7 +219,7 @@ ensure_file(File) ->
         false ->
             DefaultPlugins = [ {emqx_management, true}
                              , {emqx_dashboard, true}
-                             , {emqx_modules, false}
+                             , {emqx_modules, true}
                              , {emqx_recon, true}
                              , {emqx_retainer, true}
                              , {emqx_telemetry, true}
@@ -265,7 +265,7 @@ load_plugins(Names, Persistent) ->
         []       -> ok;
         NotFound -> ?LOG(alert, "cannot_find_plugins: ~p", [NotFound])
     end,
-    NeedToLoad = Names -- NotFound -- names(started_app),
+    NeedToLoad = (Names -- NotFound) -- names(started_app),
     lists:foreach(fun(Name) ->
                       Plugin = find_plugin(Name, Plugins),
                       load_plugin(Plugin#plugin.name, Persistent)
diff --git a/test/emqx_broker_sup_SUITE.erl b/test/emqx_broker_sup_SUITE.erl
index 04881f308..20bcfbbf6 100644
--- a/test/emqx_broker_sup_SUITE.erl
+++ b/test/emqx_broker_sup_SUITE.erl
@@ -61,5 +61,5 @@ t_restart_shared_sub(Config) when is_list(Config) ->
        after 2000 ->
                  false
        end);
-t_restart_shared_sub({'end', Config}) ->
+t_restart_shared_sub({'end', _Config}) ->
     emqx:unsubscribe(<<"$share/grpa/t/a">>).
diff --git a/test/emqx_plugins_SUITE.erl b/test/emqx_plugins_SUITE.erl
index cd4b68d86..66a88a047 100644
--- a/test/emqx_plugins_SUITE.erl
+++ b/test/emqx_plugins_SUITE.erl
@@ -100,7 +100,7 @@ t_ensure_default_loaded_plugins_file(Config) ->
        [ {emqx_bridge_mqtt, false}
        , {emqx_dashboard, true}
        , {emqx_management, true}
-       , {emqx_modules, false}
+       , {emqx_modules, true}
        , {emqx_recon, true}
        , {emqx_retainer, true}
        , {emqx_rule_engine, true}