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

Re: [PATCH-4.16 v2] xen/efi: Fix Grub2 boot on arm64



Hi,

On 16/11/2021 15: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@xxxxxxxxxx> 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.
(I have a vague memory of some conversation about this on irc but
nothing about this seems to have made it into email.)

AIUI from the thread, most of this discussion is about a followup
patch.  I don't understand the nature of the problem the followup
patch fixes, or the risk of the followup patch.

Does the current state of staging have a regression or serious bug ?
Who is affected by this bug and what are the consequences ?

I think there was some confusion from my side, I thought the original path was not committed and therefore we would respin it and fold the fix discussed. Apologies for that.

Given the original patch is merged, my view on the new patch is different. This is improving performance during boot. While it would be a nice patch to merge, I am less inclined to get it merged in 4.16.

That said, the 4.16 tree will be re-opened for backports soon after 4.16.0 is released. In the past, we backported performance improvement patch and we could consider to do the same in this case as well.

Cheers,

--
Julien Grall



 


Rackspace

Lists.xenproject.org is hosted with RackSpace, monitoring our
servers 24x7x365 and backed by RackSpace's Fanatical Support®.