[PATCH v17 00/25] remove use of VLAs for Windows
Andre Muezerie
andremue at linux.microsoft.com
Thu Jan 30 15:32:16 CET 2025
As per guidance technical board meeting 2024/04/17. This series
removes the use of VLAs from code built for Windows for all 3
toolchains. If there are additional opportunities to convert VLAs
to regular C arrays please provide the details for incorporation
into the series.
MSVC does not support VLAs, replace VLAs with standard C arrays
or alloca(). alloca() is available for all toolchain/platform
combinations officially supported by DPDK.
v17:
* define no_vla_cflag in a top directory (config)
* add no_vla_cflag to directories that are not VLA-free
* add -Wvla project-wide (VLAs not allowed by default, except for
directories using no_vla_cflag)
v16:
* remove -Wvla from drivers/common/mlx5/meson.build and
drivers/common/qat/meson.build
v15:
* inverted some of the logic added during v14:
add -Wvla to meson build files in app and lib directories, adding
-Wno-vla to the few subdirectories which are not yet VLA free
v14:
* add -Wvla to meson build for directories that are VLA free
under app, lib, drivers. This is to ensure that new VLAs are
not added to these directories in the future.
v13:
* increase stack allocated buffer size in
ipv4_reassembly_interleaved_flows_perf and
ipv6_reassembly_interleaved_flows_perf to avoid
STATUS_STACK_BUFFER_OVERRUN on Windows using MSVC
v12:
* update commit message for patch 06/21 to avoid warning
v11:
* add include stdlib.h for alloca() declaration on FreeBSD
* zero-initialize array without code loop
* increase maximum tuple length
v10:
* add ifdef to scope gcc's diagnostic error down to gcc only
v9:
* fix sender's email address
* fix gcc's diagnostic error string to make clang happy
v8:
* rebase
* reduce scope for disabling error "-Warray-bounds=" to only
the place that needs it
* remove parentesis around numbers from defines in test_bitset.c
v7:
* remove use of VLA from new file which sneaked in during review
v6:
* remove use of VLA from new test code added recently
* fix title for patch 08/20
v5:
* add patches for net/ice, net/ixgbe and gro
from Konstantin Ananyev from
https://patchwork.dpdk.org/project/dpdk/list/?series=31972&archive=both&state=*
* address debug_autotest ASan failure
* address array-bound error in bitset_autotest with gcc-13
v4:
* rebase and adapt for changes made in main since v3 was sent
* use fixed maximum array size instead of VLA when doable
v3:
* address checkpatch/check git log warnings (minor typos)
v2:
* replace patches for ethdev, hash, rcu and include new
patches for eal from Konstantin Ananyev
from https://patchwork.dpdk.org/project/dpdk/list/?series=31781
Andre Muezerie (8):
test: remove use of VLAs for Windows built code in bitset tests
app/testpmd: remove use of VLAs for Windows built code in
shared_rxq_fwd
hash: remove use of VLAs by using standard arrays
config: define no_vla_cflag
app: add no_vla_cflag to directories that are not VLA-free
drivers: add no_vla_cflag to directories that are not VLA-free
lib: add no_vla_cflag to directories that are not VLA-free
config: add -Wvla project-wide
Konstantin Ananyev (10):
eal/linux: remove use of VLAs
eal/common: remove use of VLAs
ethdev: remove use of VLAs for Windows built code
hash: remove use of VLAs for Windows built code
hash/thash: remove use of VLAs for Windows built
rcu: remove use of VLAs for Windows built code
gro: fix overwrite unprocessed packets
gro: remove use of VLAs
net/ixgbe: remove use of VLAs
net/ice: remove use of VLAs
Tyler Retzlaff (7):
eal: include header required for alloca
app/testpmd: remove use of VLAs for Windows built
test: remove use of VLAs for Windows built code
common/idpf: remove use of VLAs for Windows built code
net/i40e: remove use of VLAs for Windows built code
common/mlx5: remove use of VLAs for Windows built code
net/mlx5: remove use of VLAs for Windows built code
app/pdump/meson.build | 2 +
app/proc-info/meson.build | 2 +
app/test-acl/meson.build | 2 +
app/test-bbdev/meson.build | 2 +
app/test-crypto-perf/meson.build | 2 +
app/test-dma-perf/meson.build | 2 +
app/test-eventdev/meson.build | 2 +
app/test-flow-perf/meson.build | 2 +
app/test-pmd/cmdline.c | 2 +-
app/test-pmd/cmdline_flow.c | 15 +-
app/test-pmd/config.c | 16 +-
app/test-pmd/meson.build | 10 +-
app/test-pmd/shared_rxq_fwd.c | 2 +-
app/test-sad/meson.build | 2 +
app/test/meson.build | 17 +-
app/test/test.c | 2 +-
app/test/test_bitset.c | 69 ++++---
app/test/test_cmdline_string.c | 2 +-
app/test/test_cryptodev.c | 34 ++--
app/test/test_cryptodev_blockcipher.c | 4 +-
app/test/test_cryptodev_crosscheck.c | 2 +-
app/test/test_dmadev.c | 9 +-
app/test/test_hash.c | 14 +-
app/test/test_lcore_var_perf.c | 2 +-
app/test/test_mempool.c | 25 +--
app/test/test_reassembly_perf.c | 22 ++-
app/test/test_reorder.c | 48 ++---
app/test/test_service_cores.c | 9 +-
app/test/test_soring_stress_impl.h | 13 +-
app/test/test_thash.c | 7 +-
config/meson.build | 9 +
drivers/common/cnxk/meson.build | 2 +
drivers/common/idpf/idpf_common_rxtx.c | 2 +-
drivers/common/idpf/idpf_common_rxtx_avx512.c | 6 +-
drivers/common/mlx5/mlx5_common.h | 4 +-
drivers/common/mlx5/mlx5_devx_cmds.c | 7 +-
drivers/crypto/cnxk/meson.build | 2 +
drivers/crypto/octeontx/meson.build | 2 +
drivers/crypto/scheduler/meson.build | 2 +
drivers/dma/dpaa/meson.build | 2 +
drivers/event/cnxk/meson.build | 2 +
drivers/event/dsw/meson.build | 1 +
drivers/event/opdl/meson.build | 1 +
drivers/event/sw/meson.build | 1 +
drivers/net/af_xdp/meson.build | 2 +
drivers/net/bnxt/meson.build | 2 +
drivers/net/bonding/meson.build | 2 +
drivers/net/cnxk/meson.build | 2 +
drivers/net/cxgbe/meson.build | 2 +
drivers/net/dpaa/meson.build | 2 +
drivers/net/dpaa2/meson.build | 2 +
drivers/net/failsafe/meson.build | 1 +
drivers/net/gve/meson.build | 2 +
drivers/net/hns3/meson.build | 2 +
drivers/net/intel/i40e/i40e_testpmd.c | 5 +-
drivers/net/intel/ice/ice_rxtx.c | 18 +-
drivers/net/intel/ice/ice_rxtx.h | 2 +
drivers/net/intel/ixgbe/ixgbe_ethdev.c | 21 +-
drivers/net/mlx5/mlx5.c | 5 +-
drivers/net/mlx5/mlx5_flow.c | 6 +-
drivers/net/netvsc/meson.build | 2 +
drivers/net/nfp/meson.build | 2 +
drivers/net/ngbe/base/meson.build | 1 +
drivers/net/ntnic/meson.build | 2 +
drivers/net/octeontx/meson.build | 2 +
drivers/net/sfc/meson.build | 1 +
drivers/net/softnic/meson.build | 1 +
drivers/net/tap/meson.build | 1 +
drivers/net/txgbe/meson.build | 2 +
drivers/net/vdev_netvsc/meson.build | 2 +
drivers/net/virtio/meson.build | 2 +
lib/acl/meson.build | 2 +
lib/bpf/meson.build | 2 +
lib/dispatcher/meson.build | 2 +
lib/eal/common/eal_common_proc.c | 5 +-
lib/eal/freebsd/include/rte_os.h | 1 +
lib/eal/linux/eal_interrupts.c | 32 ++-
lib/eal/linux/include/rte_os.h | 1 +
lib/eal/windows/include/rte_os.h | 1 +
lib/ethdev/rte_ethdev.c | 183 +++++++++++-------
lib/eventdev/meson.build | 2 +
lib/gro/rte_gro.c | 42 ++--
lib/hash/rte_cuckoo_hash.c | 4 +-
lib/hash/rte_thash.c | 2 +-
lib/hash/rte_thash.h | 7 +
lib/hash/rte_thash_gf2_poly_math.c | 9 +-
lib/ipsec/meson.build | 2 +
lib/member/meson.build | 2 +
lib/metrics/meson.build | 2 +
lib/pdcp/meson.build | 2 +
lib/pdump/meson.build | 2 +
lib/pipeline/meson.build | 2 +
lib/power/meson.build | 3 +
lib/rcu/rte_rcu_qsbr.c | 7 +-
lib/rcu/rte_rcu_qsbr.h | 5 +
lib/table/meson.build | 2 +
lib/vhost/meson.build | 15 +-
97 files changed, 531 insertions(+), 287 deletions(-)
--
2.47.2.vfs.0.1
More information about the dev
mailing list