Re: [PATCH v3 3/3] cmdline: parse multiple instances of the vga option

2023-07-04 Thread Roger Pau Monné
On Wed, Jun 07, 2023 at 12:07:54PM +0200, Jan Beulich wrote: > On 01.06.2023 15:05, Roger Pau Monne wrote: > > --- a/docs/misc/xen-command-line.pandoc > > +++ b/docs/misc/xen-command-line.pandoc > > @@ -2628,6 +2628,9 @@ with the specified width, height and depth. > > `ask` option. (N.B menu mode

Re: [PATCH v3 3/3] cmdline: parse multiple instances of the vga option

2023-06-07 Thread Andrew Cooper
On 07/06/2023 11:07 am, Jan Beulich wrote: > On 01.06.2023 15:05, Roger Pau Monne wrote: >> --- a/docs/misc/xen-command-line.pandoc >> +++ b/docs/misc/xen-command-line.pandoc >> @@ -2628,6 +2628,9 @@ with the specified width, height and depth. >> `ask` option. (N.B menu modes are displayed in hex

Re: [PATCH v3 3/3] cmdline: parse multiple instances of the vga option

2023-06-07 Thread Jan Beulich
On 01.06.2023 15:05, Roger Pau Monne wrote: > --- a/docs/misc/xen-command-line.pandoc > +++ b/docs/misc/xen-command-line.pandoc > @@ -2628,6 +2628,9 @@ with the specified width, height and depth. > `ask` option. (N.B menu modes are displayed in hex, so `` > should be a hexadecimal number) > >

[PATCH v3 3/3] cmdline: parse multiple instances of the vga option

2023-06-01 Thread Roger Pau Monne
Parse all instances of the vga= option on the command line, in order to always enforce the last selection on the command line. Note that it's not safe to parse just the last occurrence of the vga= option, as then a command line with `vga=current vga=keep` would result in current being ignored. Ad