On Tue, Mar 18, 2025 at 6:35 PM Bruce Richardson <bruce.richard...@intel.com> wrote: > > A number of libs and drivers had special optimized AVX2 and AVX512 code > paths for performance reasons, and these tended to have copy-pasted > logic to build those files. Centralise that logic in the main > drivers/ and lib/ meson.build files to avoid duplication. > > v3: add patch for event/dlb2 AVX512 handling. > add common code for libraries as well as drivers. > v2: add patch 4 to remove use of unnecessary CC_AVX2_SUPPORT flag > > > Bruce Richardson (11): > build: add generalized AVX handling for drivers > net/intel: use common AVX build code > drivers/net: build use common AVX handling > drivers/net: remove AVX2 build-time define > event/dlb2: build using common AVX handling > build: add generalized AVX handling for libs > acl: use common AVX build handling > fib: use common AVX build handling > net: simplify build-time logic for x86 > net: use common AVX512 build code > member: use common AVX512 build support > > drivers/event/dlb2/dlb2_sse.c | 4 ++ > drivers/event/dlb2/meson.build | 16 +------- > drivers/meson.build | 30 ++++++++++++++ > drivers/net/bnxt/bnxt_ethdev.c | 2 - > drivers/net/bnxt/meson.build | 10 +---- > drivers/net/enic/meson.build | 10 +---- > drivers/net/intel/i40e/meson.build | 26 +----------- > drivers/net/intel/iavf/meson.build | 25 +----------- > drivers/net/intel/ice/meson.build | 25 +----------- > drivers/net/intel/idpf/meson.build | 25 +----------- > drivers/net/nfp/meson.build | 10 +---- > drivers/net/octeon_ep/meson.build | 13 +----- > drivers/net/octeon_ep/otx_ep_ethdev.c | 4 -- > drivers/net/virtio/meson.build | 9 +---- > lib/acl/meson.build | 54 ++----------------------- > lib/fib/dir24_8.c | 6 +-- > lib/fib/meson.build | 18 +-------- > lib/fib/trie.c | 6 +-- > lib/member/meson.build | 46 ++++----------------- > lib/meson.build | 34 +++++++++++++++- > lib/net/meson.build | 58 +++------------------------ > lib/net/rte_net_crc.c | 16 ++++---- > 22 files changed, 114 insertions(+), 333 deletions(-)
I just have some nits (and we need a rebase), otherwise this is a nice cleanup. Thanks Bruce. -- David Marchand
