// Luanti // SPDX-License-Identifier: LGPL-2.1-or-later // Copyright (C) 2013 celeron55, Perttu Ahola #include "test.h" #include #include #if defined(IPC_CHANNEL_IMPLEMENTATION_WIN32) || defined(_WIN32) #include #endif #include "threading/ipc_channel.h" #include "threading/semaphore.h" #include "threading/thread.h" class TestThreading : public TestBase { public: TestThreading() { TestManager::registerTestModule(this); } const char *getName() { return "TestThreading"; } void runTests(IGameDef *gamedef); void testStartStopWait(); void testAtomicSemaphoreThread(); void testTLS(); void testIPCChannel(); }; static TestThreading g_test_instance; void TestThreading::runTests(IGameDef *gamedef) { TEST(testStartStopWait); TEST(testAtomicSemaphoreThread); TEST(testTLS); TEST(testIPCChannel); } class SimpleTestThread : public Thread { public: SimpleTestThread(unsigned int interval) : Thread("SimpleTest"), m_interval(interval) { } private: void *run() { void *retval = this; if (isCurrentThread() == false) retval = (void *)0xBAD; while (!stopRequested()) sleep_ms(m_interval); return retval; } unsigned int m_interval; }; void TestThreading::testStartStopWait() { void *thread_retval; SimpleTestThread *thread = new SimpleTestThread(25); // Try this a couple times, since a Thread should be reusable after waiting for (size_t i = 0; i != 5; i++) { // Can't wait() on a joined, stopped thread UASSERT(thread->wait() == false); // start() should work the first time, but not the second. UASSERT(thread->start() == true); UASSERT(thread->start() == false); UASSERT(thread->isRunning() == true); UASSERT(thread->isCurrentThread() == false); // Let it loop a few times... sleep_ms(70); // It's still running, so the return value shouldn't be available to us. UASSERT(thread->getReturnValue(&thread_retval) == false); // stop() should always succeed UASSERT(thread->stop() == true); // wait() only needs to wait the first time - the other two are no-ops. UASSERT(thread->wait() == true); UASSERT(thread->wait() == false); UASSERT(thread->wait() == false); // Now that the thread is stopped, we should be able to get the // return value, and it should be the object itself. thread_retval = NULL; UASSERT(thread->getReturnValue(&thread_retval) == true); UASSERT(thread_retval == thread); } delete thread; } class AtomicTestThread : public Thread { public: AtomicTestThread(std::atomic &v, Semaphore &trigger) : Thread("AtomicTest"), val(v), trigger(trigger) { } private: void *run() { trigger.wait(); for (u32 i = 0; i < 0x10000; ++i) ++val; return NULL; } std::atomic &val; Semaphore &trigger; }; void TestThreading::testAtomicSemaphoreThread() { std::atomic val; val = 0; Semaphore trigger; static const u8 num_threads = 4; AtomicTestThread *threads[num_threads]; for (auto &thread : threads) { thread = new AtomicTestThread(val, trigger); UASSERT(thread->start()); } trigger.post(num_threads); for (AtomicTestThread *thread : threads) { thread->wait(); delete thread; } UASSERT(val == num_threads * 0x10000); } static std::atomic g_tls_broken; class TLSTestThread : public Thread { public: TLSTestThread() : Thread("TLSTest") { } private: struct TestObject { TestObject() { for (u32 i = 0; i < buffer_size; i++) buffer[i] = (i % 2) ? 0xa1 : 0x1a; } ~TestObject() { for (u32 i = 0; i < buffer_size; i++) { const u8 expect = (i % 2) ? 0xa1 : 0x1a; if (buffer[i] != expect) { std::cout << "At offset " << i << " expected " << (int)expect << " but found " << (int)buffer[i] << std::endl; g_tls_broken = true; break; } // If we're unlucky the loop might actually just crash. // probably the user will realize the test failure :) } } // larger objects seem to surface corruption more easily static constexpr u32 buffer_size = 576; u8 buffer[buffer_size]; }; void *run() { thread_local TestObject foo; while (!stopRequested()) sleep_ms(1); return nullptr; } }; /* What are we actually testing here? MinGW with gcc has a long-standing unsolved bug where memory belonging to thread-local variables is freed *before* the destructors are called. Needless to say this leads to unreliable crashes whenever a thread exits. It does not affect MSVC or MinGW+clang and as far as we know no other platforms are affected either. Related reports and information: * (2018) * (2017) * maybe Note that this is different from , which affected only 32-bit MinGW. It was caused by incorrect calling convention and fixed in GCC in 2020. Occurrences in Luanti: * (2020) * (2022) * (2023) */ void TestThreading::testTLS() { constexpr int num_threads = 10; for (int j = 0; j < num_threads; j++) { g_tls_broken = false; TLSTestThread thread; thread.start(); thread.stop(); thread.wait(); if (g_tls_broken) { std::cout << "While running test thread " << j << std::endl; UASSERT(!g_tls_broken); } } } void TestThreading::testIPCChannel() { struct Stuff { IPCChannelShared shared{}; #if defined(IPC_CHANNEL_IMPLEMENTATION_WIN32) HANDLE sem_a; HANDLE sem_b; #endif Stuff() { #ifdef IPC_CHANNEL_IMPLEMENTATION_WIN32 HANDLE sem_a = CreateSemaphoreA(nullptr, 0, 1, nullptr); UASSERT(sem_a != INVALID_HANDLE_VALUE); HANDLE sem_b = CreateSemaphoreA(nullptr, 0, 1, nullptr); UASSERT(sem_b != INVALID_HANDLE_VALUE); #endif } ~Stuff() { #ifdef IPC_CHANNEL_IMPLEMENTATION_WIN32 CloseHandle(sem_b); CloseHandle(sem_a); #endif } }; struct IPCChannelStuffSingleProcess final : public IPCChannelStuff { std::shared_ptr stuff; IPCChannelStuffSingleProcess(std::shared_ptr stuff) : stuff(std::move(stuff)) {} ~IPCChannelStuffSingleProcess() override = default; IPCChannelShared *getShared() override { return &stuff->shared; } #if defined(IPC_CHANNEL_IMPLEMENTATION_WIN32) HANDLE getSemA() override { return stuff->sem_a; } HANDLE getSemB() override { return stuff->sem_b; } #endif }; auto stuff = std::make_shared(); IPCChannelEnd end_a = IPCChannelEnd::makeA(std::make_unique(stuff)); std::thread thread_b([=] { IPCChannelEnd end_b = IPCChannelEnd::makeB(std::make_unique(stuff)); for (;;) { UASSERT(end_b.recv()); UASSERT(end_b.send(end_b.getRecvData(), end_b.getRecvSize())); if (end_b.getRecvSize() == 0) break; } }); char buf[20000] = {}; for (int i = sizeof(buf); i > 0; i -= 1000) { buf[i - 1] = 123; UASSERT(end_a.exchange(buf, i)); UASSERTEQ(int, end_a.getRecvSize(), i); UASSERTEQ(int, ((const char *)end_a.getRecvData())[i - 1], 123); } UASSERT(end_a.exchange(buf, 0)); UASSERTEQ(int, end_a.getRecvSize(), 0); thread_b.join(); UASSERT(!end_a.exchange(buf, 0, 1000)); }