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

feat(CMO): merged CMO requests into TileLink A Channel #282

Merged
merged 18 commits into from
Nov 30, 2024

Conversation

Kumonda221-CrO3
Copy link
Contributor

@Kumonda221-CrO3 Kumonda221-CrO3 commented Nov 6, 2024

* Removed 'hasCMO' parameter.

* Removed all SinkCMO related modules.

* Merged CMO requests into TileLink A Channel with extended
  4-bit opcode and custom encodings.

* Bump OpenXiangShan/rocket-chip.
@Kumonda221-CrO3
Copy link
Contributor Author

This PR was currently for unit tests and tl-test-new tests, don't merge before we support this in L1 DCache.

* Added support for **WriteCleanFull**.

* Expanded all opcode decoding to be compatible with new 4-bit opcode
encoding.

* Enabled datapath of ```mp_release``` -> ```ReleaseBuffer``` ->
```DataStorage```.

* Reuse ```mp_release``` for CMO-derived **WriteBackFull**,
**WriteCleanFull** and **Evict**.

* Reuse ```mp_acquire``` for CMO-derived **CleanShared**,
**CleanInvalid** and **MakeInvalid*.
@Kumonda221-CrO3
Copy link
Contributor Author

  • Added support for WriteCleanFull.
  • Expanded all opcode decoding to be compatible with new 4-bit opcode encoding.
  • Enabled datapath of mp_release -> ReleaseBuffer -> DataStorage.
  • Reuse mp_release for CMO-derived WriteBackFull, WriteCleanFull and Evict.
  • Reuse mp_acquire for CMO-derived CleanShared, CleanInvalid and MakeInvalid.

@Kumonda221-CrO3 Kumonda221-CrO3 merged commit c4ce81f into master Nov 30, 2024
1 check passed
@Kumonda221-CrO3 Kumonda221-CrO3 deleted the feat-Achannel-cmo branch November 30, 2024 12:55
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