[Buildroot] [PATCH v5 2/5] qemu: add qemu_user_opts
Yann E. MORIN
yann.morin.1998 at free.fr
Sun Mar 18 19:18:45 UTC 2018
Adam, all,
On 2018-03-18 15:01 -0400, Adam Duskett spake thusly:
> Add a new variable in qemu.mk: QEMU_USER_OPTS. When qemu-$(ARCH) is
> called, some architectures require a -cpu flag to be passed, or else a
> segfault may occure when building introspection files.
> For PowerPC, many of these cpu's need a specific CPU flag to be passed
> via the command line.
>
> Signed-off-by: Adam Duskett <aduskett at gmail.com>
Reviewed-by: "Yann E. MORIN" <yann.morin.1998 at free.fr>
However, see a little comment below...
> ---
> Changes v1 -> v4:
> - Add patch to series.
>
> Changes v4 -> v5:
> - Change HOST_QEMU_CPU_TUNE to QEMU_USER_OPTS. (Yann)
> - Add $(QEMU_USER_OPTS) to end of QEMU_USER (Yann)
>
> package/qemu/qemu.mk | 33 ++++++++++++++++++++++++++++++++-
> 1 file changed, 32 insertions(+), 1 deletion(-)
>
> diff --git a/package/qemu/qemu.mk b/package/qemu/qemu.mk
> index 345ef52668..6a0fcabbce 100644
> --- a/package/qemu/qemu.mk
> +++ b/package/qemu/qemu.mk
> @@ -70,6 +70,37 @@ endif
> ifeq ($(HOST_QEMU_ARCH),sh4aeb)
> HOST_QEMU_ARCH = sh4eb
> endif
> +
> +# Add cpu specific callouts for qemu.
> +# In some cases (e.g. ppc) simply being arch specific (apparently) isn't good
> +# enough and a PACKAGE_ARCH specific -cpu option is needed
What did you meant with "PACKAGE_ARCH"? I guess it's a leftover from the
previous iteration, right?
Just removing it looks like the correct phrasing to me, right?
Regards,
Yann E. MORIN.
> +# For others (e.g. arm) a -cpu option is not necessary, since the
> +# qemu-arm default CPU supports all required architecture levels.
> +ifeq ($(BR2_x86_64),y)
> +QEMU_USER_OPTS = -cpu Nehalem,check=false
> +endif
> +
> +# PowerPC architecture. Currently, the 620, 630, and 970 are not supported.
> +ifeq ($(BR2_powerpc_e5500),y)
> +QEMU_USER_OPTS = -cpu e5500
> +endif
> +
> +ifeq ($(BR2_powerpc_e6500),y)
> +QEMU_USER_OPTS = -cpu e6500
> +endif
> +
> +ifeq ($(BR2_powerpc_power7),y)
> +QEMU_USER_OPTS = -cpu 7400
> +endif
> +
> +ifeq ($(BR2_powerpc64)$(BR2_powerpc_e5500),yy)
> +QEMU_USER_OPTS = -cpu e500mc
> +endif
> +
> +ifeq ($(BR2_powerpc64)$(BR2_powerpc_e6500),yy)
> +QEMU_USER_OPTS = -cpu e500mc
> +endif
> +
> HOST_QEMU_SYS_ARCH ?= $(HOST_QEMU_ARCH)
>
> ifeq ($(BR2_PACKAGE_HOST_QEMU_SYSTEM_MODE),y)
> @@ -147,7 +178,7 @@ endef
> $(eval $(host-generic-package))
>
> # variable used by other packages
> -QEMU_USER = $(HOST_DIR)/bin/qemu-$(HOST_QEMU_ARCH)
> +QEMU_USER = $(HOST_DIR)/bin/qemu-$(HOST_QEMU_ARCH) $(QEMU_USER_OPTS)
>
> #-------------------------------------------------------------
> # Target-qemu
> --
> 2.14.3
>
> _______________________________________________
> buildroot mailing list
> buildroot at busybox.net
> http://lists.busybox.net/mailman/listinfo/buildroot
--
.-----------------.--------------------.------------------.--------------------.
| Yann E. MORIN | Real-Time Embedded | /"\ ASCII RIBBON | Erics' conspiracy: |
| +33 662 376 056 | Software Designer | \ / CAMPAIGN | ___ |
| +33 223 225 172 `------------.-------: X AGAINST | \e/ There is no |
| http://ymorin.is-a-geek.org/ | _/*\_ | / \ HTML MAIL | v conspiracy. |
'------------------------------^-------^------------------^--------------------'
More information about the buildroot
mailing list