All of lore.kernel.org
 help / color / mirror / Atom feed
* [kvm-unit-tests PATCH] Make the powerpc tests compilable with GCC 8
@ 2018-06-13 14:17 Thomas Huth
  2018-06-14  7:50 ` Laurent Vivier
  0 siblings, 1 reply; 2+ messages in thread
From: Thomas Huth @ 2018-06-13 14:17 UTC (permalink / raw)
  To: kvm-ppc

The new GCC 8.1 is optimizing the code with vector instructions.
However, we have not turned on the vector extension in our kvm-unit-test
code, so the tests crash when they hit such an instruction. Compile
the unit tests without vector extensions to avoid this problem.

Signed-off-by: Thomas Huth <thuth@redhat.com>
---
 powerpc/Makefile.common | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/powerpc/Makefile.common b/powerpc/Makefile.common
index 63dc166..81c5074 100644
--- a/powerpc/Makefile.common
+++ b/powerpc/Makefile.common
@@ -19,7 +19,7 @@ all: directories $(TEST_DIR)/boot_rom.bin $(tests-all)
 
 CFLAGS += -std=gnu99
 CFLAGS += -ffreestanding
-CFLAGS += -O2
+CFLAGS += -O2 -msoft-float -mabi=no-altivec -mno-altivec
 CFLAGS += -I $(SRCDIR)/lib -I $(SRCDIR)/lib/libfdt -I lib
 CFLAGS += -Wa,-mregnames
 
-- 
1.8.3.1


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

* Re: [kvm-unit-tests PATCH] Make the powerpc tests compilable with GCC 8
  2018-06-13 14:17 [kvm-unit-tests PATCH] Make the powerpc tests compilable with GCC 8 Thomas Huth
@ 2018-06-14  7:50 ` Laurent Vivier
  0 siblings, 0 replies; 2+ messages in thread
From: Laurent Vivier @ 2018-06-14  7:50 UTC (permalink / raw)
  To: kvm-ppc

On 13/06/2018 16:17, Thomas Huth wrote:
> The new GCC 8.1 is optimizing the code with vector instructions.
> However, we have not turned on the vector extension in our kvm-unit-test
> code, so the tests crash when they hit such an instruction. Compile
> the unit tests without vector extensions to avoid this problem.
> 
> Signed-off-by: Thomas Huth <thuth@redhat.com>
> ---
>  powerpc/Makefile.common | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
> 
> diff --git a/powerpc/Makefile.common b/powerpc/Makefile.common
> index 63dc166..81c5074 100644
> --- a/powerpc/Makefile.common
> +++ b/powerpc/Makefile.common
> @@ -19,7 +19,7 @@ all: directories $(TEST_DIR)/boot_rom.bin $(tests-all)
>  
>  CFLAGS += -std=gnu99
>  CFLAGS += -ffreestanding
> -CFLAGS += -O2
> +CFLAGS += -O2 -msoft-float -mabi=no-altivec -mno-altivec
>  CFLAGS += -I $(SRCDIR)/lib -I $(SRCDIR)/lib/libfdt -I lib
>  CFLAGS += -Wa,-mregnames
>  
> 

Reviewed-by: Laurent Vivier <lvivier@redhat.com>


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

end of thread, other threads:[~2018-06-14  7:50 UTC | newest]

Thread overview: 2+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2018-06-13 14:17 [kvm-unit-tests PATCH] Make the powerpc tests compilable with GCC 8 Thomas Huth
2018-06-14  7:50 ` Laurent Vivier

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.