Re: [PATCH v14 03/81] eal/common: remove use of VLAs

2025-01-10 Thread Stephen Hemminger
On Fri, 10 Jan 2025 12:22:22 -0800 Andre Muezerie wrote: > diff --git a/lib/eal/meson.build b/lib/eal/meson.build > index e1d6c4cf17..352db049e9 100644 > --- a/lib/eal/meson.build > +++ b/lib/eal/meson.build > @@ -31,3 +31,11 @@ endif > if is_freebsd > annotate_locks = false > endif > + >

[PATCH v11 16/30] drivers/regex: replace packed attributes

2025-01-10 Thread Andre Muezerie
MSVC struct packing is not compatible with GCC. Replace macro __rte_packed with __rte_packed_begin to push existing pack value and set packing to 1-byte and macro __rte_packed_end to restore the pack value prior to the push. Macro __rte_packed_end is deliberately utilized to trigger a MSVC compile

[PATCH v11 27/30] net: replace packed attributes

2025-01-10 Thread Andre Muezerie
MSVC struct packing is not compatible with GCC. Replace macro __rte_packed with __rte_packed_begin to push existing pack value and set packing to 1-byte and macro __rte_packed_end to restore the pack value prior to the push. Macro __rte_packed_end is deliberately utilized to trigger a MSVC compile

[PATCH v11 23/30] examples/ptpclient: replace packed attributes

2025-01-10 Thread Andre Muezerie
MSVC struct packing is not compatible with GCC. Replace macro __rte_packed with __rte_packed_begin to push existing pack value and set packing to 1-byte and macro __rte_packed_end to restore the pack value prior to the push. Macro __rte_packed_end is deliberately utilized to trigger a MSVC compile

[PATCH v11 24/30] examples/vhost_blk: replace packed attributes

2025-01-10 Thread Andre Muezerie
MSVC struct packing is not compatible with GCC. Replace macro __rte_packed with __rte_packed_begin to push existing pack value and set packing to 1-byte and macro __rte_packed_end to restore the pack value prior to the push. Macro __rte_packed_end is deliberately utilized to trigger a MSVC compile

[PATCH v11 28/30] pipeline: replace packed attributes

2025-01-10 Thread Andre Muezerie
MSVC struct packing is not compatible with GCC. Replace macro __rte_packed with __rte_packed_begin to push existing pack value and set packing to 1-byte and macro __rte_packed_end to restore the pack value prior to the push. Macro __rte_packed_end is deliberately utilized to trigger a MSVC compile

[PATCH v11 30/30] eal/include: deprecate macro __rte_packed

2025-01-10 Thread Andre Muezerie
Macro __rte_packed was marked as deprecated. Signed-off-by: Andre Muezerie --- lib/eal/include/rte_common.h | 13 + 1 file changed, 13 insertions(+) diff --git a/lib/eal/include/rte_common.h b/lib/eal/include/rte_common.h index d72f8590bd..40592f71b1 100644 --- a/lib/eal/include/rte

[PATCH v11 21/30] examples/l3fwd-power: replace packed attributes

2025-01-10 Thread Andre Muezerie
MSVC struct packing is not compatible with GCC. Replace macro __rte_packed with __rte_packed_begin to push existing pack value and set packing to 1-byte and macro __rte_packed_end to restore the pack value prior to the push. Macro __rte_packed_end is deliberately utilized to trigger a MSVC compile

[PATCH v11 25/30] eal: replace packed attributes

2025-01-10 Thread Andre Muezerie
MSVC struct packing is not compatible with GCC. Replace macro __rte_packed with __rte_packed_begin to push existing pack value and set packing to 1-byte and macro __rte_packed_end to restore the pack value prior to the push. Macro __rte_packed_end is deliberately utilized to trigger a MSVC compile

[PATCH v11 29/30] vhost: replace packed attributes

2025-01-10 Thread Andre Muezerie
MSVC struct packing is not compatible with GCC. Replace macro __rte_packed with __rte_packed_begin to push existing pack value and set packing to 1-byte and macro __rte_packed_end to restore the pack value prior to the push. Macro __rte_packed_end is deliberately utilized to trigger a MSVC compile

[PATCH v11 26/30] ipsec: replace packed attributes

2025-01-10 Thread Andre Muezerie
MSVC struct packing is not compatible with GCC. Replace macro __rte_packed with __rte_packed_begin to push existing pack value and set packing to 1-byte and macro __rte_packed_end to restore the pack value prior to the push. Macro __rte_packed_end is deliberately utilized to trigger a MSVC compile

[PATCH v11 22/30] examples/l3fwd: replace packed attributes

2025-01-10 Thread Andre Muezerie
MSVC struct packing is not compatible with GCC. Replace macro __rte_packed with __rte_packed_begin to push existing pack value and set packing to 1-byte and macro __rte_packed_end to restore the pack value prior to the push. Macro __rte_packed_end is deliberately utilized to trigger a MSVC compile

[PATCH v11 19/30] examples/ip-pipeline: remove packed attributes

2025-01-10 Thread Andre Muezerie
Removed packed attributes from structs that are naturally packed already, or don't require packing. Signed-off-by: Andre Muezerie --- examples/ip_pipeline/cli.c | 10 +- 1 file changed, 5 insertions(+), 5 deletions(-) diff --git a/examples/ip_pipeline/cli.c b/examples/ip_pipeline/cli.c

