Misinterpretation of dashes used as separators for init arguments#226
Open
iwereth wants to merge 3 commits intocirosantilli:masterfrom
Open
Misinterpretation of dashes used as separators for init arguments#226iwereth wants to merge 3 commits intocirosantilli:masterfrom
iwereth wants to merge 3 commits intocirosantilli:masterfrom
Conversation
Owner
|
Oh, God, I remember being so confused by this... thanks :-) Unfortunately it will also need a few more fixes in other parts of LKMC as I relied on this buggy behavior in some other places too, I'll try to make a more complete fix later, some points:
|
|
|
||
| === Init environment | ||
|
|
||
| Documented at https://www.kernel.org/doc/html/v4.14/admin-guide/kernel-parameters.html[]: |
Owner
There was a problem hiding this comment.
BTW, never ever use latest on a link. It WILL break one day. I'll fix this later on with my commit.
Author
There was a problem hiding this comment.
Oh alright! I will keep that in mind
ca85405 to
9724633
Compare
This comment was marked as spam.
This comment was marked as spam.
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
I was going through the guides since I am quite a newbie to whole Linux ecosystem (thank you so much!) and I found that there is a misinterpretation of dash separators for kernel boot parameters, which was probably caused by the formatting bug the official kernel documentation had torvalds/linux@62ee81b
I tried to make some fixes to the README.adoc to the main page. Moreover, some of the section of guide may have the content rendered as incorrectly making some conclusion about how parameters are parsed, for example the one where you mention about how
a.bwas parsed