[dpdk-dev] [PATCH 13/22] lib: fix uninitialized variables

Thomas Monjalon thomas.monjalon at 6wind.com
Wed Mar 20 17:05:01 CET 2013


From: Zijie Pan <zijie.pan at 6wind.com>

Fix compilation errors caused by uninitialized variables.

Acked-by: Ivan Boule <ivan.boule at 6wind.com>
Acked-by: Adrien Mazarguil <adrien.mazarguil at 6wind.com>
Signed-off-by: Krzysztof Witek <krzysztof.witek at 6wind.com>
Signed-off-by: Zijie Pan <zijie.pan at 6wind.com>
---
 lib/librte_pmd_igb/e1000_rxtx.c   |    4 ++--
 lib/librte_pmd_igb/igb/e1000_vf.c |    3 ++-
 lib/librte_pmd_ixgbe/ixgbe_rxtx.c |    2 +-
 3 files changed, 5 insertions(+), 4 deletions(-)

diff --git a/lib/librte_pmd_igb/e1000_rxtx.c b/lib/librte_pmd_igb/e1000_rxtx.c
index 96b0682..ef2b88b 100644
--- a/lib/librte_pmd_igb/e1000_rxtx.c
+++ b/lib/librte_pmd_igb/e1000_rxtx.c
@@ -330,8 +330,8 @@ eth_igb_xmit_pkts(struct igb_tx_queue *txq, struct rte_mbuf **tx_pkts,
 	uint16_t tx_last;
 	uint16_t nb_tx;
 	uint16_t tx_ol_req;
-	uint32_t new_ctx;
-	uint32_t ctx;
+	uint32_t new_ctx = 0;
+	uint32_t ctx = 0;
 	uint32_t vlan_macip_lens;
 
 	sw_ring = txq->sw_ring;
diff --git a/lib/librte_pmd_igb/igb/e1000_vf.c b/lib/librte_pmd_igb/igb/e1000_vf.c
index 8b81e4b..5d71210 100644
--- a/lib/librte_pmd_igb/igb/e1000_vf.c
+++ b/lib/librte_pmd_igb/igb/e1000_vf.c
@@ -405,12 +405,13 @@ void e1000_update_mc_addr_list_vf(struct e1000_hw *hw,
 
 	DEBUGOUT1("MC Addr Count = %d\n", mc_addr_count);
 
+	msgbuf[0] = E1000_VF_SET_MULTICAST;
+
 	if (mc_addr_count > 30) {
 		msgbuf[0] |= E1000_VF_SET_MULTICAST_OVERFLOW;
 		mc_addr_count = 30;
 	}
 
-	msgbuf[0] = E1000_VF_SET_MULTICAST;
 	msgbuf[0] |= mc_addr_count << E1000_VT_MSGINFO_SHIFT;
 
 	for (i = 0; i < mc_addr_count; i++) {
diff --git a/lib/librte_pmd_ixgbe/ixgbe_rxtx.c b/lib/librte_pmd_ixgbe/ixgbe_rxtx.c
index aa698a3..47922c9 100644
--- a/lib/librte_pmd_ixgbe/ixgbe_rxtx.c
+++ b/lib/librte_pmd_ixgbe/ixgbe_rxtx.c
@@ -399,7 +399,7 @@ ixgbe_xmit_pkts(struct igb_tx_queue *txq, struct rte_mbuf **tx_pkts,
 	uint16_t nb_used;
 	uint16_t tx_ol_req;
 	uint32_t vlan_macip_lens;
-	uint32_t ctx;
+	uint32_t ctx = 0;
 	uint32_t new_ctx;
 
 	sw_ring = txq->sw_ring;
-- 
1.7.2.5




More information about the dev mailing list