Ian Campbell
2013-Oct-22 16:12 UTC
[PATCH] xen: arm: correct XEN_COMPILE_ARCH autodetection for arm64
From: Ian Campbell <ian.campbell@citrix.com>
At least on aarch64 openSUSE running with qemu-user-aarch64 "uname -m"
reports
"aarch64" and not "armv8" so include that in the seddery.
There''s no harm
leaving the existing armv8 rune too so do so.
Signed-off-by: Ian Campbell <ian.campbell@citrix.com>
---
Config.mk | 3 ++-
1 file changed, 2 insertions(+), 1 deletion(-)
diff --git a/Config.mk b/Config.mk
index 1879454..7f3d8c7 100644
--- a/Config.mk
+++ b/Config.mk
@@ -24,7 +24,8 @@ coverage ?= n
XEN_COMPILE_ARCH ?= $(shell uname -m | sed -e s/i.86/x86_32/ \
-e s/i86pc/x86_32/ -e s/amd64/x86_64/ \
- -e s/armv7.*/arm32/ -e s/armv8.*/arm64/)
+ -e s/armv7.*/arm32/ -e s/armv8.*/arm64/ \
+ -e s/aarch64/arm64/)
XEN_TARGET_ARCH ?= $(XEN_COMPILE_ARCH)
XEN_OS ?= $(shell uname -s)
--
1.8.4.rc3
Julien Grall
2013-Oct-23 10:37 UTC
Re: [PATCH] xen: arm: correct XEN_COMPILE_ARCH autodetection for arm64
On 10/22/2013 05:12 PM, Ian Campbell wrote:> From: Ian Campbell <ian.campbell@citrix.com> > > At least on aarch64 openSUSE running with qemu-user-aarch64 "uname -m" reports > "aarch64" and not "armv8" so include that in the seddery. There''s no harm > leaving the existing armv8 rune too so do so. > > Signed-off-by: Ian Campbell <ian.campbell@citrix.com>Acked-by: Julien Grall <julien.grall@linaro.org>> --- > Config.mk | 3 ++- > 1 file changed, 2 insertions(+), 1 deletion(-) > > diff --git a/Config.mk b/Config.mk > index 1879454..7f3d8c7 100644 > --- a/Config.mk > +++ b/Config.mk > @@ -24,7 +24,8 @@ coverage ?= n > > XEN_COMPILE_ARCH ?= $(shell uname -m | sed -e s/i.86/x86_32/ \ > -e s/i86pc/x86_32/ -e s/amd64/x86_64/ \ > - -e s/armv7.*/arm32/ -e s/armv8.*/arm64/) > + -e s/armv7.*/arm32/ -e s/armv8.*/arm64/ \ > + -e s/aarch64/arm64/) > > XEN_TARGET_ARCH ?= $(XEN_COMPILE_ARCH) > XEN_OS ?= $(shell uname -s) >-- Julien Grall
Ian Campbell
2013-Oct-24 14:06 UTC
Re: [PATCH] xen: arm: correct XEN_COMPILE_ARCH autodetection for arm64
On Wed, 2013-10-23 at 11:37 +0100, Julien Grall wrote:> > On 10/22/2013 05:12 PM, Ian Campbell wrote: > > From: Ian Campbell <ian.campbell@citrix.com> > > > > At least on aarch64 openSUSE running with qemu-user-aarch64 "uname -m" reports > > "aarch64" and not "armv8" so include that in the seddery. There''s no harm > > leaving the existing armv8 rune too so do so. > > > > Signed-off-by: Ian Campbell <ian.campbell@citrix.com> > Acked-by: Julien Grall <julien.grall@linaro.org>Applied, thanks.