mbuf: extend meaning of QinQ stripped bit

[dpdk.git] / lib / librte_bitratestats / rte_bitrate.c

diff --git a/lib/librte_bitratestats/rte_bitrate.c b/lib/librte_bitratestats/rte_bitrate.c
index 193aa69..d181523 100644 (file)
--- a/lib/librte_bitratestats/rte_bitrate.c
+++ b/lib/librte_bitratestats/rte_bitrate.c
@@ -1,34 +1,5 @@
-/*-
- *   BSD LICENSE
- *
- *   Copyright(c) 2017 Intel Corporation. All rights reserved.
- *   All rights reserved.
- *
- *   Redistribution and use in source and binary forms, with or without
- *   modification, are permitted provided that the following conditions
- *   are met:
- *
- *     * Redistributions of source code must retain the above copyright
- *       notice, this list of conditions and the following disclaimer.
- *     * Redistributions in binary form must reproduce the above copyright
- *       notice, this list of conditions and the following disclaimer in
- *       the documentation and/or other materials provided with the
- *       distribution.
- *     * Neither the name of Intel Corporation nor the names of its
- *       contributors may be used to endorse or promote products derived
- *       from this software without specific prior written permission.
- *
- *   THIS SOFTWARE IS PROVIDED BY THE COPYRIGHT HOLDERS AND CONTRIBUTORS
- *   "AS IS" AND ANY EXPRESS OR IMPLIED WARRANTIES, INCLUDING, BUT NOT
- *   LIMITED TO, THE IMPLIED WARRANTIES OF MERCHANTABILITY AND FITNESS FOR
- *   A PARTICULAR PURPOSE ARE DISCLAIMED. IN NO EVENT SHALL THE COPYRIGHT
- *   OWNER OR CONTRIBUTORS BE LIABLE FOR ANY DIRECT, INDIRECT, INCIDENTAL,
- *   SPECIAL, EXEMPLARY, OR CONSEQUENTIAL DAMAGES (INCLUDING, BUT NOT
- *   LIMITED TO, PROCUREMENT OF SUBSTITUTE GOODS OR SERVICES; LOSS OF USE,
- *   DATA, OR PROFITS; OR BUSINESS INTERRUPTION) HOWEVER CAUSED AND ON ANY
- *   THEORY OF LIABILITY, WHETHER IN CONTRACT, STRICT LIABILITY, OR TORT
- *   (INCLUDING NEGLIGENCE OR OTHERWISE) ARISING IN ANY WAY OUT OF THE USE
- *   OF THIS SOFTWARE, EVEN IF ADVISED OF THE POSSIBILITY OF SUCH DAMAGE.
+/* SPDX-License-Identifier: BSD-3-Clause
+ * Copyright(c) 2017 Intel Corporation
  */
 
 #include <rte_common.h>
@@ -37,8 +8,6 @@
 #include <rte_metrics.h>
 #include <rte_bitrate.h>
 
-#define ARRAY_SIZE(x) (sizeof(x) / sizeof((x)[0]))
-
 /*
  * Persistent bit-rate data.
  * @internal
@@ -76,7 +45,10 @@ rte_stats_bitrate_reg(struct rte_stats_bitrates *bitrate_data)
        };
        int return_value;
 
-       return_value = rte_metrics_reg_names(&names[0], ARRAY_SIZE(names));
+       if (bitrate_data == NULL)
+               return -EINVAL;
+
+       return_value = rte_metrics_reg_names(&names[0], RTE_DIM(names));
        if (return_value >= 0)
                bitrate_data->id_stats_set = return_value;
        return return_value;
@@ -84,7 +56,7 @@ rte_stats_bitrate_reg(struct rte_stats_bitrates *bitrate_data)
 
 int
 rte_stats_bitrate_calc(struct rte_stats_bitrates *bitrate_data,
-       uint8_t port_id)
+                       uint16_t port_id)
 {
        struct rte_stats_bitrate *port_data;
        struct rte_eth_stats eth_stats;
@@ -93,6 +65,10 @@ rte_stats_bitrate_calc(struct rte_stats_bitrates *bitrate_data,
        int64_t delta;
        const int64_t alpha_percent = 20;
        uint64_t values[6];
+       int ret;
+
+       if (bitrate_data == NULL)
+               return -EINVAL;
 
        ret_code = rte_eth_stats_get(port_id, &eth_stats);
        if (ret_code != 0)
@@ -112,7 +88,7 @@ rte_stats_bitrate_calc(struct rte_stats_bitrates *bitrate_data,
                port_data->peak_ibits = cnt_bits;
        delta = cnt_bits;
        delta -= port_data->ewma_ibits;
-       /* The +-50 fixes integer rounding during divison */
+       /* The +-50 fixes integer rounding during division */
        if (delta > 0)
                delta = (delta * alpha_percent + 50) / 100;
        else
@@ -147,7 +123,10 @@ rte_stats_bitrate_calc(struct rte_stats_bitrates *bitrate_data,
        values[3] = port_data->mean_obits;
        values[4] = port_data->peak_ibits;
        values[5] = port_data->peak_obits;
-       rte_metrics_update_values(port_id, bitrate_data->id_stats_set,
-               values, ARRAY_SIZE(values));
+       ret = rte_metrics_update_values(port_id, bitrate_data->id_stats_set,
+               values, RTE_DIM(values));
+       if (ret < 0)
+               return ret;
+
        return 0;
 }