diff options
| -rw-r--r-- | src/core/hle/service/audio/hwopus.cpp | 39 |
1 files changed, 35 insertions, 4 deletions
diff --git a/src/core/hle/service/audio/hwopus.cpp b/src/core/hle/service/audio/hwopus.cpp index dffb890d5..11eba4a12 100644 --- a/src/core/hle/service/audio/hwopus.cpp +++ b/src/core/hle/service/audio/hwopus.cpp | |||
| @@ -37,7 +37,7 @@ public: | |||
| 37 | {3, nullptr, "SetContextForMultiStream"}, | 37 | {3, nullptr, "SetContextForMultiStream"}, |
| 38 | {4, &IHardwareOpusDecoderManager::DecodeInterleavedWithPerfOld, "DecodeInterleavedWithPerfOld"}, | 38 | {4, &IHardwareOpusDecoderManager::DecodeInterleavedWithPerfOld, "DecodeInterleavedWithPerfOld"}, |
| 39 | {5, nullptr, "DecodeInterleavedForMultiStreamWithPerfOld"}, | 39 | {5, nullptr, "DecodeInterleavedForMultiStreamWithPerfOld"}, |
| 40 | {6, nullptr, "DecodeInterleaved"}, | 40 | {6, &IHardwareOpusDecoderManager::DecodeInterleaved, "DecodeInterleaved"}, |
| 41 | {7, nullptr, "DecodeInterleavedForMultiStream"}, | 41 | {7, nullptr, "DecodeInterleavedForMultiStream"}, |
| 42 | }; | 42 | }; |
| 43 | // clang-format on | 43 | // clang-format on |
| @@ -46,24 +46,49 @@ public: | |||
| 46 | } | 46 | } |
| 47 | 47 | ||
| 48 | private: | 48 | private: |
| 49 | /// Describes extra behavior that may be asked of the decoding context. | ||
| 50 | enum class ExtraBehavior { | ||
| 51 | /// No extra behavior. | ||
| 52 | None, | ||
| 53 | |||
| 54 | /// Resets the decoder context back to a freshly initialized state. | ||
| 55 | ResetContext, | ||
| 56 | }; | ||
| 57 | |||
| 49 | void DecodeInterleavedOld(Kernel::HLERequestContext& ctx) { | 58 | void DecodeInterleavedOld(Kernel::HLERequestContext& ctx) { |
| 50 | LOG_DEBUG(Audio, "called"); | 59 | LOG_DEBUG(Audio, "called"); |
| 51 | 60 | ||
| 52 | DecodeInterleavedHelper(ctx, nullptr); | 61 | DecodeInterleavedHelper(ctx, nullptr, ExtraBehavior::None); |
| 53 | } | 62 | } |
| 54 | 63 | ||
| 55 | void DecodeInterleavedWithPerfOld(Kernel::HLERequestContext& ctx) { | 64 | void DecodeInterleavedWithPerfOld(Kernel::HLERequestContext& ctx) { |
| 56 | LOG_DEBUG(Audio, "called"); | 65 | LOG_DEBUG(Audio, "called"); |
| 57 | 66 | ||
| 58 | u64 performance = 0; | 67 | u64 performance = 0; |
| 59 | DecodeInterleavedHelper(ctx, &performance); | 68 | DecodeInterleavedHelper(ctx, &performance, ExtraBehavior::None); |
| 69 | } | ||
| 70 | |||
| 71 | void DecodeInterleaved(Kernel::HLERequestContext& ctx) { | ||
| 72 | LOG_DEBUG(Audio, "called"); | ||
| 73 | |||
| 74 | IPC::RequestParser rp{ctx}; | ||
| 75 | const auto extra_behavior = | ||
| 76 | rp.Pop<bool>() ? ExtraBehavior::ResetContext : ExtraBehavior::None; | ||
| 77 | |||
| 78 | u64 performance = 0; | ||
| 79 | DecodeInterleavedHelper(ctx, &performance, extra_behavior); | ||
| 60 | } | 80 | } |
| 61 | 81 | ||
| 62 | void DecodeInterleavedHelper(Kernel::HLERequestContext& ctx, u64* performance) { | 82 | void DecodeInterleavedHelper(Kernel::HLERequestContext& ctx, u64* performance, |
| 83 | ExtraBehavior extra_behavior) { | ||
| 63 | u32 consumed = 0; | 84 | u32 consumed = 0; |
| 64 | u32 sample_count = 0; | 85 | u32 sample_count = 0; |
| 65 | std::vector<opus_int16> samples(ctx.GetWriteBufferSize() / sizeof(opus_int16)); | 86 | std::vector<opus_int16> samples(ctx.GetWriteBufferSize() / sizeof(opus_int16)); |
| 66 | 87 | ||
| 88 | if (extra_behavior == ExtraBehavior::ResetContext) { | ||
| 89 | ResetDecoderContext(); | ||
| 90 | } | ||
| 91 | |||
| 67 | if (!Decoder_DecodeInterleaved(consumed, sample_count, ctx.ReadBuffer(), samples, | 92 | if (!Decoder_DecodeInterleaved(consumed, sample_count, ctx.ReadBuffer(), samples, |
| 68 | performance)) { | 93 | performance)) { |
| 69 | LOG_ERROR(Audio, "Failed to decode opus data"); | 94 | LOG_ERROR(Audio, "Failed to decode opus data"); |
| @@ -136,6 +161,12 @@ private: | |||
| 136 | return true; | 161 | return true; |
| 137 | } | 162 | } |
| 138 | 163 | ||
| 164 | void ResetDecoderContext() { | ||
| 165 | ASSERT(decoder != nullptr); | ||
| 166 | |||
| 167 | opus_decoder_ctl(decoder.get(), OPUS_RESET_STATE); | ||
| 168 | } | ||
| 169 | |||
| 139 | struct OpusHeader { | 170 | struct OpusHeader { |
| 140 | u32_be sz; // Needs to be BE for some odd reason | 171 | u32_be sz; // Needs to be BE for some odd reason |
| 141 | INSERT_PADDING_WORDS(1); | 172 | INSERT_PADDING_WORDS(1); |