[PATCH v11 20/30] examples/ipsec_secgw: replace packed attributes

2025-01-10 Thread Andre Muezerie
MSVC struct packing is not compatible with GCC. Replace macro __rte_packed with __rte_packed_begin to push existing pack value and set packing to 1-byte and macro __rte_packed_end to restore the pack value prior to the push. Macro __rte_packed_end is deliberately utilized to trigger a MSVC compile

[PATCH v11 17/30] drivers/vdpa: replace packed attributes

2025-01-10 Thread Andre Muezerie
MSVC struct packing is not compatible with GCC. Replace macro __rte_packed with __rte_packed_begin to push existing pack value and set packing to 1-byte and macro __rte_packed_end to restore the pack value prior to the push. Macro __rte_packed_end is deliberately utilized to trigger a MSVC compile

[PATCH v11 18/30] examples/common: replace packed attributes

2025-01-10 Thread Andre Muezerie
MSVC struct packing is not compatible with GCC. Replace macro __rte_packed with __rte_packed_begin to push existing pack value and set packing to 1-byte and macro __rte_packed_end to restore the pack value prior to the push. Macro __rte_packed_end is deliberately utilized to trigger a MSVC compile

[PATCH v11 15/30] drivers/raw: replace packed attributes

2025-01-10 Thread Andre Muezerie
MSVC struct packing is not compatible with GCC. Replace macro __rte_packed with __rte_packed_begin to push existing pack value and set packing to 1-byte and macro __rte_packed_end to restore the pack value prior to the push. Macro __rte_packed_end is deliberately utilized to trigger a MSVC compile

[PATCH v11 13/30] drivers/mempool: replace packed attributes

2025-01-10 Thread Andre Muezerie
MSVC struct packing is not compatible with GCC. Replace macro __rte_packed with __rte_packed_begin to push existing pack value and set packing to 1-byte and macro __rte_packed_end to restore the pack value prior to the push. Macro __rte_packed_end is deliberately utilized to trigger a MSVC compile

[PATCH v11 08/30] drivers/common: replace packed attributes

2025-01-10 Thread Andre Muezerie
MSVC struct packing is not compatible with GCC. Replace macro __rte_packed with __rte_packed_begin to push existing pack value and set packing to 1-byte and macro __rte_packed_end to restore the pack value prior to the push. Macro __rte_packed_end is deliberately utilized to trigger a MSVC compile

[PATCH v11 12/30] drivers/event: replace packed attributes

2025-01-10 Thread Andre Muezerie
MSVC struct packing is not compatible with GCC. Replace macro __rte_packed with __rte_packed_begin to push existing pack value and set packing to 1-byte and macro __rte_packed_end to restore the pack value prior to the push. Macro __rte_packed_end is deliberately utilized to trigger a MSVC compile

[PATCH v11 07/30] drivers/bus: replace packed attributes

2025-01-10 Thread Andre Muezerie
MSVC struct packing is not compatible with GCC. Replace macro __rte_packed with __rte_packed_begin to push existing pack value and set packing to 1-byte and macro __rte_packed_end to restore the pack value prior to the push. Macro __rte_packed_end is deliberately utilized to trigger a MSVC compile

[PATCH v11 11/30] drivers/dma: replace packed attributes

2025-01-10 Thread Andre Muezerie
MSVC struct packing is not compatible with GCC. Replace macro __rte_packed with __rte_packed_begin to push existing pack value and set packing to 1-byte and macro __rte_packed_end to restore the pack value prior to the push. Macro __rte_packed_end is deliberately utilized to trigger a MSVC compile

[PATCH v11 06/30] drivers/baseband: replace packed attributes

2025-01-10 Thread Andre Muezerie
MSVC struct packing is not compatible with GCC. Replace macro __rte_packed with __rte_packed_begin to push existing pack value and set packing to 1-byte and macro __rte_packed_end to restore the pack value prior to the push. Macro __rte_packed_end is deliberately utilized to trigger a MSVC compile

[PATCH v11 05/30] doc/guides: replace packed attributes

2025-01-10 Thread Andre Muezerie
MSVC struct packing is not compatible with GCC. Replace macro __rte_packed with __rte_packed_begin to push existing pack value and set packing to 1-byte and macro __rte_packed_end to restore the pack value prior to the push. Macro __rte_packed_end is deliberately utilized to trigger a MSVC compile

[PATCH v11 10/30] drivers/crypto: replace packed attributes

2025-01-10 Thread Andre Muezerie
MSVC struct packing is not compatible with GCC. Replace macro __rte_packed with __rte_packed_begin to push existing pack value and set packing to 1-byte and macro __rte_packed_end to restore the pack value prior to the push. Macro __rte_packed_end is deliberately utilized to trigger a MSVC compile

[PATCH v11 09/30] drivers/compress: replace packed attributes

2025-01-10 Thread Andre Muezerie
MSVC struct packing is not compatible with GCC. Replace macro __rte_packed with __rte_packed_begin to push existing pack value and set packing to 1-byte and macro __rte_packed_end to restore the pack value prior to the push. Macro __rte_packed_end is deliberately utilized to trigger a MSVC compile

[PATCH v11 00/30] fix packing of structs when building with MSVC

2025-01-10 Thread Andre Muezerie
MSVC struct packing is not compatible with GCC. Different alternatives were considered: 1) Have a macro __RTE_PACKED(decl) to which the struct/union is passed and the macro would define the struct/union with the appropriate packing attribute for the compiler in use. Advantages: * Can be pl

