-
-
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
base: main
Are you sure you want to change the base?
Conversation
d1808f0 to
070b999
Compare
| if state.cflag { | ||
| return if state.selinux_supported { | ||
| // print SElinux context and exit | ||
| #[cfg(all(any(target_os = "linux", target_os = "android"), feature = "selinux"))] |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This is unrelated to the SMACK fixes, but I am unsure as to why we need this linux and android flag here if we are targeting the SELINUX feature
a6003eb to
f2fb5e4
Compare
CodSpeed Performance ReportMerging #9910 will degrade performance by 5.89%Comparing Summary
Benchmarks breakdown
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: |
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.