Skip to content

Add matrix AB ReadNoSnp path#4

Closed
zykucas wants to merge 1 commit into
hbl2-newfrom
feature/matrix-ab-readnosnp
Closed

Add matrix AB ReadNoSnp path#4
zykucas wants to merge 1 commit into
hbl2-newfrom
feature/matrix-ab-readnosnp

Conversation

@zykucas

@zykucas zykucas commented Jul 2, 2026

Copy link
Copy Markdown

Summary

This PR adds support for routing Matrix A/B Get requests through the CHI ReadNoSnp path when enabled by configuration.

The feature is controlled by a new EnableMatrixABNoSnpGet parameter. When enabled, Matrix A/B reads are marked in the L2 request path and later converted to ReadNoSnp on the TL-CHI side.

Changes

  • Added EnableMatrixABNoSnpGet as a configurable L2 parameter.
  • Added Matrix A/B identification helpers.
  • Added a matrixAB bit to TaskBundle and MetaEntry.
  • Marked Matrix A/B Get requests in SinkA.
  • Converted eligible Matrix A/B Get requests to CHI ReadNoSnp in tl2chi/MSHR.
  • Disabled unnecessary CompAck/snoop attributes for Matrix A/B ReadNoSnp.
  • Preserved matrixAB metadata through MainPipe metadata updates.
  • Added silent replacement handling for clean Matrix A/B ReadNoSnp cachelines.
  • Fixed the TL-to-TL MainPipe path to initialize and preserve the new matrixAB field.

Validation

  • coupledL2-test-matrix-trace: passed
    • Trace summary: sent{R/W/E}=160/32/0, recv{R/W/E}=160/32/0
    • Finalized at cycle 2222

This PR is expected to be used together with the corresponding OpenLLC PR that adds LLC-side ReadNoSnp handling.

Copy link
Copy Markdown
Member

This work has been migrated to OpenXiangShan/XSAICache#5. Closing this PR in favor of the consolidated implementation there.

@Ivyfeather Ivyfeather closed this Jul 20, 2026
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.

2 participants