[PATCH v11 04/30] app/test: replace packed attributes

2025-01-10 Thread Andre Muezerie
MSVC struct packing is not compatible with GCC. Replace macro __rte_packed with __rte_packed_begin to push existing pack value and set packing to 1-byte and macro __rte_packed_end to restore the pack value prior to the push. Macro __rte_packed_end is deliberately utilized to trigger a MSVC compile

[PATCH v11 01/30] devtools: check packed attributes

2025-01-10 Thread Andre Muezerie
Ensure __rte_packed_begin and __rte_packed_end show up in pairs when checking patches. Signed-off-by: Andre Muezerie Acked-by: Tyler Retzlaff --- devtools/checkpatches.sh | 43 1 file changed, 43 insertions(+) diff --git a/devtools/checkpatches.sh b/dev

[PATCH v11 02/30] eal/include: add new packing macros

2025-01-10 Thread Andre Muezerie
MSVC struct packing is not compatible with GCC. Add macro __rte_packed_begin which can be used to push existing pack value and set packing to 1-byte. Add macro __rte_packed_end to restore the pack value prior to the push. Macro __rte_packed_end is deliberately utilized to trigger a MSVC compiler w

[PATCH v11 03/30] app/test-pmd: remove unnecessary packed attributes

2025-01-10 Thread Andre Muezerie
Removed __rte_packed attribute from structure that is naturally packed already. Signed-off-by: Andre Muezerie --- app/test-pmd/csumonly.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/app/test-pmd/csumonly.c b/app/test-pmd/csumonly.c index 2246c22e8e..d77a140641 100644 ---

Re: [PATCH v13 00/21] remove use of VLAs for Windows

2025-01-10 Thread Andre Muezerie
On Fri, Jan 10, 2025 at 09:27:01AM +, Konstantin Ananyev wrote: > > > > On Wed, Jan 08, 2025 at 01:30:13PM +, Konstantin Ananyev wrote: > > > > > > > > > > As per guidance technical board meeting 2024/04/17. This series > > > > removes the use of VLAs from code built for Windows for all 3

[PATCH v14 81/81] drivers/vdpa: add compile warning about use of VLAs

2025-01-10 Thread Andre Muezerie
MSVC does not support VLAs, so we want to prevent VLAs from being introduced under this path. Signed-off-by: Andre Muezerie --- drivers/vdpa/meson.build | 8 1 file changed, 8 insertions(+) diff --git a/drivers/vdpa/meson.build b/drivers/vdpa/meson.build index 896e8e0304..49e94584bc 10

[PATCH v14 68/81] lib/stack: add compile warning about use of VLAs

2025-01-10 Thread Andre Muezerie
MSVC does not support VLAs, so we want to prevent VLAs from being introduced under this path. Signed-off-by: Andre Muezerie --- lib/stack/meson.build | 8 1 file changed, 8 insertions(+) diff --git a/lib/stack/meson.build b/lib/stack/meson.build index 7631a14784..e5daf96e0e 100644 ---

[PATCH v14 72/81] drivers/bus: add compile warning about use of VLAs

2025-01-10 Thread Andre Muezerie
MSVC does not support VLAs, so we want to prevent VLAs from being introduced under this path. Signed-off-by: Andre Muezerie --- drivers/bus/meson.build | 8 1 file changed, 8 insertions(+) diff --git a/drivers/bus/meson.build b/drivers/bus/meson.build index d67db8576b..59af0f920e 10064

[PATCH v14 70/81] lib/timer: add compile warning about use of VLAs

2025-01-10 Thread Andre Muezerie
MSVC does not support VLAs, so we want to prevent VLAs from being introduced under this path. Signed-off-by: Andre Muezerie --- lib/timer/meson.build | 8 1 file changed, 8 insertions(+) diff --git a/lib/timer/meson.build b/lib/timer/meson.build index 66ef89ac00..7546fc9838 100644 ---

[PATCH v14 78/81] drivers/power: add compile warning about use of VLAs

2025-01-10 Thread Andre Muezerie
MSVC does not support VLAs, so we want to prevent VLAs from being introduced under this path. Signed-off-by: Andre Muezerie --- drivers/power/meson.build | 8 1 file changed, 8 insertions(+) diff --git a/drivers/power/meson.build b/drivers/power/meson.build index 0a703bce38..e7c16e2d47

[PATCH v14 73/81] drivers/common: add compile warning about use of VLAs

2025-01-10 Thread Andre Muezerie
MSVC does not support VLAs, so we want to prevent VLAs from being introduced under this path. Signed-off-by: Andre Muezerie --- drivers/common/mlx5/meson.build| 1 + drivers/common/nfp/meson.build | 8 drivers/common/nitrox/meson.build | 8 drivers/common/qat/meson.bui

[PATCH v14 80/81] drivers/regex: add compile warning about use of VLAs

2025-01-10 Thread Andre Muezerie
MSVC does not support VLAs, so we want to prevent VLAs from being introduced under this path. Signed-off-by: Andre Muezerie --- drivers/regex/meson.build | 8 1 file changed, 8 insertions(+) diff --git a/drivers/regex/meson.build b/drivers/regex/meson.build index ff2a8fea89..06614f34e2

[PATCH v14 67/81] lib/security: add compile warning about use of VLAs

