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(TL2CHICoupledL2): support TraceTag #288

Merged
merged 1 commit into from
Nov 14, 2024
Merged

feat(TL2CHICoupledL2): support TraceTag #288

merged 1 commit into from
Nov 14, 2024

Conversation

linjuanZ
Copy link
Member

According to CHI issue E.b spec, a component that receives a packet with a TraceTag bit set in every received packet must preserve and reflect the value back in any response packet or spawned packet generated in response to the received packet. This commit specifies the Request-Response pairs that should follow this rule, including:

  • A snoop response (including DCT response), with or without data, in response to a snoop request.
  • A CompAck from RN in response to CompData, Comp, or RespSepData.
  • A write data in response to DBIDResp, DBIDRespOrd, or CompDBIDResp.

According to CHI issue E.b spec, a component that receives a packet with
a TraceTag bit set in every received packet must preserve and reflect
the value back in any response packet or spawned packet generated in
response to the received packet. This commit specifies the
Request-Response pairs that should follow this rule, including:

* A snoop response (including DCT response), with or without data, in
  response to a snoop request.
* A CompAck from RN in response to CompData, Comp, or RespSepData.
* A write data in response to DBIDResp, DBIDRespOrd, or CompDBIDResp.
@linjuanZ linjuanZ marked this pull request as ready for review November 14, 2024 12:06
@linjuanZ linjuanZ merged commit 73c84cc into master Nov 14, 2024
1 check passed
@linjuanZ linjuanZ deleted the feat-tracetag branch November 14, 2024 12:16
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