Page MenuHomeFreeBSD

PVH: support whitespace cmdline splitting
ClosedPublic

Authored by cperciva on Aug 13 2022, 12:56 AM.
Tags
None
Referenced Files
F125054567: D36190.id109284.diff
Sat, Aug 2, 10:25 PM
Unknown Object (File)
Thu, Jul 31, 2:08 PM
Unknown Object (File)
Sun, Jul 20, 1:43 AM
Unknown Object (File)
Sun, Jul 13, 5:45 AM
Unknown Object (File)
Sat, Jul 12, 8:01 PM
Unknown Object (File)
Jun 18 2025, 11:20 AM
Unknown Object (File)
Jun 18 2025, 8:03 AM
Unknown Object (File)
Jun 6 2025, 12:13 PM
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.