Page MenuHomeFreeBSD

PVH: support whitespace cmdline splitting
ClosedPublic

Authored by cperciva on Aug 13 2022, 12:56 AM.
Tags
None
Referenced Files
F61901786: D36190.diff
Sun, May 28, 1:48 AM
Unknown Object (File)
Wed, May 3, 8:59 PM
Unknown Object (File)
Apr 10 2023, 3:31 PM
Unknown Object (File)
Apr 8 2023, 3:44 PM
Unknown Object (File)
Mar 27 2023, 12:33 AM
Unknown Object (File)
Mar 24 2023, 9:54 PM
Unknown Object (File)
Mar 22 2023, 10:51 PM
Unknown Object (File)
Mar 5 2023, 3:47 AM
Subscribers

Details

Summary

For historical reasons, Xen kernel command lines have options
separated by commas. Every other FreeBSD platform uses whitespace;
this is also necessary in PVH in order to support the Firecracker
VMM. Allow options to be separated by any combination of commas
and whitespace.

Diff Detail

Repository
rG FreeBSD src repository
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

Though some things are like "gee=woo foo=bar,baz,bing" will now be misparsed.

This revision is now accepted and ready to land.Aug 13 2022, 3:10 PM
In D36190#821461, @imp wrote:

Though some things are like "gee=woo foo=bar,baz,bing" will now be misparsed.

Yeah, that's an unavoidable consequence of wanting to support Xen's historical use of commas as argument separators. Maybe at some point we should switch to only using commas here if we're booting under Xen, but accepting both commas and whitespace is good enough for now, I think.

This revision was automatically updated to reflect the committed changes.