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

2021-07-21 Thread Scott Davis
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 dr

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

2021-07-21 Thread Ian Jackson
Jason Andryuk writes ("Re: [PATCH] tools/xl: Add stubdomain_cmdline option to xl.cfg"): > Yes, they can both be useful. Append is sufficient until you want to > override or remove an option that is already included. Set can be > tedious since you have to copy the exis

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

2021-07-21 Thread Jason Andryuk
On Wed, Jul 21, 2021 at 8:50 AM Ian Jackson wrote: > > Jason Andryuk writes ("Re: [PATCH] tools/xl: Add stubdomain_cmdline option to > xl.cfg"): > > I think this option actually sets the string, so you want "Set > > B as the device-model stubdomain ke

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

2021-07-21 Thread Ian Jackson
Jason Andryuk writes ("Re: [PATCH] tools/xl: Add stubdomain_cmdline option to xl.cfg"): > I think this option actually sets the string, so you want "Set > B as the device-model stubdomain kernel command line." or > something equivalent? > > With a suitable c

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

2021-07-21 Thread Jason Andryuk
On Tue, Jul 20, 2021 at 1:57 PM Scott Davis wrote: > > This adds an option to the xl domain configuration file syntax for specifying > a kernel command line for device-model stubdomains. It is intended for use > with > Linux-based stubdomains. > > Signed-off-by: Scott Davis > --- > docs/man/xl.

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

2021-07-21 Thread Ian Jackson
Julien Grall writes ("Re: [PATCH] tools/xl: Add stubdomain_cmdline option to xl.cfg"): > Everyone has a different perspective. I don't see the problem of asking > the question... Maybe I should have add "OOI" to make clear with wasn't > a complain. Yes, I

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

2021-07-21 Thread Julien Grall
Hi, On 21/07/2021 10:06, Andrew Cooper wrote: On 21/07/2021 09:21, Julien Grall wrote: Hi Scott, On 20/07/2021 18:56, Scott Davis wrote: This adds an option to the xl domain configuration file syntax for specifying a kernel command line for device-model stubdomains. It is intended for use wit

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

2021-07-21 Thread Andrew Cooper
On 21/07/2021 09:21, Julien Grall wrote: > Hi Scott, > > On 20/07/2021 18:56, Scott Davis wrote: >> This adds an option to the xl domain configuration file syntax for >> specifying >> a kernel command line for device-model stubdomains. It is intended >> for use with >> Linux-based stubdomains. > >

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

2021-07-21 Thread Juergen Gross
On 21.07.21 10:21, Julien Grall wrote: Hi Scott, On 20/07/2021 18:56, Scott Davis wrote: This adds an option to the xl domain configuration file syntax for specifying a kernel command line for device-model stubdomains. It is intended for use with Linux-based stubdomains. May I ask why embed

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

2021-07-21 Thread Julien Grall
Hi Scott, On 20/07/2021 18:56, Scott Davis wrote: This adds an option to the xl domain configuration file syntax for specifying a kernel command line for device-model stubdomains. It is intended for use with Linux-based stubdomains. May I ask why embedding the command line in the kernel would

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

2021-07-20 Thread Scott Davis
This adds an option to the xl domain configuration file syntax for specifying a kernel command line for device-model stubdomains. It is intended for use with Linux-based stubdomains. Signed-off-by: Scott Davis --- docs/man/xl.cfg.5.pod.in | 4 tools/golang/xenlight/helpers.gen.g