mbox series

[Linaro-uefi,v8,0/3] arm64, xen: add xen_boot support into grub-mkconfig

Message ID 20170514074347.18787-1-fu.wei@linaro.org
Headers show
Series arm64, xen: add xen_boot support into grub-mkconfig | expand

Message

Fu Wei Fu May 14, 2017, 7:43 a.m. UTC
From: Fu Wei <fu.wei@linaro.org>

This patchset add xen_boot support into grub-mkconfig for
generating xen boot entrances automatically

Also update the docs/grub.texi for new xen_boot commands.

ChangeLog:
v8: http://lists.gnu.org/archive/html/grub-devel/2017-05/
    Delete the redundant "#include <grub/env.h>" in patch 0001.
    Use $grub_file instead of "feature_xen_boot" mechanism to
    determine which xen boot commands we should use.

v7: http://lists.gnu.org/archive/html/grub-devel/2017-05/msg00000.html
    Delete patch 0001(v6) which is wrong.
    Improve the commit message of 0002(v6).
    Use "machine" variable instead of "feature_xen_boot" mechanism to
    determine which xen boot commands we should use.
    Update the introduction of xen_module commands in docs/grub.texi,
    emphasize xen_hypervisor and xen_module are only for AArch64.

v6: http://lists.gnu.org/archive/html/grub-devel/2016-07/msg00034.html
    Fix Coding style of util/grub.d/20_linux_xen.in, use soft tab.

v5: http://lists.gnu.org/archive/html/grub-devel/2016-07/msg00008.html
    Update the introduction of xen_module commands in docs/grub.texi,
    according to the suggestion from Julien Grall

v4: http://lists.gnu.org/archive/html/grub-devel/2016-05/
    according to the XSM loading mechanism of Xen(upstreamed),
    update the introduction of xen_module commands in docs/grub.texi

v3: http://lists.gnu.org/archive/html/grub-devel/2016-02/msg00314.html
    reorder the patches
    update the introduction of xen_module commands in docs/grub.texi

v2: http://lists.gnu.org/archive/html/grub-devel/2016-02/msg00282.html
    add "--nounzip" option support in xen_module
    use "feature_xen_boot" instead of "grub_xen_boot"
    update the introduction of xen boot commands in docs/grub.texi

v1 :first upstream patchset:
    http://lists.gnu.org/archive/html/grub-devel/2016-02/msg00264.html
Fu Wei (3):
  arm64: add "--nounzip" option support in xen_module command
  * util/grub.d/20_linux_xen.in: Add xen_boot command support for
    aarch64
  arm64: update the introduction of xen boot commands in docs/grub.texi

 docs/grub.texi                    | 38 +++++++++++++-------------------------
 grub-core/loader/arm64/xen_boot.c | 16 ++++++++++++++++
 util/grub.d/20_linux_xen.in       | 14 +++++++++++---
 3 files changed, 40 insertions(+), 28 deletions(-)

Comments

Julien Grall May 15, 2017, 1:43 p.m. UTC | #1
Hi Daniel,

On 15/05/17 14:38, Daniel Kiper wrote:
> On Sun, May 14, 2017 at 03:43:44PM +0800, fu.wei@linaro.org wrote:
>> From: Fu Wei <fu.wei@linaro.org>
>>
>> This patchset add xen_boot support into grub-mkconfig for
>> generating xen boot entrances automatically
>>
>> Also update the docs/grub.texi for new xen_boot commands.
>
> LGTM, if there are no objections I will commit it at the end
> of this week or the beginning of next one.

Thank you!

Can you also please commit patch [1] which has been sitting on the grub 
ML for more than a year? This is preventing to boot Xen ARM with GRUB.

Cheers,

[1] https://lists.gnu.org/archive/html/grub-devel/2016-02/msg00205.html
Daniel Kiper May 15, 2017, 1:46 p.m. UTC | #2
Hi Julien,

On Mon, May 15, 2017 at 02:43:28PM +0100, Julien Grall wrote:
> Hi Daniel,
>
> On 15/05/17 14:38, Daniel Kiper wrote:
> >On Sun, May 14, 2017 at 03:43:44PM +0800, fu.wei@linaro.org wrote:
> >>From: Fu Wei <fu.wei@linaro.org>
> >>
> >>This patchset add xen_boot support into grub-mkconfig for
> >>generating xen boot entrances automatically
> >>
> >>Also update the docs/grub.texi for new xen_boot commands.
> >
> >LGTM, if there are no objections I will commit it at the end
> >of this week or the beginning of next one.
>
> Thank you!
>
> Can you also please commit patch [1] which has been sitting on the grub
> ML for more than a year? This is preventing to boot Xen ARM with GRUB.
>
> Cheers,
>
> [1] https://lists.gnu.org/archive/html/grub-devel/2016-02/msg00205.html

Will do with this patch series.

Daniel
Fu Wei Fu May 15, 2017, 10:27 p.m. UTC | #3
Hi Daniel,

On 15 May 2017 at 21:46, Daniel Kiper <dkiper@net-space.pl> wrote:
> Hi Julien,
>
> On Mon, May 15, 2017 at 02:43:28PM +0100, Julien Grall wrote:
>> Hi Daniel,
>>
>> On 15/05/17 14:38, Daniel Kiper wrote:
>> >On Sun, May 14, 2017 at 03:43:44PM +0800, fu.wei@linaro.org wrote:
>> >>From: Fu Wei <fu.wei@linaro.org>
>> >>
>> >>This patchset add xen_boot support into grub-mkconfig for
>> >>generating xen boot entrances automatically
>> >>
>> >>Also update the docs/grub.texi for new xen_boot commands.
>> >
>> >LGTM, if there are no objections I will commit it at the end
>> >of this week or the beginning of next one.
>>
>> Thank you!
>>
>> Can you also please commit patch [1] which has been sitting on the grub
>> ML for more than a year? This is preventing to boot Xen ARM with GRUB.
>>
>> Cheers,
>>
>> [1] https://lists.gnu.org/archive/html/grub-devel/2016-02/msg00205.html
>
> Will do with this patch series.
>

Great thanks ! :-)


> Daniel