checkpoint-restore / criu

Checkpoint/Restore tool
criu.org
Other
2.86k stars 576 forks source link

amdgpu_plugin: remove duplicated log prefix #2239

Closed rst0git closed 1 year ago

rst0git commented 1 year ago

The log prefix "amdgpu_plugin:" is defined with LOG_PREFIX in amdgpu_plugin.c. However, the prefix is also included in each log message. As a result it appears duplicated in the log messages:

(00.044324) amdgpu_plugin: amdgpu_plugin: devices:1 bos:58 objects:148 priv_data:45696
(00.045376) amdgpu_plugin: amdgpu_plugin: Thread[0x5589] started
(00.167172) amdgpu_plugin: amdgpu_plugin: img_path = amdgpu-kfd-62.img
(00.083739) amdgpu_plugin: amdgpu_plugin : amdgpu_plugin_dump_file() called for fd = 235
codecov-commenter commented 1 year ago

Codecov Report

Patch coverage has no change and project coverage change: -0.02% :warning:

Comparison is base (72494ed) 70.65% compared to head (a45c44d) 70.64%.

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

Additional details and impacted files ```diff @@ Coverage Diff @@ ## criu-dev #2239 +/- ## ============================================ - Coverage 70.65% 70.64% -0.02% ============================================ Files 133 133 Lines 33317 33317 ============================================ - Hits 23540 23536 -4 - Misses 9777 9781 +4 ``` [see 5 files with indirect coverage changes](https://app.codecov.io/gh/checkpoint-restore/criu/pull/2239/indirect-changes?src=pr&el=tree-more&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=None)

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