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: [intel_rdt] Added support for LLC misses monitoring event #4296
Conversation
e3865d3
to
949c1a3
Compare
949c1a3
to
5e218ea
Compare
b3473d7
to
2bde020
Compare
3d597dc
to
846e064
Compare
846e064
to
8799589
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
After the couple of (now) redundant ifdefs are removed, I think this is good to go!
src/intel_rdt.c
Outdated
#if PQOS_VERSION < 40000 | ||
const enum pqos_mon_event events = group->event; | ||
#endif |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This is unnecessary and confuses code. Just remove it and use group->event
directly in that one place below.
src/intel_rdt.c
Outdated
#if PQOS_VERSION < 40400 | ||
const struct pqos_event_values *values = &group->values; | ||
#endif |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This is unnecessary now. Just use group->values
directly in that one place below.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks! Approved.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks, looks good to me!
Added llc_miss metric
ChangeLog: intel_rdt: Added support for LLC misses monitoring event