From: Kefu Chai Date: Wed, 5 Sep 2018 11:06:40 +0000 (+0800) Subject: test/crimson: do not use unit.cc as the driver of unittest_seastar_denc X-Git-Tag: v14.0.1~406^2 X-Git-Url: http://git-server-git.apps.pok.os.sepia.ceph.com/?a=commitdiff_plain;h=refs%2Fpull%2F23937%2Fhead;p=ceph.git 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 --- diff --git a/src/test/crimson/CMakeLists.txt b/src/test/crimson/CMakeLists.txt index 618b94236a43..4e8c789ea78b 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)