Re: [PATCH v2] xen: arm: Update arm64 image header

[Stefano Stabellini <sstabellini@kernel.org>]

On Wed, 22 Jun 2016, Julien Grall wrote:
> Hi Konrad,
> 
> On 22/06/16 15:29, Konrad Rzeszutek Wilk wrote:
> > > > Oh, indeed!
> > > > > 
> > > > > 
> > > > > My patch doesn't change anything regarding the fact that if the magics
> > > > > are
> > > > > not valid (due to quite old kernel version) the code does exit with an
> > > > > error.
> > > > 
> > > > /me nods
> > > > > 
> > > > > While the review of v1 of this patch, Julien asked for an error
> > > > > message to
> > > > > be added here. Fine.
> > > > > 
> > > > > But if there is the request to change the behavior regarding which
> > > > > kernels
> > > > > are supported I'd think that this is independent on this patch and
> > > > > should be
> > > > > done in an additional patch.
> > > > 
> > > > Right, and I think it is fine to skip that  - because as you say - it
> > > > already ignores older kernels.
> > > 
> > > This is not right. The current version of Xen is not ignoring older
> > > kernels.
> > > 
> > > The current check allows kernels with only MAGIC_V0 (any Linux older than
> > > 3.12) to be detected by Xen.
> > > 
> > > However, this is impacting only one version because support for Xen on
> > > ARM64
> > > was added in Linux 3.11.
> > 
> > And you ARM maintainers are OK ditching that support?
> 
> The number of board supported by Linux 3.11 on ARM64 is very limited: ARM
> models and X-gene.
> 
> For the latter it was an early support with only the serial and timer
> upstreamed (see [1]).
> 
> So personally I am fine with that. Any opinions?

I agree with that

_______________________________________________
Xen-devel mailing list
Xen-devel@lists.xen.org
http://lists.xen.org/xen-devel