xen-devel.lists.xenproject.org archive mirror
 help / color / mirror / Atom feed
* [PATCH] xen: arm: correct XEN_COMPILE_ARCH autodetection for arm64
@ 2013-10-22 16:12 Ian Campbell
  2013-10-23 10:37 ` Julien Grall
  0 siblings, 1 reply; 3+ messages in thread
From: Ian Campbell @ 2013-10-22 16:12 UTC (permalink / raw)
  To: xen-devel; +Cc: julien.grall, tim, Ian Campbell, stefano.stabellini

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

^ permalink raw reply related	[flat|nested] 3+ messages in thread

* Re: [PATCH] xen: arm: correct XEN_COMPILE_ARCH autodetection for arm64
  2013-10-22 16:12 [PATCH] xen: arm: correct XEN_COMPILE_ARCH autodetection for arm64 Ian Campbell
@ 2013-10-23 10:37 ` Julien Grall
  2013-10-24 14:06   ` Ian Campbell
  0 siblings, 1 reply; 3+ messages in thread
From: Julien Grall @ 2013-10-23 10:37 UTC (permalink / raw)
  To: Ian Campbell, xen-devel; +Cc: tim, Ian Campbell, stefano.stabellini



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

^ permalink raw reply	[flat|nested] 3+ messages in thread

* Re: [PATCH] xen: arm: correct XEN_COMPILE_ARCH autodetection for arm64
  2013-10-23 10:37 ` Julien Grall
@ 2013-10-24 14:06   ` Ian Campbell
  0 siblings, 0 replies; 3+ messages in thread
From: Ian Campbell @ 2013-10-24 14:06 UTC (permalink / raw)
  To: Julien Grall; +Cc: stefano.stabellini, tim, xen-devel

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.

^ permalink raw reply	[flat|nested] 3+ messages in thread

end of thread, other threads:[~2013-10-24 14:06 UTC | newest]

Thread overview: 3+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2013-10-22 16:12 [PATCH] xen: arm: correct XEN_COMPILE_ARCH autodetection for arm64 Ian Campbell
2013-10-23 10:37 ` Julien Grall
2013-10-24 14:06   ` Ian Campbell

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).