From aa6532cf34a57830dd839d685691ae26e96e1bc5 Mon Sep 17 00:00:00 2001 From: FearlessTobi Date: Mon, 19 Feb 2024 16:36:24 +0100 Subject: [PATCH 1/3] core/aoc: Move IPurchaseEventManager to separate file --- src/core/CMakeLists.txt | 2 + src/core/hle/service/aoc/aoc_u.cpp | 79 +------------------ .../service/aoc/purchase_event_manager.cpp | 79 +++++++++++++++++++ .../hle/service/aoc/purchase_event_manager.h | 28 +++++++ 4 files changed, 110 insertions(+), 78 deletions(-) create mode 100644 src/core/hle/service/aoc/purchase_event_manager.cpp create mode 100644 src/core/hle/service/aoc/purchase_event_manager.h diff --git a/src/core/CMakeLists.txt b/src/core/CMakeLists.txt index fa5c4de37e..d923a9a550 100644 --- a/src/core/CMakeLists.txt +++ b/src/core/CMakeLists.txt @@ -483,6 +483,8 @@ add_library(core STATIC hle/service/am/service/window_controller.h hle/service/aoc/aoc_u.cpp hle/service/aoc/aoc_u.h + hle/service/aoc/purchase_event_manager.cpp + hle/service/aoc/purchase_event_manager.h hle/service/apm/apm.cpp hle/service/apm/apm.h hle/service/apm/apm_controller.cpp diff --git a/src/core/hle/service/aoc/aoc_u.cpp b/src/core/hle/service/aoc/aoc_u.cpp index 486719cc09..31c7385ee7 100644 --- a/src/core/hle/service/aoc/aoc_u.cpp +++ b/src/core/hle/service/aoc/aoc_u.cpp @@ -16,14 +16,13 @@ #include "core/file_sys/registered_cache.h" #include "core/hle/kernel/k_event.h" #include "core/hle/service/aoc/aoc_u.h" +#include "core/hle/service/aoc/purchase_event_manager.h" #include "core/hle/service/ipc_helpers.h" #include "core/hle/service/server_manager.h" #include "core/loader/loader.h" namespace Service::AOC { -constexpr Result ResultNoPurchasedProductInfoAvailable{ErrorModule::NIMShop, 400}; - static bool CheckAOCTitleIDMatchesBase(u64 title_id, u64 base) { return FileSys::GetBaseTitleID(title_id) == base; } @@ -46,82 +45,6 @@ static std::vector AccumulateAOCTitleIDs(Core::System& system) { return add_on_content; } -class IPurchaseEventManager final : public ServiceFramework { -public: - explicit IPurchaseEventManager(Core::System& system_) - : ServiceFramework{system_, "IPurchaseEventManager"}, service_context{ - system, "IPurchaseEventManager"} { - // clang-format off - static const FunctionInfo functions[] = { - {0, &IPurchaseEventManager::SetDefaultDeliveryTarget, "SetDefaultDeliveryTarget"}, - {1, &IPurchaseEventManager::SetDeliveryTarget, "SetDeliveryTarget"}, - {2, &IPurchaseEventManager::GetPurchasedEventReadableHandle, "GetPurchasedEventReadableHandle"}, - {3, &IPurchaseEventManager::PopPurchasedProductInfo, "PopPurchasedProductInfo"}, - {4, &IPurchaseEventManager::PopPurchasedProductInfoWithUid, "PopPurchasedProductInfoWithUid"}, - }; - // clang-format on - - RegisterHandlers(functions); - - purchased_event = service_context.CreateEvent("IPurchaseEventManager:PurchasedEvent"); - } - - ~IPurchaseEventManager() override { - service_context.CloseEvent(purchased_event); - } - -private: - void SetDefaultDeliveryTarget(HLERequestContext& ctx) { - IPC::RequestParser rp{ctx}; - - const auto unknown_1 = rp.Pop(); - [[maybe_unused]] const auto unknown_2 = ctx.ReadBuffer(); - - LOG_WARNING(Service_AOC, "(STUBBED) called, unknown_1={}", unknown_1); - - IPC::ResponseBuilder rb{ctx, 2}; - rb.Push(ResultSuccess); - } - - void SetDeliveryTarget(HLERequestContext& ctx) { - IPC::RequestParser rp{ctx}; - - const auto unknown_1 = rp.Pop(); - [[maybe_unused]] const auto unknown_2 = ctx.ReadBuffer(); - - LOG_WARNING(Service_AOC, "(STUBBED) called, unknown_1={}", unknown_1); - - IPC::ResponseBuilder rb{ctx, 2}; - rb.Push(ResultSuccess); - } - - void GetPurchasedEventReadableHandle(HLERequestContext& ctx) { - LOG_WARNING(Service_AOC, "called"); - - IPC::ResponseBuilder rb{ctx, 2, 1}; - rb.Push(ResultSuccess); - rb.PushCopyObjects(purchased_event->GetReadableEvent()); - } - - void PopPurchasedProductInfo(HLERequestContext& ctx) { - LOG_DEBUG(Service_AOC, "(STUBBED) called"); - - IPC::ResponseBuilder rb{ctx, 2}; - rb.Push(ResultNoPurchasedProductInfoAvailable); - } - - void PopPurchasedProductInfoWithUid(HLERequestContext& ctx) { - LOG_DEBUG(Service_AOC, "(STUBBED) called"); - - IPC::ResponseBuilder rb{ctx, 2}; - rb.Push(ResultNoPurchasedProductInfoAvailable); - } - - KernelHelpers::ServiceContext service_context; - - Kernel::KEvent* purchased_event; -}; - AOC_U::AOC_U(Core::System& system_) : ServiceFramework{system_, "aoc:u"}, add_on_content{AccumulateAOCTitleIDs(system)}, service_context{system_, "aoc:u"} { diff --git a/src/core/hle/service/aoc/purchase_event_manager.cpp b/src/core/hle/service/aoc/purchase_event_manager.cpp new file mode 100644 index 0000000000..83124cd21d --- /dev/null +++ b/src/core/hle/service/aoc/purchase_event_manager.cpp @@ -0,0 +1,79 @@ +// SPDX-FileCopyrightText: Copyright 2018 yuzu Emulator Project +// SPDX-License-Identifier: GPL-2.0-or-later + +#include "core/hle/service/aoc/purchase_event_manager.h" +#include "core/hle/service/ipc_helpers.h" + +namespace Service::AOC { + +constexpr Result ResultNoPurchasedProductInfoAvailable{ErrorModule::NIMShop, 400}; + +IPurchaseEventManager::IPurchaseEventManager(Core::System& system_) + : ServiceFramework{system_, "IPurchaseEventManager"}, + service_context{system, "IPurchaseEventManager"} { + // clang-format off + static const FunctionInfo functions[] = { + {0, &IPurchaseEventManager::SetDefaultDeliveryTarget, "SetDefaultDeliveryTarget"}, + {1, &IPurchaseEventManager::SetDeliveryTarget, "SetDeliveryTarget"}, + {2, &IPurchaseEventManager::GetPurchasedEventReadableHandle, "GetPurchasedEventReadableHandle"}, + {3, &IPurchaseEventManager::PopPurchasedProductInfo, "PopPurchasedProductInfo"}, + {4, &IPurchaseEventManager::PopPurchasedProductInfoWithUid, "PopPurchasedProductInfoWithUid"}, + }; + // clang-format on + + RegisterHandlers(functions); + + purchased_event = service_context.CreateEvent("IPurchaseEventManager:PurchasedEvent"); +} + +IPurchaseEventManager::~IPurchaseEventManager() { + service_context.CloseEvent(purchased_event); +} + +void IPurchaseEventManager::SetDefaultDeliveryTarget(HLERequestContext& ctx) { + IPC::RequestParser rp{ctx}; + + const auto unknown_1 = rp.Pop(); + [[maybe_unused]] const auto unknown_2 = ctx.ReadBuffer(); + + LOG_WARNING(Service_AOC, "(STUBBED) called, unknown_1={}", unknown_1); + + IPC::ResponseBuilder rb{ctx, 2}; + rb.Push(ResultSuccess); +} + +void IPurchaseEventManager::SetDeliveryTarget(HLERequestContext& ctx) { + IPC::RequestParser rp{ctx}; + + const auto unknown_1 = rp.Pop(); + [[maybe_unused]] const auto unknown_2 = ctx.ReadBuffer(); + + LOG_WARNING(Service_AOC, "(STUBBED) called, unknown_1={}", unknown_1); + + IPC::ResponseBuilder rb{ctx, 2}; + rb.Push(ResultSuccess); +} + +void IPurchaseEventManager::GetPurchasedEventReadableHandle(HLERequestContext& ctx) { + LOG_WARNING(Service_AOC, "called"); + + IPC::ResponseBuilder rb{ctx, 2, 1}; + rb.Push(ResultSuccess); + rb.PushCopyObjects(purchased_event->GetReadableEvent()); +} + +void IPurchaseEventManager::PopPurchasedProductInfo(HLERequestContext& ctx) { + LOG_DEBUG(Service_AOC, "(STUBBED) called"); + + IPC::ResponseBuilder rb{ctx, 2}; + rb.Push(ResultNoPurchasedProductInfoAvailable); +} + +void IPurchaseEventManager::PopPurchasedProductInfoWithUid(HLERequestContext& ctx) { + LOG_DEBUG(Service_AOC, "(STUBBED) called"); + + IPC::ResponseBuilder rb{ctx, 2}; + rb.Push(ResultNoPurchasedProductInfoAvailable); +} + +} // namespace Service::AOC diff --git a/src/core/hle/service/aoc/purchase_event_manager.h b/src/core/hle/service/aoc/purchase_event_manager.h new file mode 100644 index 0000000000..efde3c8f3a --- /dev/null +++ b/src/core/hle/service/aoc/purchase_event_manager.h @@ -0,0 +1,28 @@ +// SPDX-FileCopyrightText: Copyright 2018 yuzu Emulator Project +// SPDX-License-Identifier: GPL-2.0-or-later + +#pragma once + +#include "core/hle/service/kernel_helpers.h" +#include "core/hle/service/os/event.h" +#include "core/hle/service/service.h" + +namespace Service::AOC { + +class IPurchaseEventManager final : public ServiceFramework { +public: + explicit IPurchaseEventManager(Core::System& system_); + ~IPurchaseEventManager() override; + + void SetDefaultDeliveryTarget(HLERequestContext& ctx); + void SetDeliveryTarget(HLERequestContext& ctx); + void GetPurchasedEventReadableHandle(HLERequestContext& ctx); + void PopPurchasedProductInfo(HLERequestContext& ctx); + void PopPurchasedProductInfoWithUid(HLERequestContext& ctx); + +private: + KernelHelpers::ServiceContext service_context; + Kernel::KEvent* purchased_event; +}; + +} // namespace Service::AOC From a2a0be424638365c9348bbe16d1db9706b690f8c Mon Sep 17 00:00:00 2001 From: FearlessTobi Date: Mon, 19 Feb 2024 17:20:02 +0100 Subject: [PATCH 2/3] aoc: Migrate to use cmif serialization --- src/core/hle/service/aoc/aoc_u.cpp | 168 +++++++----------- src/core/hle/service/aoc/aoc_u.h | 30 ++-- .../service/aoc/purchase_event_manager.cpp | 56 +++--- .../hle/service/aoc/purchase_event_manager.h | 12 +- 4 files changed, 109 insertions(+), 157 deletions(-) diff --git a/src/core/hle/service/aoc/aoc_u.cpp b/src/core/hle/service/aoc/aoc_u.cpp index 31c7385ee7..82b95b704a 100644 --- a/src/core/hle/service/aoc/aoc_u.cpp +++ b/src/core/hle/service/aoc/aoc_u.cpp @@ -17,6 +17,7 @@ #include "core/hle/kernel/k_event.h" #include "core/hle/service/aoc/aoc_u.h" #include "core/hle/service/aoc/purchase_event_manager.h" +#include "core/hle/service/cmif_serialization.h" #include "core/hle/service/ipc_helpers.h" #include "core/hle/service/server_manager.h" #include "core/loader/loader.h" @@ -52,21 +53,21 @@ AOC_U::AOC_U(Core::System& system_) static const FunctionInfo functions[] = { {0, nullptr, "CountAddOnContentByApplicationId"}, {1, nullptr, "ListAddOnContentByApplicationId"}, - {2, &AOC_U::CountAddOnContent, "CountAddOnContent"}, - {3, &AOC_U::ListAddOnContent, "ListAddOnContent"}, + {2, D<&AOC_U::CountAddOnContent>, "CountAddOnContent"}, + {3, D<&AOC_U::ListAddOnContent>, "ListAddOnContent"}, {4, nullptr, "GetAddOnContentBaseIdByApplicationId"}, - {5, &AOC_U::GetAddOnContentBaseId, "GetAddOnContentBaseId"}, + {5, D<&AOC_U::GetAddOnContentBaseId>, "GetAddOnContentBaseId"}, {6, nullptr, "PrepareAddOnContentByApplicationId"}, - {7, &AOC_U::PrepareAddOnContent, "PrepareAddOnContent"}, - {8, &AOC_U::GetAddOnContentListChangedEvent, "GetAddOnContentListChangedEvent"}, + {7, D<&AOC_U::PrepareAddOnContent>, "PrepareAddOnContent"}, + {8, D<&AOC_U::GetAddOnContentListChangedEvent>, "GetAddOnContentListChangedEvent"}, {9, nullptr, "GetAddOnContentLostErrorCode"}, - {10, &AOC_U::GetAddOnContentListChangedEventWithProcessId, "GetAddOnContentListChangedEventWithProcessId"}, - {11, &AOC_U::NotifyMountAddOnContent, "NotifyMountAddOnContent"}, - {12, &AOC_U::NotifyUnmountAddOnContent, "NotifyUnmountAddOnContent"}, + {10, D<&AOC_U::GetAddOnContentListChangedEventWithProcessId>, "GetAddOnContentListChangedEventWithProcessId"}, + {11, D<&AOC_U::NotifyMountAddOnContent>, "NotifyMountAddOnContent"}, + {12, D<&AOC_U::NotifyUnmountAddOnContent>, "NotifyUnmountAddOnContent"}, {13, nullptr, "IsAddOnContentMountedForDebug"}, - {50, &AOC_U::CheckAddOnContentMountStatus, "CheckAddOnContentMountStatus"}, - {100, &AOC_U::CreateEcPurchasedEventManager, "CreateEcPurchasedEventManager"}, - {101, &AOC_U::CreatePermanentEcPurchasedEventManager, "CreatePermanentEcPurchasedEventManager"}, + {50, D<&AOC_U::CheckAddOnContentMountStatus>, "CheckAddOnContentMountStatus"}, + {100, D<&AOC_U::CreateEcPurchasedEventManager>, "CreateEcPurchasedEventManager"}, + {101, D<&AOC_U::CreatePermanentEcPurchasedEventManager>, "CreatePermanentEcPurchasedEventManager"}, {110, nullptr, "CreateContentsServiceManager"}, {200, nullptr, "SetRequiredAddOnContentsOnContentsAvailabilityTransition"}, {300, nullptr, "SetupHostAddOnContent"}, @@ -84,46 +85,28 @@ AOC_U::~AOC_U() { service_context.CloseEvent(aoc_change_event); } -void AOC_U::CountAddOnContent(HLERequestContext& ctx) { - struct Parameters { - u64 process_id; - }; - static_assert(sizeof(Parameters) == 8); - - IPC::RequestParser rp{ctx}; - const auto params = rp.PopRaw(); - - LOG_DEBUG(Service_AOC, "called. process_id={}", params.process_id); - - IPC::ResponseBuilder rb{ctx, 3}; - rb.Push(ResultSuccess); +Result AOC_U::CountAddOnContent(Out out_count, ClientProcessId process_id) { + LOG_DEBUG(Service_AOC, "called. process_id={}", process_id.pid); const auto current = system.GetApplicationProcessProgramID(); const auto& disabled = Settings::values.disabled_addons[current]; if (std::find(disabled.begin(), disabled.end(), "DLC") != disabled.end()) { - rb.Push(0); - return; + *out_count = 0; + R_SUCCEED(); } - rb.Push(static_cast( + *out_count = static_cast( std::count_if(add_on_content.begin(), add_on_content.end(), - [current](u64 tid) { return CheckAOCTitleIDMatchesBase(tid, current); }))); + [current](u64 tid) { return CheckAOCTitleIDMatchesBase(tid, current); })); + + R_SUCCEED(); } -void AOC_U::ListAddOnContent(HLERequestContext& ctx) { - struct Parameters { - u32 offset; - u32 count; - u64 process_id; - }; - static_assert(sizeof(Parameters) == 16); - - IPC::RequestParser rp{ctx}; - const auto [offset, count, process_id] = rp.PopRaw(); - +Result AOC_U::ListAddOnContent(Out out_count, OutBuffer out_addons, + u32 offset, u32 count, ClientProcessId process_id) { LOG_DEBUG(Service_AOC, "called with offset={}, count={}, process_id={}", offset, count, - process_id); + process_id.pid); const auto current = FileSys::GetBaseTitleID(system.GetApplicationProcessProgramID()); @@ -139,37 +122,19 @@ void AOC_U::ListAddOnContent(HLERequestContext& ctx) { } } - if (out.size() < offset) { - IPC::ResponseBuilder rb{ctx, 2}; - // TODO(DarkLordZach): Find the correct error code. - rb.Push(ResultUnknown); - return; - } + // TODO(DarkLordZach): Find the correct error code. + R_UNLESS(out.size() >= offset, ResultUnknown); - const auto out_count = static_cast(std::min(out.size() - offset, count)); + *out_count = static_cast(std::min(out.size() - offset, count)); std::rotate(out.begin(), out.begin() + offset, out.end()); - out.resize(out_count); - ctx.WriteBuffer(out); + std::memcpy(out_addons.data(), out.data(), *out_count); - IPC::ResponseBuilder rb{ctx, 3}; - rb.Push(ResultSuccess); - rb.Push(out_count); + R_SUCCEED(); } -void AOC_U::GetAddOnContentBaseId(HLERequestContext& ctx) { - struct Parameters { - u64 process_id; - }; - static_assert(sizeof(Parameters) == 8); - - IPC::RequestParser rp{ctx}; - const auto params = rp.PopRaw(); - - LOG_DEBUG(Service_AOC, "called. process_id={}", params.process_id); - - IPC::ResponseBuilder rb{ctx, 4}; - rb.Push(ResultSuccess); +Result AOC_U::GetAddOnContentBaseId(Out out_title_id, ClientProcessId process_id) { + LOG_DEBUG(Service_AOC, "called. process_id={}", process_id.pid); const auto title_id = system.GetApplicationProcessProgramID(); const FileSys::PatchManager pm{title_id, system.GetFileSystemController(), @@ -177,81 +142,72 @@ void AOC_U::GetAddOnContentBaseId(HLERequestContext& ctx) { const auto res = pm.GetControlMetadata(); if (res.first == nullptr) { - rb.Push(FileSys::GetAOCBaseTitleID(title_id)); - return; + *out_title_id = FileSys::GetAOCBaseTitleID(title_id); + R_SUCCEED(); } - rb.Push(res.first->GetDLCBaseTitleId()); + *out_title_id = res.first->GetDLCBaseTitleId(); + + R_SUCCEED(); } -void AOC_U::PrepareAddOnContent(HLERequestContext& ctx) { - struct Parameters { - s32 addon_index; - u64 process_id; - }; - static_assert(sizeof(Parameters) == 16); - - IPC::RequestParser rp{ctx}; - const auto [addon_index, process_id] = rp.PopRaw(); - +Result AOC_U::PrepareAddOnContent(s32 addon_index, ClientProcessId process_id) { LOG_WARNING(Service_AOC, "(STUBBED) called with addon_index={}, process_id={}", addon_index, - process_id); + process_id.pid); - IPC::ResponseBuilder rb{ctx, 2}; - rb.Push(ResultSuccess); + R_SUCCEED(); } -void AOC_U::GetAddOnContentListChangedEvent(HLERequestContext& ctx) { +Result AOC_U::GetAddOnContentListChangedEvent(OutCopyHandle out_event) { LOG_WARNING(Service_AOC, "(STUBBED) called"); - IPC::ResponseBuilder rb{ctx, 2, 1}; - rb.Push(ResultSuccess); - rb.PushCopyObjects(aoc_change_event->GetReadableEvent()); + *out_event = &aoc_change_event->GetReadableEvent(); + + R_SUCCEED(); } -void AOC_U::GetAddOnContentListChangedEventWithProcessId(HLERequestContext& ctx) { +Result AOC_U::GetAddOnContentListChangedEventWithProcessId( + OutCopyHandle out_event, ClientProcessId process_id) { LOG_WARNING(Service_AOC, "(STUBBED) called"); - IPC::ResponseBuilder rb{ctx, 2, 1}; - rb.Push(ResultSuccess); - rb.PushCopyObjects(aoc_change_event->GetReadableEvent()); + *out_event = &aoc_change_event->GetReadableEvent(); + + R_SUCCEED(); } -void AOC_U::NotifyMountAddOnContent(HLERequestContext& ctx) { +Result AOC_U::NotifyMountAddOnContent() { LOG_WARNING(Service_AOC, "(STUBBED) called"); - IPC::ResponseBuilder rb{ctx, 2}; - rb.Push(ResultSuccess); + R_SUCCEED(); } -void AOC_U::NotifyUnmountAddOnContent(HLERequestContext& ctx) { +Result AOC_U::NotifyUnmountAddOnContent() { LOG_WARNING(Service_AOC, "(STUBBED) called"); - IPC::ResponseBuilder rb{ctx, 2}; - rb.Push(ResultSuccess); + R_SUCCEED(); } -void AOC_U::CheckAddOnContentMountStatus(HLERequestContext& ctx) { +Result AOC_U::CheckAddOnContentMountStatus() { LOG_WARNING(Service_AOC, "(STUBBED) called"); - IPC::ResponseBuilder rb{ctx, 2}; - rb.Push(ResultSuccess); + R_SUCCEED(); } -void AOC_U::CreateEcPurchasedEventManager(HLERequestContext& ctx) { +Result AOC_U::CreateEcPurchasedEventManager(OutInterface out_interface) { LOG_WARNING(Service_AOC, "(STUBBED) called"); - IPC::ResponseBuilder rb{ctx, 2, 0, 1}; - rb.Push(ResultSuccess); - rb.PushIpcInterface(system); + *out_interface = std::make_shared(system); + + R_SUCCEED(); } -void AOC_U::CreatePermanentEcPurchasedEventManager(HLERequestContext& ctx) { +Result AOC_U::CreatePermanentEcPurchasedEventManager( + OutInterface out_interface) { LOG_WARNING(Service_AOC, "(STUBBED) called"); - IPC::ResponseBuilder rb{ctx, 2, 0, 1}; - rb.Push(ResultSuccess); - rb.PushIpcInterface(system); + *out_interface = std::make_shared(system); + + R_SUCCEED(); } void LoopProcess(Core::System& system) { diff --git a/src/core/hle/service/aoc/aoc_u.h b/src/core/hle/service/aoc/aoc_u.h index 12ccfeb6a3..c7b94a9335 100644 --- a/src/core/hle/service/aoc/aoc_u.h +++ b/src/core/hle/service/aoc/aoc_u.h @@ -3,6 +3,7 @@ #pragma once +#include "core/hle/service/cmif_types.h" #include "core/hle/service/kernel_helpers.h" #include "core/hle/service/service.h" @@ -16,24 +17,29 @@ class KEvent; namespace Service::AOC { +class IPurchaseEventManager; + class AOC_U final : public ServiceFramework { public: explicit AOC_U(Core::System& system); ~AOC_U() override; -private: - void CountAddOnContent(HLERequestContext& ctx); - void ListAddOnContent(HLERequestContext& ctx); - void GetAddOnContentBaseId(HLERequestContext& ctx); - void PrepareAddOnContent(HLERequestContext& ctx); - void GetAddOnContentListChangedEvent(HLERequestContext& ctx); - void GetAddOnContentListChangedEventWithProcessId(HLERequestContext& ctx); - void NotifyMountAddOnContent(HLERequestContext& ctx); - void NotifyUnmountAddOnContent(HLERequestContext& ctx); - void CheckAddOnContentMountStatus(HLERequestContext& ctx); - void CreateEcPurchasedEventManager(HLERequestContext& ctx); - void CreatePermanentEcPurchasedEventManager(HLERequestContext& ctx); + Result CountAddOnContent(Out out_count, ClientProcessId process_id); + Result ListAddOnContent(Out out_count, OutBuffer out_addons, + u32 offset, u32 count, ClientProcessId process_id); + Result GetAddOnContentBaseId(Out out_title_id, ClientProcessId process_id); + Result PrepareAddOnContent(s32 addon_index, ClientProcessId process_id); + Result GetAddOnContentListChangedEvent(OutCopyHandle out_event); + Result GetAddOnContentListChangedEventWithProcessId( + OutCopyHandle out_event, ClientProcessId process_id); + Result NotifyMountAddOnContent(); + Result NotifyUnmountAddOnContent(); + Result CheckAddOnContentMountStatus(); + Result CreateEcPurchasedEventManager(OutInterface out_interface); + Result CreatePermanentEcPurchasedEventManager( + OutInterface out_interface); +private: std::vector add_on_content; KernelHelpers::ServiceContext service_context; diff --git a/src/core/hle/service/aoc/purchase_event_manager.cpp b/src/core/hle/service/aoc/purchase_event_manager.cpp index 83124cd21d..1192263f8a 100644 --- a/src/core/hle/service/aoc/purchase_event_manager.cpp +++ b/src/core/hle/service/aoc/purchase_event_manager.cpp @@ -2,7 +2,7 @@ // SPDX-License-Identifier: GPL-2.0-or-later #include "core/hle/service/aoc/purchase_event_manager.h" -#include "core/hle/service/ipc_helpers.h" +#include "core/hle/service/cmif_serialization.h" namespace Service::AOC { @@ -13,11 +13,11 @@ IPurchaseEventManager::IPurchaseEventManager(Core::System& system_) service_context{system, "IPurchaseEventManager"} { // clang-format off static const FunctionInfo functions[] = { - {0, &IPurchaseEventManager::SetDefaultDeliveryTarget, "SetDefaultDeliveryTarget"}, - {1, &IPurchaseEventManager::SetDeliveryTarget, "SetDeliveryTarget"}, - {2, &IPurchaseEventManager::GetPurchasedEventReadableHandle, "GetPurchasedEventReadableHandle"}, - {3, &IPurchaseEventManager::PopPurchasedProductInfo, "PopPurchasedProductInfo"}, - {4, &IPurchaseEventManager::PopPurchasedProductInfoWithUid, "PopPurchasedProductInfoWithUid"}, + {0, D<&IPurchaseEventManager::SetDefaultDeliveryTarget>, "SetDefaultDeliveryTarget"}, + {1, D<&IPurchaseEventManager::SetDeliveryTarget>, "SetDeliveryTarget"}, + {2, D<&IPurchaseEventManager::GetPurchasedEvent>, "GetPurchasedEvent"}, + {3, D<&IPurchaseEventManager::PopPurchasedProductInfo>, "PopPurchasedProductInfo"}, + {4, D<&IPurchaseEventManager::PopPurchasedProductInfoWithUid>, "PopPurchasedProductInfoWithUid"}, }; // clang-format on @@ -30,50 +30,38 @@ IPurchaseEventManager::~IPurchaseEventManager() { service_context.CloseEvent(purchased_event); } -void IPurchaseEventManager::SetDefaultDeliveryTarget(HLERequestContext& ctx) { - IPC::RequestParser rp{ctx}; +Result IPurchaseEventManager::SetDefaultDeliveryTarget( + ClientProcessId process_id, InBuffer in_buffer) { + LOG_WARNING(Service_AOC, "(STUBBED) called, process_id={}", process_id.pid); - const auto unknown_1 = rp.Pop(); - [[maybe_unused]] const auto unknown_2 = ctx.ReadBuffer(); - - LOG_WARNING(Service_AOC, "(STUBBED) called, unknown_1={}", unknown_1); - - IPC::ResponseBuilder rb{ctx, 2}; - rb.Push(ResultSuccess); + R_SUCCEED(); } -void IPurchaseEventManager::SetDeliveryTarget(HLERequestContext& ctx) { - IPC::RequestParser rp{ctx}; +Result IPurchaseEventManager::SetDeliveryTarget(u64 unknown, + InBuffer in_buffer) { + LOG_WARNING(Service_AOC, "(STUBBED) called, unknown={}", unknown); - const auto unknown_1 = rp.Pop(); - [[maybe_unused]] const auto unknown_2 = ctx.ReadBuffer(); - - LOG_WARNING(Service_AOC, "(STUBBED) called, unknown_1={}", unknown_1); - - IPC::ResponseBuilder rb{ctx, 2}; - rb.Push(ResultSuccess); + R_SUCCEED(); } -void IPurchaseEventManager::GetPurchasedEventReadableHandle(HLERequestContext& ctx) { +Result IPurchaseEventManager::GetPurchasedEvent(OutCopyHandle out_event) { LOG_WARNING(Service_AOC, "called"); - IPC::ResponseBuilder rb{ctx, 2, 1}; - rb.Push(ResultSuccess); - rb.PushCopyObjects(purchased_event->GetReadableEvent()); + *out_event = &purchased_event->GetReadableEvent(); + + R_SUCCEED(); } -void IPurchaseEventManager::PopPurchasedProductInfo(HLERequestContext& ctx) { +Result IPurchaseEventManager::PopPurchasedProductInfo() { LOG_DEBUG(Service_AOC, "(STUBBED) called"); - IPC::ResponseBuilder rb{ctx, 2}; - rb.Push(ResultNoPurchasedProductInfoAvailable); + R_RETURN(ResultNoPurchasedProductInfoAvailable); } -void IPurchaseEventManager::PopPurchasedProductInfoWithUid(HLERequestContext& ctx) { +Result IPurchaseEventManager::PopPurchasedProductInfoWithUid() { LOG_DEBUG(Service_AOC, "(STUBBED) called"); - IPC::ResponseBuilder rb{ctx, 2}; - rb.Push(ResultNoPurchasedProductInfoAvailable); + R_RETURN(ResultNoPurchasedProductInfoAvailable); } } // namespace Service::AOC diff --git a/src/core/hle/service/aoc/purchase_event_manager.h b/src/core/hle/service/aoc/purchase_event_manager.h index efde3c8f3a..ea3836bc93 100644 --- a/src/core/hle/service/aoc/purchase_event_manager.h +++ b/src/core/hle/service/aoc/purchase_event_manager.h @@ -3,6 +3,7 @@ #pragma once +#include "core/hle/service/cmif_types.h" #include "core/hle/service/kernel_helpers.h" #include "core/hle/service/os/event.h" #include "core/hle/service/service.h" @@ -14,11 +15,12 @@ public: explicit IPurchaseEventManager(Core::System& system_); ~IPurchaseEventManager() override; - void SetDefaultDeliveryTarget(HLERequestContext& ctx); - void SetDeliveryTarget(HLERequestContext& ctx); - void GetPurchasedEventReadableHandle(HLERequestContext& ctx); - void PopPurchasedProductInfo(HLERequestContext& ctx); - void PopPurchasedProductInfoWithUid(HLERequestContext& ctx); + Result SetDefaultDeliveryTarget(ClientProcessId process_id, + InBuffer in_buffer); + Result SetDeliveryTarget(u64 unknown, InBuffer in_buffer); + Result GetPurchasedEvent(OutCopyHandle out_event); + Result PopPurchasedProductInfo(); + Result PopPurchasedProductInfoWithUid(); private: KernelHelpers::ServiceContext service_context; From 2786d34dd7c03f493b0cdc220189446f63646dfe Mon Sep 17 00:00:00 2001 From: FearlessTobi Date: Mon, 19 Feb 2024 18:58:57 +0100 Subject: [PATCH 3/3] aoc: Rename AOC_U to IAddOnContentManager --- src/core/CMakeLists.txt | 4 +- .../{aoc_u.cpp => addon_content_manager.cpp} | 60 ++++++++++--------- .../aoc/{aoc_u.h => addon_content_manager.h} | 6 +- .../service/aoc/purchase_event_manager.cpp | 4 +- src/core/hle/service/services.cpp | 2 +- 5 files changed, 40 insertions(+), 36 deletions(-) rename src/core/hle/service/aoc/{aoc_u.cpp => addon_content_manager.cpp} (70%) rename src/core/hle/service/aoc/{aoc_u.h => addon_content_manager.h} (89%) diff --git a/src/core/CMakeLists.txt b/src/core/CMakeLists.txt index d923a9a550..a54325f682 100644 --- a/src/core/CMakeLists.txt +++ b/src/core/CMakeLists.txt @@ -481,8 +481,8 @@ add_library(core STATIC hle/service/am/service/system_applet_proxy.h hle/service/am/service/window_controller.cpp hle/service/am/service/window_controller.h - hle/service/aoc/aoc_u.cpp - hle/service/aoc/aoc_u.h + hle/service/aoc/addon_content_manager.cpp + hle/service/aoc/addon_content_manager.h hle/service/aoc/purchase_event_manager.cpp hle/service/aoc/purchase_event_manager.h hle/service/apm/apm.cpp diff --git a/src/core/hle/service/aoc/aoc_u.cpp b/src/core/hle/service/aoc/addon_content_manager.cpp similarity index 70% rename from src/core/hle/service/aoc/aoc_u.cpp rename to src/core/hle/service/aoc/addon_content_manager.cpp index 82b95b704a..d47f57d645 100644 --- a/src/core/hle/service/aoc/aoc_u.cpp +++ b/src/core/hle/service/aoc/addon_content_manager.cpp @@ -15,7 +15,7 @@ #include "core/file_sys/patch_manager.h" #include "core/file_sys/registered_cache.h" #include "core/hle/kernel/k_event.h" -#include "core/hle/service/aoc/aoc_u.h" +#include "core/hle/service/aoc/addon_content_manager.h" #include "core/hle/service/aoc/purchase_event_manager.h" #include "core/hle/service/cmif_serialization.h" #include "core/hle/service/ipc_helpers.h" @@ -46,28 +46,28 @@ static std::vector AccumulateAOCTitleIDs(Core::System& system) { return add_on_content; } -AOC_U::AOC_U(Core::System& system_) +IAddOnContentManager::IAddOnContentManager(Core::System& system_) : ServiceFramework{system_, "aoc:u"}, add_on_content{AccumulateAOCTitleIDs(system)}, service_context{system_, "aoc:u"} { // clang-format off static const FunctionInfo functions[] = { {0, nullptr, "CountAddOnContentByApplicationId"}, {1, nullptr, "ListAddOnContentByApplicationId"}, - {2, D<&AOC_U::CountAddOnContent>, "CountAddOnContent"}, - {3, D<&AOC_U::ListAddOnContent>, "ListAddOnContent"}, + {2, D<&IAddOnContentManager::CountAddOnContent>, "CountAddOnContent"}, + {3, D<&IAddOnContentManager::ListAddOnContent>, "ListAddOnContent"}, {4, nullptr, "GetAddOnContentBaseIdByApplicationId"}, - {5, D<&AOC_U::GetAddOnContentBaseId>, "GetAddOnContentBaseId"}, + {5, D<&IAddOnContentManager::GetAddOnContentBaseId>, "GetAddOnContentBaseId"}, {6, nullptr, "PrepareAddOnContentByApplicationId"}, - {7, D<&AOC_U::PrepareAddOnContent>, "PrepareAddOnContent"}, - {8, D<&AOC_U::GetAddOnContentListChangedEvent>, "GetAddOnContentListChangedEvent"}, + {7, D<&IAddOnContentManager::PrepareAddOnContent>, "PrepareAddOnContent"}, + {8, D<&IAddOnContentManager::GetAddOnContentListChangedEvent>, "GetAddOnContentListChangedEvent"}, {9, nullptr, "GetAddOnContentLostErrorCode"}, - {10, D<&AOC_U::GetAddOnContentListChangedEventWithProcessId>, "GetAddOnContentListChangedEventWithProcessId"}, - {11, D<&AOC_U::NotifyMountAddOnContent>, "NotifyMountAddOnContent"}, - {12, D<&AOC_U::NotifyUnmountAddOnContent>, "NotifyUnmountAddOnContent"}, + {10, D<&IAddOnContentManager::GetAddOnContentListChangedEventWithProcessId>, "GetAddOnContentListChangedEventWithProcessId"}, + {11, D<&IAddOnContentManager::NotifyMountAddOnContent>, "NotifyMountAddOnContent"}, + {12, D<&IAddOnContentManager::NotifyUnmountAddOnContent>, "NotifyUnmountAddOnContent"}, {13, nullptr, "IsAddOnContentMountedForDebug"}, - {50, D<&AOC_U::CheckAddOnContentMountStatus>, "CheckAddOnContentMountStatus"}, - {100, D<&AOC_U::CreateEcPurchasedEventManager>, "CreateEcPurchasedEventManager"}, - {101, D<&AOC_U::CreatePermanentEcPurchasedEventManager>, "CreatePermanentEcPurchasedEventManager"}, + {50, D<&IAddOnContentManager::CheckAddOnContentMountStatus>, "CheckAddOnContentMountStatus"}, + {100, D<&IAddOnContentManager::CreateEcPurchasedEventManager>, "CreateEcPurchasedEventManager"}, + {101, D<&IAddOnContentManager::CreatePermanentEcPurchasedEventManager>, "CreatePermanentEcPurchasedEventManager"}, {110, nullptr, "CreateContentsServiceManager"}, {200, nullptr, "SetRequiredAddOnContentsOnContentsAvailabilityTransition"}, {300, nullptr, "SetupHostAddOnContent"}, @@ -81,11 +81,11 @@ AOC_U::AOC_U(Core::System& system_) aoc_change_event = service_context.CreateEvent("GetAddOnContentListChanged:Event"); } -AOC_U::~AOC_U() { +IAddOnContentManager::~IAddOnContentManager() { service_context.CloseEvent(aoc_change_event); } -Result AOC_U::CountAddOnContent(Out out_count, ClientProcessId process_id) { +Result IAddOnContentManager::CountAddOnContent(Out out_count, ClientProcessId process_id) { LOG_DEBUG(Service_AOC, "called. process_id={}", process_id.pid); const auto current = system.GetApplicationProcessProgramID(); @@ -103,8 +103,9 @@ Result AOC_U::CountAddOnContent(Out out_count, ClientProcessId process_id) R_SUCCEED(); } -Result AOC_U::ListAddOnContent(Out out_count, OutBuffer out_addons, - u32 offset, u32 count, ClientProcessId process_id) { +Result IAddOnContentManager::ListAddOnContent(Out out_count, + OutBuffer out_addons, + u32 offset, u32 count, ClientProcessId process_id) { LOG_DEBUG(Service_AOC, "called with offset={}, count={}, process_id={}", offset, count, process_id.pid); @@ -128,12 +129,13 @@ Result AOC_U::ListAddOnContent(Out out_count, OutBuffer(std::min(out.size() - offset, count)); std::rotate(out.begin(), out.begin() + offset, out.end()); - std::memcpy(out_addons.data(), out.data(), *out_count); + std::memcpy(out_addons.data(), out.data(), *out_count * sizeof(u32)); R_SUCCEED(); } -Result AOC_U::GetAddOnContentBaseId(Out out_title_id, ClientProcessId process_id) { +Result IAddOnContentManager::GetAddOnContentBaseId(Out out_title_id, + ClientProcessId process_id) { LOG_DEBUG(Service_AOC, "called. process_id={}", process_id.pid); const auto title_id = system.GetApplicationProcessProgramID(); @@ -151,14 +153,15 @@ Result AOC_U::GetAddOnContentBaseId(Out out_title_id, ClientProcessId proce R_SUCCEED(); } -Result AOC_U::PrepareAddOnContent(s32 addon_index, ClientProcessId process_id) { +Result IAddOnContentManager::PrepareAddOnContent(s32 addon_index, ClientProcessId process_id) { LOG_WARNING(Service_AOC, "(STUBBED) called with addon_index={}, process_id={}", addon_index, process_id.pid); R_SUCCEED(); } -Result AOC_U::GetAddOnContentListChangedEvent(OutCopyHandle out_event) { +Result IAddOnContentManager::GetAddOnContentListChangedEvent( + OutCopyHandle out_event) { LOG_WARNING(Service_AOC, "(STUBBED) called"); *out_event = &aoc_change_event->GetReadableEvent(); @@ -166,7 +169,7 @@ Result AOC_U::GetAddOnContentListChangedEvent(OutCopyHandle out_event, ClientProcessId process_id) { LOG_WARNING(Service_AOC, "(STUBBED) called"); @@ -175,25 +178,26 @@ Result AOC_U::GetAddOnContentListChangedEventWithProcessId( R_SUCCEED(); } -Result AOC_U::NotifyMountAddOnContent() { +Result IAddOnContentManager::NotifyMountAddOnContent() { LOG_WARNING(Service_AOC, "(STUBBED) called"); R_SUCCEED(); } -Result AOC_U::NotifyUnmountAddOnContent() { +Result IAddOnContentManager::NotifyUnmountAddOnContent() { LOG_WARNING(Service_AOC, "(STUBBED) called"); R_SUCCEED(); } -Result AOC_U::CheckAddOnContentMountStatus() { +Result IAddOnContentManager::CheckAddOnContentMountStatus() { LOG_WARNING(Service_AOC, "(STUBBED) called"); R_SUCCEED(); } -Result AOC_U::CreateEcPurchasedEventManager(OutInterface out_interface) { +Result IAddOnContentManager::CreateEcPurchasedEventManager( + OutInterface out_interface) { LOG_WARNING(Service_AOC, "(STUBBED) called"); *out_interface = std::make_shared(system); @@ -201,7 +205,7 @@ Result AOC_U::CreateEcPurchasedEventManager(OutInterface R_SUCCEED(); } -Result AOC_U::CreatePermanentEcPurchasedEventManager( +Result IAddOnContentManager::CreatePermanentEcPurchasedEventManager( OutInterface out_interface) { LOG_WARNING(Service_AOC, "(STUBBED) called"); @@ -212,7 +216,7 @@ Result AOC_U::CreatePermanentEcPurchasedEventManager( void LoopProcess(Core::System& system) { auto server_manager = std::make_unique(system); - server_manager->RegisterNamedService("aoc:u", std::make_shared(system)); + server_manager->RegisterNamedService("aoc:u", std::make_shared(system)); ServerManager::RunServer(std::move(server_manager)); } diff --git a/src/core/hle/service/aoc/aoc_u.h b/src/core/hle/service/aoc/addon_content_manager.h similarity index 89% rename from src/core/hle/service/aoc/aoc_u.h rename to src/core/hle/service/aoc/addon_content_manager.h index c7b94a9335..91857df4c7 100644 --- a/src/core/hle/service/aoc/aoc_u.h +++ b/src/core/hle/service/aoc/addon_content_manager.h @@ -19,10 +19,10 @@ namespace Service::AOC { class IPurchaseEventManager; -class AOC_U final : public ServiceFramework { +class IAddOnContentManager final : public ServiceFramework { public: - explicit AOC_U(Core::System& system); - ~AOC_U() override; + explicit IAddOnContentManager(Core::System& system); + ~IAddOnContentManager() override; Result CountAddOnContent(Out out_count, ClientProcessId process_id); Result ListAddOnContent(Out out_count, OutBuffer out_addons, diff --git a/src/core/hle/service/aoc/purchase_event_manager.cpp b/src/core/hle/service/aoc/purchase_event_manager.cpp index 1192263f8a..9e718510b4 100644 --- a/src/core/hle/service/aoc/purchase_event_manager.cpp +++ b/src/core/hle/service/aoc/purchase_event_manager.cpp @@ -9,8 +9,8 @@ namespace Service::AOC { constexpr Result ResultNoPurchasedProductInfoAvailable{ErrorModule::NIMShop, 400}; IPurchaseEventManager::IPurchaseEventManager(Core::System& system_) - : ServiceFramework{system_, "IPurchaseEventManager"}, - service_context{system, "IPurchaseEventManager"} { + : ServiceFramework{system_, "IPurchaseEventManager"}, service_context{system, + "IPurchaseEventManager"} { // clang-format off static const FunctionInfo functions[] = { {0, D<&IPurchaseEventManager::SetDefaultDeliveryTarget>, "SetDefaultDeliveryTarget"}, diff --git a/src/core/hle/service/services.cpp b/src/core/hle/service/services.cpp index d6c6eff500..a2a3e556c1 100644 --- a/src/core/hle/service/services.cpp +++ b/src/core/hle/service/services.cpp @@ -5,7 +5,7 @@ #include "core/hle/service/acc/acc.h" #include "core/hle/service/am/am.h" -#include "core/hle/service/aoc/aoc_u.h" +#include "core/hle/service/aoc/addon_content_manager.h" #include "core/hle/service/apm/apm.h" #include "core/hle/service/audio/audio.h" #include "core/hle/service/bcat/bcat.h"