-
Notifications
You must be signed in to change notification settings - Fork 51
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
document sysctl settings / kernel parameters using KSPP=yes / KSPP=no #256
Comments
Yes this is a good idea. Not sure what would be the best and most clear presentation style. As a random example where to insert the KSPP recommendation notice:
Top:
Bottom:
Or some other alternative? |
I guess below is more user friendly? Maybe later we'll add more (machine-readable) or other metadata. |
Ok sounds good.
For example, for Current:
Proposal 1:
Proposal 2:
Which one would be better? I would personally go with the second.
For example, for IOMMU, KSPP currently has:
We have:
Should we still treat our settings as "KSPP=yes"? My opinion is that we probably should say they are similar enough. However, a valid argument can be made saying that kernel compilation with the CONFIGs sets a far higher standard and so we should not consider our approach equivalent. |
Yes, until we hear a strong argument for the contrary.
Right. That we can address by having a good definition what |
Is this complete? |
Yes it should be good for now. Just for the record there are 4 areas where we do not adhere to the KSPP. Partial compliance:
Disable
Force immediate reboot upon a kernel panic #264 #268. Can enable but may cause system crashes.
Disable user namespaces entirely #263. Unadvisable due to numerous potential breakages. Non-compliance:
Disable registering interpreters for miscellaneous binary formats #249 #267. Currently unadvisable due to breakage with Firefox. |
Rewritten your comment a bit and added to readme. |
Would be useful if each setting was documented.
Meaning, as recommended by KSPP.
Not mentioned by KSPP.
The text was updated successfully, but these errors were encountered: