All of lore.kernel.org
 help / color / mirror / Atom feed
* [Buildroot] [PATCH v2, 1/2] package/xvisor: drop ARM32 without virtualization support
@ 2022-01-26 22:51 Fabrice Fontaine
  2022-01-26 22:51 ` [Buildroot] [PATCH v2,2/2] package/xvisor: add riscv support Fabrice Fontaine
  2022-02-01 22:04 ` [Buildroot] [PATCH v2, 1/2] package/xvisor: drop ARM32 without virtualization support Arnout Vandecappelle
  0 siblings, 2 replies; 3+ messages in thread
From: Fabrice Fontaine @ 2022-01-26 22:51 UTC (permalink / raw)
  To: buildroot; +Cc: Eric Le Bihan, Fabrice Fontaine

Support for old 32-bit ARM boards (i.e. ARMv5, ARMv6, ARMv7a) without
virtualization has been dropped since
https://github.com/xvisor/xvisor/commit/9fcd69692484e0f6aa5036c27196f55c797582c5
resulting in the following build failure since bump to version 0.3.1 in
commit c4f8b8968770ecbf6444a5921c6472f126717626:

*** Can't find default configuration "/home/giuliobenetti/autobuild/run/instance-3/output-1/build/xvisor-0.3.1/arch/arm/configs/generic-v6-defconfig"!

*** Can't find default configuration "/home/giuliobenetti/autobuild/run/instance-0/output-1/build/xvisor-0.3.1/arch/arm/configs/generic-v7-defconfig"!

As spotted by Thomas Petazzoni in
https://patchwork.ozlabs.org/project/buildroot/patch/20220125190650.107677-1-fontaine.fabrice@gmail.com,
"ARM32 cores that support the ARMv7ve instruction set are still
supported by Xvisor. This means that Cortex-A7, Cortex-A12, Cortex-A15
and Cortex-A17, which are all ARM32 cores, are still supported"

Fixes:
 - http://autobuild.buildroot.org/results/1211bf6ff10c75815fa3ac320532fab5fe649a2b
 - http://autobuild.buildroot.org/results/2bcbbb270df71d2489b7bc83e56c898c58cc90d2

Signed-off-by: Fabrice Fontaine <fontaine.fabrice@gmail.com>
---
Changes v1 -> v2:
 - Keep ARM32 with virtualization

 package/xvisor/Config.in | 10 ++++------
 1 file changed, 4 insertions(+), 6 deletions(-)

diff --git a/package/xvisor/Config.in b/package/xvisor/Config.in
index eb7475b86d..71628e633e 100644
--- a/package/xvisor/Config.in
+++ b/package/xvisor/Config.in
@@ -1,9 +1,9 @@
 config BR2_PACKAGE_XVISOR_ARCH_SUPPORTS
 	bool
-	default y
+	default y if BR2_aarch64
+	default y if BR2_cortex_a7 || BR2_cortex_a12 || BR2_cortex_a15 || BR2_cortex_a17
+	default y if BR2_x86_64
 	depends on BR2_USE_MMU
-	depends on BR2_arm || BR2_aarch64 || BR2_x86_64
-	depends on !BR2_ARM_CPU_ARMV4 && !BR2_ARM_CPU_ARMV7M
 
 menuconfig BR2_PACKAGE_XVISOR
 	bool "xvisor"
@@ -32,9 +32,7 @@ endchoice
 
 config BR2_PACKAGE_XVISOR_DEFCONFIG
 	string "Defconfig name"
-	default "generic-v5" if BR2_ARM_CPU_ARMV5
-	default "generic-v6" if BR2_ARM_CPU_ARMV6
-	default "generic-v7" if BR2_ARM_CPU_ARMV7A
+	default "generic-v7-ve" if BR2_ARM_CPU_ARMV7A
 	default "generic-v8" if BR2_aarch64
 	default "x86_64_generic" if BR2_x86_64
 	depends on BR2_PACKAGE_XVISOR_USE_DEFCONFIG
-- 
2.34.1

_______________________________________________
buildroot mailing list
buildroot@buildroot.org
https://lists.buildroot.org/mailman/listinfo/buildroot

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

