lib: fix unused values
authorZijie Pan <zijie.pan@6wind.com>
Tue, 4 Dec 2012 09:17:43 +0000 (17:17 +0800)
committerThomas Monjalon <thomas.monjalon@6wind.com>
Thu, 25 Jul 2013 14:07:51 +0000 (16:07 +0200)
Fix warnings of type "Value stored to 'xxx' is never read".

Signed-off-by: Zijie Pan <zijie.pan@6wind.com>
Acked-by: Ivan Boule <ivan.boule@6wind.com>
Acked-by: Adrien Mazarguil <adrien.mazarguil@6wind.com>
lib/librte_cmdline/cmdline_parse_ipaddr.c
lib/librte_eal/linuxapp/eal/eal_thread.c
lib/librte_lpm/rte_lpm.c
lib/librte_pmd_e1000/igb_rxtx.c
lib/librte_pmd_ixgbe/ixgbe_ethdev.c

index 7659a1d..4ab3ac3 100644 (file)
@@ -266,7 +266,6 @@ inet_pton6(const char *src, unsigned char *dst)
                    inet_pton4(curtok, tp) > 0) {
                        tp += INADDRSZ;
                        saw_xdigit = 0;
-                       count_xdigit = 0;
                        dbloct_count += 2;
                        break;  /* '\0' was seen by inet_pton4(). */
                }
index 6016142..69bc10e 100644 (file)
@@ -85,7 +85,6 @@ rte_eal_remote_launch(int (*f)(void *), void *arg, unsigned slave_id)
                rte_panic("cannot write on configuration pipe\n");
 
        /* wait ack */
-       n = 0;
        do {
                n = read(s2m, &c, 1);
        } while (n < 0 && errno == EINTR);
@@ -202,7 +201,6 @@ eal_thread_loop(__attribute__((unused)) void *arg)
                void *fct_arg;
 
                /* wait command */
-               n = 0;
                do {
                        n = read(m2s, &c, 1);
                } while (n < 0 && errno == EINTR);
index 10705fc..f816fdd 100644 (file)
@@ -253,7 +253,7 @@ rule_add(struct rte_lpm *lpm, uint32_t ip_masked, uint8_t depth,
        last_rule = rule_gindex + lpm->used_rules_at_depth[depth - 1];
                
        /* Scan through rule group to see if rule already exists. */
-       for (rule_index = rule_gindex; rule_index < last_rule; rule_index++) {
+       for (; rule_index < last_rule; rule_index++) {
 
                /* If rule already exists update its next_hop and return. */
                if (lpm->rules_tbl[rule_index].ip == ip_masked) {
index c942e1e..f70ea5a 100644 (file)
@@ -1231,8 +1231,6 @@ eth_igb_tx_queue_setup(struct rte_eth_dev *dev,
        txq->tx_ring_phys_addr = (uint64_t) tz->phys_addr;
        txq->tx_ring = (union e1000_adv_tx_desc *) tz->addr;
 
-       size = sizeof(union e1000_adv_tx_desc) * nb_desc;
-
        /* Allocate software ring */
        txq->sw_ring = rte_zmalloc("txq->sw_ring",
                                   sizeof(struct igb_tx_entry) * nb_desc,
index 31434a7..8350ee2 100644 (file)
@@ -708,6 +708,10 @@ eth_ixgbevf_dev_init(__attribute__((unused)) struct eth_driver *eth_drv,
 
        hw->mac.num_rar_entries = hw->mac.max_rx_queues;
        diag = hw->mac.ops.reset_hw(hw);
+       if (diag != IXGBE_SUCCESS) {
+               PMD_INIT_LOG(ERR, "VF Initialization Failure: %d", diag);
+               return (diag);
+       }
 
        /* Allocate memory for storing MAC addresses */
        eth_dev->data->mac_addrs = rte_zmalloc("ixgbevf", ETHER_ADDR_LEN *