From: "Kumar, Ravi1" <Ravi1.Kumar@amd.com>
To: "Namburu, Chandu-babu" <chandu@amd.com>, "dev@dpdk.org" <dev@dpdk.org>
Subject: Re: [dpdk-dev] [PATCH v1] net/axgbe: enhance device stats reporting
Date: Fri, 10 Jan 2020 09:38:06 +0000 [thread overview]
Message-ID: <MW2PR12MB25703CF328BD3BB54E6D0E5EAE380@MW2PR12MB2570.namprd12.prod.outlook.com> (raw)
In-Reply-To: <MW2PR12MB25703D99A13013659323F806AE380@MW2PR12MB2570.namprd12.prod.outlook.com>
[AMD Official Use Only - Internal Distribution Only]
Looks good to me.
Reviewed-by: Ravi Kumar <ravi1.kumar@amd.com>
Acked-by: Ravi Kumar <ravi1.kumar@amd.com>
Regards,
Ravi
>
>-----Original Message-----
>From: dev <dev-bounces@dpdk.org> On Behalf Of cnamburu@amd.com
>Sent: Friday, December 20, 2019 6:13 PM
>To: dev@dpdk.org
>Subject: [dpdk-dev] [PATCH v1] net/axgbe: enhance device stats
>reporting
>
>[CAUTION: External Email]
>
>From: Chandu Babu N <chandu@amd.com>
>
>Implements eth dev ops xstats_get, xstats_reset, xstats_get_names,
>xstats_get_names_by_id, xstats_get_by_id
>
>Signed-off-by: Chandu Babu N <chandu@amd.com>
>---
> drivers/net/axgbe/axgbe_dev.c | 15 ++
> drivers/net/axgbe/axgbe_ethdev.c | 408 +++++++++++++++++++++++++++++++ drivers/net/axgbe/axgbe_ethdev.h | 49 ++++
> drivers/net/axgbe/axgbe_rxtx.c | 1 +
> drivers/net/axgbe/axgbe_rxtx.h | 1 +
> 5 files changed, 474 insertions(+)
>
>diff --git a/drivers/net/axgbe/axgbe_dev.c
>b/drivers/net/axgbe/axgbe_dev.c index b1f0bbc8e..83089f20d 100644
>--- a/drivers/net/axgbe/axgbe_dev.c
>+++ b/drivers/net/axgbe/axgbe_dev.c
>@@ -1036,6 +1036,20 @@ static void axgbe_config_checksum_offload(struct axgbe_port *pdata)
> AXGMAC_IOWRITE_BITS(pdata, MAC_RCR, IPC, 0); }
>
>+static void axgbe_config_mmc(struct axgbe_port *pdata) {
>+ struct axgbe_mmc_stats *stats = &pdata->mmc_stats;
>+
>+ /* Reset stats */
>+ memset(stats, 0, sizeof(*stats));
>+
>+ /* Set counters to reset on read */
>+ AXGMAC_IOWRITE_BITS(pdata, MMC_CR, ROR, 1);
>+
>+ /* Reset the counters */
>+ AXGMAC_IOWRITE_BITS(pdata, MMC_CR, CR, 1); }
>+
> static int axgbe_init(struct axgbe_port *pdata) {
> int ret;
>@@ -1078,6 +1092,7 @@ static int axgbe_init(struct axgbe_port *pdata)
> axgbe_config_flow_control(pdata);
> axgbe_config_mac_speed(pdata);
> axgbe_config_checksum_offload(pdata);
>+ axgbe_config_mmc(pdata);
>
> return 0;
> }
>diff --git a/drivers/net/axgbe/axgbe_ethdev.c
>b/drivers/net/axgbe/axgbe_ethdev.c
>index d1f160e79..750a768e4 100644
>--- a/drivers/net/axgbe/axgbe_ethdev.c
>+++ b/drivers/net/axgbe/axgbe_ethdev.c
>@@ -24,9 +24,79 @@ static int axgbe_dev_link_update(struct rte_eth_dev *dev, static int axgbe_dev_stats_get(struct rte_eth_dev *dev,
> struct rte_eth_stats *stats); static
>int axgbe_dev_stats_reset(struct rte_eth_dev *dev);
>+static int axgbe_dev_xstats_get(struct rte_eth_dev *dev,
>+ struct rte_eth_xstat *stats,
>+ unsigned int n); static int
>+axgbe_dev_xstats_get_names(struct rte_eth_dev *dev,
>+ struct rte_eth_xstat_name *xstats_names,
>+ unsigned int size); static int
>+axgbe_dev_xstats_get_by_id(struct rte_eth_dev *dev,
>+ const uint64_t *ids,
>+ uint64_t *values,
>+ unsigned int n); static int
>+axgbe_dev_xstats_get_names_by_id(struct rte_eth_dev *dev,
>+ struct rte_eth_xstat_name *xstats_names,
>+ const uint64_t *ids,
>+ unsigned int size); static int
>+axgbe_dev_xstats_reset(struct rte_eth_dev *dev);
> static int axgbe_dev_info_get(struct rte_eth_dev *dev,
> struct rte_eth_dev_info *dev_info);
>
>+struct axgbe_xstats {
>+ char name[RTE_ETH_XSTATS_NAME_SIZE];
>+ int offset;
>+};
>+
>+#define AXGMAC_MMC_STAT(_string, _var) \
>+ { _string, \
>+ offsetof(struct axgbe_mmc_stats, _var), \
>+ }
>+
>+static const struct axgbe_xstats axgbe_xstats_strings[] = {
>+ AXGMAC_MMC_STAT("tx_bytes", txoctetcount_gb),
>+ AXGMAC_MMC_STAT("tx_packets", txframecount_gb),
>+ AXGMAC_MMC_STAT("tx_unicast_packets", txunicastframes_gb),
>+ AXGMAC_MMC_STAT("tx_broadcast_packets", txbroadcastframes_gb),
>+ AXGMAC_MMC_STAT("tx_multicast_packets", txmulticastframes_gb),
>+ AXGMAC_MMC_STAT("tx_vlan_packets", txvlanframes_g),
>+ AXGMAC_MMC_STAT("tx_64_byte_packets", tx64octets_gb),
>+ AXGMAC_MMC_STAT("tx_65_to_127_byte_packets", tx65to127octets_gb),
>+ AXGMAC_MMC_STAT("tx_128_to_255_byte_packets", tx128to255octets_gb),
>+ AXGMAC_MMC_STAT("tx_256_to_511_byte_packets", tx256to511octets_gb),
>+ AXGMAC_MMC_STAT("tx_512_to_1023_byte_packets", tx512to1023octets_gb),
>+ AXGMAC_MMC_STAT("tx_1024_to_max_byte_packets", tx1024tomaxoctets_gb),
>+ AXGMAC_MMC_STAT("tx_underflow_errors", txunderflowerror),
>+ AXGMAC_MMC_STAT("tx_pause_frames", txpauseframes),
>+
>+ AXGMAC_MMC_STAT("rx_bytes", rxoctetcount_gb),
>+ AXGMAC_MMC_STAT("rx_packets", rxframecount_gb),
>+ AXGMAC_MMC_STAT("rx_unicast_packets", rxunicastframes_g),
>+ AXGMAC_MMC_STAT("rx_broadcast_packets", rxbroadcastframes_g),
>+ AXGMAC_MMC_STAT("rx_multicast_packets", rxmulticastframes_g),
>+ AXGMAC_MMC_STAT("rx_vlan_packets", rxvlanframes_gb),
>+ AXGMAC_MMC_STAT("rx_64_byte_packets", rx64octets_gb),
>+ AXGMAC_MMC_STAT("rx_65_to_127_byte_packets", rx65to127octets_gb),
>+ AXGMAC_MMC_STAT("rx_128_to_255_byte_packets", rx128to255octets_gb),
>+ AXGMAC_MMC_STAT("rx_256_to_511_byte_packets", rx256to511octets_gb),
>+ AXGMAC_MMC_STAT("rx_512_to_1023_byte_packets", rx512to1023octets_gb),
>+ AXGMAC_MMC_STAT("rx_1024_to_max_byte_packets", rx1024tomaxoctets_gb),
>+ AXGMAC_MMC_STAT("rx_undersize_packets", rxundersize_g),
>+ AXGMAC_MMC_STAT("rx_oversize_packets", rxoversize_g),
>+ AXGMAC_MMC_STAT("rx_crc_errors", rxcrcerror),
>+ AXGMAC_MMC_STAT("rx_crc_errors_small_packets", rxrunterror),
>+ AXGMAC_MMC_STAT("rx_crc_errors_giant_packets", rxjabbererror),
>+ AXGMAC_MMC_STAT("rx_length_errors", rxlengtherror),
>+ AXGMAC_MMC_STAT("rx_out_of_range_errors", rxoutofrangetype),
>+ AXGMAC_MMC_STAT("rx_fifo_overflow_errors", rxfifooverflow),
>+ AXGMAC_MMC_STAT("rx_watchdog_errors", rxwatchdogerror),
>+ AXGMAC_MMC_STAT("rx_pause_frames", rxpauseframes), };
>+
>+#define AXGBE_XSTATS_COUNT ARRAY_SIZE(axgbe_xstats_strings)
>+
> /* The set of PCI devices this driver supports */
> #define AMD_PCI_VENDOR_ID 0x1022
> #define AMD_PCI_AXGBE_DEVICE_V2A 0x1458 @@ -89,6 +159,11 @@ static const struct eth_dev_ops axgbe_eth_dev_ops = {
> .link_update = axgbe_dev_link_update,
> .stats_get = axgbe_dev_stats_get,
> .stats_reset = axgbe_dev_stats_reset,
>+ .xstats_get = axgbe_dev_xstats_get,
>+ .xstats_reset = axgbe_dev_xstats_reset,
>+ .xstats_get_names = axgbe_dev_xstats_get_names,
>+ .xstats_get_names_by_id = axgbe_dev_xstats_get_names_by_id,
>+ .xstats_get_by_id = axgbe_dev_xstats_get_by_id,
> .dev_infos_get = axgbe_dev_info_get,
> .rx_queue_setup = axgbe_dev_rx_queue_setup,
> .rx_queue_release = axgbe_dev_rx_queue_release,
>@@ -315,27 +390,359 @@ axgbe_dev_link_update(struct rte_eth_dev *dev,
> return ret;
> }
>
>+static void axgbe_read_mmc_stats(struct axgbe_port *pdata) {
>+ struct axgbe_mmc_stats *stats = &pdata->mmc_stats;
>+
>+ /* Freeze counters */
>+ AXGMAC_IOWRITE_BITS(pdata, MMC_CR, MCF, 1);
>+
>+ /* Tx counters */
>+ stats->txoctetcount_gb +=
>+ AXGMAC_IOREAD(pdata, MMC_TXOCTETCOUNT_GB_LO);
>+ stats->txoctetcount_gb +=
>+ ((uint64_t)AXGMAC_IOREAD(pdata, MMC_TXOCTETCOUNT_GB_HI) << 32);
>+
>+ stats->txframecount_gb +=
>+ AXGMAC_IOREAD(pdata, MMC_TXFRAMECOUNT_GB_LO);
>+ stats->txframecount_gb +=
>+ ((uint64_t)AXGMAC_IOREAD(pdata, MMC_TXFRAMECOUNT_GB_HI) << 32);
>+
>+ stats->txbroadcastframes_g +=
>+ AXGMAC_IOREAD(pdata, MMC_TXBROADCASTFRAMES_G_LO);
>+ stats->txbroadcastframes_g +=
>+ ((uint64_t)AXGMAC_IOREAD(pdata, MMC_TXBROADCASTFRAMES_G_HI) <<
>+ 32);
>+
>+ stats->txmulticastframes_g +=
>+ AXGMAC_IOREAD(pdata, MMC_TXMULTICASTFRAMES_G_LO);
>+ stats->txmulticastframes_g +=
>+ ((uint64_t)AXGMAC_IOREAD(pdata, MMC_TXMULTICASTFRAMES_G_HI) <<
>+ 32);
>+
>+ stats->tx64octets_gb +=
>+ AXGMAC_IOREAD(pdata, MMC_TX64OCTETS_GB_LO);
>+ stats->tx64octets_gb +=
>+ ((uint64_t)AXGMAC_IOREAD(pdata, MMC_TX64OCTETS_GB_HI) << 32);
>+
>+ stats->tx65to127octets_gb +=
>+ AXGMAC_IOREAD(pdata, MMC_TX65TO127OCTETS_GB_LO);
>+ stats->tx65to127octets_gb +=
>+ ((uint64_t)AXGMAC_IOREAD(pdata, MMC_TX65TO127OCTETS_GB_HI) <<
>+ 32);
>+
>+ stats->tx128to255octets_gb +=
>+ AXGMAC_IOREAD(pdata, MMC_TX128TO255OCTETS_GB_LO);
>+ stats->tx128to255octets_gb +=
>+ ((uint64_t)AXGMAC_IOREAD(pdata, MMC_TX128TO255OCTETS_GB_HI) <<
>+ 32);
>+
>+ stats->tx256to511octets_gb +=
>+ AXGMAC_IOREAD(pdata, MMC_TX256TO511OCTETS_GB_LO);
>+ stats->tx256to511octets_gb +=
>+ ((uint64_t)AXGMAC_IOREAD(pdata, MMC_TX256TO511OCTETS_GB_HI) <<
>+ 32);
>+
>+ stats->tx512to1023octets_gb +=
>+ AXGMAC_IOREAD(pdata, MMC_TX512TO1023OCTETS_GB_LO);
>+ stats->tx512to1023octets_gb +=
>+ ((uint64_t)AXGMAC_IOREAD(pdata, MMC_TX512TO1023OCTETS_GB_HI) <<
>+ 32);
>+
>+ stats->tx1024tomaxoctets_gb +=
>+ AXGMAC_IOREAD(pdata, MMC_TX1024TOMAXOCTETS_GB_LO);
>+ stats->tx1024tomaxoctets_gb +=
>+ ((uint64_t)AXGMAC_IOREAD(pdata, MMC_TX1024TOMAXOCTETS_GB_HI) <<
>+ 32);
>+
>+ stats->txunicastframes_gb +=
>+ AXGMAC_IOREAD(pdata, MMC_TXUNICASTFRAMES_GB_LO);
>+ stats->txunicastframes_gb +=
>+ ((uint64_t)AXGMAC_IOREAD(pdata, MMC_TXUNICASTFRAMES_GB_HI) <<
>+ 32);
>+
>+ stats->txmulticastframes_gb +=
>+ AXGMAC_IOREAD(pdata, MMC_TXMULTICASTFRAMES_GB_LO);
>+ stats->txmulticastframes_gb +=
>+ ((uint64_t)AXGMAC_IOREAD(pdata, MMC_TXMULTICASTFRAMES_GB_HI) <<
>+ 32);
>+
>+ stats->txbroadcastframes_g +=
>+ AXGMAC_IOREAD(pdata, MMC_TXBROADCASTFRAMES_GB_LO);
>+ stats->txbroadcastframes_g +=
>+ ((uint64_t)AXGMAC_IOREAD(pdata, MMC_TXBROADCASTFRAMES_GB_HI) <<
>+ 32);
>+
>+ stats->txunderflowerror +=
>+ AXGMAC_IOREAD(pdata, MMC_TXUNDERFLOWERROR_LO);
>+ stats->txunderflowerror +=
>+ ((uint64_t)AXGMAC_IOREAD(pdata, MMC_TXUNDERFLOWERROR_HI) <<
>+ 32);
>+
>+ stats->txoctetcount_g +=
>+ AXGMAC_IOREAD(pdata, MMC_TXOCTETCOUNT_G_LO);
>+ stats->txoctetcount_g +=
>+ ((uint64_t)AXGMAC_IOREAD(pdata, MMC_TXOCTETCOUNT_G_HI) << 32);
>+
>+ stats->txframecount_g +=
>+ AXGMAC_IOREAD(pdata, MMC_TXFRAMECOUNT_G_LO);
>+ stats->txframecount_g +=
>+ ((uint64_t)AXGMAC_IOREAD(pdata, MMC_TXFRAMECOUNT_G_HI) << 32);
>+
>+ stats->txpauseframes +=
>+ AXGMAC_IOREAD(pdata, MMC_TXPAUSEFRAMES_LO);
>+ stats->txpauseframes +=
>+ ((uint64_t)AXGMAC_IOREAD(pdata, MMC_TXPAUSEFRAMES_HI) << 32);
>+
>+ stats->txvlanframes_g +=
>+ AXGMAC_IOREAD(pdata, MMC_TXVLANFRAMES_G_LO);
>+ stats->txvlanframes_g +=
>+ ((uint64_t)AXGMAC_IOREAD(pdata, MMC_TXVLANFRAMES_G_HI) << 32);
>+
>+ /* Rx counters */
>+ stats->rxframecount_gb +=
>+ AXGMAC_IOREAD(pdata, MMC_RXFRAMECOUNT_GB_LO);
>+ stats->rxframecount_gb +=
>+ ((uint64_t)AXGMAC_IOREAD(pdata, MMC_RXFRAMECOUNT_GB_HI) << 32);
>+
>+ stats->rxoctetcount_gb +=
>+ AXGMAC_IOREAD(pdata, MMC_RXOCTETCOUNT_GB_LO);
>+ stats->rxoctetcount_gb +=
>+ ((uint64_t)AXGMAC_IOREAD(pdata, MMC_RXOCTETCOUNT_GB_HI) << 32);
>+
>+ stats->rxoctetcount_g +=
>+ AXGMAC_IOREAD(pdata, MMC_RXOCTETCOUNT_G_LO);
>+ stats->rxoctetcount_g +=
>+ ((uint64_t)AXGMAC_IOREAD(pdata, MMC_RXOCTETCOUNT_G_HI) << 32);
>+
>+ stats->rxbroadcastframes_g +=
>+ AXGMAC_IOREAD(pdata, MMC_RXBROADCASTFRAMES_G_LO);
>+ stats->rxbroadcastframes_g +=
>+ ((uint64_t)AXGMAC_IOREAD(pdata, MMC_RXBROADCASTFRAMES_G_HI) <<
>+ 32);
>+
>+ stats->rxmulticastframes_g +=
>+ AXGMAC_IOREAD(pdata, MMC_RXMULTICASTFRAMES_G_LO);
>+ stats->rxmulticastframes_g +=
>+ ((uint64_t)AXGMAC_IOREAD(pdata, MMC_RXMULTICASTFRAMES_G_HI) <<
>+ 32);
>+
>+ stats->rxcrcerror +=
>+ AXGMAC_IOREAD(pdata, MMC_RXCRCERROR_LO);
>+ stats->rxcrcerror +=
>+ ((uint64_t)AXGMAC_IOREAD(pdata, MMC_RXCRCERROR_HI) << 32);
>+
>+ stats->rxrunterror +=
>+ AXGMAC_IOREAD(pdata, MMC_RXRUNTERROR);
>+
>+ stats->rxjabbererror +=
>+ AXGMAC_IOREAD(pdata, MMC_RXJABBERERROR);
>+
>+ stats->rxundersize_g +=
>+ AXGMAC_IOREAD(pdata, MMC_RXUNDERSIZE_G);
>+
>+ stats->rxoversize_g +=
>+ AXGMAC_IOREAD(pdata, MMC_RXOVERSIZE_G);
>+
>+ stats->rx64octets_gb +=
>+ AXGMAC_IOREAD(pdata, MMC_RX64OCTETS_GB_LO);
>+ stats->rx64octets_gb +=
>+ ((uint64_t)AXGMAC_IOREAD(pdata, MMC_RX64OCTETS_GB_HI) << 32);
>+
>+ stats->rx65to127octets_gb +=
>+ AXGMAC_IOREAD(pdata, MMC_RX65TO127OCTETS_GB_LO);
>+ stats->rx65to127octets_gb +=
>+ ((uint64_t)AXGMAC_IOREAD(pdata, MMC_RX65TO127OCTETS_GB_HI) <<
>+ 32);
>+
>+ stats->rx128to255octets_gb +=
>+ AXGMAC_IOREAD(pdata, MMC_RX128TO255OCTETS_GB_LO);
>+ stats->rx128to255octets_gb +=
>+ ((uint64_t)AXGMAC_IOREAD(pdata, MMC_RX128TO255OCTETS_GB_HI) <<
>+ 32);
>+
>+ stats->rx256to511octets_gb +=
>+ AXGMAC_IOREAD(pdata, MMC_RX256TO511OCTETS_GB_LO);
>+ stats->rx256to511octets_gb +=
>+ ((uint64_t)AXGMAC_IOREAD(pdata, MMC_RX256TO511OCTETS_GB_HI) <<
>+ 32);
>+
>+ stats->rx512to1023octets_gb +=
>+ AXGMAC_IOREAD(pdata, MMC_RX512TO1023OCTETS_GB_LO);
>+ stats->rx512to1023octets_gb +=
>+ ((uint64_t)AXGMAC_IOREAD(pdata, MMC_RX512TO1023OCTETS_GB_HI) <<
>+ 32);
>+
>+ stats->rx1024tomaxoctets_gb +=
>+ AXGMAC_IOREAD(pdata, MMC_RX1024TOMAXOCTETS_GB_LO);
>+ stats->rx1024tomaxoctets_gb +=
>+ ((uint64_t)AXGMAC_IOREAD(pdata, MMC_RX1024TOMAXOCTETS_GB_HI) <<
>+ 32);
>+
>+ stats->rxunicastframes_g +=
>+ AXGMAC_IOREAD(pdata, MMC_RXUNICASTFRAMES_G_LO);
>+ stats->rxunicastframes_g +=
>+ ((uint64_t)AXGMAC_IOREAD(pdata, MMC_RXUNICASTFRAMES_G_HI) <<
>+ 32);
>+
>+ stats->rxlengtherror +=
>+ AXGMAC_IOREAD(pdata, MMC_RXLENGTHERROR_LO);
>+ stats->rxlengtherror +=
>+ ((uint64_t)AXGMAC_IOREAD(pdata, MMC_RXLENGTHERROR_HI) << 32);
>+
>+ stats->rxoutofrangetype +=
>+ AXGMAC_IOREAD(pdata, MMC_RXOUTOFRANGETYPE_LO);
>+ stats->rxoutofrangetype +=
>+ ((uint64_t)AXGMAC_IOREAD(pdata, MMC_RXOUTOFRANGETYPE_HI) <<
>+ 32);
>+
>+ stats->rxpauseframes +=
>+ AXGMAC_IOREAD(pdata, MMC_RXPAUSEFRAMES_LO);
>+ stats->rxpauseframes +=
>+ ((uint64_t)AXGMAC_IOREAD(pdata, MMC_RXPAUSEFRAMES_HI) << 32);
>+
>+ stats->rxfifooverflow +=
>+ AXGMAC_IOREAD(pdata, MMC_RXFIFOOVERFLOW_LO);
>+ stats->rxfifooverflow +=
>+ ((uint64_t)AXGMAC_IOREAD(pdata, MMC_RXFIFOOVERFLOW_HI) << 32);
>+
>+ stats->rxvlanframes_gb +=
>+ AXGMAC_IOREAD(pdata, MMC_RXVLANFRAMES_GB_LO);
>+ stats->rxvlanframes_gb +=
>+ ((uint64_t)AXGMAC_IOREAD(pdata, MMC_RXVLANFRAMES_GB_HI) << 32);
>+
>+ stats->rxwatchdogerror +=
>+ AXGMAC_IOREAD(pdata, MMC_RXWATCHDOGERROR);
>+
>+ /* Un-freeze counters */
>+ AXGMAC_IOWRITE_BITS(pdata, MMC_CR, MCF, 0); }
>+
>+static int
>+axgbe_dev_xstats_get(struct rte_eth_dev *dev, struct rte_eth_xstat *stats,
>+ unsigned int n)
>+{
>+ struct axgbe_port *pdata = dev->data->dev_private;
>+ unsigned int i;
>+
>+ if (!stats)
>+ return 0;
>+
>+ axgbe_read_mmc_stats(pdata);
>+
>+ for (i = 0; i < n && i < AXGBE_XSTATS_COUNT; i++) {
>+ stats[i].id = i;
>+ stats[i].value = *(u64 *)((uint8_t *)&pdata->mmc_stats +
>+ axgbe_xstats_strings[i].offset);
>+ }
>+
>+ return i;
>+}
>+
>+static int
>+axgbe_dev_xstats_get_names(__rte_unused struct rte_eth_dev *dev,
>+ struct rte_eth_xstat_name *xstats_names,
>+ unsigned int n) {
>+ unsigned int i;
>+
>+ if (n >= AXGBE_XSTATS_COUNT && xstats_names) {
>+ for (i = 0; i < AXGBE_XSTATS_COUNT; ++i) {
>+ snprintf(xstats_names[i].name,
>+ RTE_ETH_XSTATS_NAME_SIZE, "%s",
>+ axgbe_xstats_strings[i].name);
>+ }
>+ }
>+
>+ return AXGBE_XSTATS_COUNT;
>+}
>+
>+static int
>+axgbe_dev_xstats_get_by_id(struct rte_eth_dev *dev, const uint64_t *ids,
>+ uint64_t *values, unsigned int n) {
>+ unsigned int i;
>+ uint64_t values_copy[AXGBE_XSTATS_COUNT];
>+
>+ if (!ids) {
>+ struct axgbe_port *pdata = dev->data->dev_private;
>+
>+ if (n < AXGBE_XSTATS_COUNT)
>+ return AXGBE_XSTATS_COUNT;
>+
>+ axgbe_read_mmc_stats(pdata);
>+
>+ for (i = 0; i < AXGBE_XSTATS_COUNT; i++) {
>+ values[i] = *(u64 *)((uint8_t *)&pdata->mmc_stats +
>+ axgbe_xstats_strings[i].offset);
>+ }
>+
>+ return i;
>+ }
>+
>+ axgbe_dev_xstats_get_by_id(dev, NULL, values_copy,
>+ AXGBE_XSTATS_COUNT);
>+
>+ for (i = 0; i < n; i++) {
>+ if (ids[i] >= AXGBE_XSTATS_COUNT) {
>+ PMD_DRV_LOG(ERR, "id value isn't valid\n");
>+ return -1;
>+ }
>+ values[i] = values_copy[ids[i]];
>+ }
>+ return n;
>+}
>+
>+static int
>+axgbe_dev_xstats_get_names_by_id(struct rte_eth_dev *dev,
>+ struct rte_eth_xstat_name *xstats_names,
>+ const uint64_t *ids,
>+ unsigned int size) {
>+ struct rte_eth_xstat_name xstats_names_copy[AXGBE_XSTATS_COUNT];
>+ unsigned int i;
>+
>+ if (!ids)
>+ return axgbe_dev_xstats_get_names(dev, xstats_names,
>+ size);
>+
>+ axgbe_dev_xstats_get_names(dev, xstats_names_copy, size);
>+
>+ for (i = 0; i < size; i++) {
>+ if (ids[i] >= AXGBE_XSTATS_COUNT) {
>+ PMD_DRV_LOG(ERR, "id value isn't valid\n");
>+ return -1;
>+ }
>+ strcpy(xstats_names[i].name, xstats_names_copy[ids[i]].name);
>+ }
>+ return size;
>+}
>+
>+static int
>+axgbe_dev_xstats_reset(struct rte_eth_dev *dev) {
>+ struct axgbe_port *pdata = dev->data->dev_private;
>+ struct axgbe_mmc_stats *stats = &pdata->mmc_stats;
>+
>+ /* MMC registers are configured for reset on read */
>+ axgbe_read_mmc_stats(pdata);
>+
>+ /* Reset stats */
>+ memset(stats, 0, sizeof(*stats));
>+
>+ return 0;
>+}
>+
> static int
> axgbe_dev_stats_get(struct rte_eth_dev *dev,
> struct rte_eth_stats *stats) {
> struct axgbe_rx_queue *rxq;
> struct axgbe_tx_queue *txq;
>+ struct axgbe_port *pdata = dev->data->dev_private;
>+ struct axgbe_mmc_stats *mmc_stats = &pdata->mmc_stats;
> unsigned int i;
>
>+ axgbe_read_mmc_stats(pdata);
>+
>+ stats->imissed = mmc_stats->rxfifooverflow;
>+
> for (i = 0; i < dev->data->nb_rx_queues; i++) {
> rxq = dev->data->rx_queues[i];
> stats->q_ipackets[i] = rxq->pkts;
> stats->ipackets += rxq->pkts;
> stats->q_ibytes[i] = rxq->bytes;
> stats->ibytes += rxq->bytes;
>+ stats->rx_nombuf += rxq->rx_mbuf_alloc_failed;
>+ stats->q_errors[i] = rxq->errors + rxq->rx_mbuf_alloc_failed;
>+ stats->ierrors += rxq->errors;
> }
>+
> for (i = 0; i < dev->data->nb_tx_queues; i++) {
> txq = dev->data->tx_queues[i];
> stats->q_opackets[i] = txq->pkts;
> stats->opackets += txq->pkts;
> stats->q_obytes[i] = txq->bytes;
> stats->obytes += txq->bytes;
>+ stats->oerrors += txq->errors;
> }
>
> return 0;
>@@ -353,6 +760,7 @@ axgbe_dev_stats_reset(struct rte_eth_dev *dev)
> rxq->pkts = 0;
> rxq->bytes = 0;
> rxq->errors = 0;
>+ rxq->rx_mbuf_alloc_failed = 0;
> }
> for (i = 0; i < dev->data->nb_tx_queues; i++) {
> txq = dev->data->tx_queues[i]; diff --git
>a/drivers/net/axgbe/axgbe_ethdev.h b/drivers/net/axgbe/axgbe_ethdev.h
>index e3cfaf36f..a1083b17b 100644
>--- a/drivers/net/axgbe/axgbe_ethdev.h
>+++ b/drivers/net/axgbe/axgbe_ethdev.h
>@@ -438,6 +438,53 @@ struct axgbe_version_data {
> unsigned int an_cdr_workaround; };
>
>+struct axgbe_mmc_stats {
>+ /* Tx Stats */
>+ uint64_t txoctetcount_gb;
>+ uint64_t txframecount_gb;
>+ uint64_t txbroadcastframes_g;
>+ uint64_t txmulticastframes_g;
>+ uint64_t tx64octets_gb;
>+ uint64_t tx65to127octets_gb;
>+ uint64_t tx128to255octets_gb;
>+ uint64_t tx256to511octets_gb;
>+ uint64_t tx512to1023octets_gb;
>+ uint64_t tx1024tomaxoctets_gb;
>+ uint64_t txunicastframes_gb;
>+ uint64_t txmulticastframes_gb;
>+ uint64_t txbroadcastframes_gb;
>+ uint64_t txunderflowerror;
>+ uint64_t txoctetcount_g;
>+ uint64_t txframecount_g;
>+ uint64_t txpauseframes;
>+ uint64_t txvlanframes_g;
>+
>+ /* Rx Stats */
>+ uint64_t rxframecount_gb;
>+ uint64_t rxoctetcount_gb;
>+ uint64_t rxoctetcount_g;
>+ uint64_t rxbroadcastframes_g;
>+ uint64_t rxmulticastframes_g;
>+ uint64_t rxcrcerror;
>+ uint64_t rxrunterror;
>+ uint64_t rxjabbererror;
>+ uint64_t rxundersize_g;
>+ uint64_t rxoversize_g;
>+ uint64_t rx64octets_gb;
>+ uint64_t rx65to127octets_gb;
>+ uint64_t rx128to255octets_gb;
>+ uint64_t rx256to511octets_gb;
>+ uint64_t rx512to1023octets_gb;
>+ uint64_t rx1024tomaxoctets_gb;
>+ uint64_t rxunicastframes_g;
>+ uint64_t rxlengtherror;
>+ uint64_t rxoutofrangetype;
>+ uint64_t rxpauseframes;
>+ uint64_t rxfifooverflow;
>+ uint64_t rxvlanframes_gb;
>+ uint64_t rxwatchdogerror;
>+};
>+
> /*
> * Structure to store private data for each port.
> */
>@@ -576,6 +623,8 @@ struct axgbe_port {
> int crc_strip_enable;
> /* csum enable to hardware */
> uint32_t rx_csum_enable;
>+
>+ struct axgbe_mmc_stats mmc_stats;
> };
>
> void axgbe_init_function_ptrs_dev(struct axgbe_hw_if *hw_if); diff
>--git a/drivers/net/axgbe/axgbe_rxtx.c b/drivers/net/axgbe/axgbe_rxtx.c
>index cfc55b359..96055c25b 100644
>--- a/drivers/net/axgbe/axgbe_rxtx.c
>+++ b/drivers/net/axgbe/axgbe_rxtx.c
>@@ -229,6 +229,7 @@ axgbe_recv_pkts(void *rx_queue, struct rte_mbuf **rx_pkts,
> (unsigned int)rxq->queue_id);
> rte_eth_devices[
>
>rxq->port_id].data->rx_mbuf_alloc_failed++;
>+ rxq->rx_mbuf_alloc_failed++;
> break;
> }
> pidx = idx + 1;
>diff --git a/drivers/net/axgbe/axgbe_rxtx.h
>b/drivers/net/axgbe/axgbe_rxtx.h index 917da58ce..a21537df9 100644
>--- a/drivers/net/axgbe/axgbe_rxtx.h
>+++ b/drivers/net/axgbe/axgbe_rxtx.h
>@@ -91,6 +91,7 @@ struct axgbe_rx_queue {
> uint64_t pkts;
> uint64_t bytes;
> uint64_t errors;
>+ uint64_t rx_mbuf_alloc_failed;
> /* Number of mbufs allocated from pool*/
> uint64_t mbuf_alloc;
>
>--
>2.17.1
next prev parent reply other threads:[~2020-01-10 9:38 UTC|newest]
Thread overview: 6+ messages / expand[flat|nested] mbox.gz Atom feed top
2019-12-20 12:43 cnamburu
2020-01-09 14:57 ` Ferruh Yigit
2020-01-10 9:11 ` Kumar, Ravi1
2020-01-10 9:09 ` Kumar, Ravi1
2020-01-10 9:38 ` Kumar, Ravi1 [this message]
2020-01-10 13:23 ` Ferruh Yigit
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=MW2PR12MB25703CF328BD3BB54E6D0E5EAE380@MW2PR12MB2570.namprd12.prod.outlook.com \
--to=ravi1.kumar@amd.com \
--cc=chandu@amd.com \
--cc=dev@dpdk.org \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).