From b7852ff594a184aa9b0f3a3e01bd1ea2bf9a2a3f Mon Sep 17 00:00:00 2001 From: Kefu Chai Date: Wed, 5 Sep 2018 19:06:40 +0800 Subject: [PATCH] test/crimson: do not use unit.cc as the driver of unittest_seastar_denc as unit.cc initializes the CephContext and all of Ceph's infratructure, which is not necessary for the denc test. also, seastar's builtin allocator only pre-allocates 32 << 20 bytes. it's enough for the denc test, but not necessarily enough for create CephContext and its friends. an option is to use seastar's app template to initialize the memory allocator properly. another option is to avoid initializing CephContext in this test. the latter is simpler. Signed-off-by: Kefu Chai --- src/test/crimson/CMakeLists.txt | 5 ++--- 1 file changed, 2 insertions(+), 3 deletions(-) diff --git a/src/test/crimson/CMakeLists.txt b/src/test/crimson/CMakeLists.txt index 618b94236a4..4e8c789ea78 100644 --- a/src/test/crimson/CMakeLists.txt +++ b/src/test/crimson/CMakeLists.txt @@ -4,10 +4,9 @@ add_ceph_unittest(unittest_seastar_buffer) target_link_libraries(unittest_seastar_buffer ceph-common crimson) add_executable(unittest_seastar_denc - test_denc.cc - $) + test_denc.cc) add_ceph_unittest(unittest_seastar_denc) -target_link_libraries(unittest_seastar_denc ceph-common global crimson) +target_link_libraries(unittest_seastar_denc crimson GTest::Main) add_executable(unittest_seastar_messenger test_messenger.cc) add_ceph_unittest(unittest_seastar_messenger) -- 2.39.5