checkpoint-restore / criu

Checkpoint/Restore tool
criu.org
Other
2.76k stars 560 forks source link

plugin/amdgpu: Also don't print 'plugin failed' in criu #2343

Closed Snorch closed 5 months ago

Snorch commented 5 months ago

We already don't treat it as error in the plugin itself, but after returning -1 from RESUME_DEVICES_LATE hook we print debug message in criu about failed plugin, let's return 0 instead.

While on it let's replace ret to exit_code.

Fixes: a9cbdad76 ("plugin/amdgpu: Don't print error for "No such process" during resume")

codecov-commenter commented 5 months ago

Codecov Report

All modified and coverable lines are covered by tests :white_check_mark:

Comparison is base (a9cbdad) 70.56% compared to head (d6e67c8) 70.48%.

:exclamation: Current head d6e67c8 differs from pull request most recent head 12108c4. Consider uploading reports for the commit 12108c4 to get more accurate results

Additional details and impacted files ```diff @@ Coverage Diff @@ ## criu-dev #2343 +/- ## ============================================ - Coverage 70.56% 70.48% -0.09% ============================================ Files 132 133 +1 Lines 33619 33452 -167 ============================================ - Hits 23723 23577 -146 + Misses 9896 9875 -21 ```

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.