net/hns3: fix error code in xstats
[dpdk.git] / drivers / net / hns3 / hns3_stats.c
index 067673c..42ec9b8 100644 (file)
@@ -2,13 +2,9 @@
  * Copyright(c) 2018-2019 Hisilicon Limited.
  */
 
-#include <stdbool.h>
-#include <stdint.h>
-#include <rte_common.h>
 #include <rte_ethdev.h>
 #include <rte_io.h>
 #include <rte_malloc.h>
-#include <rte_spinlock.h>
 
 #include "hns3_ethdev.h"
 #include "hns3_rxtx.h"
@@ -330,6 +326,8 @@ static const struct hns3_xstats_name_offset hns3_tx_queue_strings[] = {
 #define HNS3_FIX_NUM_STATS (HNS3_NUM_MAC_STATS + HNS3_NUM_ERROR_INT_XSTATS + \
                            HNS3_NUM_RESET_XSTATS)
 
+static void hns3_tqp_stats_clear(struct hns3_hw *hw);
+
 /*
  * Query all the MAC statistics data of Network ICL command ,opcode id: 0x0034.
  * This command is used before send 'query_mac_stat command', the descriptor
@@ -456,11 +454,10 @@ hns3_update_tqp_stats(struct hns3_hw *hw)
                hns3_cmd_setup_basic_desc(&desc, HNS3_OPC_QUERY_RX_STATUS,
                                          true);
 
-               desc.data[0] = rte_cpu_to_le_32((uint32_t)i &
-                                               HNS3_QUEUE_ID_MASK);
+               desc.data[0] = rte_cpu_to_le_32((uint32_t)i);
                ret = hns3_cmd_send(hw, &desc, 1);
                if (ret) {
-                       hns3_err(hw, "Failed to query RX No.%d queue stat: %d",
+                       hns3_err(hw, "Failed to query RX No.%u queue stat: %d",
                                 i, ret);
                        return ret;
                }
@@ -471,11 +468,10 @@ hns3_update_tqp_stats(struct hns3_hw *hw)
                hns3_cmd_setup_basic_desc(&desc, HNS3_OPC_QUERY_TX_STATUS,
                                          true);
 
-               desc.data[0] = rte_cpu_to_le_32((uint32_t)i &
-                                               HNS3_QUEUE_ID_MASK);
+               desc.data[0] = rte_cpu_to_le_32((uint32_t)i);
                ret = hns3_cmd_send(hw, &desc, 1);
                if (ret) {
-                       hns3_err(hw, "Failed to query TX No.%d queue stat: %d",
+                       hns3_err(hw, "Failed to query TX No.%u queue stat: %d",
                                 i, ret);
                        return ret;
                }
@@ -553,7 +549,6 @@ hns3_stats_reset(struct rte_eth_dev *eth_dev)
 {
        struct hns3_adapter *hns = eth_dev->data->dev_private;
        struct hns3_hw *hw = &hns->hw;
-       struct hns3_tqp_stats *stats = &hw->tqp_stats;
        struct hns3_cmd_desc desc_reset;
        struct hns3_rx_queue *rxq;
        struct hns3_tx_queue *txq;
@@ -561,28 +556,26 @@ hns3_stats_reset(struct rte_eth_dev *eth_dev)
        int ret;
 
        /*
-        * If this is a reset xstats is NULL, and we have cleared the
-        * registers by reading them.
+        * Note: Reading hardware statistics of rx/tx queue packet number
+        * will clear them.
         */
        for (i = 0; i < hw->tqps_num; i++) {
                hns3_cmd_setup_basic_desc(&desc_reset, HNS3_OPC_QUERY_RX_STATUS,
                                          true);
-               desc_reset.data[0] = rte_cpu_to_le_32((uint32_t)i &
-                                                     HNS3_QUEUE_ID_MASK);
+               desc_reset.data[0] = rte_cpu_to_le_32((uint32_t)i);
                ret = hns3_cmd_send(hw, &desc_reset, 1);
                if (ret) {
-                       hns3_err(hw, "Failed to reset RX No.%d queue stat: %d",
+                       hns3_err(hw, "Failed to reset RX No.%u queue stat: %d",
                                 i, ret);
                        return ret;
                }
 
                hns3_cmd_setup_basic_desc(&desc_reset, HNS3_OPC_QUERY_TX_STATUS,
                                          true);
-               desc_reset.data[0] = rte_cpu_to_le_32((uint32_t)i &
-                                                     HNS3_QUEUE_ID_MASK);
+               desc_reset.data[0] = rte_cpu_to_le_32((uint32_t)i);
                ret = hns3_cmd_send(hw, &desc_reset, 1);
                if (ret) {
-                       hns3_err(hw, "Failed to reset TX No.%d queue stat: %d",
+                       hns3_err(hw, "Failed to reset TX No.%u queue stat: %d",
                                 i, ret);
                        return ret;
                }
@@ -614,7 +607,7 @@ hns3_stats_reset(struct rte_eth_dev *eth_dev)
                }
        }
 
-       memset(stats, 0, sizeof(struct hns3_tqp_stats));
+       hns3_tqp_stats_clear(hw);
 
        return 0;
 }
