On 01/14, Ferruh Yigit wrote: >Build error: > In function ‘ixgbe_recv_pkts_lro_bulk_alloc’: >../drivers/net/ixgbe/ixgbe_rxtx.c:2209:24: > error: ‘next_sc_entry’ may be used uninitialized in this function > [-Werror=maybe-uninitialized] > next_sc_entry->fbuf = first_seg; > ^ >http://mails.dpdk.org/archives/test-report/2020-January/113891.html > >This is a compiler false positive and error not seen by newer compilers, >or clang but to fix the warning initializing the complained variable. > >According git bisect, no idea how: >Fixes: 8a2e7ce976fd ("net/ixgbe: avoid multiple definitions of bool") > >Reported-by: Maxime Coquelin <maxime.coque...@redhat.com> >Signed-off-by: Ferruh Yigit <ferruh.yi...@intel.com> >--- >Cc: dharmik.thak...@arm.com >--- > drivers/net/ixgbe/ixgbe_rxtx.c | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > >diff --git a/drivers/net/ixgbe/ixgbe_rxtx.c b/drivers/net/ixgbe/ixgbe_rxtx.c >index 13b06eb9c..30f890572 100644 >--- a/drivers/net/ixgbe/ixgbe_rxtx.c >+++ b/drivers/net/ixgbe/ixgbe_rxtx.c >@@ -2028,7 +2028,7 @@ ixgbe_recv_pkts_lro(void *rx_queue, struct rte_mbuf >**rx_pkts, uint16_t nb_pkts, > bool eop; > struct ixgbe_rx_entry *rxe; > struct ixgbe_scattered_rx_entry *sc_entry; >- struct ixgbe_scattered_rx_entry *next_sc_entry; >+ struct ixgbe_scattered_rx_entry *next_sc_entry = NULL; > struct ixgbe_rx_entry *next_rxe = NULL; > struct rte_mbuf *first_seg; > struct rte_mbuf *rxm; >-- >2.24.1 >
Acked-by: Xiaolong Ye <xiaolong...@intel.com>
