[PATCH] acl: fix build with GCC 15 on aarch64

[David Marchand]

On Thu, Mar 27, 2025 at 7:06 PM Bruce Richardson
<bruce.richardson at intel.com> wrote:
>
> On Wed, Mar 26, 2025 at 11:39:28AM +0100, David Marchand wrote:
> > Caught in OBS for Fedora Rawhide on aarch64:
> >
> > [  198s] In file included from ../lib/acl/acl_run_neon.h:7,
> > [  198s]                  from ../lib/acl/acl_run_neon.c:5:
> > [  198s] In function ‘alloc_completion’,
> > [  198s]     inlined from ‘acl_start_next_trie’ at
> >       ../lib/acl/acl_run.h:140:24,
> > [  198s]     inlined from ‘search_neon_4.isra’ at
> >       ../lib/acl/acl_run_neon.h:239:20:
> > [  198s] ../lib/acl/acl_run.h:93:25: error: ‘cmplt’ may be used
> >       uninitialized [-Werror=maybe-uninitialized]
> > [  198s]    93 |                 if (p[n].count == 0) {
> > [  198s]       |                     ~~~~^~~~~~
> > [  198s] ../lib/acl/acl_run_neon.h: In function ‘search_neon_4.isra’:
> > [  198s] ../lib/acl/acl_run_neon.h:230:27: note: ‘cmplt’ declared here
> > [  198s]   230 |         struct completion cmplt[4];
> > [  198s]       |                           ^~~~~
> >
> > The code was resetting sequentially cmpl[].count at the exact index that
> > later call to alloc_completion uses.
> > While this code seems correct, GCC 15 does not understand this (probably
> > when applying some optimisations).
> >
> > Instead, reset cmpl[].count all at once in acl_set_flow, and cleanup the
> > various vectorized implementations accordingly.
> >
> > Bugzilla ID: 1678
> > Cc: stable at dpdk.org
> >
> > Signed-off-by: David Marchand <david.marchand at redhat.com>
> Acked-by: Bruce Richardson <bruce.richardson at intel.com>

Applied as is, thanks.


-- 
David Marchand