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

Implement invalid NV4097_NOTIFY context handling #5540

Merged
merged 2 commits into from Jan 13, 2019

Conversation

@elad335
Copy link
Contributor

elad335 commented Jan 12, 2019

fixes #5538 , invalid context is set at startup pointing into system reserved area. simply ignore it instead of throwing an exception.
old HW tests showed that even context = 0 is passable in this method.

@AniLeo AniLeo requested a review from kd-11 Jan 12, 2019
@AniLeo AniLeo added RSX Bugfix labels Jan 12, 2019
@kd-11 kd-11 merged commit bc27f5f into RPCS3:master Jan 13, 2019
2 checks passed
2 checks passed
continuous-integration/appveyor/pr AppVeyor build succeeded
Details
continuous-integration/travis-ci/pr The Travis CI build passed
Details
@elad335 elad335 deleted the elad335:notify branch Jan 14, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
3 participants
You can’t perform that action at this time.