2025-01-10 Thread Andre Muezerie
MSVC does not support VLAs, so we want to prevent VLAs from being introduced under this path. Signed-off-by: Andre Muezerie --- lib/security/meson.build | 8 1 file changed, 8 insertions(+) diff --git a/lib/security/meson.build b/lib/security/meson.build index 1034a7a299..6d0cefc943 10

[PATCH v14 75/81] drivers/gpu: add compile warning about use of VLAs

2025-01-10 Thread Andre Muezerie
MSVC does not support VLAs, so we want to prevent VLAs from being introduced under this path. Signed-off-by: Andre Muezerie --- drivers/gpu/meson.build | 8 1 file changed, 8 insertions(+) diff --git a/drivers/gpu/meson.build b/drivers/gpu/meson.build index b6edd12678..610151e9b1 10064

[PATCH v14 77/81] drivers/ml: add compile warning about use of VLAs

2025-01-10 Thread Andre Muezerie
MSVC does not support VLAs, so we want to prevent VLAs from being introduced under this path. Signed-off-by: Andre Muezerie --- drivers/ml/meson.build | 8 1 file changed, 8 insertions(+) diff --git a/drivers/ml/meson.build b/drivers/ml/meson.build index 54bc394c47..ae07c5292e 100644 -

[PATCH v14 74/81] drivers/compress: add compile warning about use of VLAs

2025-01-10 Thread Andre Muezerie
MSVC does not support VLAs, so we want to prevent VLAs from being introduced under this path. Signed-off-by: Andre Muezerie --- drivers/compress/meson.build | 8 1 file changed, 8 insertions(+) diff --git a/drivers/compress/meson.build b/drivers/compress/meson.build index 91d7800a4a..2

[PATCH v14 79/81] drivers/raw: add compile warning about use of VLAs

2025-01-10 Thread Andre Muezerie
MSVC does not support VLAs, so we want to prevent VLAs from being introduced under this path. Signed-off-by: Andre Muezerie --- drivers/raw/meson.build | 8 1 file changed, 8 insertions(+) diff --git a/drivers/raw/meson.build b/drivers/raw/meson.build index d173ac6097..726a786e0d 10064

[PATCH v14 65/81] lib/ring: add compile warning about use of VLAs

2025-01-10 Thread Andre Muezerie
MSVC does not support VLAs, so we want to prevent VLAs from being introduced under this path. Signed-off-by: Andre Muezerie --- lib/ring/meson.build | 8 1 file changed, 8 insertions(+) diff --git a/lib/ring/meson.build b/lib/ring/meson.build index 7fca958ed7..c18033c205 100644 --- a/l

[PATCH v14 69/81] lib/telemetry: add compile warning about use of VLAs

2025-01-10 Thread Andre Muezerie
MSVC does not support VLAs, so we want to prevent VLAs from being introduced under this path. Signed-off-by: Andre Muezerie --- lib/telemetry/meson.build | 8 1 file changed, 8 insertions(+) diff --git a/lib/telemetry/meson.build b/lib/telemetry/meson.build index 489d47..188792fbfc

[PATCH v14 71/81] drivers/baseband: add compile warning about use of VLAs

2025-01-10 Thread Andre Muezerie
MSVC does not support VLAs, so we want to prevent VLAs from being introduced under this path. Signed-off-by: Andre Muezerie --- drivers/baseband/meson.build | 8 1 file changed, 8 insertions(+) diff --git a/drivers/baseband/meson.build b/drivers/baseband/meson.build index 3420d98564..6

[PATCH v14 66/81] lib/sched: add compile warning about use of VLAs

2025-01-10 Thread Andre Muezerie
MSVC does not support VLAs, so we want to prevent VLAs from being introduced under this path. Signed-off-by: Andre Muezerie --- lib/sched/meson.build | 8 1 file changed, 8 insertions(+) diff --git a/lib/sched/meson.build b/lib/sched/meson.build index df75db51ed..78524c7e07 100644 ---

[PATCH v14 76/81] drivers/mempool: add compile warning about use of VLAs

2025-01-10 Thread Andre Muezerie
MSVC does not support VLAs, so we want to prevent VLAs from being introduced under this path. Signed-off-by: Andre Muezerie --- drivers/mempool/meson.build | 8 1 file changed, 8 insertions(+) diff --git a/drivers/mempool/meson.build b/drivers/mempool/meson.build index dc88812585..4f74

[PATCH v14 53/81] lib/meter: add compile warning about use of VLAs

2025-01-10 Thread Andre Muezerie
MSVC does not support VLAs, so we want to prevent VLAs from being introduced under this path. Signed-off-by: Andre Muezerie --- lib/meter/meson.build | 8 1 file changed, 8 insertions(+) diff --git a/lib/meter/meson.build b/lib/meter/meson.build index 646fd4d43f..320e09e53f 100644 ---

[PATCH v14 59/81] lib/pci: add compile warning about use of VLAs

2025-01-10 Thread Andre Muezerie
MSVC does not support VLAs, so we want to prevent VLAs from being introduced under this path. Signed-off-by: Andre Muezerie --- lib/pci/meson.build | 8 1 file changed, 8 insertions(+) diff --git a/lib/pci/meson.build b/lib/pci/meson.build index dd41cd5068..88cfd2a238 100644 --- a/lib/

[PATCH v14 46/81] lib/jobstats: add compile warning about use of VLAs

