Messages in this thread | | | Date | Tue, 15 Nov 2022 19:58:41 +0100 | From | Borislav Petkov <> | Subject | Re: [PATCH v8 2/5] x86: Add cmdline_prepare() helper |
| |
On Mon, Nov 14, 2022 at 03:28:55PM +0100, Borislav Petkov wrote: > So now it is my turn: I'll do it how I think it should be done and you > can review it.
Ok, here are two patches as a reply to this message.
I was able to test them as much as I can in a VM here but I'd need more details/testing in your configuration with earlyprintk as a builtin cmdline.
cmdline_prepare() has grown a bit hairy in the end but I've tried hard to comment what happens there so that it is clear for the future. The main goal being to concentrate all command line strings processing in that function and not have it spread around the tree. And yes, there are more cleanups possible.
In the compressed stage I'm using the cmdline which is in boot_params as source and destination to basically add only the builtin cmdline.
In kernel proper the boot_command_line comes from generic code and that is a whole another way of crazy in itself when I look at init/main.c
And as previously stated - the goal is to have everything in one place and documented as good as possible so that trying to figure out how command line parsing is done doesn't send you on grepping spree around the tree.
Suggestions how to simplify this even more are always welcome, ofc.
Thx.
-- Regards/Gruss, Boris.
https://people.kernel.org/tglx/notes-about-netiquette
| |