* [Buildroot] [PATCH v2,2/2] package/xvisor: add riscv support
  2022-01-26 22:51 [Buildroot] [PATCH v2, 1/2] package/xvisor: drop ARM32 without virtualization support Fabrice Fontaine
@ 2022-01-26 22:51 ` Fabrice Fontaine
  2022-02-01 22:04 ` [Buildroot] [PATCH v2, 1/2] package/xvisor: drop ARM32 without virtualization support Arnout Vandecappelle
  1 sibling, 0 replies; 3+ messages in thread
From: Fabrice Fontaine @ 2022-01-26 22:51 UTC (permalink / raw)
  To: buildroot; +Cc: Eric Le Bihan, Alistair Francis, Fabrice Fontaine

riscv is supported since version 0.3.0 and
https://github.com/xvisor/xvisor/commit/d6feda4e80eefd372294b081111ce709d08ee4c0

Signed-off-by: Fabrice Fontaine <fontaine.fabrice@gmail.com>
Reviewed-by: Alistair Francis <alistair.francis@wdc.com>
---
 package/xvisor/Config.in | 3 +++
 package/xvisor/xvisor.mk | 2 ++
 2 files changed, 5 insertions(+)

diff --git a/package/xvisor/Config.in b/package/xvisor/Config.in
index 71628e633e..1eb50c8abe 100644
--- a/package/xvisor/Config.in
+++ b/package/xvisor/Config.in
@@ -2,6 +2,7 @@ config BR2_PACKAGE_XVISOR_ARCH_SUPPORTS
 	bool
 	default y if BR2_aarch64
 	default y if BR2_cortex_a7 || BR2_cortex_a12 || BR2_cortex_a15 || BR2_cortex_a17
+	default y if BR2_riscv
 	default y if BR2_x86_64
 	depends on BR2_USE_MMU
 
@@ -34,6 +35,8 @@ config BR2_PACKAGE_XVISOR_DEFCONFIG
 	string "Defconfig name"
 	default "generic-v7-ve" if BR2_ARM_CPU_ARMV7A
 	default "generic-v8" if BR2_aarch64
+	default "generic-32b" if BR2_RISCV_32
+	default "generic-64b" if BR2_RISCV_64
 	default "x86_64_generic" if BR2_x86_64
 	depends on BR2_PACKAGE_XVISOR_USE_DEFCONFIG
 	help
diff --git a/package/xvisor/xvisor.mk b/package/xvisor/xvisor.mk
index dc49258400..d7bb288ea8 100644
--- a/package/xvisor/xvisor.mk
+++ b/package/xvisor/xvisor.mk
@@ -32,6 +32,8 @@ ifeq ($(BR2_x86_64),y)
 XVISOR_ARCH = x86
 else ifeq ($(BR2_arm)$(BR2_aarch64),y)
 XVISOR_ARCH = arm
+else ifeq ($(BR2_riscv),y)
+XVISOR_ARCH = riscv
 endif
 
 ifeq ($(BR2_PACKAGE_XVISOR)$(BR_BUILDING),yy)
-- 
2.34.1

_______________________________________________
buildroot mailing list
buildroot@buildroot.org
https://lists.buildroot.org/mailman/listinfo/buildroot

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

* Re: [Buildroot] [PATCH v2, 1/2] package/xvisor: drop ARM32 without virtualization support
  2022-01-26 22:51 [Buildroot] [PATCH v2, 1/2] package/xvisor: drop ARM32 without virtualization support Fabrice Fontaine
  2022-01-26 22:51 ` [Buildroot] [PATCH v2,2/2] package/xvisor: add riscv support Fabrice Fontaine
@ 2022-02-01 22:04 ` Arnout Vandecappelle
  1 sibling, 0 replies; 3+ messages in thread
From: Arnout Vandecappelle @ 2022-02-01 22:04 UTC (permalink / raw)
  To: Fabrice Fontaine, buildroot; +Cc: Eric Le Bihan



