checkpoint-restore / criu

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

irmap: scan user-provided paths in order #2236

Closed osctobe closed 1 year ago

osctobe commented 1 year ago

Make the scan use the order of paths that came from the user.

This assumes that the order actually matters and makes it easier to construct the list as to speed up inotify-using binaries.

codecov-commenter commented 1 year ago

Codecov Report

Patch coverage: 100.00% and project coverage change: -0.08% :warning:

Comparison is base (1db922f) 70.69% compared to head (6d79e1e) 70.62%.

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

Additional details and impacted files ```diff @@ Coverage Diff @@ ## criu-dev #2236 +/- ## ============================================ - Coverage 70.69% 70.62% -0.08% ============================================ Files 133 134 +1 Lines 33319 33220 -99 ============================================ - Hits 23555 23461 -94 + Misses 9764 9759 -5 ``` | [Files Changed](https://app.codecov.io/gh/checkpoint-restore/criu/pull/2236?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=None) | Coverage Δ | | |---|---|---| | [criu/irmap.c](https://app.codecov.io/gh/checkpoint-restore/criu/pull/2236?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=None#diff-Y3JpdS9pcm1hcC5j) | `85.33% <100.00%> (ø)` | | ... and [14 files with indirect coverage changes](https://app.codecov.io/gh/checkpoint-restore/criu/pull/2236/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.