grub-devel
[Top][All Lists]
Advanced

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

Re: Bug#617538: update-grub (mk-grubconfig) does not support old "xenkop


From: Vladimir 'φ-coder/phcoder' Serbinenko
Subject: Re: Bug#617538: update-grub (mk-grubconfig) does not support old "xenkopt"
Date: Wed, 23 Mar 2011 13:42:56 +0100
User-agent: Mozilla/5.0 (X11; U; Linux x86_64; en-US; rv:1.9.1.16) Gecko/20110303 Iceowl/1.0b1 Icedove/3.0.11

The patch is ok after 1.99.
On 09.03.2011 18:48, Colin Watson wrote:
> On Wed, Mar 09, 2011 at 04:51:05PM +0000, Ian Jackson wrote:
>   
>> The grub 2 config generator does not appear to support an equivalent
>> to the "xenkopt" option from old (grub 1) update-grub.
>>
>> This is used to provide kernel command line arguments that apply only
>> when booting the kernel under Xen.  For example, a pvops kernel which
>> can boot dom0 vs. baremetal, and where a serial console is in use,
>> would normally want
>>    kopt="console=ttyS0,115200,8n1"
>>    xenkopt="console=hvc0"
>> since when booting under Xen the kernel needs to use the Xen console
>> rather than direct access to the serial port.
>>
>> I suggest adding two variables
>>   GRUB_CMDLINE_LINUX_XEN
>>   GRUB_CMDLINE_LINUX_XEN_DEFAULT
>> which when specified /override/ the values of
>>   GRUB_CMDLINE_LINUX
>>   GRUB_CMDLINE_LINUX_DEFAULT
>>     
> Seems reasonable, although I think the naming is confusing; it's not
> necessarily obvious that GRUB_CMDLINE_LINUX_XEN overrides
> GRUB_CMDLINE_LINUX while GRUB_CMDLINE_XEN appends to it.  How about
> GRUB_CMDLINE_LINUX_XEN_REPLACE and
> GRUB_CMDLINE_LINUX_XEN_REPLACE_DEFAULT?  They're a bit long, but ...
>
> (CCing patch to grub-devel.)
>
> 2011-03-09  Colin Watson  <address@hidden>
>
>       * util/grub.d/20_linux_xen.in: Honour GRUB_CMDLINE_LINUX_XEN_REPLACE
>       and GRUB_CMDLINE_LINUX_XEN_REPLACE_DEFAULT, which replace
>       GRUB_CMDLINE_LINUX and GRUB_CMDLINE_LINUX_DEFAULT (complementing the
>       existing options which append).
>       * docs/grub.texi (Simple configuration): Document new options.
>
> === modified file 'docs/grub.texi'
> --- docs/grub.texi    2011-03-09 17:35:16 +0000
> +++ docs/grub.texi    2011-03-09 17:46:20 +0000
> @@ -1114,8 +1114,14 @@ NetBSD.
>  
>  @item GRUB_CMDLINE_XEN
>  @itemx GRUB_CMDLINE_XEN_DEFAULT
> -As @samp{GRUB_CMDLINE_LINUX} and @samp{GRUB_CMDLINE_LINUX_DEFAULT}, but for
> -Linux and Xen.
> +The values of these options are appended to the values of
> address@hidden and @samp{GRUB_CMDLINE_LINUX_DEFAULT} for Linux
> +and Xen menu entries.
> +
> address@hidden GRUB_CMDLINE_LINUX_XEN_REPLACE
> address@hidden GRUB_CMDLINE_LINUX_XEN_REPLACE_DEFAULT
> +The values of these options replace the values of @samp{GRUB_CMDLINE_LINUX}
> +and @samp{GRUB_CMDLINE_LINUX_DEFAULT} for Linux and Xen menu entries.
>  
>  @item GRUB_DISABLE_LINUX_UUID
>  Normally, @command{grub-mkconfig} will generate menu entries that use
>
> === modified file 'util/grub.d/20_linux_xen.in'
> --- util/grub.d/20_linux_xen.in       2011-02-11 13:00:54 +0000
> +++ util/grub.d/20_linux_xen.in       2011-03-09 17:44:27 +0000
> @@ -51,6 +51,14 @@ else
>    LINUX_ROOT_DEVICE=UUID=${GRUB_DEVICE_UUID}
>  fi
>  
> +# Allow overriding GRUB_CMDLINE_LINUX and GRUB_CMDLINE_LINUX_DEFAULT.
> +if [ "${GRUB_CMDLINE_LINUX_XEN_REPLACE}" ]; then
> +  GRUB_CMDLINE_LINUX="${GRUB_CMDLINE_LINUX_XEN_REPLACE}"
> +fi
> +if [ "${GRUB_CMDLINE_LINUX_XEN_REPLACE_DEFAULT}" ]; then
> +  GRUB_CMDLINE_LINUX_DEFAULT="${GRUB_CMDLINE_LINUX_XEN_REPLACE_DEFAULT}"
> +fi
> +
>  linux_entry ()
>  {
>    os="$1"
>
>   


-- 
Regards
Vladimir 'φ-coder/phcoder' Serbinenko


Attachment: signature.asc
Description: OpenPGP digital signature


reply via email to

[Prev in Thread] Current Thread [Next in Thread]