Re: [PATCH] tools/xl: Add stubdomain_cmdline option to xl.cfg

[Scott Davis <scott.davis@starlab.io>]

Thanks for the feedback, all.

On 7/21/21, 4:21 AM, Julien Grall wrote:
> May I ask why embedding the command line in the kernel would not be a
> solution? Do you expect it to change from stubdom to stubdom?

Of course. For Crucible, we're using a common kernel and initramfs for 
stubdomains and driver domains. The command line lets us tell each domain 
type how to configure itself on boot via the tool stack.

> Please add a LIBXL_HAVE_... in include/libxl.h. This is used by external
> toolstack (e.g. libvirt) to know whether a given version of libxl
> provide the field.

Ack. It appears there was not an entry added there for the other 
Linux-based "stubdomain_*" fields when they were inserted last year, so I 
will add a single entry in v2 to cover those three (memkb, kernel, and 
ramdisk) plus cmdline, unless there is an objection.

On 7/21/21, 8:42 AM, Jason Andryuk wrote:
> I think this option actually sets the string, so you want "Set
> B<STRING> as the device-model stubdomain kernel command line." or
> something equivalent?

Ack. As you later noted, I copied the "Append" wording from the cmdline 
option on the assumption that xl was leaving room for itself to add other 
items to the command line in the future. Will use "Set" in v2 for clarity, 
though.

Good day,

-Scott Davis