On Wed, 18 May 2022 12:16:50 +0200 David Marchand <david.march...@redhat.com> wrote:
> GCC 12 raises the following warning: > > ../drivers/net/enetfec/enet_ethdev.c: In function > ‘enetfec_rx_queue_setup’: > ../drivers/net/enetfec/enet_ethdev.c:473:9: error: array > subscript 1 is > above array bounds of ‘uint32_t[1]’ {aka ‘unsigned int[1]’} > [-Werror=array-bounds] > 473 | rte_write32(rte_cpu_to_le_32(fep->bd_addr_p_r[queue_idx]), > | ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~ > 474 | (uint8_t *)fep->hw_baseaddr_v + ENETFEC_RD_START(queue_idx)); > | ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~ > In file included from ../drivers/net/enetfec/enet_ethdev.c:9: > ../drivers/net/enetfec/enet_ethdev.h:113:33: note: while referencing > ‘bd_addr_p_r’ > 113 | uint32_t bd_addr_p_r[ENETFEC_MAX_Q]; > | ^~~~~~~~~~~ > > This driver properly announces that it only supports 1 rxq. > Silence this warning by adding an explicit check on the queue id. > > Cc: sta...@dpdk.org > > Signed-off-by: David Marchand <david.march...@redhat.com> Acked-by: Stephen Hemminger <step...@networkplumber.org>