2025-01-10 Thread Andre Muezerie
MSVC does not support VLAs, so we want to prevent VLAs from being introduced under this path. Signed-off-by: Andre Muezerie --- lib/jobstats/meson.build | 8 1 file changed, 8 insertions(+) diff --git a/lib/jobstats/meson.build b/lib/jobstats/meson.build index 6da2a95b75..e43e229247 10

[PATCH v14 54/81] lib/metrics: add compile warning about use of VLAs

2025-01-10 Thread Andre Muezerie
MSVC does not support VLAs, so we want to prevent VLAs from being introduced under this path. Signed-off-by: Andre Muezerie --- lib/metrics/meson.build | 8 1 file changed, 8 insertions(+) diff --git a/lib/metrics/meson.build b/lib/metrics/meson.build index 8c1c4b4b49..86f320f90c 10064

[PATCH v14 50/81] lib/lpm: add compile warning about use of VLAs

2025-01-10 Thread Andre Muezerie
MSVC does not support VLAs, so we want to prevent VLAs from being introduced under this path. Signed-off-by: Andre Muezerie --- lib/lpm/meson.build | 8 1 file changed, 8 insertions(+) diff --git a/lib/lpm/meson.build b/lib/lpm/meson.build index fae4f79fb9..06525b76c1 100644 --- a/lib/

[PATCH v14 56/81] lib/net: add compile warning about use of VLAs

2025-01-10 Thread Andre Muezerie
MSVC does not support VLAs, so we want to prevent VLAs from being introduced under this path. Signed-off-by: Andre Muezerie --- lib/net/meson.build | 8 1 file changed, 8 insertions(+) diff --git a/lib/net/meson.build b/lib/net/meson.build index 8afcc4ed37..3194681521 100644 --- a/lib/

[PATCH v14 51/81] lib/mbuf: add compile warning about use of VLAs

2025-01-10 Thread Andre Muezerie
MSVC does not support VLAs, so we want to prevent VLAs from being introduced under this path. Signed-off-by: Andre Muezerie --- lib/mbuf/meson.build | 8 1 file changed, 8 insertions(+) diff --git a/lib/mbuf/meson.build b/lib/mbuf/meson.build index 2cee9057a5..1b94b286a6 100644 --- a/l

[PATCH v14 52/81] lib/mempool: add compile warning about use of VLAs

2025-01-10 Thread Andre Muezerie
MSVC does not support VLAs, so we want to prevent VLAs from being introduced under this path. Signed-off-by: Andre Muezerie --- lib/mempool/meson.build | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/lib/mempool/meson.build b/lib/mempool/meson.build index 8099a56466..3921945f

[PATCH v14 47/81] lib/kvargs: add compile warning about use of VLAs

2025-01-10 Thread Andre Muezerie
MSVC does not support VLAs, so we want to prevent VLAs from being introduced under this path. Signed-off-by: Andre Muezerie --- lib/kvargs/meson.build | 8 1 file changed, 8 insertions(+) diff --git a/lib/kvargs/meson.build b/lib/kvargs/meson.build index 7eae744a8f..c94ddb2fe3 100644 -

[PATCH v14 45/81] lib/ip_frag: add compile warning about use of VLAs

2025-01-10 Thread Andre Muezerie
MSVC does not support VLAs, so we want to prevent VLAs from being introduced under this path. Signed-off-by: Andre Muezerie --- lib/ip_frag/meson.build | 8 1 file changed, 8 insertions(+) diff --git a/lib/ip_frag/meson.build b/lib/ip_frag/meson.build index ea2de09f75..f5c6cb4769 10064

[PATCH v14 49/81] lib/log: add compile warning about use of VLAs

2025-01-10 Thread Andre Muezerie
MSVC does not support VLAs, so we want to prevent VLAs from being introduced under this path. Signed-off-by: Andre Muezerie --- lib/log/meson.build | 8 1 file changed, 8 insertions(+) diff --git a/lib/log/meson.build b/lib/log/meson.build index b3de57b9c7..5d7261a899 100644 --- a/lib/

[PATCH v14 44/81] lib/gso: add compile warning about use of VLAs

2025-01-10 Thread Andre Muezerie
MSVC does not support VLAs, so we want to prevent VLAs from being introduced under this path. Signed-off-by: Andre Muezerie --- lib/gso/meson.build | 8 1 file changed, 8 insertions(+) diff --git a/lib/gso/meson.build b/lib/gso/meson.build index 622411df8f..f5702e200f 100644 --- a/lib/

[PATCH v14 48/81] lib/latencystats: add compile warning about use of VLAs

2025-01-10 Thread Andre Muezerie
MSVC does not support VLAs, so we want to prevent VLAs from being introduced under this path. Signed-off-by: Andre Muezerie --- lib/latencystats/meson.build | 8 1 file changed, 8 insertions(+) diff --git a/lib/latencystats/meson.build b/lib/latencystats/meson.build index 286558dd79..e

[PATCH v14 42/81] lib/fib: add compile warning about use of VLAs

2025-01-10 Thread Andre Muezerie
MSVC does not support VLAs, so we want to prevent VLAs from being introduced under this path. Signed-off-by: Andre Muezerie --- lib/fib/meson.build | 8 1 file changed, 8 insertions(+) diff --git a/lib/fib/meson.build b/lib/fib/meson.build index 0c19cc8201..5a1cf0226a 100644 --- a/lib/

