diff options
| author | Amlal El Mahrouss <amlal.elmahrouss@icloud.com> | 2024-01-27 12:05:27 +0100 |
|---|---|---|
| committer | Amlal El Mahrouss <amlal.elmahrouss@icloud.com> | 2024-01-27 12:05:27 +0100 |
| commit | d6bc3461106b9c8f0f29b4eea487ff59b0cc6206 (patch) | |
| tree | 82630d94b81dac8504acb54a62c2d51975b7cc93 /Private/Source/SMPManager.cxx | |
| parent | de4af9d8929da048ab714b75d2af23ccebe3f6cb (diff) | |
KernelKit: SMP: Fix SMPManager::Switch (IsBusy())
KernelKit: TLS: Refactor and add documentation to it.
Signed-off-by: Amlal El Mahrouss <amlal.elmahrouss@icloud.com>
Diffstat (limited to 'Private/Source/SMPManager.cxx')
| -rw-r--r-- | Private/Source/SMPManager.cxx | 23 |
1 files changed, 19 insertions, 4 deletions
diff --git a/Private/Source/SMPManager.cxx b/Private/Source/SMPManager.cxx index 1c73cb82..4897165b 100644 --- a/Private/Source/SMPManager.cxx +++ b/Private/Source/SMPManager.cxx @@ -82,13 +82,15 @@ namespace hCore ///! @brief Default destructor. SMPManager::~SMPManager() = default; + /// @brief Shared singleton function Ref<SMPManager> SMPManager::Shared() { static SMPManager manager; return { manager }; } - HAL::StackFramePtr SMPManager::GetStack() noexcept + /// @brief Get Stack Frame of Core + HAL::StackFramePtr SMPManager::GetStackFrame() noexcept { if (m_ThreadList[m_CurrentThread].Leak() && ProcessHelper::GetCurrentPID() == m_ThreadList[m_CurrentThread].Leak().Leak().m_PID) @@ -97,7 +99,7 @@ namespace hCore return nullptr; } - // @brief Finds and switch to a free core. + /// @brief Finds and switch to a free core. bool SMPManager::Switch(HAL::StackFrame* stack) { if (stack == nullptr) @@ -108,7 +110,7 @@ namespace hCore // stack != nullptr -> if core is used, then continue. if (!m_ThreadList[idx].Leak() || !m_ThreadList[idx].Leak().Leak().IsWakeup() || - !m_ThreadList[idx].Leak().Leak().IsBusy()) + m_ThreadList[idx].Leak().Leak().IsBusy()) continue; m_ThreadList[idx].Leak().Leak().m_ID = idx; @@ -117,7 +119,7 @@ namespace hCore m_ThreadList[idx].Leak().Leak().Busy(true); - bool ret = rt_do_context_switch(rt_get_current_context(), stack) == 0; + Boolean ret = (rt_do_context_switch(rt_get_current_context(), stack) == 0); m_ThreadList[idx].Leak().Leak().Busy(false); @@ -127,9 +129,22 @@ namespace hCore return false; } + /** + * Index hart + * @param idx + * @return + */ Ref<ProcessorCore> SMPManager::operator[](const SizeT& idx) { return m_ThreadList[idx].Leak(); } + /** + * Check if thread pool isn't empty. + * @return + */ SMPManager::operator bool() noexcept { return !m_ThreadList.Empty(); } + /** + * Reverse operator bool + * @return + */ bool SMPManager::operator!() noexcept { return m_ThreadList.Empty(); } } // namespace hCore |