diff mbox series

[v2,2/3] package/qemu: do not support x86_steamroller or x86_core_avx2

Message ID 20200208211512.994361-2-aduskett@gmail.com
State Accepted
Headers show
Series [v2,1/3] package/qemu: Bump to version 4.2.0 | expand

Commit Message

Adam Duskett Feb. 8, 2020, 9:15 p.m. UTC 
From: Adam Duskett <Aduskett@gmail.com>

These CPU's cause segfaults with qemu.

Signed-off-by: Adam Duskett <Aduskett@gmail.com>
---
 package/qemu/Config.in.host | 1 +
 1 file changed, 1 insertion(+)

Comments

Yann E. MORIN Feb. 8, 2020, 9:33 p.m. UTC | #1 
Adam, All,

On 2020-02-08 13:15 -0800, aduskett@gmail.com spake thusly:
> From: Adam Duskett <Aduskett@gmail.com>
> 
> These CPU's cause segfaults with qemu.
> 
> Signed-off-by: Adam Duskett <Aduskett@gmail.com>

If the issue is with the excistign qemu version, then that patch should
be first. If that's qemu-4.2 that brings the failure, the patch should
be squashed into the bump patch.

No need to resend, we can re-order when applying.

Regards,
Yann E. MORIN.

> ---
>  package/qemu/Config.in.host | 1 +
>  1 file changed, 1 insertion(+)
> 
> diff --git a/package/qemu/Config.in.host b/package/qemu/Config.in.host
> index d6b4bf6dd4..ea9281c5fb 100644
> --- a/package/qemu/Config.in.host
> +++ b/package/qemu/Config.in.host
> @@ -20,6 +20,7 @@ config BR2_PACKAGE_HOST_QEMU_ARCH_SUPPORTS
>  	default y if BR2_sparc64
>  	default y if BR2_xtensa
>  	default y if BR2_x86_64
> +	depends on !BR2_x86_steamroller && !BR2_x86_core_avx2
>  	depends on !BR2_powerpc_620 && !BR2_powerpc_630 && !BR2_powerpc_970
>  
>  config BR2_PACKAGE_HOST_QEMU_SYSTEM_ARCH_SUPPORTS
> -- 
> 2.24.1
> 
> _______________________________________________
> buildroot mailing list
> buildroot@busybox.net
> http://lists.busybox.net/mailman/listinfo/buildroot
Yann E. MORIN Feb. 9, 2020, 8:49 a.m. UTC | #2 
Adam, All,

On 2020-02-08 13:15 -0800, aduskett@gmail.com spake thusly:
> From: Adam Duskett <Aduskett@gmail.com>
> 
> These CPU's cause segfaults with qemu.
> 
> Signed-off-by: Adam Duskett <Aduskett@gmail.com>

Applied to master, thanks.

Regards,
Yann E. MORIN.

> ---
>  package/qemu/Config.in.host | 1 +
>  1 file changed, 1 insertion(+)
> 
> diff --git a/package/qemu/Config.in.host b/package/qemu/Config.in.host
> index d6b4bf6dd4..ea9281c5fb 100644
> --- a/package/qemu/Config.in.host
> +++ b/package/qemu/Config.in.host
> @@ -20,6 +20,7 @@ config BR2_PACKAGE_HOST_QEMU_ARCH_SUPPORTS
>  	default y if BR2_sparc64
>  	default y if BR2_xtensa
>  	default y if BR2_x86_64
> +	depends on !BR2_x86_steamroller && !BR2_x86_core_avx2
>  	depends on !BR2_powerpc_620 && !BR2_powerpc_630 && !BR2_powerpc_970
>  
>  config BR2_PACKAGE_HOST_QEMU_SYSTEM_ARCH_SUPPORTS
> -- 
> 2.24.1
> 
> _______________________________________________
> buildroot mailing list
> buildroot@busybox.net
> http://lists.busybox.net/mailman/listinfo/buildroot
diff mbox series

Patch

diff --git a/package/qemu/Config.in.host b/package/qemu/Config.in.host
index d6b4bf6dd4..ea9281c5fb 100644
--- a/package/qemu/Config.in.host
+++ b/package/qemu/Config.in.host
@@ -20,6 +20,7 @@  config BR2_PACKAGE_HOST_QEMU_ARCH_SUPPORTS
 	default y if BR2_sparc64
 	default y if BR2_xtensa
 	default y if BR2_x86_64
+	depends on !BR2_x86_steamroller && !BR2_x86_core_avx2
 	depends on !BR2_powerpc_620 && !BR2_powerpc_630 && !BR2_powerpc_970
 
 config BR2_PACKAGE_HOST_QEMU_SYSTEM_ARCH_SUPPORTS