[PATCH v14 40/81] lib/dmadev: add compile warning about use of VLAs

2025-01-10 Thread Andre Muezerie
MSVC does not support VLAs, so we want to prevent VLAs from being introduced under this path. Signed-off-by: Andre Muezerie --- lib/dmadev/meson.build | 8 1 file changed, 8 insertions(+) diff --git a/lib/dmadev/meson.build b/lib/dmadev/meson.build index e66dcb66b0..61c99696ae 100644 -

[PATCH v14 41/81] lib/efd: add compile warning about use of VLAs

2025-01-10 Thread Andre Muezerie
MSVC does not support VLAs, so we want to prevent VLAs from being introduced under this path. Signed-off-by: Andre Muezerie --- lib/efd/meson.build | 8 1 file changed, 8 insertions(+) diff --git a/lib/efd/meson.build b/lib/efd/meson.build index 343f14e1f3..5475c05765 100644 --- a/lib/

[PATCH v14 43/81] lib/gpudev: add compile warning about use of VLAs

2025-01-10 Thread Andre Muezerie
MSVC does not support VLAs, so we want to prevent VLAs from being introduced under this path. Signed-off-by: Andre Muezerie --- lib/gpudev/meson.build | 8 1 file changed, 8 insertions(+) diff --git a/lib/gpudev/meson.build b/lib/gpudev/meson.build index d21fadc052..57022d75a4 100644 -

[PATCH v14 39/81] lib/distributor: add compile warning about use of VLAs

2025-01-10 Thread Andre Muezerie
MSVC does not support VLAs, so we want to prevent VLAs from being introduced under this path. Signed-off-by: Andre Muezerie --- lib/distributor/meson.build | 8 1 file changed, 8 insertions(+) diff --git a/lib/distributor/meson.build b/lib/distributor/meson.build index 24988c5827..e41f

[PATCH v14 36/81] lib/cmdline: add compile warning about use of VLAs

2025-01-10 Thread Andre Muezerie
MSVC does not support VLAs, so we want to prevent VLAs from being introduced under this path. Signed-off-by: Andre Muezerie --- lib/cmdline/meson.build | 8 1 file changed, 8 insertions(+) diff --git a/lib/cmdline/meson.build b/lib/cmdline/meson.build index 63fb69100d..da7a578d05 10064

[PATCH v14 27/81] app/test-gpudev: add compile warning about use of VLAs

2025-01-10 Thread Andre Muezerie
MSVC does not support VLAs, so we want to prevent VLAs from being introduced under this path. Signed-off-by: Andre Muezerie --- app/test-gpudev/meson.build | 8 1 file changed, 8 insertions(+) diff --git a/app/test-gpudev/meson.build b/app/test-gpudev/meson.build index 17bdef3646..9d11

[PATCH v14 35/81] lib/cfgfile: add compile warning about use of VLAs

2025-01-10 Thread Andre Muezerie
MSVC does not support VLAs, so we want to prevent VLAs from being introduced under this path. Signed-off-by: Andre Muezerie --- lib/cfgfile/meson.build | 8 1 file changed, 8 insertions(+) diff --git a/lib/cfgfile/meson.build b/lib/cfgfile/meson.build index 88eb819856..afc0a87c00 10064

[PATCH v14 38/81] lib/cryptodev: add compile warning about use of VLAs

2025-01-10 Thread Andre Muezerie
MSVC does not support VLAs, so we want to prevent VLAs from being introduced under this path. Signed-off-by: Andre Muezerie --- lib/cryptodev/meson.build | 8 1 file changed, 8 insertions(+) diff --git a/lib/cryptodev/meson.build b/lib/cryptodev/meson.build index 4734acf321..02bf7cec8e

[PATCH v14 37/81] lib/compressdev: add compile warning about use of VLAs

2025-01-10 Thread Andre Muezerie
MSVC does not support VLAs, so we want to prevent VLAs from being introduced under this path. Signed-off-by: Andre Muezerie --- lib/compressdev/meson.build | 8 1 file changed, 8 insertions(+) diff --git a/lib/compressdev/meson.build b/lib/compressdev/meson.build index c80295dc0d..b354

[PATCH v14 62/81] lib/rawdev: add compile warning about use of VLAs

2025-01-10 Thread Andre Muezerie
MSVC does not support VLAs, so we want to prevent VLAs from being introduced under this path. Signed-off-by: Andre Muezerie --- lib/rawdev/meson.build | 8 1 file changed, 8 insertions(+) diff --git a/lib/rawdev/meson.build b/lib/rawdev/meson.build index 7dfc3d5cf9..5e60255676 100644 -

[PATCH v14 26/81] app/test-fib: add compile warning about use of VLAs

2025-01-10 Thread Andre Muezerie
MSVC does not support VLAs, so we want to prevent VLAs from being introduced under this path. Signed-off-by: Andre Muezerie --- app/test-fib/meson.build | 8 1 file changed, 8 insertions(+) diff --git a/app/test-fib/meson.build b/app/test-fib/meson.build index eb36772cf3..f639171068 10

[PATCH v14 33/81] lib/bbdev: add compile warning about use of VLAs

2025-01-10 Thread Andre Muezerie
MSVC does not support VLAs, so we want to prevent VLAs from being introduced under this path. Signed-off-by: Andre Muezerie --- lib/bbdev/meson.build | 8 1 file changed, 8 insertions(+) diff --git a/lib/bbdev/meson.build b/lib/bbdev/meson.build index 07685e7578..542372d29b 100644 ---

