Re: [Minios-devel] [UNIKRAFT PATCH 6/9] build: Add arm64 architecture config to menuconfig

[Wei Chen <Wei.Chen@xxxxxxx>]

Hi Simon,

> -----Original Message-----
> From: Simon Kuenzer <simon.kuenzer@xxxxxxxxx>
> Sent: 2018年4月5日 5:50
> To: Wei Chen <Wei.Chen@xxxxxxx>; minios-devel@xxxxxxxxxxxxxxxxxxxx
> Cc: Shijie Huang <Shijie.Huang@xxxxxxx>; Kaly Xin <Kaly.Xin@xxxxxxx>; nd
> <nd@xxxxxxx>
> Subject: Re: [UNIKRAFT PATCH 6/9] build: Add arm64 architecture config to
> menuconfig
> 
> On 15.03.2018 04:39, Wei Chen wrote:
> > Add the arm64 entry for menuconfig. As different silicon vendors may
> > have different 64-bit ARMv8 SoCs. If we want to add them to Config,
> > it will be a large list. So we only provide ARM's cortex A53~A75 CPUs
> > for "Processor Optimization"
> >
> > If we use MARCH_ as the prefix for ARM64 CPUs as x86, when we select
> > "generic", the MARCH_GENERIC will conflict with x86's MARCH_GENERIC.
> > So, we use MARCH_ARM64_ for ARM64 as the prefix.
> 
> Hum, the same should happen with ARM32, right?

No, luckily, the ARM32 doesn’t have "generic" options ; )
But, ARM32 has lots of variants like: arm32v6, arm32v7. So it's not easy
to give a "generic" option. But for arm32v7, the most popular arm32 arch,
we can give a "generic-armv7-a" option. So a MARCH_ARM32_V7 can be added
to ARM32.

> And we should even rename the MARCH_* for x86. Can you add these patches
> to your series?
> 

Ok, I will add a patch to rename the MARCH_* for x86.

