summaryrefslogtreecommitdiff
path: root/src
diff options
context:
space:
mode:
authorGravatar David Marcec2019-07-13 01:22:08 +1000
committerGravatar David Marcec2019-07-13 01:22:08 +1000
commit73b37886c10f771abe05b57f53d8a570ee80b178 (patch)
tree769f3d823a4783d42a1621798debb598e106466d /src
parentMerge pull request #2609 from FernandoS27/new-scan (diff)
downloadyuzu-73b37886c10f771abe05b57f53d8a570ee80b178.tar.gz
yuzu-73b37886c10f771abe05b57f53d8a570ee80b178.tar.xz
yuzu-73b37886c10f771abe05b57f53d8a570ee80b178.zip
"AudioRenderer" thread should have a unique name
Creating multiple "AudioRenderer" threads cause the previous thread to be overwritten. The thread will name be renamed to AudioRenderer-InstanceX, where X is the current instance number.
Diffstat (limited to 'src')
-rw-r--r--src/audio_core/audio_renderer.cpp5
-rw-r--r--src/audio_core/audio_renderer.h2
-rw-r--r--src/core/hle/service/audio/audren_u.cpp7
-rw-r--r--src/core/hle/service/audio/audren_u.h1
4 files changed, 8 insertions, 7 deletions
diff --git a/src/audio_core/audio_renderer.cpp b/src/audio_core/audio_renderer.cpp
index 4882a6cd8..969d8f2a5 100644
--- a/src/audio_core/audio_renderer.cpp
+++ b/src/audio_core/audio_renderer.cpp
@@ -73,13 +73,14 @@ private:
73 EffectInStatus info{}; 73 EffectInStatus info{};
74}; 74};
75AudioRenderer::AudioRenderer(Core::Timing::CoreTiming& core_timing, AudioRendererParameter params, 75AudioRenderer::AudioRenderer(Core::Timing::CoreTiming& core_timing, AudioRendererParameter params,
76 Kernel::SharedPtr<Kernel::WritableEvent> buffer_event) 76 Kernel::SharedPtr<Kernel::WritableEvent> buffer_event,
77 const std::size_t instance_number)
77 : worker_params{params}, buffer_event{buffer_event}, voices(params.voice_count), 78 : worker_params{params}, buffer_event{buffer_event}, voices(params.voice_count),
78 effects(params.effect_count) { 79 effects(params.effect_count) {
79 80
80 audio_out = std::make_unique<AudioCore::AudioOut>(); 81 audio_out = std::make_unique<AudioCore::AudioOut>();
81 stream = audio_out->OpenStream(core_timing, STREAM_SAMPLE_RATE, STREAM_NUM_CHANNELS, 82 stream = audio_out->OpenStream(core_timing, STREAM_SAMPLE_RATE, STREAM_NUM_CHANNELS,
82 "AudioRenderer", [=]() { buffer_event->Signal(); }); 83 fmt::format("AudioRenderer-Instance{}", instance_number), [=]() { buffer_event->Signal(); });
83 audio_out->StartStream(stream); 84 audio_out->StartStream(stream);
84 85
85 QueueMixedBuffer(0); 86 QueueMixedBuffer(0);
diff --git a/src/audio_core/audio_renderer.h b/src/audio_core/audio_renderer.h
index b2e5d336c..1bdf709e5 100644
--- a/src/audio_core/audio_renderer.h
+++ b/src/audio_core/audio_renderer.h
@@ -215,7 +215,7 @@ static_assert(sizeof(UpdateDataHeader) == 0x40, "UpdateDataHeader has wrong size
215class AudioRenderer { 215class AudioRenderer {
216public: 216public:
217 AudioRenderer(Core::Timing::CoreTiming& core_timing, AudioRendererParameter params, 217 AudioRenderer(Core::Timing::CoreTiming& core_timing, AudioRendererParameter params,
218 Kernel::SharedPtr<Kernel::WritableEvent> buffer_event); 218 Kernel::SharedPtr<Kernel::WritableEvent> buffer_event, const std::size_t instance_number);
219 ~AudioRenderer(); 219 ~AudioRenderer();
220 220
221 std::vector<u8> UpdateAudioRenderer(const std::vector<u8>& input_params); 221 std::vector<u8> UpdateAudioRenderer(const std::vector<u8>& input_params);
diff --git a/src/core/hle/service/audio/audren_u.cpp b/src/core/hle/service/audio/audren_u.cpp
index 3711e1ea1..b6158d16e 100644
--- a/src/core/hle/service/audio/audren_u.cpp
+++ b/src/core/hle/service/audio/audren_u.cpp
@@ -25,7 +25,7 @@ namespace Service::Audio {
25 25
26class IAudioRenderer final : public ServiceFramework<IAudioRenderer> { 26class IAudioRenderer final : public ServiceFramework<IAudioRenderer> {
27public: 27public:
28 explicit IAudioRenderer(AudioCore::AudioRendererParameter audren_params) 28 explicit IAudioRenderer(AudioCore::AudioRendererParameter audren_params, const std::size_t instance_number)
29 : ServiceFramework("IAudioRenderer") { 29 : ServiceFramework("IAudioRenderer") {
30 // clang-format off 30 // clang-format off
31 static const FunctionInfo functions[] = { 31 static const FunctionInfo functions[] = {
@@ -48,8 +48,7 @@ public:
48 auto& system = Core::System::GetInstance(); 48 auto& system = Core::System::GetInstance();
49 system_event = Kernel::WritableEvent::CreateEventPair( 49 system_event = Kernel::WritableEvent::CreateEventPair(
50 system.Kernel(), Kernel::ResetType::Manual, "IAudioRenderer:SystemEvent"); 50 system.Kernel(), Kernel::ResetType::Manual, "IAudioRenderer:SystemEvent");
51 renderer = std::make_unique<AudioCore::AudioRenderer>(system.CoreTiming(), audren_params, 51 renderer = std::make_unique<AudioCore::AudioRenderer>(system.CoreTiming(), audren_params, system_event.writable, instance_number);
52 system_event.writable);
53 } 52 }
54 53
55private: 54private:
@@ -607,7 +606,7 @@ void AudRenU::OpenAudioRendererImpl(Kernel::HLERequestContext& ctx) {
607 IPC::ResponseBuilder rb{ctx, 2, 0, 1}; 606 IPC::ResponseBuilder rb{ctx, 2, 0, 1};
608 607
609 rb.Push(RESULT_SUCCESS); 608 rb.Push(RESULT_SUCCESS);
610 rb.PushIpcInterface<IAudioRenderer>(params); 609 rb.PushIpcInterface<IAudioRenderer>(params, audren_instance_count++);
611} 610}
612 611
613bool AudRenU::IsFeatureSupported(AudioFeatures feature, u32_le revision) const { 612bool AudRenU::IsFeatureSupported(AudioFeatures feature, u32_le revision) const {
diff --git a/src/core/hle/service/audio/audren_u.h b/src/core/hle/service/audio/audren_u.h
index 1d3c8df61..49f2733cf 100644
--- a/src/core/hle/service/audio/audren_u.h
+++ b/src/core/hle/service/audio/audren_u.h
@@ -33,6 +33,7 @@ private:
33 }; 33 };
34 34
35 bool IsFeatureSupported(AudioFeatures feature, u32_le revision) const; 35 bool IsFeatureSupported(AudioFeatures feature, u32_le revision) const;
36 std::size_t audren_instance_count = 0;
36}; 37};
37 38
38} // namespace Service::Audio 39} // namespace Service::Audio