[PATCH v14 34/81] lib/bitratestats: add compile warning about use of VLAs

2025-01-10 Thread Andre Muezerie
MSVC does not support VLAs, so we want to prevent VLAs from being introduced under this path. Signed-off-by: Andre Muezerie --- lib/bitratestats/meson.build | 8 1 file changed, 8 insertions(+) diff --git a/lib/bitratestats/meson.build b/lib/bitratestats/meson.build index ede7e0a579..7

[PATCH v14 31/81] app/test-security-perf: add compile warning about use of VLAs

2025-01-10 Thread Andre Muezerie
MSVC does not support VLAs, so we want to prevent VLAs from being introduced under this path. Signed-off-by: Andre Muezerie --- app/test-security-perf/meson.build | 8 1 file changed, 8 insertions(+) diff --git a/app/test-security-perf/meson.build b/app/test-security-perf/meson.build

[PATCH v14 30/81] app/test-regex: add compile warning about use of VLAs

2025-01-10 Thread Andre Muezerie
MSVC does not support VLAs, so we want to prevent VLAs from being introduced under this path. Signed-off-by: Andre Muezerie --- app/test-regex/meson.build | 8 1 file changed, 8 insertions(+) diff --git a/app/test-regex/meson.build b/app/test-regex/meson.build index e3ba060e55..a0c5bce

[PATCH v14 60/81] lib/port: add compile warning about use of VLAs

2025-01-10 Thread Andre Muezerie
MSVC does not support VLAs, so we want to prevent VLAs from being introduced under this path. Signed-off-by: Andre Muezerie --- lib/port/meson.build | 8 1 file changed, 8 insertions(+) diff --git a/lib/port/meson.build b/lib/port/meson.build index b597772872..4a09af361f 100644 --- a/l

[PATCH v14 32/81] lib/argparse: add compile warning about use of VLAs

2025-01-10 Thread Andre Muezerie
MSVC does not support VLAs, so we want to prevent VLAs from being introduced under this path. Signed-off-by: Andre Muezerie --- lib/argparse/meson.build | 8 1 file changed, 8 insertions(+) diff --git a/lib/argparse/meson.build b/lib/argparse/meson.build index b6a08ca049..3ef537b958 10

[PATCH v14 58/81] lib/pcapng: add compile warning about use of VLAs

2025-01-10 Thread Andre Muezerie
MSVC does not support VLAs, so we want to prevent VLAs from being introduced under this path. Signed-off-by: Andre Muezerie --- lib/pcapng/meson.build | 8 1 file changed, 8 insertions(+) diff --git a/lib/pcapng/meson.build b/lib/pcapng/meson.build index 4549925d41..0ccaf2dcfd 100644 -

[PATCH v14 63/81] lib/reorder: add compile warning about use of VLAs

2025-01-10 Thread Andre Muezerie
MSVC does not support VLAs, so we want to prevent VLAs from being introduced under this path. Signed-off-by: Andre Muezerie --- lib/reorder/meson.build | 8 1 file changed, 8 insertions(+) diff --git a/lib/reorder/meson.build b/lib/reorder/meson.build index 03aed53d90..5197da9298 10064

[PATCH v14 28/81] app/test-mldev: add compile warning about use of VLAs

2025-01-10 Thread Andre Muezerie
MSVC does not support VLAs, so we want to prevent VLAs from being introduced under this path. Signed-off-by: Andre Muezerie --- app/test-mldev/meson.build | 8 1 file changed, 8 insertions(+) diff --git a/app/test-mldev/meson.build b/app/test-mldev/meson.build index 2922888e9f..06fba62

[PATCH v14 24/81] app/test-cmdline: add compile warning about use of VLAs

2025-01-10 Thread Andre Muezerie
MSVC does not support VLAs, so we want to prevent VLAs from being introduced under this path. Signed-off-by: Andre Muezerie --- app/test-cmdline/meson.build | 8 1 file changed, 8 insertions(+) diff --git a/app/test-cmdline/meson.build b/app/test-cmdline/meson.build index 6027f67919..0

[PATCH v14 29/81] app/test-pipeline: add compile warning about use of VLAs

2025-01-10 Thread Andre Muezerie
MSVC does not support VLAs, so we want to prevent VLAs from being introduced under this path. Signed-off-by: Andre Muezerie --- app/test-pipeline/meson.build | 8 1 file changed, 8 insertions(+) diff --git a/app/test-pipeline/meson.build b/app/test-pipeline/meson.build index 3add6b637b

[PATCH v14 57/81] lib/node: add compile warning about use of VLAs

2025-01-10 Thread Andre Muezerie
MSVC does not support VLAs, so we want to prevent VLAs from being introduced under this path. Signed-off-by: Andre Muezerie --- lib/node/meson.build | 8 1 file changed, 8 insertions(+) diff --git a/lib/node/meson.build b/lib/node/meson.build index 0bed97a96c..4c7361a53a 100644 --- a/l

[PATCH v14 18/81] build: enable vla warnings on Windows built code

2025-01-10 Thread Andre Muezerie
From: Tyler Retzlaff MSVC does not support optional C11 VLAs. When building for Windows enable -Wvla so that mingw and clang also fail if a VLA is used. Signed-off-by: Tyler Retzlaff Acked-by: Bruce Richardson Acked-by: Chengwen Feng --- config/meson.build | 4 1 file changed, 4 inserti

