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

proc: add amdgpu support #2009

Merged
merged 1 commit into from
Jul 12, 2024
Merged

Conversation

fberat
Copy link
Contributor

@fberat fberat commented Jul 8, 2024

This is a first shot for new AMD GPU process metrics.
This patch collects information retrieved from /proc/<pid>/fdinfo.

The current implementation accumulates data for each process, walking through all file descriptors and looking for drm and amd entries.

A future patch may be considered to separate the data per drm client ID.

@fberat fberat changed the title draft: proc: add amdgpu support proc: add amdgpu support Jul 10, 2024
@fberat
Copy link
Contributor Author

fberat commented Jul 10, 2024

Rebased on main, description amended.

@natoscott
Copy link
Member

Looking good Fred. There's something not-quite-right causing the help text to go missing. I haven't been able to pin it down though (there's some tweaks needed in root_proc but that's not enough it seems). I'll send you a patch of what I have, maybe you can see what I'm missing here.

We'll also probably need some filtering in common QA routines, similar to the way [hot]proc.smaps.* is done.

Add per process GPU metrics for AMD GPUs.
The data is retrieved from `/proc/<pid>/fdinfo`.

The current implementation accumulates data for each process, walking
through all file descriptors and looking for drm and amd entries.

A future patch may be considered to separate the data per drm client ID.

Signed-off-by: Frédéric Bérat <[email protected]>
@natoscott natoscott merged commit 31a881f into performancecopilot:main Jul 12, 2024
24 checks passed
@natoscott
Copy link
Member

@fberat thanks Fred - I've made some QA fixups on top of this (please double check), but there may be others I've missed (please keep an eye on CI results tomorrow).

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.

None yet

2 participants