From: Rakesh Kudurumalla Date: Fri, 4 Mar 2022 14:23:37 +0000 (+0530) Subject: net/cnxk: fix build with optimization X-Git-Url: http://git.droids-corp.org/?a=commitdiff_plain;h=68f8a52a6b0ad6b77772d4564928aebb21c2ca66;p=dpdk.git net/cnxk: fix build with optimization Fix the following build error seen with --optimization=1 and GCC 10.3.0. drivers/net/cnxk/cnxk_ethdev_mtr.c: In function ‘cnxk_nix_mtr_policy_validate’: lib/ethdev/rte_mtr_driver.h:188:10: error: ‘str’ may be used uninitialized in this function [-Werror=maybe-uninitialized] drivers/net/cnxk/cn10k_rx.h:149:2 error: ‘frag_ptr’ may be used uninitialized in this function Bugzilla ID: 939 Fixes: b526599020ef ("net/cnxk: fix build with GCC 12") Cc: stable@dpdk.org Reported-by: Ferruh Yigit Signed-off-by: Rakesh Kudurumalla Tested-by: Daxue Gao Acked-by: Jerin Jacob --- diff --git a/drivers/net/cnxk/cn10k_rx.h b/drivers/net/cnxk/cn10k_rx.h index 7f219e5ddd..e4f5a553a6 100644 --- a/drivers/net/cnxk/cn10k_rx.h +++ b/drivers/net/cnxk/cn10k_rx.h @@ -146,7 +146,8 @@ nix_sec_attach_frags(const struct cpt_parse_hdr_s *hdr, uint32_t offset = hdr->w2.fi_offset; union nix_rx_parse_u *frag_rx; struct cpt_frag_info_s *finfo; - uint64_t *frag_ptr, ol_flags; + uint64_t *frag_ptr = NULL; + uint64_t ol_flags; uint16_t frag_size; uint16_t rlen; uint64_t *wqe; @@ -276,8 +277,8 @@ nix_sec_reassemble_frags(const struct cpt_parse_hdr_s *hdr, uint64_t cq_w1, union nix_rx_parse_u *frag_rx; struct cpt_frag_info_s *finfo; struct rte_mbuf *head, *mbuf; + uint64_t *frag_ptr = NULL; rte_iova_t *inner_iova; - uint64_t *frag_ptr; uint16_t frag_size; uint64_t *wqe; diff --git a/drivers/net/cnxk/cnxk_ethdev_mtr.c b/drivers/net/cnxk/cnxk_ethdev_mtr.c index c8183aa12d..02803bdf75 100644 --- a/drivers/net/cnxk/cnxk_ethdev_mtr.c +++ b/drivers/net/cnxk/cnxk_ethdev_mtr.c @@ -288,7 +288,7 @@ cnxk_nix_mtr_profile_delete(struct rte_eth_dev *eth_dev, uint32_t profile_id, static int update_mtr_err(uint32_t act_color, struct rte_mtr_error *error, bool action) { - const char *str; + const char *str = NULL; switch (act_color) { case RTE_COLOR_GREEN: if (action) {