[PATCH v14 22/81] app/dumpcap: add compile warning about use of VLAs

2025-01-10 Thread Andre Muezerie
MSVC does not support VLAs, so we want to prevent VLAs from being introduced under this path. Signed-off-by: Andre Muezerie --- app/dumpcap/meson.build | 8 1 file changed, 8 insertions(+) diff --git a/app/dumpcap/meson.build b/app/dumpcap/meson.build index 69c016c780..675f5d7448 10064

[PATCH v14 64/81] lib/rib: add compile warning about use of VLAs

2025-01-10 Thread Andre Muezerie
MSVC does not support VLAs, so we want to prevent VLAs from being introduced under this path. Signed-off-by: Andre Muezerie --- lib/rib/meson.build | 8 1 file changed, 8 insertions(+) diff --git a/lib/rib/meson.build b/lib/rib/meson.build index e98f708481..58fcb825a8 100644 --- a/lib/

[PATCH v14 61/81] lib/ptr_compress: add compile warning about use of VLAs

2025-01-10 Thread Andre Muezerie
MSVC does not support VLAs, so we want to prevent VLAs from being introduced under this path. Signed-off-by: Andre Muezerie --- lib/ptr_compress/meson.build | 8 1 file changed, 8 insertions(+) diff --git a/lib/ptr_compress/meson.build b/lib/ptr_compress/meson.build index e92706a45f..7

[PATCH v14 17/81] net/mlx5: remove use of VLAs for Windows built code

2025-01-10 Thread Andre Muezerie
From: Tyler Retzlaff 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. Signed-off-by: Tyler Retzlaff --- drivers/net/meson.build | 8 drivers/net/mlx5/mlx5.c

[PATCH v14 19/81] test: remove use of VLAs for Windows built code in bitset tests

2025-01-10 Thread Andre Muezerie
1) MSVC does not support VLAs. Use standard fixed C arrays of maximum size required instead. 2) ../usr/lib/gcc/x86_64-redhat-linux/13/include/emmintrin.h:742:8: error: array subscript 9 is outside array bounds of 'uint64_t[16]' {aka 'long unsigned int[16]'} [-Werror=array-bounds=] 3695

[PATCH v14 55/81] lib/mldev: add compile warning about use of VLAs

2025-01-10 Thread Andre Muezerie
MSVC does not support VLAs, so we want to prevent VLAs from being introduced under this path. Signed-off-by: Andre Muezerie --- lib/mldev/meson.build | 8 1 file changed, 8 insertions(+) diff --git a/lib/mldev/meson.build b/lib/mldev/meson.build index 2c933baad6..d49302f021 100644 ---

[PATCH v14 21/81] hash: remove use of VLAs by using standard arrays

2025-01-10 Thread Andre Muezerie
MSVC does not support VLAs, replace VLAs with standard C arrays. Signed-off-by: Andre Muezerie --- lib/hash/meson.build | 8 lib/hash/rte_thash_gf2_poly_math.c | 9 - 2 files changed, 12 insertions(+), 5 deletions(-) diff --git a/lib/hash/meson.build b/lib/hash/me

[PATCH v14 15/81] net/i40e: remove use of VLAs for Windows built code

2025-01-10 Thread Andre Muezerie
From: Tyler Retzlaff 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. Signed-off-by: Tyler Retzlaff Reviewed-by: Bruce Richardson --- drivers/net/i40e/i40e_testpmd.c | 5 ++-

[PATCH v14 25/81] app/test-compress-perf: add compile warning about use of VLAs

2025-01-10 Thread Andre Muezerie
MSVC does not support VLAs, so we want to prevent VLAs from being introduced under this path. Signed-off-by: Andre Muezerie --- app/test-compress-perf/meson.build | 8 1 file changed, 8 insertions(+) diff --git a/app/test-compress-perf/meson.build b/app/test-compress-perf/meson.build

[PATCH v14 23/81] app/graph: add compile warning about use of VLAs

2025-01-10 Thread Andre Muezerie
MSVC does not support VLAs, so we want to prevent VLAs from being introduced under this path. Signed-off-by: Andre Muezerie --- app/graph/meson.build | 8 1 file changed, 8 insertions(+) diff --git a/app/graph/meson.build b/app/graph/meson.build index 344e4a418f..28ad533498 100644 ---

[PATCH v14 16/81] common/mlx5: remove use of VLAs for Windows built code

2025-01-10 Thread Andre Muezerie
From: Tyler Retzlaff 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. Signed-off-by: Tyler Retzlaff --- drivers/common/meson.build | 8 drivers/common/mlx5

[PATCH v14 13/81] test: remove use of VLAs for Windows built code

2025-01-10 Thread Andre Muezerie
From: Tyler Retzlaff 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. Signed-off-by: Tyler Retzlaff Acked-by: Chengwen Feng --- app/test/test.c | 2 +-

[PATCH v14 11/81] net/ice: remove use of VLAs

2025-01-10 Thread Andre Muezerie
From: Konstantin Ananyev ../drivers/net/ice/ice_rxtx.c:1871:29: warning: variable length array used [-Wvla] Here VLA is used as a temp array for mbufs that will be used as a split RX data buffers. As at any given time only one thread can do RX from particular queue, at rx_queue_setup() we ca

  1   2   >