totallyregularhuman/Ryujinx
1
0
Fork 0
forked from Mirror/Ryujinx
Code Pull requests Activity

amadeus: Fix wrong SendCommands logic (#3969)

Browse source 
* amadeus: Fix wrong SendCommands logic

Might help with audio desync, might cause audio stutters, will see!

* Address gdkchan's comment
This commit is contained in:
Mary-nyan 2022-12-02 14:01:19 +01:00 committed by GitHub
parent 21a081b185
commit c25e8427aa
No known key found for this signature in database
GPG key ID: 4AEE18F83AFDEB23
2 changed files with 18 additions and 6 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

5
Ryujinx.Audio/Renderer/Dsp/AudioProcessor.cs
View file

@ -116,6 +116,11 @@ namespace Ryujinx.Audio.Renderer.Dsp
}; };
} }
public bool HasRemainingCommands(int sessionId)
{
return _sessionCommandList[sessionId] != null;
}
public void Signal() public void Signal()
{ {
_mailbox.SendMessage(MailboxMessage.RenderStart); _mailbox.SendMessage(MailboxMessage.RenderStart);

19
Ryujinx.Audio/Renderer/Server/AudioRenderSystem.cs
View file

@ -670,14 +670,21 @@ namespace Ryujinx.Audio.Renderer.Server
{ {
_terminationEvent.Reset(); _terminationEvent.Reset();
GenerateCommandList(out CommandList commands); if (!_manager.Processor.HasRemainingCommands(_sessionId))
{
GenerateCommandList(out CommandList commands);
_manager.Processor.Send(_sessionId, _manager.Processor.Send(_sessionId,
commands, commands,
GetMaxAllocatedTimeForDsp(), GetMaxAllocatedTimeForDsp(),
_appletResourceId); _appletResourceId);
_systemEvent.Signal(); _systemEvent.Signal();
}
else
{
_isDspRunningBehind = true;
}
} }
else else
{ {