[core] define core/instance folder for instance modules (openthre…

…ad#9561)

This commit moves the `instance` module to a newly added folder
`core/instance` (from `core/common`.  Header file `extension.hpp`
and its example is also moved to the same folder.

script/check-simulation-build-cmake

@@ -188,7 +188,7 @@ build_all_features()
     reset_source
     "$(dirname "$0")"/cmake-build simulation \
         -DOT_THREAD_VERSION=1.1 \
-        -DOT_VENDOR_EXTENSION=../../src/core/common/extension_example.cpp
+        -DOT_VENDOR_EXTENSION=../../src/core/instance/extension_example.cpp
 
     # Build Thread 1.3 with no additional features
     reset_source
@@ -249,7 +249,7 @@ build_nest_common()
     mkdir build && cd build
     CFLAGS="${cppflags[*]} ${CFLAGS}" CXXFLAGS="${cppflags[*]} ${CXXFLAGS}" \
         cmake -GNinja -DOT_PLATFORM=simulation \
-        -DOT_VENDOR_EXTENSION=common/extension_example.cpp \
+        -DOT_VENDOR_EXTENSION=instance/extension_example.cpp \
         -DOT_NCP_VENDOR_HOOK_SOURCE=example_vendor_hook.cpp \
         ..
     ninja

src/cli/cli.hpp

@@ -82,8 +82,8 @@
 #include "common/array.hpp"
 #include "common/code_utils.hpp"
 #include "common/debug.hpp"
-#include "common/instance.hpp"
 #include "common/type_traits.hpp"
+#include "instance/instance.hpp"
 
 namespace ot {
 

src/core/BUILD.gn

@@ -396,7 +396,6 @@ openthread_core_files = [
   "common/equatable.hpp",
   "common/error.cpp",
   "common/error.hpp",
-  "common/extension.hpp",
   "common/frame_builder.cpp",
   "common/frame_builder.hpp",
   "common/frame_data.cpp",
@@ -409,8 +408,6 @@ openthread_core_files = [
   "common/heap_data.hpp",
   "common/heap_string.cpp",
   "common/heap_string.hpp",
-  "common/instance.cpp",
-  "common/instance.hpp",
   "common/iterator_utils.hpp",
   "common/linked_list.hpp",
   "common/locator.hpp",
@@ -474,6 +471,9 @@ openthread_core_files = [
   "crypto/storage.hpp",
   "diags/factory_diags.cpp",
   "diags/factory_diags.hpp",
+  "instance/extension.hpp",
+  "instance/instance.cpp",
+  "instance/instance.hpp",
   "mac/channel_mask.cpp",
   "mac/channel_mask.hpp",
   "mac/data_poll_handler.cpp",
@@ -737,7 +737,6 @@ openthread_radio_sources = [
   "common/error.hpp",
   "common/frame_builder.cpp",
   "common/frame_builder.hpp",
-  "common/instance.cpp",
   "common/log.cpp",
   "common/random.cpp",
   "common/string.cpp",
@@ -749,6 +748,7 @@ openthread_radio_sources = [
   "crypto/crypto_platform.cpp",
   "crypto/storage.cpp",
   "diags/factory_diags.cpp",
+  "instance/instance.cpp",
   "mac/link_raw.cpp",
   "mac/mac_frame.cpp",
   "mac/mac_types.cpp",

src/core/CMakeLists.txt

@@ -106,7 +106,6 @@ set(COMMON_SOURCES
     common/heap.cpp
     common/heap_data.cpp
     common/heap_string.cpp
-    common/instance.cpp
     common/log.cpp
     common/message.cpp
     common/notifier.cpp
@@ -129,6 +128,7 @@ set(COMMON_SOURCES
     crypto/sha256.cpp
     crypto/storage.cpp
     diags/factory_diags.cpp
+    instance/instance.cpp
     mac/channel_mask.cpp
     mac/data_poll_handler.cpp
     mac/data_poll_sender.cpp
@@ -265,7 +265,6 @@ set(RADIO_COMMON_SOURCES
     common/binary_search.cpp
     common/error.cpp
     common/frame_builder.cpp
-    common/instance.cpp
     common/log.cpp
     common/random.cpp
     common/string.cpp
@@ -277,6 +276,7 @@ set(RADIO_COMMON_SOURCES
     crypto/crypto_platform.cpp
     crypto/storage.cpp
     diags/factory_diags.cpp
+    instance/instance.cpp
     mac/link_raw.cpp
     mac/mac_frame.cpp
     mac/mac_types.cpp

src/core/api/border_router_api.cpp

@@ -39,7 +39,7 @@
 
 #include "border_router/routing_manager.hpp"
 #include "common/debug.hpp"
-#include "common/instance.hpp"
+#include "instance/instance.hpp"
 
 using namespace ot;
 

src/core/api/border_routing_api.cpp

@@ -39,7 +39,7 @@
 #include <openthread/platform/border_routing.h>
 
 #include "border_router/routing_manager.hpp"
-#include "common/instance.hpp"
+#include "instance/instance.hpp"
 
 using namespace ot;
 

src/core/api/dns_api.cpp

@@ -35,7 +35,7 @@
 
 #include <openthread/dns_client.h>
 
-#include "common/instance.hpp"
+#include "instance/instance.hpp"
 #include "net/dns_types.hpp"
 
 using namespace ot;

src/core/api/dns_server_api.cpp

@@ -33,7 +33,7 @@
 
 #include "openthread-core-config.h"
 
-#include "common/instance.hpp"
+#include "instance/instance.hpp"
 #include "net/dns_types.hpp"
 #include "net/dnssd_server.hpp"
 

src/core/api/logging_api.cpp

@@ -35,10 +35,10 @@
 
 #include "common/code_utils.hpp"
 #include "common/debug.hpp"
-#include "common/instance.hpp"
 #include "common/locator_getters.hpp"
 #include "common/log.hpp"
 #include "common/string.hpp"
+#include "instance/instance.hpp"
 
 using namespace ot;
 

src/core/api/nat64_api.cpp

@@ -39,7 +39,7 @@
 
 #include "border_router/routing_manager.hpp"
 #include "common/debug.hpp"
-#include "common/instance.hpp"
+#include "instance/instance.hpp"
 #include "net/ip4_types.hpp"
 #include "net/ip6_headers.hpp"
 #include "net/nat64_translator.hpp"

src/core/api/trel_api.cpp

@@ -39,7 +39,7 @@
 
 #include "common/as_core_type.hpp"
 #include "common/code_utils.hpp"
-#include "common/instance.hpp"
+#include "instance/instance.hpp"
 
 using namespace ot;
 

src/core/backbone_router/bbr_leader.cpp

@@ -35,8 +35,8 @@
 
 #if (OPENTHREAD_CONFIG_THREAD_VERSION >= OT_THREAD_VERSION_1_2)
 
-#include "common/instance.hpp"
 #include "common/locator_getters.hpp"
+#include "instance/instance.hpp"
 
 namespace ot {
 namespace BackboneRouter {

src/core/backbone_router/bbr_local.cpp

@@ -36,10 +36,10 @@
 #if OPENTHREAD_FTD && OPENTHREAD_CONFIG_BACKBONE_ROUTER_ENABLE
 
 #include "common/code_utils.hpp"
-#include "common/instance.hpp"
 #include "common/locator_getters.hpp"
 #include "common/log.hpp"
 #include "common/random.hpp"
+#include "instance/instance.hpp"
 #include "thread/mle_types.hpp"
 #include "thread/thread_netif.hpp"
 

src/core/backbone_router/bbr_manager.cpp

@@ -37,11 +37,11 @@
 
 #include "common/as_core_type.hpp"
 #include "common/code_utils.hpp"
-#include "common/instance.hpp"
 #include "common/locator_getters.hpp"
 #include "common/log.hpp"
 #include "common/num_utils.hpp"
 #include "common/random.hpp"
+#include "instance/instance.hpp"
 #include "thread/mle_types.hpp"
 #include "thread/thread_netif.hpp"
 #include "thread/thread_tlvs.hpp"

src/core/backbone_router/multicast_listeners_table.cpp

@@ -37,10 +37,10 @@
 
 #include "common/array.hpp"
 #include "common/code_utils.hpp"
-#include "common/instance.hpp"
 #include "common/locator_getters.hpp"
 #include "common/log.hpp"
 #include "common/random.hpp"
+#include "instance/instance.hpp"
 #include "thread/mle_types.hpp"
 #include "thread/thread_netif.hpp"
 #include "thread/uri_paths.hpp"

src/core/border_router/infra_if.cpp

@@ -37,9 +37,9 @@
 
 #include "border_router/routing_manager.hpp"
 #include "common/as_core_type.hpp"
-#include "common/instance.hpp"
 #include "common/locator_getters.hpp"
 #include "common/logging.hpp"
+#include "instance/instance.hpp"
 #include "net/icmp6.hpp"
 
 namespace ot {

src/core/border_router/routing_manager.cpp

@@ -44,13 +44,13 @@
 
 #include "common/code_utils.hpp"
 #include "common/debug.hpp"
-#include "common/instance.hpp"
 #include "common/locator_getters.hpp"
 #include "common/log.hpp"
 #include "common/num_utils.hpp"
 #include "common/numeric_limits.hpp"
 #include "common/random.hpp"
 #include "common/settings.hpp"
+#include "instance/instance.hpp"
 #include "meshcop/extended_panid.hpp"
 #include "net/ip6.hpp"
 #include "net/nat64_translator.hpp"

src/core/coap/coap.cpp

@@ -32,10 +32,10 @@
 #include "common/as_core_type.hpp"
 #include "common/code_utils.hpp"
 #include "common/debug.hpp"
-#include "common/instance.hpp"
 #include "common/locator_getters.hpp"
 #include "common/log.hpp"
 #include "common/random.hpp"
+#include "instance/instance.hpp"
 #include "net/ip6.hpp"
 #include "net/udp6.hpp"
 #include "thread/thread_netif.hpp"

src/core/coap/coap_message.cpp

@@ -38,9 +38,9 @@
 #include "common/code_utils.hpp"
 #include "common/debug.hpp"
 #include "common/encoding.hpp"
-#include "common/instance.hpp"
 #include "common/random.hpp"
 #include "common/string.hpp"
+#include "instance/instance.hpp"
 
 namespace ot {
 namespace Coap {

src/core/coap/coap_secure.cpp

@@ -30,10 +30,10 @@
 
 #if OPENTHREAD_CONFIG_DTLS_ENABLE
 
-#include "common/instance.hpp"
 #include "common/locator_getters.hpp"
 #include "common/log.hpp"
 #include "common/new.hpp"
+#include "instance/instance.hpp"
 #include "meshcop/dtls.hpp"
 #include "thread/thread_netif.hpp"
 

src/core/common/heap.cpp

@@ -33,7 +33,7 @@
 
 #include "heap.hpp"
 
-#include "common/instance.hpp"
+#include "instance/instance.hpp"
 
 namespace ot {
 namespace Heap {

src/core/common/locator_getters.hpp

@@ -37,9 +37,9 @@
 
 #include "openthread-core-config.h"
 
-#include "common/instance.hpp"
 #include "common/locator.hpp"
 #include "common/tasklet.hpp"
+#include "instance/instance.hpp"
 
 namespace ot {
 

src/core/common/log.cpp

@@ -38,10 +38,10 @@
 #include <openthread/platform/logging.h>
 
 #include "common/code_utils.hpp"
-#include "common/instance.hpp"
 #include "common/num_utils.hpp"
 #include "common/numeric_limits.hpp"
 #include "common/string.hpp"
+#include "instance/instance.hpp"
 
 /*
  * Verify debug UART dependency.

src/core/common/message.cpp

@@ -37,11 +37,11 @@
 #include "common/code_utils.hpp"
 #include "common/debug.hpp"
 #include "common/heap.hpp"
-#include "common/instance.hpp"
 #include "common/locator_getters.hpp"
 #include "common/log.hpp"
 #include "common/num_utils.hpp"
 #include "common/numeric_limits.hpp"
+#include "instance/instance.hpp"
 #include "net/checksum.hpp"
 #include "net/ip6.hpp"
 

src/core/common/settings.cpp

@@ -35,9 +35,9 @@
 
 #include "common/array.hpp"
 #include "common/code_utils.hpp"
-#include "common/instance.hpp"
 #include "common/locator_getters.hpp"
 #include "common/num_utils.hpp"
+#include "instance/instance.hpp"
 #include "meshcop/dataset.hpp"
 #include "thread/mle.hpp"
 

src/core/common/time_ticker.cpp

@@ -35,9 +35,9 @@
 
 #include "common/code_utils.hpp"
 #include "common/debug.hpp"
-#include "common/instance.hpp"
 #include "common/locator_getters.hpp"
 #include "common/random.hpp"
+#include "instance/instance.hpp"
 #include "thread/mle_router.hpp"
 
 namespace ot {

src/core/common/timer.cpp

@@ -36,8 +36,8 @@
 #include "common/as_core_type.hpp"
 #include "common/code_utils.hpp"
 #include "common/debug.hpp"
-#include "common/instance.hpp"
 #include "common/locator_getters.hpp"
+#include "instance/instance.hpp"
 
 namespace ot {
 

src/core/common/uptime.cpp

@@ -37,8 +37,8 @@
 
 #include "common/code_utils.hpp"
 #include "common/debug.hpp"
-#include "common/instance.hpp"
 #include "common/locator_getters.hpp"
+#include "instance/instance.hpp"
 
 namespace ot {
 

src/core/crypto/crypto_platform.cpp

@@ -51,12 +51,12 @@
 
 #include "common/code_utils.hpp"
 #include "common/debug.hpp"
-#include "common/instance.hpp"
 #include "common/new.hpp"
 #include "config/crypto.h"
 #include "crypto/ecdsa.hpp"
 #include "crypto/hmac_sha256.hpp"
 #include "crypto/storage.hpp"
+#include "instance/instance.hpp"
 
 using namespace ot;
 using namespace Crypto;

src/core/diags/factory_diags.cpp

@@ -43,8 +43,8 @@
 
 #include "common/as_core_type.hpp"
 #include "common/code_utils.hpp"
-#include "common/instance.hpp"
 #include "common/locator_getters.hpp"
+#include "instance/instance.hpp"
 #include "radio/radio.hpp"
 #include "utils/parse_cmdline.hpp"
 

src/core/common/extension_example.cpp → src/core/instance/extension_example.cpp

@@ -37,8 +37,8 @@
 #include <stdint.h>
 
 #include "common/code_utils.hpp"
-#include "common/extension.hpp"
 #include "common/new.hpp"
+#include "instance/extension.hpp"
 
 namespace ot {
 namespace Extension {