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.

Sign up for GitHub

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

Jump to bottom

SPU: Mask SRR0 at write #6685

Merged
merged 2 commits into from
Oct 7, 2019
Merged

SPU: Mask SRR0 at write #6685

merged 2 commits into from
Oct 7, 2019

Conversation

elad335
Copy link
Contributor

@elad335 elad335 commented Oct 1, 2019 •
edited

testcase : https://github.com/elad335/myps3tests/tree/master/spu_tests/SRR0%20masking
Testcase write -1 to srr0, read value is 0x3fffc.

@elad335 elad335 force-pushed the srr0 branch 2 times, most recently from 61ebe68 to 2f88009 Compare October 1, 2019 10:45
@Nekotekina
Copy link
Member

I wonder what srr0 value sys_dbg_read_spu_thread_context may print. In case it's only masked on reads.

@elad335
Copy link
Contributor Author

elad335 commented Oct 3, 2019 •
edited 

SRR0 as read from SPU: 0x3fffc
SRR0 as read from PPU: 0x3fffc

Updated testcase.

@elad335 elad335 changed the title spu: Mask SRR0 at write SPU: Mask SRR0 at write Oct 4, 2019
@elad335 elad335 force-pushed the srr0 branch 2 times, most recently from bee74c2 to 28db875 Compare October 4, 2019 10:05
@Nekotekina Nekotekina merged commit 36de3d4 into RPCS3:master Oct 7, 2019
@elad335 elad335 deleted the srr0 branch October 8, 2019 02:56
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@andrew-boyarshin andrew-boyarshin andrew-boyarshin left review comments

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

None yet
3 participants
@elad335 @Nekotekina @andrew-boyarshin