No instructions from the AVX512VL ISA are used. Compilation for the AVX512F ISA is already enabled in lib/automake.mk for the dpif-netdev-lookup-avx512-gather.c file because it's part of the libopenvswitchavx512.la library. It doesn't need to be enabled at a function level.
Remove both of these unnecessary function-level compiler target attributes. Signed-off-by: Cian Ferriter <cian.ferri...@intel.com> --- v4: * Remove the 'avx512f' target since it's also unnecessary. * Sunil acked the v3 version of this commit, but since it's changed, I'm not carrying the ack over. --- lib/dpif-netdev-extract-avx512.c | 4 ---- 1 file changed, 4 deletions(-) diff --git a/lib/dpif-netdev-extract-avx512.c b/lib/dpif-netdev-extract-avx512.c index a0fedb137..2815bf480 100644 --- a/lib/dpif-netdev-extract-avx512.c +++ b/lib/dpif-netdev-extract-avx512.c @@ -628,8 +628,6 @@ mfex_avx512_process(struct dp_packet_batch *packets, #define DECLARE_MFEX_FUNC(name, profile) \ uint32_t \ -__attribute__((__target__("avx512f"))) \ -__attribute__((__target__("avx512vl"))) \ __attribute__((__target__("avx512vbmi"))) \ mfex_avx512_vbmi_##name(struct dp_packet_batch *packets, \ struct netdev_flow_key *keys, uint32_t keys_size,\ @@ -641,8 +639,6 @@ mfex_avx512_vbmi_##name(struct dp_packet_batch *packets, \ } \ \ uint32_t \ -__attribute__((__target__("avx512f"))) \ -__attribute__((__target__("avx512vl"))) \ mfex_avx512_##name(struct dp_packet_batch *packets, \ struct netdev_flow_key *keys, uint32_t keys_size, \ odp_port_t in_port, struct dp_netdev_pmd_thread \ -- 2.25.1 _______________________________________________ dev mailing list d...@openvswitch.org https://mail.openvswitch.org/mailman/listinfo/ovs-dev