xen-devel.lists.xenproject.org archive mirror
 help / color / mirror / Atom feed
From: Jan Beulich <jbeulich@suse.com>
To: Ian Jackson <iwj@xenproject.org>
Cc: Stefano Stabellini <sstabellini@kernel.org>,
	Julien Grall <julien@xen.org>,
	Bertrand Marquis <bertrand.marquis@arm.com>,
	wei.chen@arm.com, Volodymyr Babchuk <Volodymyr_Babchuk@epam.com>,
	Xen-devel <xen-devel@lists.xenproject.org>,
	Luca Fancellu <luca.fancellu@arm.com>
Subject: Re: [PATCH-4.16 v2] xen/efi: Fix Grub2 boot on arm64
Date: Tue, 16 Nov 2021 17:11:44 +0100	[thread overview]
Message-ID: <db75423a-bb59-6536-20ce-fc9df89149fe@suse.com> (raw)
In-Reply-To: <24979.51557.253278.46402@mariner.uk.xensource.com>

On 16.11.2021 16:08, Ian Jackson wrote:
> Luca Fancellu writes ("Re: [PATCH-4.16 v2] xen/efi: Fix Grub2 boot on arm64"):
>>> On 15 Nov 2021, at 22:00, Stefano Stabellini <sstabellini@kernel.org> wrote:
>>> On Mon, 15 Nov 2021, Julien Grall wrote:
>>>> That would indeed be better. I'd like this patch to be merged in 4.16. Would
>>>> you be able to send a new version in the next couple of days?
>>>
>>> I'd love that too; adding Ian so that he is aware.
>>
>> Hi, yes I will prepare it and push very soon.
> 
> Can someone explain to me what is going on here in management-level-speak ?
> I have read the thread and, as far as I can tell:
> 
> There was an actual regression with Grub2 on ARM64.  This was fixed by
> 9bc9fff04ba0 "xen/efi: Fix Grub2 boot on arm64" (committed on the 5th
> of Novwmber).
> 
> But there are some objections to parts of that patch, from Jan.  It is
> not clear to me what the status of those objections is.
> 
> Was I wrong to think that Jan had given an R-b ?  Had it been
> withdrawn ?  I apologise if I committed a patch I shouldn't have.

FTAOD that patch was fine to commit afaic. My concerns were addressed,
but further concerns had been left pending.

Jan



  reply	other threads:[~2021-11-16 16:12 UTC|newest]

Thread overview: 31+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2021-11-04 14:12 [PATCH-4.16 v2] xen/efi: Fix Grub2 boot on arm64 Luca Fancellu
2021-11-04 14:33 ` Bertrand Marquis
2021-11-04 14:44   ` Ian Jackson
2021-11-04 16:36 ` Jan Beulich
2021-11-04 20:56   ` Stefano Stabellini
2021-11-04 21:07     ` Luca Fancellu
2021-11-04 21:35       ` Stefano Stabellini
2021-11-04 21:43         ` Luca Fancellu
2021-11-04 21:50           ` Stefano Stabellini
2021-11-05  7:35             ` Jan Beulich
2021-11-05 15:33               ` Stefano Stabellini
2021-11-08  7:25                 ` Jan Beulich
2021-11-09  2:11                   ` Stefano Stabellini
2021-11-09  9:23                     ` Luca Fancellu
2021-11-09 11:01                       ` Jan Beulich
2021-11-09 11:00                     ` Jan Beulich
2021-11-09 21:52                       ` Stefano Stabellini
2021-11-09 22:31                         ` Julien Grall
2021-11-10  7:40                         ` Jan Beulich
2021-11-10 13:05                         ` Luca Fancellu
2021-11-10 13:36                           ` Julien Grall
2021-11-10 14:02                             ` Luca Fancellu
2021-11-15 18:57                               ` Julien Grall
2021-11-15 22:00                                 ` Stefano Stabellini
2021-11-16  8:36                                   ` Luca Fancellu
2021-11-16 15:08                                     ` Ian Jackson
2021-11-16 16:11                                       ` Jan Beulich [this message]
2021-11-16 16:23                                       ` Julien Grall
2021-11-05  7:32           ` Jan Beulich
2021-11-05  7:27     ` Jan Beulich
2021-11-04 20:51 ` Stefano Stabellini

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=db75423a-bb59-6536-20ce-fc9df89149fe@suse.com \
    --to=jbeulich@suse.com \
    --cc=Volodymyr_Babchuk@epam.com \
    --cc=bertrand.marquis@arm.com \
    --cc=iwj@xenproject.org \
    --cc=julien@xen.org \
    --cc=luca.fancellu@arm.com \
    --cc=sstabellini@kernel.org \
    --cc=wei.chen@arm.com \
    --cc=xen-devel@lists.xenproject.org \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).