Re: [PATCH v2 1/5] x86/setup: Clean up cmdline handling in create_dom0()

2023-11-22 Thread Andrew Cooper
On 22/11/2023 4:20 pm, Andrew Cooper wrote: > On 22/11/2023 9:02 am, Jan Beulich wrote: >> On 21.11.2023 21:15, Andrew Cooper wrote: >>> @@ -913,33 +914,30 @@ static struct domain *__init create_dom0(const >>> module_t *image, >>> panic("Error creating d%uv0\n", domid); >>> >>> /*

Re: [PATCH v2 1/5] x86/setup: Clean up cmdline handling in create_dom0()

2023-11-22 Thread Andrew Cooper
On 22/11/2023 9:02 am, Jan Beulich wrote: > On 21.11.2023 21:15, Andrew Cooper wrote: >> There's a confusing mix of variables; a static dom0_cmdline[], and a cmdline >> pointer which points to image->string before being pointed at the static >> buffer in order to be passed into construct_dom0(). c

Re: [PATCH v2 1/5] x86/setup: Clean up cmdline handling in create_dom0()

2023-11-22 Thread Jan Beulich
On 21.11.2023 21:15, Andrew Cooper wrote: > There's a confusing mix of variables; a static dom0_cmdline[], and a cmdline > pointer which points to image->string before being pointed at the static > buffer in order to be passed into construct_dom0(). cmdline being a mutable > pointer falls over -Ww

[PATCH v2 1/5] x86/setup: Clean up cmdline handling in create_dom0()

2023-11-21 Thread Andrew Cooper
There's a confusing mix of variables; a static dom0_cmdline[], and a cmdline pointer which points to image->string before being pointed at the static buffer in order to be passed into construct_dom0(). cmdline being a mutable pointer falls over -Wwrite-strings builds. Delete the cmdline pointer,