From 4f7ea612982a9ee83023e43724ff53f8dce65fe0 Mon Sep 17 00:00:00 2001 From: Fabian Imhof Date: Tue, 3 Dec 2024 08:06:40 +0000 Subject: improve mutex file structure --- arch/x86_64/src/shared/mutex.cpp | 27 +++++++++++++++++++++++++++ 1 file changed, 27 insertions(+) create mode 100644 arch/x86_64/src/shared/mutex.cpp (limited to 'arch/x86_64/src/shared/mutex.cpp') diff --git a/arch/x86_64/src/shared/mutex.cpp b/arch/x86_64/src/shared/mutex.cpp new file mode 100644 index 0000000..65cd095 --- /dev/null +++ b/arch/x86_64/src/shared/mutex.cpp @@ -0,0 +1,27 @@ +#include "arch/shared/mutex.hpp" + +namespace teachos::arch::shared +{ + auto mutex::lock() -> void + { + while (true) + { + if (!locked.exchange(true, std::memory_order_acquire)) + { + return; + } + } + } + + /** + * @brief Try to lock the mutex (non-blocking) + * + * @return true if lock has been acquired and false otherwise + */ + auto mutex::try_lock() -> bool { return !locked.exchange(true, std::memory_order_acquire); } + + /** + * @brief Unlock the mutex + */ + auto mutex::unlock() -> void { locked.store(false, std::memory_order_release); } +} // namespace teachos::arch::shared \ No newline at end of file -- cgit v1.2.3 From 6dff0ff5bcdd63de4a68f9c361acd0bace39b5ca Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?Matteo=20Gm=C3=BCr?= Date: Tue, 3 Dec 2024 08:52:59 +0000 Subject: Fix minor typos in mutex --- arch/x86_64/src/shared/mutex.cpp | 17 +++-------------- 1 file changed, 3 insertions(+), 14 deletions(-) (limited to 'arch/x86_64/src/shared/mutex.cpp') diff --git a/arch/x86_64/src/shared/mutex.cpp b/arch/x86_64/src/shared/mutex.cpp index 65cd095..6598255 100644 --- a/arch/x86_64/src/shared/mutex.cpp +++ b/arch/x86_64/src/shared/mutex.cpp @@ -4,24 +4,13 @@ namespace teachos::arch::shared { auto mutex::lock() -> void { - while (true) + while (!try_lock()) { - if (!locked.exchange(true, std::memory_order_acquire)) - { - return; - } + // Nothing to do } } - /** - * @brief Try to lock the mutex (non-blocking) - * - * @return true if lock has been acquired and false otherwise - */ auto mutex::try_lock() -> bool { return !locked.exchange(true, std::memory_order_acquire); } - /** - * @brief Unlock the mutex - */ auto mutex::unlock() -> void { locked.store(false, std::memory_order_release); } -} // namespace teachos::arch::shared \ No newline at end of file +} // namespace teachos::arch::shared -- cgit v1.2.3