diff options
author | bunnei <bunneidev@gmail.com> | 2021-09-14 08:32:43 +0200 |
---|---|---|
committer | GitHub <noreply@github.com> | 2021-09-14 08:32:43 +0200 |
commit | 081ccc644126fc339a8c11ca97a5a715256f125e (patch) | |
tree | e1dd7195d56f4e49c4c188c1cfa5ec01a9eaea39 /src | |
parent | Merge pull request #6943 from FernandoS27/omae-wa-mou-shindeiru (diff) | |
parent | core: Destroy main_process during shutdown (diff) | |
download | yuzu-081ccc644126fc339a8c11ca97a5a715256f125e.tar yuzu-081ccc644126fc339a8c11ca97a5a715256f125e.tar.gz yuzu-081ccc644126fc339a8c11ca97a5a715256f125e.tar.bz2 yuzu-081ccc644126fc339a8c11ca97a5a715256f125e.tar.lz yuzu-081ccc644126fc339a8c11ca97a5a715256f125e.tar.xz yuzu-081ccc644126fc339a8c11ca97a5a715256f125e.tar.zst yuzu-081ccc644126fc339a8c11ca97a5a715256f125e.zip |
Diffstat (limited to '')
-rw-r--r-- | src/core/core.cpp | 15 |
1 files changed, 12 insertions, 3 deletions
diff --git a/src/core/core.cpp b/src/core/core.cpp index ba4629993..b13350f6e 100644 --- a/src/core/core.cpp +++ b/src/core/core.cpp @@ -83,6 +83,12 @@ FileSys::StorageId GetStorageIdForFrontendSlot( } } +void KProcessDeleter(Kernel::KProcess* process) { + process->Destroy(); +} + +using KProcessPtr = std::unique_ptr<Kernel::KProcess, decltype(&KProcessDeleter)>; + } // Anonymous namespace FileSys::VirtualFile GetGameFileFromPath(const FileSys::VirtualFilesystem& vfs, @@ -233,8 +239,8 @@ struct System::Impl { } telemetry_session->AddInitialInfo(*app_loader, fs_controller, *content_provider); - auto main_process = Kernel::KProcess::Create(system.Kernel()); - ASSERT(Kernel::KProcess::Initialize(main_process, system, "main", + main_process = KProcessPtr{Kernel::KProcess::Create(system.Kernel()), KProcessDeleter}; + ASSERT(Kernel::KProcess::Initialize(main_process.get(), system, "main", Kernel::KProcess::ProcessType::Userland) .IsSuccess()); main_process->Open(); @@ -247,7 +253,7 @@ struct System::Impl { static_cast<u32>(load_result)); } AddGlueRegistrationForProcess(*app_loader, *main_process); - kernel.MakeCurrentProcess(main_process); + kernel.MakeCurrentProcess(main_process.get()); kernel.InitializeCores(); // Initialize cheat engine @@ -316,6 +322,8 @@ struct System::Impl { kernel.Shutdown(); memory.Reset(); applet_manager.ClearAll(); + // TODO: The main process should be freed based on KAutoObject ref counting. + main_process.reset(); LOG_DEBUG(Core, "Shutdown OK"); } @@ -374,6 +382,7 @@ struct System::Impl { std::unique_ptr<Tegra::GPU> gpu_core; std::unique_ptr<Hardware::InterruptManager> interrupt_manager; std::unique_ptr<Core::DeviceMemory> device_memory; + KProcessPtr main_process{nullptr, KProcessDeleter}; Core::Memory::Memory memory; CpuManager cpu_manager; std::atomic_bool is_powered_on{}; |