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

snapshot: collect data related to GPU devices #243

Merged
merged 1 commit into from
May 12, 2021

Conversation

ffromani
Copy link
Collaborator

@ffromani ffromani commented May 3, 2021

make sure the snapshot code collects data related to GPU devices.
Turns out not much was missing, most notably the /sys/class/drm link;
nevertheless, let's make sure to collect this data, so now also the
gpu package can consume snapshots.

We choose to refactor the code from net devices collection more
for clarity -what we collect and what we filter out is a bit more
explicit now- than to avoid duplication.

The PCI collection code is a bit more elaborate, because it needs to
deal with PCI roots and subtrees (subforests?), so it was intentionally
left out for the time being.

Signed-off-by: Francesco Romani [email protected]

@ffromani
Copy link
Collaborator Author

ffromani commented May 3, 2021

split from #235

Copy link
Owner

@jaypipes jaypipes left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Small nits inline, otherwise looks great, thank you @fromanirh!

pkg/snapshot/clonetree.go Outdated Show resolved Hide resolved
pkg/snapshot/clonetree.go Outdated Show resolved Hide resolved
make sure the snapshot code collects data related to GPU devices.
Turns out not much was missing, most notably the `/sys/class/drm` link;
nevertheless, let's make sure to collect this data, so now also the
gpu package can consume snapshots.

We choose to refactor the code from net devices collection more
for clarity -what we collect and what we filter out is a bit more
explicit now- than to avoid duplication.

The PCI collection code is a bit more elaborate, because it needs to
deal with PCI roots and subtrees (subforests?), so it was intentionally
left out for the time being.

Signed-off-by: Francesco Romani <[email protected]>
Copy link
Owner

@jaypipes jaypipes left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

w00t :) thanks @fromanirh!

@jaypipes jaypipes merged commit a7823c0 into jaypipes:master May 12, 2021
@ffromani ffromani deleted the snapshots-gpu branch May 24, 2021 06:52
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