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

Correct error codes and dup checks in intr registration #6389

Merged
merged 2 commits into from
Jun 22, 2014

Conversation

unknownbrackets
Copy link
Collaborator

I noticed a game in a log registered the same subintr twice. I checked and that is not allowed.

This corrects a bunch of error codes and some of the handling there, which could fix games with bugs or that check error codes.

-[Unknown]

This checks for duplicate registrations and other thigns that could've
been causing wrong behavior.
Could've caused interrupts, primarily vblanks, not to run because we
implicitly disabled them on register.
@sum2012
Copy link
Collaborator

sum2012 commented Jun 22, 2014

@unknownbrackets Well done
Fixed SBK-09 Superbike World Champinoship video
(06:37:058 user_main E[INTC]: HLE\sceKernelInterrupt.cpp:516 sceKernelRegisterSubIntrHandler(30, 0, 08a22378, 08fb3600): duplicate handler)
Info log:
https://gist.github.com/sum2012/44349283f2d7a965f39c
debug log
https://drive.google.com/file/d/0B3OaSdeV0L8kSmdodjZqTThMZlk/edit?usp=sharing

hrydgard added a commit that referenced this pull request Jun 22, 2014
Correct error codes and dup checks in inter registration
@hrydgard hrydgard merged commit b543526 into hrydgard:master Jun 22, 2014
@unknownbrackets unknownbrackets changed the title Correct error codes and dup checks in inter registration Correct error codes and dup checks in intr registration Jun 22, 2014
@unknownbrackets unknownbrackets deleted the intrman branch June 22, 2014 17:30
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.

3 participants