refactor: make aio decoupled from dsn_runtime

src/replica/replica_stub.cpp

@@ -60,6 +60,7 @@
 #endif
 #include <dsn/utility/fail_point.h>
 #include <dsn/dist/remote_command.h>
+#include <aio/disk_engine.h>
 
 namespace dsn {
 namespace replication {
@@ -87,6 +88,11 @@ replica_stub::replica_stub(replica_state_subscriber subscriber /*= nullptr*/,
       _fs_manager(false),
       _bulk_load_downloading_count(0)
 {
+
+    // make disk_engine destructed after replica_stub,
+    // because replica_stub relies on the former to close files.
+    disk_engine::instance();
+
 #ifdef DSN_ENABLE_GPERF
     _release_tcmalloc_memory_command = nullptr;
     _max_reserved_memory_percentage_command = nullptr;

src/runtime/CMakeLists.txt

@@ -30,5 +30,5 @@ add_library(dsn_runtime STATIC
         tracer.cpp
         zlocks.cpp
         )
-target_link_libraries(dsn_runtime PRIVATE dsn_utils sasl2 gssapi_krb5 krb5 dsn_aio)
+target_link_libraries(dsn_runtime PRIVATE dsn_utils sasl2 gssapi_krb5 krb5)
 install(TARGETS dsn_runtime DESTINATION "lib")

src/runtime/service_api_c.cpp

@@ -24,7 +24,6 @@
  * THE SOFTWARE.
  */
 
-#include "aio/disk_engine.h"
 #include "service_engine.h"
 #include "utils/coredump.h"
 #include "runtime/rpc/rpc_engine.h"
@@ -292,11 +291,10 @@ extern void dsn_core_init();
 
 inline void dsn_global_init()
 {
-    // make perf_counters/disk_engine destructed after service_engine,
+    // make perf_counters destructed after service_engine,
     // because service_engine relies on the former to monitor
-    // task queues length and close files.
+    // task queues length.
     dsn::perf_counters::instance();
-    dsn::disk_engine::instance();
     dsn::service_engine::instance();
 }