Skip to content

Add standalone CHI HN-F functional node implementation#4

Draft
Copilot wants to merge 3 commits intomainfrom
copilot/add-chi-home-node-implementation
Draft

Add standalone CHI HN-F functional node implementation#4
Copilot wants to merge 3 commits intomainfrom
copilot/add-chi-home-node-implementation

Conversation

Copy link
Copy Markdown

Copilot AI commented Mar 31, 2026

  • Update hn/chi_hn_header.hpp — add includes for chi_joint.hpp and chi_xact_state.hpp
  • Update hn/chi_hn.hpp:
    • Add addr_value_t typedef and std::shared_ptr<Xact::Xaction<config>> xaction field to PendingTxn
    • Add Xact::Global<config> global, rnJoints map (RNFJoint per RN), rnStateMaps map (RNCacheStateMap per RN) to Node
    • Add AddrValue() helper and RNJoint() / RNStateMap() accessor implementations
    • Wire CreateTransactionrnJoint.NextTXREQ + rnStateMap.NextTXREQ
    • Wire IssueSnooprnJoint.NextRXSNP + rnStateMap.NextRXSNP
    • Wire HandleSnpResprnJoint.NextTXRSP + rnStateMap.NextTXRSP
    • Wire HandleCompAckrnJoint.NextTXRSP
    • Wire HandleSnpRespDatarnJoint.NextTXDAT + rnStateMap.NextTXDAT
    • Wire HandleRNWriteDatarnJoint.NextTXDAT + rnStateMap.NextTXDAT
    • Wire SendComprnJoint.NextRXRSP + rnStateMap.NextRXRSP
    • Wire SendCompDBIDResprnJoint.NextRXRSP + rnStateMap.NextRXRSP
    • Wire SendCompDatarnJoint.NextRXDAT + rnStateMap.NextRXDAT
  • Fix pre-existing framework bugs blocking compilation:
    • chi_joint.hpp: Joint::Joint() missing OnDeniedRequest(0) and OnDeniedResponse(0) initializers
    • chi_joint.hpp: JointDenied*Event constructors and member fields updated to use const& to properly handle temporaries (eliminates const_cast)
    • chi_xact_flit.hpp: SAMScope .value should be ->value (pointer type)
    • chi_xactions_base.hpp: GetFirstFrom/GetLastFrom/GetFirstTo/GetLastTo called IsFrom/IsTo on SubsequenceKey — should use subsequence[index].IsFrom/IsTo
    • chi_xactions_impl_AllocatingRead.hpp: missing GetDWTSrcIDSource / GetDWTTgtIDSource implementations
  • Compile-verify and smoke test pass; CodeQL: 0 alerts

Agent-Logs-Url: https://github.com/RISMicroDevices/CHIron/sessions/068d934b-736a-4e64-a68c-8d70c9348448

Co-authored-by: Kumonda221-CrO3 <10897073+Kumonda221-CrO3@users.noreply.github.com>
Copilot AI and others added 2 commits March 31, 2026 13:47
Agent-Logs-Url: https://github.com/RISMicroDevices/CHIron/sessions/3ea67173-c9f9-4ee6-8944-8f2f611d3bc4

Co-authored-by: Kumonda221-CrO3 <10897073+Kumonda221-CrO3@users.noreply.github.com>
…tors

Agent-Logs-Url: https://github.com/RISMicroDevices/CHIron/sessions/3ea67173-c9f9-4ee6-8944-8f2f611d3bc4

Co-authored-by: Kumonda221-CrO3 <10897073+Kumonda221-CrO3@users.noreply.github.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants