Re: [Xen-devel] Backport candidate for Arm

[Julien Grall <julien.grall@xxxxxxx>]

Hi,

On 28/01/2019 22:58, Stefano Stabellini wrote:
> On Mon, 28 Jan 2019, Julien Grall wrote:
> > Hi,
> >
> > On 1/26/19 1:30 AM, Stefano Stabellini wrote:
> > > On Mon, 21 Jan 2019, Julien Grall wrote:
> > > > Hi,
> > > >
> > > > Ping?
> > > >
> > > > Cheers,
> > > >
> > > > On 30/11/2018 17:25, Julien Grall wrote:
> > > > > Hi,
> > > > >
> > > > > Below a list of backport candidate for Arm.
> > > > >
> > > > >
> > > > > For Xen 4.10+ to handle correctly SMC call parameters and result
> > > > >
> > > > > 35fc608612    xen/arm: smccc-1.1: Make return values unsigned long
> > > > > fa7974f743      xen/arm: smccc-1.1: Handle function result as parameters
> > > > >
> > > > > For Xen 4.9+ to avoid Dom0 crash when using less vCPUs than pCPUs on
> > > > > GICv3
> > > > >
> > > > > 703d9d5ec1      xen/arm: vgic-v3: Delay the initialization of the domain
> > > > > information
> > > > > 54ec59f6b0       xen/arm: vgic-v3: Don't create empty re-distributor
> > > > > regions
> > > > >
> > > > > The following patch is required in Xen 4.11 to avoid breaking the new
> > > > > vGIC
> > > > > after applying the 2 previous patches.
> > > > >
> > > > > 62aa9e7f1b    xen/arm: Don't build GICv3 with the new vGIC
> > > For the moment I skipped 54ec59f6b0 and 62aa9e7f1b because 62aa9e7f1b is
> > > not a trivial backport. Everything else is done.
> > Thank you for backporting the patches. I think it was a bit odd to apply patch
> > 703d9d5ec1 without the 2 patches. Anyway, below a replacement patch for
> > 62aa9e7f1b.
> >
> > diff --cc xen/arch/arm/Kconfig
> > index 8174c0c635,581de67b6b..0000000000
> > --- a/xen/arch/arm/Kconfig
> > +++ b/xen/arch/arm/Kconfig
> > @@@ -12,7 -12,6 +12,7 @@@ config ARM_3
> >    config ARM_64
> >          def_bool y
> >          depends on 64BIT
> > -       select HAS_GICV3
> > ++      select HAS_GICV3 if !NEW_VGIC
> >
> >    config ARM
> >          def_bool y
> Thank you, and that is fine for 4.11. I did that. However for 4.9 and
> 4.10 we also need to make significant changes to 54ec59f6b0, or backport
> a lot more. I went with the former approach -- does the backport commit
> below look good to you for 4.10?
I have tested the patch and confirm it works as expected.

Thank you for the rebase!

Cheers,

--
Julien Grall

_______________________________________________
Xen-devel mailing list
Xen-devel@xxxxxxxxxxxxxxxxxxxx
https://lists.xenproject.org/mailman/listinfo/xen-devel