From 0fb9481e59b5507ba5d6efc44343da73cf878995 Mon Sep 17 00:00:00 2001 From: Michael Scire Date: Wed, 12 Nov 2025 17:32:43 -0700 Subject: [PATCH] kern: fix debug build for [[nodiscard]] changes --- .../include/mesosphere/arch/arm64/kern_k_page_table.hpp | 2 +- .../libmesosphere/include/mesosphere/kern_k_handle_table.hpp | 2 +- .../libmesosphere/source/arch/arm64/kern_k_page_table.cpp | 4 +--- .../source/board/nintendo/nx/kern_k_device_page_table.cpp | 2 +- libraries/libmesosphere/source/kern_k_handle_table.cpp | 4 +--- 5 files changed, 5 insertions(+), 9 deletions(-) diff --git a/libraries/libmesosphere/include/mesosphere/arch/arm64/kern_k_page_table.hpp b/libraries/libmesosphere/include/mesosphere/arch/arm64/kern_k_page_table.hpp index b08fe1625..2ab8527c9 100644 --- a/libraries/libmesosphere/include/mesosphere/arch/arm64/kern_k_page_table.hpp +++ b/libraries/libmesosphere/include/mesosphere/arch/arm64/kern_k_page_table.hpp @@ -199,7 +199,7 @@ namespace ams::kern::arch::arm64 { NOINLINE void InitializeForKernel(void *table, KVirtualAddress start, KVirtualAddress end); NOINLINE Result InitializeForProcess(ams::svc::CreateProcessFlag flags, bool from_back, KMemoryManager::Pool pool, KProcessAddress code_address, size_t code_size, KSystemResource *system_resource, KResourceLimit *resource_limit, size_t process_index); - Result Finalize(); + void Finalize(); static void NoteUpdatedCallback(const void *pt) { /* Note the update. */ diff --git a/libraries/libmesosphere/include/mesosphere/kern_k_handle_table.hpp b/libraries/libmesosphere/include/mesosphere/kern_k_handle_table.hpp index 440616aa6..0b176f482 100644 --- a/libraries/libmesosphere/include/mesosphere/kern_k_handle_table.hpp +++ b/libraries/libmesosphere/include/mesosphere/kern_k_handle_table.hpp @@ -105,7 +105,7 @@ namespace ams::kern { constexpr ALWAYS_INLINE size_t GetCount() const { return m_count; } constexpr ALWAYS_INLINE size_t GetMaxCount() const { return m_max_count; } - MESOSPHERE_NOINLINE_IF_DEBUG Result Finalize(); + MESOSPHERE_NOINLINE_IF_DEBUG void Finalize(); MESOSPHERE_NOINLINE_IF_DEBUG bool Remove(ams::svc::Handle handle); template diff --git a/libraries/libmesosphere/source/arch/arm64/kern_k_page_table.cpp b/libraries/libmesosphere/source/arch/arm64/kern_k_page_table.cpp index 4607272be..b8bce7b56 100644 --- a/libraries/libmesosphere/source/arch/arm64/kern_k_page_table.cpp +++ b/libraries/libmesosphere/source/arch/arm64/kern_k_page_table.cpp @@ -151,7 +151,7 @@ namespace ams::kern::arch::arm64 { R_SUCCEED(); } - Result KPageTable::Finalize() { + void KPageTable::Finalize() { /* Only process tables should be finalized. */ MESOSPHERE_ASSERT(!this->IsKernel()); @@ -269,8 +269,6 @@ namespace ams::kern::arch::arm64 { /* Perform inherited finalization. */ KPageTableBase::Finalize(); } - - R_SUCCEED(); } Result KPageTable::OperateImpl(PageLinkedList *page_list, KProcessAddress virt_addr, size_t num_pages, KPhysicalAddress phys_addr, bool is_pa_valid, const KPageProperties properties, OperationType operation, bool reuse_ll) { diff --git a/libraries/libmesosphere/source/board/nintendo/nx/kern_k_device_page_table.cpp b/libraries/libmesosphere/source/board/nintendo/nx/kern_k_device_page_table.cpp index f02bbee51..cdbe148f4 100644 --- a/libraries/libmesosphere/source/board/nintendo/nx/kern_k_device_page_table.cpp +++ b/libraries/libmesosphere/source/board/nintendo/nx/kern_k_device_page_table.cpp @@ -709,7 +709,7 @@ namespace ams::kern::board::nintendo::nx { /* Install interrupt handler. */ #if defined(MESOSPHERE_ENABLE_MEMORY_CONTROLLER_INTERRUPT) { - Kernel::GetInterruptManager().BindHandler(std::addressof(g_mc_interrupt_task), KInterruptName_MemoryController, GetCurrentCoreId(), KInterruptController::PriorityLevel_High, true, true); + MESOSPHERE_R_ABORT_UNLESS(Kernel::GetInterruptManager().BindHandler(std::addressof(g_mc_interrupt_task), KInterruptName_MemoryController, GetCurrentCoreId(), KInterruptController::PriorityLevel_High, true, true)); } #endif } diff --git a/libraries/libmesosphere/source/kern_k_handle_table.cpp b/libraries/libmesosphere/source/kern_k_handle_table.cpp index 421df9281..a21f93f7a 100644 --- a/libraries/libmesosphere/source/kern_k_handle_table.cpp +++ b/libraries/libmesosphere/source/kern_k_handle_table.cpp @@ -17,7 +17,7 @@ namespace ams::kern { - Result KHandleTable::Finalize() { + void KHandleTable::Finalize() { MESOSPHERE_ASSERT_THIS(); /* Get the table and clear our record of it. */ @@ -35,8 +35,6 @@ namespace ams::kern { obj->Close(); } } - - R_SUCCEED(); } bool KHandleTable::Remove(ams::svc::Handle handle) {