On 26/01/2022 23:51, Fabrice Fontaine wrote:
> Support for old 32-bit ARM boards (i.e. ARMv5, ARMv6, ARMv7a) without
> virtualization has been dropped since
> https://github.com/xvisor/xvisor/commit/9fcd69692484e0f6aa5036c27196f55c797582c5
> resulting in the following build failure since bump to version 0.3.1 in
> commit c4f8b8968770ecbf6444a5921c6472f126717626:
> 
> *** Can't find default configuration "/home/giuliobenetti/autobuild/run/instance-3/output-1/build/xvisor-0.3.1/arch/arm/configs/generic-v6-defconfig"!
> 
> *** Can't find default configuration "/home/giuliobenetti/autobuild/run/instance-0/output-1/build/xvisor-0.3.1/arch/arm/configs/generic-v7-defconfig"!
> 
> As spotted by Thomas Petazzoni in
> https://patchwork.ozlabs.org/project/buildroot/patch/20220125190650.107677-1-fontaine.fabrice@gmail.com,
> "ARM32 cores that support the ARMv7ve instruction set are still
> supported by Xvisor. This means that Cortex-A7, Cortex-A12, Cortex-A15
> and Cortex-A17, which are all ARM32 cores, are still supported"
> 
> Fixes:
>   - http://autobuild.buildroot.org/results/1211bf6ff10c75815fa3ac320532fab5fe649a2b
>   - http://autobuild.buildroot.org/results/2bcbbb270df71d2489b7bc83e56c898c58cc90d2
> 
> Signed-off-by: Fabrice Fontaine <fontaine.fabrice@gmail.com>

  Both applied ot master, thanks.

  Regards,
  Arnout

> ---
> Changes v1 -> v2:
>   - Keep ARM32 with virtualization
> 
>   package/xvisor/Config.in | 10 ++++------
>   1 file changed, 4 insertions(+), 6 deletions(-)
> 
> diff --git a/package/xvisor/Config.in b/package/xvisor/Config.in
> index eb7475b86d..71628e633e 100644
> --- a/package/xvisor/Config.in
> +++ b/package/xvisor/Config.in
> @@ -1,9 +1,9 @@
>   config BR2_PACKAGE_XVISOR_ARCH_SUPPORTS
>   	bool
> -	default y
> +	default y if BR2_aarch64
> +	default y if BR2_cortex_a7 || BR2_cortex_a12 || BR2_cortex_a15 || BR2_cortex_a17
> +	default y if BR2_x86_64
>   	depends on BR2_USE_MMU
> -	depends on BR2_arm || BR2_aarch64 || BR2_x86_64
> -	depends on !BR2_ARM_CPU_ARMV4 && !BR2_ARM_CPU_ARMV7M
>   
>   menuconfig BR2_PACKAGE_XVISOR
>   	bool "xvisor"
> @@ -32,9 +32,7 @@ endchoice
>   
>   config BR2_PACKAGE_XVISOR_DEFCONFIG
>   	string "Defconfig name"
> -	default "generic-v5" if BR2_ARM_CPU_ARMV5
> -	default "generic-v6" if BR2_ARM_CPU_ARMV6
> -	default "generic-v7" if BR2_ARM_CPU_ARMV7A
> +	default "generic-v7-ve" if BR2_ARM_CPU_ARMV7A
>   	default "generic-v8" if BR2_aarch64
>   	default "x86_64_generic" if BR2_x86_64
>   	depends on BR2_PACKAGE_XVISOR_USE_DEFCONFIG
_______________________________________________
buildroot mailing list
buildroot@buildroot.org
https://lists.buildroot.org/mailman/listinfo/buildroot

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

end of thread, other threads:[~2022-02-01 22:04 UTC | newest]

Thread overview: 3+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2022-01-26 22:51 [Buildroot] [PATCH v2, 1/2] package/xvisor: drop ARM32 without virtualization support Fabrice Fontaine
2022-01-26 22:51 ` [Buildroot] [PATCH v2,2/2] package/xvisor: add riscv support Fabrice Fontaine
2022-02-01 22:04 ` [Buildroot] [PATCH v2, 1/2] package/xvisor: drop ARM32 without virtualization support Arnout Vandecappelle

This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.