Skip to content

Support profile-defined SJRMT UNI storage - #228

Open
GTTeancum wants to merge 1 commit into
ran-j:mainfrom
GTTeancum:codex/profile-sjrmt-uni-rings
Open

Support profile-defined SJRMT UNI storage#228
GTTeancum wants to merge 1 commit into
ran-j:mainfrom
GTTeancum:codex/profile-sjrmt-uni-rings

Conversation

@GTTeancum

Copy link
Copy Markdown

Summary

  • add optional profile fields for the SJRMT UNI payload offset and ring capacity
  • preserve the existing 256-byte normalization when a profile does not opt in
  • validate the profile-defined layout with a direct CRI DTX RPC test

Why

X-Men Legends initializes SJRMT UNI with a work area whose first 0x100 bytes are metadata and whose following 0x4000 bytes form the audio ring. Treating the whole request as the existing normalized 256-byte ring aliases metadata and payload storage. Keeping the layout profile-defined avoids changing established profiles while supporting this observed CRI variant.

Test plan

  • cmake --build out/sjrmt-test --config Release --target ps2x_tests -- /m:1
  • out/sjrmt-test/ps2xTest/Release/ps2x_tests.exe
  • 426 passed, 0 failed

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.

1 participant