[Top][All Lists]
[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
signature.asc
Description: OpenPGP digital signature