File 0001-m4-Test-avx512-for-x86-only.patch of Package openvswitch

Overview Repositories Revisions Requests Users Attributes Meta

File 0001-m4-Test-avx512-for-x86-only.patch of Package openvswitch (Revision d58b34ae354ca500e9fb303c788be866)

Currently displaying revision d58b34ae354ca500e9fb303c788be866 , Show latest

From edf699ec6404da3612b58aab85d7da12f0dc9733 Mon Sep 17 00:00:00 2001
From: Cheng Li <lic121@chinatelecom.cn>
Date: Fri, 16 Sep 2022 09:56:18 +0000
Subject: [PATCH] m4: Test avx512 for x86 only.

'as' command of arm version may don't support option '--64', this
patch is to move the avx512 test into x86 branch to avoid this.

Fixes: 352b6c7116cd ("dpif-lookup: add avx512 gather implementation.")
Tested-by: Harry van Haaren <harry.van.haaren@intel.com>
Signed-off-by: Cheng Li <lic121@chinatelecom.cn>
Signed-off-by: Ilya Maximets <i.maximets@ovn.org>
---
 m4/openvswitch.m4 | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/m4/openvswitch.m4 b/m4/openvswitch.m4
index 21808483e..09134feca 100644
--- a/m4/openvswitch.m4
+++ b/m4/openvswitch.m4
@@ -436,8 +436,8 @@ AC_DEFUN([OVS_CHECK_BINUTILS_AVX512],
      mkdir -p build-aux
      OBJFILE=build-aux/binutils_avx512_check.o
      GATHER_PARAMS='0x8(,%ymm1,1),%ymm0{%k2}'
-     echo "vpgatherqq $GATHER_PARAMS" | as --64 -o $OBJFILE -
      if ($CC -dumpmachine | grep x86_64) >/dev/null 2>&1; then
+       echo "vpgatherqq $GATHER_PARAMS" | as --64 -o $OBJFILE -
        if (objdump -d  --no-show-raw-insn $OBJFILE | grep -q $GATHER_PARAMS) >/dev/null 2>&1; then
          ovs_cv_binutils_avx512_good=yes
        else
@@ -446,11 +446,11 @@ AC_DEFUN([OVS_CHECK_BINUTILS_AVX512],
          dnl and causing zmm usage with buggy binutils versions.
          CFLAGS="$CFLAGS -mno-avx512f"
        fi
+       rm $OBJFILE
      else
        dnl non x86_64 architectures don't have avx512, so not affected
        ovs_cv_binutils_avx512_good=no
      fi])
-     rm $OBJFILE
    if test "$ovs_cv_binutils_avx512_good" = yes; then
      AC_DEFINE([HAVE_LD_AVX512_GOOD], [1],
                [Define to 1 if binutils correctly supports AVX512.])
-- 
2.37.3

Places

File 0001-m4-Test-avx512-for-x86-only.patch of Package openvswitch (Revision d58b34ae354ca500e9fb303c788be866)

Places