@@ -668,8 +661,7 @@ hns3_get_queue_stats(struct rte_eth_dev *dev, struct rte_eth_xstat *xstats,
        /* Get rx queue stats */
        for (j = 0; j < dev->data->nb_rx_queues; j++) {
                for (i = 0; i < HNS3_NUM_RX_QUEUE_STATS; i++) {
-                       reg_offset = HNS3_TQP_REG_OFFSET +
-                                       HNS3_TQP_REG_SIZE * j;
+                       reg_offset = hns3_get_tqp_reg_offset(j);
                        xstats[*count].value = hns3_read_dev(hw,
                                reg_offset + hns3_rx_queue_strings[i].offset);
                        xstats[*count].id = *count;
@@ -680,15 +672,13 @@ hns3_get_queue_stats(struct rte_eth_dev *dev, struct rte_eth_xstat *xstats,
        /* Get tx queue stats */
        for (j = 0; j < dev->data->nb_tx_queues; j++) {
                for (i = 0; i < HNS3_NUM_TX_QUEUE_STATS; i++) {
-                       reg_offset = HNS3_TQP_REG_OFFSET +
-                                       HNS3_TQP_REG_SIZE * j;
+                       reg_offset = hns3_get_tqp_reg_offset(j);
                        xstats[*count].value = hns3_read_dev(hw,
                                reg_offset + hns3_tx_queue_strings[i].offset);
                        xstats[*count].id = *count;
                        (*count)++;
                }
        }
-
 }
 
 void
@@ -703,6 +693,7 @@ hns3_error_int_stats_add(struct hns3_adapter *hns, const char *err)
                        addr = (char *)&pf->abn_int_stats +
                                hns3_error_int_stats_strings[i].offset;
                        *(uint64_t *)addr += 1;
+                       break;
                }
        }
 }
