aboutsummaryrefslogtreecommitdiff
path: root/src/audio_core
diff options
context:
space:
mode:
authorlizzie <lizzie@eden-emu.dev>2026-06-23 06:31:25 +0200
committercrueter <crueter@eden-emu.dev>2026-06-23 06:31:25 +0200
commit3aa0d462598beb2021308eee1d757d74fe33b312 (patch)
treeed425c3845ba8fa61184d97b0588cb049b518bd7 /src/audio_core
parentf8facda35fe7f17143f84bbfbdacc4cc48678946 (diff)
[video_core, hle] remove redundant parent references in system structs (#3908)
reworked a bit to remove references of parent objects and instead pass as arguments to methods to prevent useless reloads Signed-off-by: lizzie <lizzie@eden-emu.dev> Co-authored-by: maufeat <sahyno1996@gmail.com> Reviewed-on: https://git.eden-emu.dev/eden-emu/eden/pulls/3908 Reviewed-by: Maufeat <sahyno1996@gmail.com> Reviewed-by: crueter <crueter@eden-emu.dev>
Diffstat (limited to 'src/audio_core')
-rw-r--r--src/audio_core/audio_core.cpp8
-rw-r--r--src/audio_core/audio_core.h23
-rw-r--r--src/audio_core/audio_in_manager.cpp6
-rw-r--r--src/audio_core/audio_manager.cpp100
-rw-r--r--src/audio_core/audio_manager.h10
-rw-r--r--src/audio_core/audio_out_manager.cpp6
-rw-r--r--src/audio_core/device/device_session.cpp7
-rw-r--r--src/audio_core/in/audio_in_system.cpp6
-rw-r--r--src/audio_core/out/audio_out_system.cpp6
-rw-r--r--src/audio_core/renderer/system.cpp6
10 files changed, 88 insertions, 90 deletions
diff --git a/src/audio_core/audio_core.cpp b/src/audio_core/audio_core.cpp
index fcaab2b320..234c831ac0 100644
--- a/src/audio_core/audio_core.cpp
+++ b/src/audio_core/audio_core.cpp
@@ -1,3 +1,6 @@
+// SPDX-FileCopyrightText: Copyright 2026 Eden Emulator Project
+// SPDX-License-Identifier: GPL-3.0-or-later
+
// SPDX-FileCopyrightText: Copyright 2022 yuzu Emulator Project
// SPDX-License-Identifier: GPL-2.0-or-later
@@ -8,10 +11,11 @@
namespace AudioCore {
-AudioCore::AudioCore(Core::System& system) : audio_manager{std::make_unique<AudioManager>()} {
+AudioCore::AudioCore(Core::System& system) {
+ audio_manager.emplace();
CreateSinks();
// Must be created after the sinks
- adsp = std::make_unique<ADSP::ADSP>(system, *output_sink);
+ adsp.emplace(system, *output_sink);
}
AudioCore ::~AudioCore() {
diff --git a/src/audio_core/audio_core.h b/src/audio_core/audio_core.h
index e4e27fc661..ababd780b1 100644
--- a/src/audio_core/audio_core.h
+++ b/src/audio_core/audio_core.h
@@ -1,3 +1,6 @@
+// SPDX-FileCopyrightText: Copyright 2026 Eden Emulator Project
+// SPDX-License-Identifier: GPL-3.0-or-later
+
// SPDX-FileCopyrightText: Copyright 2022 yuzu Emulator Project
// SPDX-License-Identifier: GPL-2.0-or-later
@@ -15,10 +18,7 @@ class System;
namespace AudioCore {
-class AudioManager;
-/**
- * Main audio class, stored inside the core, and holding the audio manager, all sinks, and the ADSP.
- */
+/// @brief Main audio class, stored inside the core, and holding the audio manager, all sinks, and the ADSP.
class AudioCore {
public:
explicit AudioCore(Core::System& system);
@@ -50,27 +50,22 @@ public:
*/
Sink::Sink& GetInputSink();
- /**
- * Get the ADSP.
- *
- * @return Ref to the ADSP.
- */
+ /// @brief Get the ADSP.
+ /// @return Ref to the ADSP.
ADSP::ADSP& ADSP();
private:
- /**
- * Create the sinks on startup.
- */
+ /// @brief Create the sinks on startup.
void CreateSinks();
/// Main audio manager for audio in/out
- std::unique_ptr<AudioManager> audio_manager;
+ std::optional<AudioManager> audio_manager;
/// Sink used for audio renderer and audio out
std::unique_ptr<Sink::Sink> output_sink;
/// Sink used for audio input
std::unique_ptr<Sink::Sink> input_sink;
/// The ADSP in the sysmodule
- std::unique_ptr<ADSP::ADSP> adsp;
+ std::optional<ADSP::ADSP> adsp;
};
} // namespace AudioCore
diff --git a/src/audio_core/audio_in_manager.cpp b/src/audio_core/audio_in_manager.cpp
index 63b064922a..6b528e9db0 100644
--- a/src/audio_core/audio_in_manager.cpp
+++ b/src/audio_core/audio_in_manager.cpp
@@ -1,3 +1,6 @@
+// SPDX-FileCopyrightText: Copyright 2026 Eden Emulator Project
+// SPDX-License-Identifier: GPL-3.0-or-later
+
// SPDX-FileCopyrightText: Copyright 2022 yuzu Emulator Project
// SPDX-License-Identifier: GPL-2.0-or-later
@@ -41,8 +44,7 @@ void Manager::ReleaseSessionId(const size_t session_id) {
Result Manager::LinkToManager() {
std::scoped_lock l{mutex};
if (!linked_to_manager) {
- AudioManager& manager{system.AudioCore().GetAudioManager()};
- manager.SetInManager(std::bind(&Manager::BufferReleaseAndRegister, this));
+ system.AudioCore().GetAudioManager().SetInManager(std::bind(&Manager::BufferReleaseAndRegister, this));
linked_to_manager = true;
}
diff --git a/src/audio_core/audio_manager.cpp b/src/audio_core/audio_manager.cpp
index 10b56f2140..93142a53fb 100644
--- a/src/audio_core/audio_manager.cpp
+++ b/src/audio_core/audio_manager.cpp
@@ -1,81 +1,77 @@
+// SPDX-FileCopyrightText: Copyright 2026 Eden Emulator Project
+// SPDX-License-Identifier: GPL-3.0-or-later
+
// SPDX-FileCopyrightText: Copyright 2022 yuzu Emulator Project
// SPDX-License-Identifier: GPL-2.0-or-later
#include "audio_core/audio_manager.h"
+#include "common/thread.h"
#include "core/core.h"
#include "core/hle/service/audio/errors.h"
namespace AudioCore {
AudioManager::AudioManager() {
- thread = std::jthread([this]() { ThreadFunc(); });
+ thread = std::jthread([this](std::stop_token stop_token) {
+ Common::SetCurrentThreadName("AudioManager");
+ std::unique_lock l{events.GetAudioEventLock()};
+ events.ClearEvents();
+ while (!stop_token.stop_requested()) {
+ const auto timed_out{events.Wait(l, std::chrono::seconds(2))};
+ if (events.CheckAudioEventSet(Event::Type::Max)) {
+ break;
+ }
+ for (size_t i = 0; i < buffer_events.size(); i++) {
+ const auto event_type = Event::Type(i);
+ if (events.CheckAudioEventSet(event_type) || timed_out) {
+ if (buffer_events[i]) {
+ buffer_events[i]();
+ }
+ }
+ events.SetAudioEvent(event_type, false);
+ }
+ }
+ });
}
void AudioManager::Shutdown() {
- running = false;
events.SetAudioEvent(Event::Type::Max, true);
- thread.join();
+ if (thread.joinable()) {
+ thread.request_stop();
+ thread.join();
+ }
}
Result AudioManager::SetOutManager(BufferEventFunc buffer_func) {
- if (!running) {
- return Service::Audio::ResultOperationFailed;
- }
-
- std::scoped_lock l{lock};
-
- const auto index{events.GetManagerIndex(Event::Type::AudioOutManager)};
- if (buffer_events[index] == nullptr) {
- buffer_events[index] = std::move(buffer_func);
- needs_update = true;
- events.SetAudioEvent(Event::Type::AudioOutManager, true);
+ if (thread.joinable()) {
+ std::scoped_lock l{lock};
+ const auto index{events.GetManagerIndex(Event::Type::AudioOutManager)};
+ if (buffer_events[index] == nullptr) {
+ buffer_events[index] = std::move(buffer_func);
+ needs_update = true;
+ events.SetAudioEvent(Event::Type::AudioOutManager, true);
+ }
+ return ResultSuccess;
}
- return ResultSuccess;
+ return Service::Audio::ResultOperationFailed;
}
Result AudioManager::SetInManager(BufferEventFunc buffer_func) {
- if (!running) {
- return Service::Audio::ResultOperationFailed;
- }
-
- std::scoped_lock l{lock};
-
- const auto index{events.GetManagerIndex(Event::Type::AudioInManager)};
- if (buffer_events[index] == nullptr) {
- buffer_events[index] = std::move(buffer_func);
- needs_update = true;
- events.SetAudioEvent(Event::Type::AudioInManager, true);
+ if (thread.joinable()) {
+ std::scoped_lock l{lock};
+ const auto index{events.GetManagerIndex(Event::Type::AudioInManager)};
+ if (buffer_events[index] == nullptr) {
+ buffer_events[index] = std::move(buffer_func);
+ needs_update = true;
+ events.SetAudioEvent(Event::Type::AudioInManager, true);
+ }
+ return ResultSuccess;
}
- return ResultSuccess;
+ return Service::Audio::ResultOperationFailed;
}
void AudioManager::SetEvent(const Event::Type type, const bool signalled) {
events.SetAudioEvent(type, signalled);
}
-void AudioManager::ThreadFunc() {
- std::unique_lock l{events.GetAudioEventLock()};
- events.ClearEvents();
- running = true;
-
- while (running) {
- const auto timed_out{events.Wait(l, std::chrono::seconds(2))};
-
- if (events.CheckAudioEventSet(Event::Type::Max)) {
- break;
- }
-
- for (size_t i = 0; i < buffer_events.size(); i++) {
- const auto event_type = static_cast<Event::Type>(i);
-
- if (events.CheckAudioEventSet(event_type) || timed_out) {
- if (buffer_events[i]) {
- buffer_events[i]();
- }
- }
- events.SetAudioEvent(event_type, false);
- }
- }
-}
-
} // namespace AudioCore
diff --git a/src/audio_core/audio_manager.h b/src/audio_core/audio_manager.h
index 02270242ac..0194aa16db 100644
--- a/src/audio_core/audio_manager.h
+++ b/src/audio_core/audio_manager.h
@@ -1,3 +1,6 @@
+// SPDX-FileCopyrightText: Copyright 2026 Eden Emulator Project
+// SPDX-License-Identifier: GPL-3.0-or-later
+
// SPDX-FileCopyrightText: Copyright 2022 yuzu Emulator Project
// SPDX-License-Identifier: GPL-2.0-or-later
@@ -66,13 +69,6 @@ public:
void SetEvent(Event::Type type, bool signalled);
private:
- /**
- * Main thread, waiting on a manager signal and calling the registered function.
- */
- void ThreadFunc();
-
- /// Is the main thread running?
- std::atomic<bool> running{};
/// Unused
bool needs_update{};
/// Events to be set and signalled
diff --git a/src/audio_core/audio_out_manager.cpp b/src/audio_core/audio_out_manager.cpp
index 316ea7c817..569df8d1e0 100644
--- a/src/audio_core/audio_out_manager.cpp
+++ b/src/audio_core/audio_out_manager.cpp
@@ -1,3 +1,6 @@
+// SPDX-FileCopyrightText: Copyright 2026 Eden Emulator Project
+// SPDX-License-Identifier: GPL-3.0-or-later
+
// SPDX-FileCopyrightText: Copyright 2022 yuzu Emulator Project
// SPDX-License-Identifier: GPL-2.0-or-later
@@ -40,8 +43,7 @@ void Manager::ReleaseSessionId(const size_t session_id) {
Result Manager::LinkToManager() {
std::scoped_lock l{mutex};
if (!linked_to_manager) {
- AudioManager& manager{system.AudioCore().GetAudioManager()};
- manager.SetOutManager(std::bind(&Manager::BufferReleaseAndRegister, this));
+ system.AudioCore().GetAudioManager().SetOutManager(std::bind(&Manager::BufferReleaseAndRegister, this));
linked_to_manager = true;
}
diff --git a/src/audio_core/device/device_session.cpp b/src/audio_core/device/device_session.cpp
index 2a1ae1bb3f..2e9da7876f 100644
--- a/src/audio_core/device/device_session.cpp
+++ b/src/audio_core/device/device_session.cpp
@@ -1,3 +1,6 @@
+// SPDX-FileCopyrightText: Copyright 2026 Eden Emulator Project
+// SPDX-License-Identifier: GPL-3.0-or-later
+
// SPDX-FileCopyrightText: Copyright 2022 yuzu Emulator Project
// SPDX-License-Identifier: GPL-2.0-or-later
@@ -39,7 +42,7 @@ Result DeviceSession::Initialize(std::string_view name_, SampleFormat sample_for
channel_count = channel_count_;
session_id = session_id_;
handle = handle_;
- handle->Open();
+ handle->Open(system.Kernel());
applet_resource_user_id = applet_resource_user_id_;
if (type == Sink::StreamType::In) {
@@ -60,7 +63,7 @@ void DeviceSession::Finalize() {
}
if (handle) {
- handle->Close();
+ handle->Close(system.Kernel());
handle = nullptr;
}
}
diff --git a/src/audio_core/in/audio_in_system.cpp b/src/audio_core/in/audio_in_system.cpp
index ec1f565736..f71a2df6be 100644
--- a/src/audio_core/in/audio_in_system.cpp
+++ b/src/audio_core/in/audio_in_system.cpp
@@ -122,7 +122,7 @@ Result System::Stop() {
session->SetVolume(0.0f);
session->ClearBuffers();
if (buffers.ReleaseBuffers(system.CoreTiming(), *session, true)) {
- buffer_event->Signal();
+ buffer_event->Signal(system.Kernel());
}
state = State::Stopped;
}
@@ -164,7 +164,7 @@ void System::ReleaseBuffers() {
if (signal) {
// Signal if any buffer was released, or if none are registered, we need more.
- buffer_event->Signal();
+ buffer_event->Signal(system.Kernel());
}
}
@@ -181,7 +181,7 @@ bool System::FlushAudioInBuffers() {
buffers.FlushBuffers(buffers_released);
if (buffers_released > 0) {
- buffer_event->Signal();
+ buffer_event->Signal(system.Kernel());
}
return true;
}
diff --git a/src/audio_core/out/audio_out_system.cpp b/src/audio_core/out/audio_out_system.cpp
index dcb0d0694e..46f30f19c6 100644
--- a/src/audio_core/out/audio_out_system.cpp
+++ b/src/audio_core/out/audio_out_system.cpp
@@ -121,7 +121,7 @@ Result System::Stop() {
session->SetVolume(0.0f);
session->ClearBuffers();
if (buffers.ReleaseBuffers(system.CoreTiming(), *session, true)) {
- buffer_event->Signal();
+ buffer_event->Signal(system.Kernel());
}
state = State::Stopped;
}
@@ -162,7 +162,7 @@ void System::ReleaseBuffers() {
bool signal{buffers.ReleaseBuffers(system.CoreTiming(), *session, false)};
if (signal) {
// Signal if any buffer was released, or if none are registered, we need more.
- buffer_event->Signal();
+ buffer_event->Signal(system.Kernel());
}
}
@@ -179,7 +179,7 @@ bool System::FlushAudioOutBuffers() {
buffers.FlushBuffers(buffers_released);
if (buffers_released > 0) {
- buffer_event->Signal();
+ buffer_event->Signal(system.Kernel());
}
return true;
}
diff --git a/src/audio_core/renderer/system.cpp b/src/audio_core/renderer/system.cpp
index 2053ec6dd4..b7675ff5de 100644
--- a/src/audio_core/renderer/system.cpp
+++ b/src/audio_core/renderer/system.cpp
@@ -1,4 +1,4 @@
-// SPDX-FileCopyrightText: Copyright 2025 Eden Emulator Project
+// SPDX-FileCopyrightText: Copyright 2026 Eden Emulator Project
// SPDX-License-Identifier: GPL-3.0-or-later
// SPDX-FileCopyrightText: Copyright 2022 yuzu Emulator Project
@@ -540,7 +540,7 @@ Result System::Update(std::span<const u8> input, std::span<u8> performance, std:
return result;
}
- adsp_rendered_event->Clear();
+ adsp_rendered_event->Clear(core.Kernel());
num_times_updated++;
const auto end_time{core.CoreTiming().GetGlobalTimeNs().count()};
@@ -624,7 +624,7 @@ void System::SendCommandToDsp() {
reset_command_buffers = false;
command_buffer_size = command_size;
if (remaining_command_count == 0) {
- adsp_rendered_event->Signal();
+ adsp_rendered_event->Signal(core.Kernel());
}
} else {
audio_renderer.ClearRemainCommandCount(session_id);