coreutils: replace /proc/ by /dev/ - #12553
Conversation
|
GNU testsuite comparison: |
|
please explain the why in the commit message |
7da01bd to
91792a8
Compare
This comment was marked as outdated.
This comment was marked as outdated.
Merging this PR will degrade performance by 13.49%
|
| Mode | Benchmark | BASE |
HEAD |
Efficiency | |
|---|---|---|---|---|---|
| ❌ | Simulation | false_consecutive_calls |
294.7 ns | 350.2 ns | -15.86% |
| ❌ | Simulation | mv_directory |
7.3 ms | 8.2 ms | -11.05% |
Tip
Investigate this regression by commenting @codspeedbot fix this regression on this PR, or directly use the CodSpeed MCP with your agent.
Comparing oech3:patch-1 (cfabc07) with main (adba541)2
Footnotes
-
50 benchmarks were skipped, so the baseline results were used instead. If they were deleted from the codebase, click here and archive them to remove them from the performance reports. ↩
-
No successful run was found on
main(1c68c61) during the generation of this report, so adba541 was used instead as the comparison base. There might be some changes unrelated to this pull request in this report. ↩
92360ed to
5cfbf32
Compare
|
selinux is sad :) |
This comment was marked as outdated.
This comment was marked as outdated.
85904e2 to
1ce1d7f
Compare
Head branch was pushed to by a user without write access
|
Binary size comparison: |
|
Test on SELinux seems fixed now. |
48a6a42 to
9ebdd28
Compare
|
@sylvestre ready to merge. (conflicted at cspell after CI passed). |
AT_EXECFN is /dev/fd/N if binary was called by fexecve(3), not /proc/self/fd/N
Closes #12552