[Buildroot] [PATCH 2/4 v3] package/kvm-unit-tests: introduce _ARCH_SUPPORTS
Thomas Petazzoni
thomas.petazzoni at free-electrons.com
Mon Jul 10 16:09:49 UTC 2017
Hello,
On Sun, 9 Jul 2017 11:29:59 +0200, Yann E. MORIN wrote:
> diff --git a/package/kvm-unit-tests/Config.in b/package/kvm-unit-tests/Config.in
> index 7eab0c25d6..3db10fc820 100644
> --- a/package/kvm-unit-tests/Config.in
> +++ b/package/kvm-unit-tests/Config.in
> @@ -1,15 +1,19 @@
> +config BR2_PACKAGE_KVM_UNIT_TESTS_ARCH_SUPPORTS
> + bool
> + # On ARM, it uses virtualization extensions
> + default y if BR2_cortex_a7 || BR2_cortex_a12 || \
> + BR2_cortex_a15 || BR2_cortex_a17
> + default y if BR2_i386 || BR2_x86_64
> + default y if BR2_powerpc64 || BR2_powerpc64le
> +
> config BR2_PACKAGE_KVM_UNIT_TESTS
> bool "kvm-unit-tests"
> + depends on BR2_PACKAGE_KVM_UNIT_TESTS_ARCH_SUPPORTS
> # on i386 and x86-64, __builtin_reachable is used, so we need
> # gcc 4.5 at least. on i386, we use the target gcc, while on
> # x86-64 we use the host gcc (see .mk file for details)
> - # On ARM, it uses virtualization extensions
> - depends on BR2_cortex_a7 || BR2_cortex_a12 || \
> - BR2_cortex_a15 || BR2_cortex_a17 || \
> - (BR2_i386 && BR2_TOOLCHAIN_GCC_AT_LEAST_4_5) || \
> - BR2_powerpc64 || \
> - BR2_powerpc64le || \
> - (BR2_x86_64 && BR2_HOST_GCC_AT_LEAST_4_5)
> + depends on BR2_TOOLCHAIN_GCC_AT_LEAST_4_5 if BR2_i386
> + depends on BR2_HOST_GCC_AT_LEAST_4_5 if BR2_x86_64
Didn't know this syntax was valid. We usually do:
depends on BR2_TOOLCHAIN_GCC_AT_LEAST_4_5 || !BR2_i386
but if the "depends on ... if ..." syntax works, I'm happy with it.
> select BR2_HOSTARCH_NEEDS_IA32_COMPILER if BR2_x86_64=y
> help
> kvm-unit-tests is a project as old as KVM. As its name
> @@ -28,3 +32,7 @@ config BR2_PACKAGE_KVM_UNIT_TESTS
> features are submitted with accompanying unit tests.
>
> http://www.linux-kvm.org/page/KVM-unit-tests
> +
> +comment "kvm-unit-tests needs a toolchain w/ gcc >= 4.5"
> + depends on BR2_i386
> + depends on !BR2_TOOLCHAIN_GCC_AT_LEAST_4_5
What about the x86_64 dependency on a host compiler >= 4.5 ?
But those Config.in comments are not really related to the introduction
of the _ARCH_SUPPORTS variable, so perhaps they should be in a separate
patch. If you don't want to respin but agree with what I said, just let
me know, I can fixup when applying.
Thanks!
Thomas
--
Thomas Petazzoni, CTO, Free Electrons
Embedded Linux and Kernel engineering
http://free-electrons.com
More information about the buildroot
mailing list