Skip to content

Added the MemEvent Id to the CacheListenerNotification in the CacheListener from MemHierarchy#2622

Open
EweLo wants to merge 4 commits intosstsimulator:develfrom
EweLo:add-memid-to-cachelistener
Open

Added the MemEvent Id to the CacheListenerNotification in the CacheListener from MemHierarchy#2622
EweLo wants to merge 4 commits intosstsimulator:develfrom
EweLo:add-memid-to-cachelistener

Conversation

@EweLo
Copy link
Contributor

@EweLo EweLo commented Jan 16, 2026

This PR extends the CacheListenerNotification interface to include the ID of the MemEvent that triggered the listener callback, enabling custom CacheListener implementations to trace the corresponding MemEvents.

I implemented this because I'm working on a complex tracer in SST that tracks memory access patterns and I therefore need to be able to correlate CacheListener notifications with specific memory events.

  • Added eventId field and getEventID() getter to CacheListenerNotification
  • Updated all notifyAccess() calls across MemHierarchy components to pass the event ID

@sst-autotester
Copy link
Contributor

Status Flag 'Pre-Test Inspection' - - This Pull Request Requires Inspection... The code must be inspected by a member of the Team before Testing/Merging
NO INSPECTION HAS BEEN PERFORMED ON THIS PULL REQUEST! - This PR must be inspected by setting label 'AT: PRE-TEST INSPECTED'.

@sst-autotester
Copy link
Contributor

Status Flag 'Pre-Test Inspection' - - This Pull Request Requires Inspection... The code must be inspected by a member of the Team before Testing/Merging
NO INSPECTION HAS BEEN PERFORMED ON THIS PULL REQUEST! - This PR must be inspected by setting label 'AT: PRE-TEST INSPECTED'.

@EweLo
Copy link
Contributor Author

EweLo commented Jan 28, 2026

@gvoskuilen would you be willing to review this? :)

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