-
-
Notifications
You must be signed in to change notification settings - Fork 1.7k
Add SMACK support to id, mkdir, mkfifo, mknod utilities #9910
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
Conversation
d1808f0 to
070b999
Compare
a6003eb to
f2fb5e4
Compare
CodSpeed Performance ReportMerging this PR will degrade performance by 6.37%Comparing Summary
Performance Changes
Footnotes
|
f2fb5e4 to
05fd505
Compare
|
I'm not sure why the GNU comment is not posting? I'm seeing this in the workflow but it didn't post the comment |
|
i can take some time also, please fix: |
|
As a note about the performance regressions, its from the default locale file growing bigger. This can be improved by duplicating it to the individual utility locales, could go either way, just leaving it in the main locale file for now. |
05fd505 to
a34b2f0
Compare
We should work on this at some point, we should not have such perf regressions for a few strings |
|
GNU testsuite comparison: |
|
Created an issue to track the shared locale files impacting performance: #9932 I think its a combination of gating the locale files based on feature flag and also coming up with mechanisms to reduce the burden that locales have on performance |
|
GNU testsuite comparison: |
|
GNU testsuite comparison: |
|
@ChrisDryden Would you switch to |
|
It seems this is not a reason for CI breakage :| |
|
It seems like they updated the default download page, will update that now |
5462855 to
6e588fe
Compare
|
GNU testsuite comparison: |
|
GNU testsuite comparison: |
This is the third PR based off of the following prototype: #9866
Now the original SMACK implementation for ls is implemented alongside the CI runner for the SMACK GNU tests and now its time to actually implement SMACK support for the different utilities.