-
Notifications
You must be signed in to change notification settings - Fork 1.2k
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
[collectd 6] gpu_sysman: use base units + other metric handling fixes #4299
Conversation
Reported-By: Florian Foster Signed-off-by: Eero Tamminen <eero.t.tamminen@intel.com>
Assumes overflowing counters to be either at 32-bit or 64-bit, which should cover most common cases, as long as overflow happens at max once during query period. Signed-off-by: Eero Tamminen <eero.t.tamminen@intel.com>
Signed-off-by: Eero Tamminen <eero.t.tamminen@intel.com>
With the new floating point counter type, joules & seconds can be used as metric units instead of microjoules & microseconds. (Frequencies are are still left as Mhz, but for convenience, and compability with Intel XPU Manager.) Signed-off-by: Eero Tamminen <eero.t.tamminen@intel.com>
As "write_prometheus" plugin already adds that unconditionally to names of all monotonic metric types. Signed-off-by: Eero Tamminen <eero.t.tamminen@intel.com>
@mrunge would you have time to review this? (Florian reviewed most of these changes before I split them to their own PR, but he's not been active lately.) |
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 looks sane to me, but I have no way to actually test it.
Sure! Sorry for not following here much closer. |
No problem, I'm not in hurry with these. It's just better to get completed things reviewed before I forget their details. :-) Thanks! |
ChangeLog: gpu_sysman: use base units + other metric handling fixes
This split the fixes from #4267 to their own PR (that should go in first).
Changes are IMHO straightforward and easiest to review from individual commits.
(Because switching to base units means metric names changing,
Feature
label could have been used instead of aFix
one.)