ice driver compiler warning

ice driver compiler warning

[Morten Brørup]

ice driver maintainers,

Is the ice base driver still defining structure fields with [0], or has it been updated to use []?
Maybe the workaround in meson.build is not required anymore:

# Bugzilla ID: 678
if (toolchain == 'gcc' and cc.version().version_compare('>=11.0.0'))
    error_cflags += ['-Wno-array-bounds']
endif

Ref:
https://github.com/DPDK/dpdk/blob/main/drivers/net/intel/ice/base/meson.build#L49


Venlig hilsen / Kind regards,
-Morten Brørup

Re: ice driver compiler warning

[Stephen Hemminger]

On Sat, 1 Aug 2026 11:00:38 +0200
Morten Brørup <mb@smartsharesystems.com> wrote:

> ice driver maintainers,
> 
> Is the ice base driver still defining structure fields with [0], or has it been updated to use []?
> Maybe the workaround in meson.build is not required anymore:
> 
> # Bugzilla ID: 678
> if (toolchain == 'gcc' and cc.version().version_compare('>=11.0.0'))
>     error_cflags += ['-Wno-array-bounds']
> endif
> 
> Ref:
> https://github.com/DPDK/dpdk/blob/main/drivers/net/intel/ice/base/meson.build#L49
> 
> 
> Venlig hilsen / Kind regards,
> -Morten Brørup
> 
> 

Looks like it could be dropped. I see no errors with Gcc 18

[PATCH] net/ice/base: allow array bounds warnings

[Bruce Richardson]

Since there are no zero-length arrays in the base code any more, remove
the -Wno-array-bounds build flag.

Signed-off-by: Bruce Richardson <bruce.richardson@intel.com>
---
 drivers/net/intel/ice/base/meson.build | 5 -----
 1 file changed, 5 deletions(-)

diff --git a/drivers/net/intel/ice/base/meson.build b/drivers/net/intel/ice/base/meson.build
index fd079c41cf..da961f0752 100644
--- a/drivers/net/intel/ice/base/meson.build
+++ b/drivers/net/intel/ice/base/meson.build
@@ -44,11 +44,6 @@ else
     ]
 endif
 
-# Bugzilla ID: 678
-if (toolchain == 'gcc' and cc.version().version_compare('>=11.0.0'))
-    error_cflags += ['-Wno-array-bounds']
-endif
-
 foreach flag: error_cflags
     if cc.has_argument(flag)
         base_cflags += flag
-- 
2.53.0

RE: [PATCH] net/ice/base: allow array bounds warnings

[Morten Brørup]

> From: Bruce Richardson [mailto:bruce.richardson@intel.com]
> Sent: Thursday, 13 August 2026 13.15
> 
> Since there are no zero-length arrays in the base code any more, remove
> the -Wno-array-bounds build flag.
> 
> Signed-off-by: Bruce Richardson <bruce.richardson@intel.com>
> ---
>  drivers/net/intel/ice/base/meson.build | 5 -----
>  1 file changed, 5 deletions(-)
> 
> diff --git a/drivers/net/intel/ice/base/meson.build
> b/drivers/net/intel/ice/base/meson.build
> index fd079c41cf..da961f0752 100644
> --- a/drivers/net/intel/ice/base/meson.build
> +++ b/drivers/net/intel/ice/base/meson.build
> @@ -44,11 +44,6 @@ else
>      ]
>  endif
> 
> -# Bugzilla ID: 678
> -if (toolchain == 'gcc' and cc.version().version_compare('>=11.0.0'))
> -    error_cflags += ['-Wno-array-bounds']
> -endif
> -

Good catch!
Consequential workarounds like this are often overlooked when the workaround is not necessary anymore.

Patchwork should have a "fixed/workaround" state for tracking workarounds, so they can be properly cleaned up like this.
The root cause has been fixed, so by removing this last detail of the workaround the bug is truly "fixed/resolved".

Acked-by: Morten Brørup <mb@smartsharesystems.com>