> >
> > Signed-off-by: Wei Chen <Wei.Chen@xxxxxxx>
> > ---
> >   arch/Arch.uk       |  2 ++
> >   arch/Config.uk     |  5 +++-
> >   arch/arm/Config.uk | 67
> +++++++++++++++++++++++++++++++++++++++++++++++++++++-
> >   3 files changed, 72 insertions(+), 2 deletions(-)
> >
> > diff --git a/arch/Arch.uk b/arch/Arch.uk
> > index 64f418c..edcbdd4 100644
> > --- a/arch/Arch.uk
> > +++ b/arch/Arch.uk
> > @@ -1,6 +1,8 @@
> >   # Selects architecture according to .config
> >   ifeq ($(ARCH_X86_64),y)
> >   UK_ARCH := x86_64
> > +else ifeq ($(ARCH_ARM_64),y)
> > +UK_ARCH := arm64
> >   else ifeq ($(ARCH_ARM_32),y)
> >   UK_ARCH := arm
> >   endif
> > diff --git a/arch/Config.uk b/arch/Config.uk
> > index a040288..92aa6c6 100644
> > --- a/arch/Config.uk
> > +++ b/arch/Config.uk
> > @@ -1,12 +1,15 @@
> >   choice
> >     prompt "Architecture"
> >     default ARCH_ARM_32 if (UK_ARCH = "arm")
> > +   default ARCH_ARM_64 if (UK_ARCH = "arm64")
> >     default ARCH_X86_64
> >     help
> >       Select the target CPU architecture.
> >
> >   config ARCH_X86_64
> >          bool "x86 compatible (64 bits)"
> > +config ARCH_ARM_64
> > +       bool "ARMv8 compatible (64 bits)"
> >   config ARCH_ARM_32
> >          bool "ARMv7 compatible (32 bits)"
> >
> > @@ -15,6 +18,6 @@ endchoice
> >   if (ARCH_X86_64)
> >     source "arch/x86/Config.uk"
> >   endif
> > -if (ARCH_ARM_32)
> > +if (ARCH_ARM_32 || ARCH_ARM_64)
> >     source "arch/arm/Config.uk"
> >   endif
> > diff --git a/arch/arm/Config.uk b/arch/arm/Config.uk
> > index 426c113..932cbd9 100644
> > --- a/arch/arm/Config.uk
> > +++ b/arch/arm/Config.uk
> > @@ -1,18 +1,83 @@
> >   choice
> >     prompt "Processor Optimization"
> > -   default MARCH_CORTEXA7
> > +   default MARCH_CORTEXA7 if ARCH_ARM_32
> > +   default MARCH_ARM64_GENERIC if ARCH_ARM_64
> >     help
> >             Optimize the code for selected target processor
> >
> >   config MARCH_CORTEXA7
> >     bool "Generic Cortex A7"
> > +   depends on ARCH_ARM_32
> >     help
> >             Compile for Cortex-A7 CPUs, no hardware FPU support
> >
> >   config MARCH_A20NEON
> >     bool "Cortex A7: AllWinner A20"
> > +   depends on ARCH_ARM_32
> >     help
> >             Compile for AllWinner A20 (Cortex-A7) CPUs
> >             The NEON FPU is enabled. Please note that NEON is
> >             not fully IEEE 754 compliant.
> > +
> > +config MARCH_ARM64_NATIVE
> > +   bool "Auto-detect host CPU"
> > +   depends on ARCH_ARM_64
> > +   help
> > +           Optimize compilation to host CPU. Please note that this
> > +           option will fail in case of cross-compilation
> > +
> > +config MARCH_ARM64_GENERIC
> > +   bool "Generic ARMv8 CPU"
> > +   depends on ARCH_ARM_64
> > +   help
> > +           Compile for Generic ARMv8 compatible CPUs
> > +
> > +config MARCH_ARM64_CORTEXA53
> > +   bool "Generic ARMv8 Cortex A53"
> > +   depends on ARCH_ARM_64
> > +   help
> > +           Compile for ARMv8 Cortex-A53 CPUs. Support TrustZone, NEON
> > +           advanced SIMD, VFPv4, hardware virtualization, dual issue,
> > +           in-order pipeline
> > +
> > +config MARCH_ARM64_CORTEXA57
> > +   bool "Generic ARMv8 Cortex A57"
> > +   depends on ARCH_ARM_64
> > +   help
> > +           Compile for ARMv8 Cortex-A57 CPUs. Support TrustZone, NEON
> > +           advanced SIMD, VFPv4, hardware virtualization, 3-way
> > +           superscalar, deeply out-of-order pipeline
> > +
> > +config MARCH_ARM64_CORTEXA72
> > +   bool "Generic ARMv8 Cortex A72"
> > +   depends on ARCH_ARM_64
> > +   help
> > +           Compile for ARMv8 Cortex-A72 CPUs. Support TrustZone, NEON
> > +           advanced SIMD, VFPv4, hardware virtualization, 3-way
> > +           superscalar, deeply out-of-order pipeline
> > +
> > +config MARCH_ARM64_CORTEXA73
> > +   bool "Generic ARMv8 Cortex A73"
> > +   depends on ARCH_ARM_64
> > +   help
> > +           Compile for ARMv8 Cortex-A73 CPUs. Support TrustZone, NEON
> > +           advanced SIMD, VFPv4, hardware virtualization, 2-way
> > +           superscalar, deeply out-of-order pipeline
> > +
> > +config MARCH_ARM64_CORTEXA55
> > +   bool "Generic ARMv8.2 Cortex A55"
> > +   depends on ARCH_ARM_64
> > +   help
> > +           Compile for ARMv8.2 Cortex-A55 CPUs. Support TrustZone, NEON
> > +           advanced SIMD, VFPv4, hardware virtualization, triple issue,
> > +           in-order pipeline
> > +
> > +config MARCH_ARM64_CORTEXA75
> > +   bool "Generic ARMv8.2 Cortex A75"
> > +   depends on ARCH_ARM_64
> > +   help
> > +           Compile for ARMv8.2 Cortex-A75 CPUs. Support TrustZone, NEON
> > +           advanced SIMD, VFPv4, hardware virtualization, 3-way
> > +           superscalar, deeply out-of-order pipeline
> > +
> >   endchoice
> >
_______________________________________________
Minios-devel mailing list
Minios-devel@xxxxxxxxxxxxxxxxxxxx
https://lists.xenproject.org/mailman/listinfo/minios-devel