Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Gpu Sync #18

Merged
merged 1 commit into from
Aug 26, 2022
Merged

Gpu Sync #18

merged 1 commit into from
Aug 26, 2022

Conversation

Amjad50
Copy link
Owner

@Amjad50 Amjad50 commented Aug 24, 2022

fixes #17

GPU_STAT had a sync issue, for example if you set the command GP1(4),
which changes the DMA_DIRECTION flags, there might be some delay for it
to take effect, since the change was happening in the backend thread.
Some games require immediate change or else the game hangs.

Now moved anything that should be synced (creating the commands,
handling any GP1 command that does not need gpu_context) in the
frontend.

GP0 commands are constructed in the frontend, and then sent to the backend which just runs them with gpu_context.

This improved the accuracy and stability of the GPU unit.

GPU_STAT had a sync issue, for example if you set the command GP1(4),
which changes the DMA_DIRECTION flags, there might be some delay for it
to take effect, since the change was happening in the backend thread.
Some games require immediate change or else the game hangs.

Now moved anything that should be synced (creating the commands,
handling any GP1 command that does not need `gpu_context`) in the
frontend.

This improved the accuracy and stability of the GPU unit.
@Amjad50 Amjad50 changed the title Gpu: Moved creation of Gp0Command to frontend to sync GPU_STAT Gpu Sync Aug 24, 2022
@Amjad50 Amjad50 merged commit 00263dd into master Aug 26, 2022
@Amjad50 Amjad50 deleted the gpu_sync branch August 26, 2022 10:38
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

GPU sync problem
1 participant