diff mbox series

[v20,27/27] config: add -Wvla project-wide

Message ID	1738686124-16173-28-git-send-email-andremue@linux.microsoft.com (mailing list archive)
State	Superseded
Delegated to:	David Marchand
Headers	From: Andre Muezerie <andremue@linux.microsoft.com> To: dev@dpdk.org Cc: konstantin.ananyev@huawei.com, thomas@monjalon.net, david.marchand@redhat.com, Andre Muezerie <andremue@linux.microsoft.com> Subject: [PATCH v20 27/27] config: add -Wvla project-wide Date: Tue, 4 Feb 2025 08:22:04 -0800 Message-Id: <1738686124-16173-28-git-send-email-andremue@linux.microsoft.com> In-Reply-To: <1738686124-16173-1-git-send-email-andremue@linux.microsoft.com> References: <1713397319-26135-1-git-send-email-roretzla@linux.microsoft.com> <1738686124-16173-1-git-send-email-andremue@linux.microsoft.com> Precedence: list Errors-To: dev-bounces@dpdk.org
Series	remove use of VLAs for Windows \| [v20,00/27] remove use of VLAs for Windows [v20,01/27] eal: include header required for alloca [v20,02/27] eal/linux: remove use of VLAs [v20,03/27] eal/common: remove use of VLAs [v20,04/27] ethdev: remove use of VLAs for Windows built code [v20,05/27] hash: remove use of VLAs for Windows built code [v20,06/27] hash/thash: remove use of VLAs for Windows built [v20,07/27] rcu: remove use of VLAs for Windows built code [v20,08/27] gro: fix overwrite unprocessed packets [v20,09/27] gro: remove use of VLAs [v20,10/27] net/ixgbe: remove use of VLAs [v20,11/27] net/ice: remove use of VLAs [v20,12/27] app/testpmd: remove use of VLAs for Windows built [v20,13/27] test: remove use of VLAs for Windows built code [v20,14/27] common/idpf: remove use of VLAs for Windows built code [v20,15/27] net/i40e: remove use of VLAs for Windows built code [v20,16/27] common/mlx5: remove use of VLAs for Windows built code [v20,17/27] net/mlx5: remove use of VLAs for Windows built code [v20,18/27] test: remove use of VLAs for Windows built code in bitset tests [v20,19/27] app/testpmd: remove use of VLAs for Windows built code in shared_rxq_fwd [v20,20/27] hash: remove use of VLAs by using standard arrays [v20,21/27] config: define no_wvla_cflag [v20,22/27] app: add no_wvla_cflag to directories that are not VLA-free [v20,23/27] drivers: add no_wvla_cflag to directories that are not VLA-free [v20,24/27] lib: add no_wvla_cflag to directories that are not VLA-free [v20,25/27] examples: add no_wvla_cflag to directories that are not VLA-free [v20,26/27] buildtools: add no_wvla_cflag to directories that are not VLA-free [v20,27/27] config: add -Wvla project-wide

Checks

Context	Check	Description
ci/checkpatch	success	coding style OK
ci/loongarch-compilation	success	Compilation OK
ci/loongarch-unit-testing	success	Unit Testing PASS
ci/iol-marvell-Functional	success	Functional Testing PASS
ci/iol-unit-amd64-testing	success	Testing PASS
ci/iol-abi-testing	success	Testing PASS
ci/iol-sample-apps-testing	success	Testing PASS
ci/iol-compile-amd64-testing	success	Testing PASS
ci/Intel-compilation	success	Compilation OK
ci/intel-Testing	success	Testing PASS
ci/github-robot: build	success	github build: passed
ci/intel-Functional	success	Functional PASS
ci/iol-compile-arm64-testing	fail	Testing issues
ci/iol-mellanox-Performance	success	Performance Testing PASS
ci/iol-unit-arm64-testing	success	Testing PASS
ci/iol-broadcom-Performance	success	Performance Testing PASS
ci/iol-intel-Performance	fail	Performance Testing issues
ci/iol-intel-Functional	success	Functional Testing PASS

Commit Message

Andre Muezerie Feb. 4, 2025, 4:22 p.m. UTC

Add -Wvla project-wide so that VLAs are not allowed by default. This
is to avoid new VLAs from being introduced.

Meson files in directories which are not yet VLA-free have already
been updated with -Wno-vla to avoid build breaks.

Signed-off-by: Andre Muezerie <andremue@linux.microsoft.com>
---
 config/meson.build | 1 +
 1 file changed, 1 insertion(+)

diff mbox series

Patch

diff --git a/config/meson.build b/config/meson.build
index a251ce49b1..710206095b 100644
--- a/config/meson.build
+++ b/config/meson.build
@@ -345,6 +345,7 @@  endif
 
 no_wvla_cflag = []
 if cc.has_argument('-Wvla')
+    add_project_arguments('-Wvla', language: 'c')
     if not is_windows
         no_wvla_cflag = '-Wno-vla'
     endif