@@ -748,9 +739,9 @@ hns3_dev_xstats_get(struct rte_eth_dev *dev, struct rte_eth_xstat *xstats,
        if (!hns->is_vf) {
                /* Update Mac stats */
                ret = hns3_query_update_mac_stats(dev);
-               if (ret) {
+               if (ret < 0) {
                        hns3_err(hw, "Update Mac stats fail : %d", ret);
-                       return 0;
+                       return ret;
                }
 
                /* Get MAC stats from hw->hw_xstats.mac_stats struct */
@@ -942,9 +933,13 @@ hns3_dev_xstats_get_by_id(struct rte_eth_dev *dev, const uint64_t *ids,
        uint32_t i;
        int ret;
 
-       if (ids == NULL || size < cnt_stats)
+       if (ids == NULL && values == NULL)
                return cnt_stats;
 
+       if (ids == NULL)
+               if (size < cnt_stats)
+                       return cnt_stats;
+
        /* Update tqp stats by read register */
        ret = hns3_update_tqp_stats(hw);
        if (ret) {
@@ -966,9 +961,18 @@ hns3_dev_xstats_get_by_id(struct rte_eth_dev *dev, const uint64_t *ids,
                return -EINVAL;
        }
 
+       if (ids == NULL && values != NULL) {
+               for (i = 0; i < cnt_stats; i++)
+                       memcpy(&values[i], &values_copy[i].value,
+                              sizeof(values[i]));
+
+               rte_free(values_copy);
+               return cnt_stats;
+       }
+
        for (i = 0; i < size; i++) {
                if (ids[i] >= cnt_stats) {
-                       hns3_err(hw, "ids[%d] (%" PRIx64 ") is invalid, "
+                       hns3_err(hw, "ids[%u] (%" PRIx64 ") is invalid, "
                                     "should < %u", i, ids[i], cnt_stats);
                        rte_free(values_copy);
                        return -EINVAL;
@@ -1014,9 +1018,16 @@ hns3_dev_xstats_get_names_by_id(struct rte_eth_dev *dev,
        uint64_t len;
        uint32_t i;
 
-       if (ids == NULL || xstats_names == NULL)
+       if (xstats_names == NULL)
                return cnt_stats;
 
+       if (ids == NULL) {
+               if (size < cnt_stats)
+                       return cnt_stats;
+
+               return hns3_dev_xstats_get_names(dev, xstats_names, cnt_stats);
+       }
+
        len = cnt_stats * sizeof(struct rte_eth_xstat_name);
        names_copy = rte_zmalloc("hns3_xstats_names", len, 0);
        if (names_copy == NULL) {
@@ -1029,7 +1040,7 @@ hns3_dev_xstats_get_names_by_id(struct rte_eth_dev *dev,
 
        for (i = 0; i < size; i++) {
                if (ids[i] >= cnt_stats) {
-                       hns3_err(hw, "ids[%d] (%" PRIx64 ") is invalid, "
+                       hns3_err(hw, "ids[%u] (%" PRIx64 ") is invalid, "
                                     "should < %u", i, ids[i], cnt_stats);
                        rte_free(names_copy);
                        return -EINVAL;
@@ -1070,3 +1081,49 @@ hns3_dev_xstats_reset(struct rte_eth_dev *dev)
 
        return 0;
 }
+
+int
+hns3_tqp_stats_init(struct hns3_hw *hw)
+{
+       struct hns3_tqp_stats *tqp_stats = &hw->tqp_stats;
+
+       tqp_stats->rcb_rx_ring_pktnum = rte_zmalloc("hns3_rx_ring_pkt_num",
+                                        sizeof(uint64_t) * hw->tqps_num, 0);
+       if (tqp_stats->rcb_rx_ring_pktnum == NULL) {
+               hns3_err(hw, "failed to allocate rx_ring pkt_num.");
+               return -ENOMEM;
+       }
+
+       tqp_stats->rcb_tx_ring_pktnum = rte_zmalloc("hns3_tx_ring_pkt_num",
+                                        sizeof(uint64_t) * hw->tqps_num, 0);
+       if (tqp_stats->rcb_tx_ring_pktnum == NULL) {
+               hns3_err(hw, "failed to allocate tx_ring pkt_num.");
+               rte_free(tqp_stats->rcb_rx_ring_pktnum);
+               tqp_stats->rcb_rx_ring_pktnum = NULL;
+               return -ENOMEM;
+       }
+
+       return 0;
+}
+
+void
+hns3_tqp_stats_uninit(struct hns3_hw *hw)
+{
+       struct hns3_tqp_stats *tqp_stats = &hw->tqp_stats;
+
+       rte_free(tqp_stats->rcb_rx_ring_pktnum);
+       tqp_stats->rcb_rx_ring_pktnum = NULL;
+       rte_free(tqp_stats->rcb_tx_ring_pktnum);
+       tqp_stats->rcb_tx_ring_pktnum = NULL;
+}
+
+static void
+hns3_tqp_stats_clear(struct hns3_hw *hw)
+{
+       struct hns3_tqp_stats *stats = &hw->tqp_stats;
+
+       stats->rcb_rx_ring_pktnum_rcd = 0;
+       stats->rcb_tx_ring_pktnum_rcd = 0;
+       memset(stats->rcb_rx_ring_pktnum, 0, sizeof(uint64_t) * hw->tqps_num);
+       memset(stats->rcb_tx_ring_pktnum, 0, sizeof(uint64_t) * hw->tqps_num);
+}