]> git.ipfire.org Git - thirdparty/kernel/linux.git/commitdiff
net: mana: Expose additional hardware counters for drop and TC via ethtool.
authorDipayaan Roy <dipayanroy@linux.microsoft.com>
Mon, 9 Jun 2025 10:01:03 +0000 (03:01 -0700)
committerJakub Kicinski <kuba@kernel.org>
Tue, 10 Jun 2025 21:23:19 +0000 (14:23 -0700)
Add support for reporting additional hardware counters for drop and
TC using the ethtool -S interface.

These counters include:

- Aggregate Rx/Tx drop counters
- Per-TC Rx/Tx packet counters
- Per-TC Rx/Tx byte counters
- Per-TC Rx/Tx pause frame counters

The counters are exposed using ethtool_ops->get_ethtool_stats and
ethtool_ops->get_strings. This feature/counters are not available
to all versions of hardware.

Signed-off-by: Dipayaan Roy <dipayanroy@linux.microsoft.com>
Reviewed-by: Subbaraya Sundeep <sbhatta@marvell.com>
Reviewed-by: Haiyang Zhang <haiyangz@microsoft.com>
Link: https://patch.msgid.link/20250609100103.GA7102@linuxonhyperv3.guj3yctzbm1etfxqx2vob5hsef.xx.internal.cloudapp.net
Signed-off-by: Jakub Kicinski <kuba@kernel.org>
drivers/net/ethernet/microsoft/mana/hw_channel.c
drivers/net/ethernet/microsoft/mana/mana_en.c
drivers/net/ethernet/microsoft/mana/mana_ethtool.c
include/net/mana/mana.h

index a8c4d8db75a563f86657183dce24b3df6b6b48e5..3d3677c0d01477d1924ff4f21f602cd33077a8da 100644 (file)
@@ -2,6 +2,7 @@
 /* Copyright (c) 2021, Microsoft Corporation. */
 
 #include <net/mana/gdma.h>
+#include <net/mana/mana.h>
 #include <net/mana/hw_channel.h>
 #include <linux/vmalloc.h>
 
@@ -890,8 +891,9 @@ int mana_hwc_send_request(struct hw_channel_context *hwc, u32 req_len,
        }
 
        if (ctx->status_code && ctx->status_code != GDMA_STATUS_MORE_ENTRIES) {
-               dev_err(hwc->dev, "HWC: Failed hw_channel req: 0x%x\n",
-                       ctx->status_code);
+               if (req_msg->req.msg_type != MANA_QUERY_PHY_STAT)
+                       dev_err(hwc->dev, "HWC: Failed hw_channel req: 0x%x\n",
+                               ctx->status_code);
                err = -EPROTO;
                goto out;
        }
index ccd2885c939e0fcf40b5e8d85f6327c85cc78864..e68b8190bb7a83827a3c86bdc36344f400be2034 100644 (file)
@@ -774,8 +774,9 @@ static int mana_send_request(struct mana_context *ac, void *in_buf,
        err = mana_gd_send_request(gc, in_len, in_buf, out_len,
                                   out_buf);
        if (err || resp->status) {
-               dev_err(dev, "Failed to send mana message: %d, 0x%x\n",
-                       err, resp->status);
+               if (req->req.msg_type != MANA_QUERY_PHY_STAT)
+                       dev_err(dev, "Failed to send mana message: %d, 0x%x\n",
+                               err, resp->status);
                return err ? err : -EPROTO;
        }
 
@@ -2611,6 +2612,88 @@ void mana_query_gf_stats(struct mana_port_context *apc)
        apc->eth_stats.hc_tx_err_gdma = resp.tx_err_gdma;
 }
 
+void mana_query_phy_stats(struct mana_port_context *apc)
+{
+       struct mana_query_phy_stat_resp resp = {};
+       struct mana_query_phy_stat_req req = {};
+       struct net_device *ndev = apc->ndev;
+       int err;
+
+       mana_gd_init_req_hdr(&req.hdr, MANA_QUERY_PHY_STAT,
+                            sizeof(req), sizeof(resp));
+       err = mana_send_request(apc->ac, &req, sizeof(req), &resp,
+                               sizeof(resp));
+       if (err)
+               return;
+
+       err = mana_verify_resp_hdr(&resp.hdr, MANA_QUERY_PHY_STAT,
+                                  sizeof(resp));
+       if (err || resp.hdr.status) {
+               netdev_err(ndev,
+                          "Failed to query PHY stats: %d, resp:0x%x\n",
+                               err, resp.hdr.status);
+               return;
+       }
+
+       /* Aggregate drop counters */
+       apc->phy_stats.rx_pkt_drop_phy = resp.rx_pkt_drop_phy;
+       apc->phy_stats.tx_pkt_drop_phy = resp.tx_pkt_drop_phy;
+
+       /* Per TC traffic Counters */
+       apc->phy_stats.rx_pkt_tc0_phy = resp.rx_pkt_tc0_phy;
+       apc->phy_stats.tx_pkt_tc0_phy = resp.tx_pkt_tc0_phy;
+       apc->phy_stats.rx_pkt_tc1_phy = resp.rx_pkt_tc1_phy;
+       apc->phy_stats.tx_pkt_tc1_phy = resp.tx_pkt_tc1_phy;
+       apc->phy_stats.rx_pkt_tc2_phy = resp.rx_pkt_tc2_phy;
+       apc->phy_stats.tx_pkt_tc2_phy = resp.tx_pkt_tc2_phy;
+       apc->phy_stats.rx_pkt_tc3_phy = resp.rx_pkt_tc3_phy;
+       apc->phy_stats.tx_pkt_tc3_phy = resp.tx_pkt_tc3_phy;
+       apc->phy_stats.rx_pkt_tc4_phy = resp.rx_pkt_tc4_phy;
+       apc->phy_stats.tx_pkt_tc4_phy = resp.tx_pkt_tc4_phy;
+       apc->phy_stats.rx_pkt_tc5_phy = resp.rx_pkt_tc5_phy;
+       apc->phy_stats.tx_pkt_tc5_phy = resp.tx_pkt_tc5_phy;
+       apc->phy_stats.rx_pkt_tc6_phy = resp.rx_pkt_tc6_phy;
+       apc->phy_stats.tx_pkt_tc6_phy = resp.tx_pkt_tc6_phy;
+       apc->phy_stats.rx_pkt_tc7_phy = resp.rx_pkt_tc7_phy;
+       apc->phy_stats.tx_pkt_tc7_phy = resp.tx_pkt_tc7_phy;
+
+       /* Per TC byte Counters */
+       apc->phy_stats.rx_byte_tc0_phy = resp.rx_byte_tc0_phy;
+       apc->phy_stats.tx_byte_tc0_phy = resp.tx_byte_tc0_phy;
+       apc->phy_stats.rx_byte_tc1_phy = resp.rx_byte_tc1_phy;
+       apc->phy_stats.tx_byte_tc1_phy = resp.tx_byte_tc1_phy;
+       apc->phy_stats.rx_byte_tc2_phy = resp.rx_byte_tc2_phy;
+       apc->phy_stats.tx_byte_tc2_phy = resp.tx_byte_tc2_phy;
+       apc->phy_stats.rx_byte_tc3_phy = resp.rx_byte_tc3_phy;
+       apc->phy_stats.tx_byte_tc3_phy = resp.tx_byte_tc3_phy;
+       apc->phy_stats.rx_byte_tc4_phy = resp.rx_byte_tc4_phy;
+       apc->phy_stats.tx_byte_tc4_phy = resp.tx_byte_tc4_phy;
+       apc->phy_stats.rx_byte_tc5_phy = resp.rx_byte_tc5_phy;
+       apc->phy_stats.tx_byte_tc5_phy = resp.tx_byte_tc5_phy;
+       apc->phy_stats.rx_byte_tc6_phy = resp.rx_byte_tc6_phy;
+       apc->phy_stats.tx_byte_tc6_phy = resp.tx_byte_tc6_phy;
+       apc->phy_stats.rx_byte_tc7_phy = resp.rx_byte_tc7_phy;
+       apc->phy_stats.tx_byte_tc7_phy = resp.tx_byte_tc7_phy;
+
+       /* Per TC pause Counters */
+       apc->phy_stats.rx_pause_tc0_phy = resp.rx_pause_tc0_phy;
+       apc->phy_stats.tx_pause_tc0_phy = resp.tx_pause_tc0_phy;
+       apc->phy_stats.rx_pause_tc1_phy = resp.rx_pause_tc1_phy;
+       apc->phy_stats.tx_pause_tc1_phy = resp.tx_pause_tc1_phy;
+       apc->phy_stats.rx_pause_tc2_phy = resp.rx_pause_tc2_phy;
+       apc->phy_stats.tx_pause_tc2_phy = resp.tx_pause_tc2_phy;
+       apc->phy_stats.rx_pause_tc3_phy = resp.rx_pause_tc3_phy;
+       apc->phy_stats.tx_pause_tc3_phy = resp.tx_pause_tc3_phy;
+       apc->phy_stats.rx_pause_tc4_phy = resp.rx_pause_tc4_phy;
+       apc->phy_stats.tx_pause_tc4_phy = resp.tx_pause_tc4_phy;
+       apc->phy_stats.rx_pause_tc5_phy = resp.rx_pause_tc5_phy;
+       apc->phy_stats.tx_pause_tc5_phy = resp.tx_pause_tc5_phy;
+       apc->phy_stats.rx_pause_tc6_phy = resp.rx_pause_tc6_phy;
+       apc->phy_stats.tx_pause_tc6_phy = resp.tx_pause_tc6_phy;
+       apc->phy_stats.rx_pause_tc7_phy = resp.rx_pause_tc7_phy;
+       apc->phy_stats.tx_pause_tc7_phy = resp.tx_pause_tc7_phy;
+}
+
 static int mana_init_port(struct net_device *ndev)
 {
        struct mana_port_context *apc = netdev_priv(ndev);
index c419626073f5f519d38dd38bc5792f03e1543a33..4fb3a04994a2dce9467e63bc21c675427ed389c8 100644 (file)
@@ -7,10 +7,12 @@
 
 #include <net/mana/mana.h>
 
-static const struct {
+struct mana_stats_desc {
        char name[ETH_GSTRING_LEN];
        u16 offset;
-} mana_eth_stats[] = {
+};
+
+static const struct mana_stats_desc mana_eth_stats[] = {
        {"stop_queue", offsetof(struct mana_ethtool_stats, stop_queue)},
        {"wake_queue", offsetof(struct mana_ethtool_stats, wake_queue)},
        {"hc_rx_discards_no_wqe", offsetof(struct mana_ethtool_stats,
@@ -75,6 +77,59 @@ static const struct {
                                        rx_cqe_unknown_type)},
 };
 
+static const struct mana_stats_desc mana_phy_stats[] = {
+       { "hc_rx_pkt_drop_phy", offsetof(struct mana_ethtool_phy_stats, rx_pkt_drop_phy) },
+       { "hc_tx_pkt_drop_phy", offsetof(struct mana_ethtool_phy_stats, tx_pkt_drop_phy) },
+       { "hc_tc0_rx_pkt_phy", offsetof(struct mana_ethtool_phy_stats, rx_pkt_tc0_phy) },
+       { "hc_tc0_rx_byte_phy", offsetof(struct mana_ethtool_phy_stats, rx_byte_tc0_phy) },
+       { "hc_tc0_tx_pkt_phy", offsetof(struct mana_ethtool_phy_stats, tx_pkt_tc0_phy) },
+       { "hc_tc0_tx_byte_phy", offsetof(struct mana_ethtool_phy_stats, tx_byte_tc0_phy) },
+       { "hc_tc1_rx_pkt_phy", offsetof(struct mana_ethtool_phy_stats, rx_pkt_tc1_phy) },
+       { "hc_tc1_rx_byte_phy", offsetof(struct mana_ethtool_phy_stats, rx_byte_tc1_phy) },
+       { "hc_tc1_tx_pkt_phy", offsetof(struct mana_ethtool_phy_stats, tx_pkt_tc1_phy) },
+       { "hc_tc1_tx_byte_phy", offsetof(struct mana_ethtool_phy_stats, tx_byte_tc1_phy) },
+       { "hc_tc2_rx_pkt_phy", offsetof(struct mana_ethtool_phy_stats, rx_pkt_tc2_phy) },
+       { "hc_tc2_rx_byte_phy", offsetof(struct mana_ethtool_phy_stats, rx_byte_tc2_phy) },
+       { "hc_tc2_tx_pkt_phy", offsetof(struct mana_ethtool_phy_stats, tx_pkt_tc2_phy) },
+       { "hc_tc2_tx_byte_phy", offsetof(struct mana_ethtool_phy_stats, tx_byte_tc2_phy) },
+       { "hc_tc3_rx_pkt_phy", offsetof(struct mana_ethtool_phy_stats, rx_pkt_tc3_phy) },
+       { "hc_tc3_rx_byte_phy", offsetof(struct mana_ethtool_phy_stats, rx_byte_tc3_phy) },
+       { "hc_tc3_tx_pkt_phy", offsetof(struct mana_ethtool_phy_stats, tx_pkt_tc3_phy) },
+       { "hc_tc3_tx_byte_phy", offsetof(struct mana_ethtool_phy_stats, tx_byte_tc3_phy) },
+       { "hc_tc4_rx_pkt_phy", offsetof(struct mana_ethtool_phy_stats, rx_pkt_tc4_phy) },
+       { "hc_tc4_rx_byte_phy", offsetof(struct mana_ethtool_phy_stats, rx_byte_tc4_phy) },
+       { "hc_tc4_tx_pkt_phy", offsetof(struct mana_ethtool_phy_stats, tx_pkt_tc4_phy) },
+       { "hc_tc4_tx_byte_phy", offsetof(struct mana_ethtool_phy_stats, tx_byte_tc4_phy) },
+       { "hc_tc5_rx_pkt_phy", offsetof(struct mana_ethtool_phy_stats, rx_pkt_tc5_phy) },
+       { "hc_tc5_rx_byte_phy", offsetof(struct mana_ethtool_phy_stats, rx_byte_tc5_phy) },
+       { "hc_tc5_tx_pkt_phy", offsetof(struct mana_ethtool_phy_stats, tx_pkt_tc5_phy) },
+       { "hc_tc5_tx_byte_phy", offsetof(struct mana_ethtool_phy_stats, tx_byte_tc5_phy) },
+       { "hc_tc6_rx_pkt_phy", offsetof(struct mana_ethtool_phy_stats, rx_pkt_tc6_phy) },
+       { "hc_tc6_rx_byte_phy", offsetof(struct mana_ethtool_phy_stats, rx_byte_tc6_phy) },
+       { "hc_tc6_tx_pkt_phy", offsetof(struct mana_ethtool_phy_stats, tx_pkt_tc6_phy) },
+       { "hc_tc6_tx_byte_phy", offsetof(struct mana_ethtool_phy_stats, tx_byte_tc6_phy) },
+       { "hc_tc7_rx_pkt_phy", offsetof(struct mana_ethtool_phy_stats, rx_pkt_tc7_phy) },
+       { "hc_tc7_rx_byte_phy", offsetof(struct mana_ethtool_phy_stats, rx_byte_tc7_phy) },
+       { "hc_tc7_tx_pkt_phy", offsetof(struct mana_ethtool_phy_stats, tx_pkt_tc7_phy) },
+       { "hc_tc7_tx_byte_phy", offsetof(struct mana_ethtool_phy_stats, tx_byte_tc7_phy) },
+       { "hc_tc0_rx_pause_phy", offsetof(struct mana_ethtool_phy_stats, rx_pause_tc0_phy) },
+       { "hc_tc0_tx_pause_phy", offsetof(struct mana_ethtool_phy_stats, tx_pause_tc0_phy) },
+       { "hc_tc1_rx_pause_phy", offsetof(struct mana_ethtool_phy_stats, rx_pause_tc1_phy) },
+       { "hc_tc1_tx_pause_phy", offsetof(struct mana_ethtool_phy_stats, tx_pause_tc1_phy) },
+       { "hc_tc2_rx_pause_phy", offsetof(struct mana_ethtool_phy_stats, rx_pause_tc2_phy) },
+       { "hc_tc2_tx_pause_phy", offsetof(struct mana_ethtool_phy_stats, tx_pause_tc2_phy) },
+       { "hc_tc3_rx_pause_phy", offsetof(struct mana_ethtool_phy_stats, rx_pause_tc3_phy) },
+       { "hc_tc3_tx_pause_phy", offsetof(struct mana_ethtool_phy_stats, tx_pause_tc3_phy) },
+       { "hc_tc4_rx_pause_phy", offsetof(struct mana_ethtool_phy_stats, rx_pause_tc4_phy) },
+       { "hc_tc4_tx_pause_phy", offsetof(struct mana_ethtool_phy_stats, tx_pause_tc4_phy) },
+       { "hc_tc5_rx_pause_phy", offsetof(struct mana_ethtool_phy_stats, rx_pause_tc5_phy) },
+       { "hc_tc5_tx_pause_phy", offsetof(struct mana_ethtool_phy_stats, tx_pause_tc5_phy) },
+       { "hc_tc6_rx_pause_phy", offsetof(struct mana_ethtool_phy_stats, rx_pause_tc6_phy) },
+       { "hc_tc6_tx_pause_phy", offsetof(struct mana_ethtool_phy_stats, tx_pause_tc6_phy) },
+       { "hc_tc7_rx_pause_phy", offsetof(struct mana_ethtool_phy_stats, rx_pause_tc7_phy) },
+       { "hc_tc7_tx_pause_phy", offsetof(struct mana_ethtool_phy_stats, tx_pause_tc7_phy) },
+};
+
 static int mana_get_sset_count(struct net_device *ndev, int stringset)
 {
        struct mana_port_context *apc = netdev_priv(ndev);
@@ -83,8 +138,8 @@ static int mana_get_sset_count(struct net_device *ndev, int stringset)
        if (stringset != ETH_SS_STATS)
                return -EINVAL;
 
-       return ARRAY_SIZE(mana_eth_stats) + num_queues *
-                               (MANA_STATS_RX_COUNT + MANA_STATS_TX_COUNT);
+       return ARRAY_SIZE(mana_eth_stats) + ARRAY_SIZE(mana_phy_stats) +
+                       num_queues * (MANA_STATS_RX_COUNT + MANA_STATS_TX_COUNT);
 }
 
 static void mana_get_strings(struct net_device *ndev, u32 stringset, u8 *data)
@@ -99,6 +154,9 @@ static void mana_get_strings(struct net_device *ndev, u32 stringset, u8 *data)
        for (i = 0; i < ARRAY_SIZE(mana_eth_stats); i++)
                ethtool_puts(&data, mana_eth_stats[i].name);
 
+       for (i = 0; i < ARRAY_SIZE(mana_phy_stats); i++)
+               ethtool_puts(&data, mana_phy_stats[i].name);
+
        for (i = 0; i < num_queues; i++) {
                ethtool_sprintf(&data, "rx_%d_packets", i);
                ethtool_sprintf(&data, "rx_%d_bytes", i);
@@ -128,6 +186,7 @@ static void mana_get_ethtool_stats(struct net_device *ndev,
        struct mana_port_context *apc = netdev_priv(ndev);
        unsigned int num_queues = apc->num_queues;
        void *eth_stats = &apc->eth_stats;
+       void *phy_stats = &apc->phy_stats;
        struct mana_stats_rx *rx_stats;
        struct mana_stats_tx *tx_stats;
        unsigned int start;
@@ -151,9 +210,18 @@ static void mana_get_ethtool_stats(struct net_device *ndev,
        /* we call mana function to update stats from GDMA */
        mana_query_gf_stats(apc);
 
+       /* We call this mana function to get the phy stats from GDMA and includes
+        * aggregate tx/rx drop counters, Per-TC(Traffic Channel) tx/rx and pause
+        * counters.
+        */
+       mana_query_phy_stats(apc);
+
        for (q = 0; q < ARRAY_SIZE(mana_eth_stats); q++)
                data[i++] = *(u64 *)(eth_stats + mana_eth_stats[q].offset);
 
+       for (q = 0; q < ARRAY_SIZE(mana_phy_stats); q++)
+               data[i++] = *(u64 *)(phy_stats + mana_phy_stats[q].offset);
+
        for (q = 0; q < num_queues; q++) {
                rx_stats = &apc->rxqs[q]->stats;
 
index 9abb66461211015af41b9966966f5259fb2a7fb2..4176edf1be719c7821bd28046bd6ff86ec70e991 100644 (file)
@@ -404,6 +404,65 @@ struct mana_ethtool_stats {
        u64 rx_cqe_unknown_type;
 };
 
+struct mana_ethtool_phy_stats {
+       /* Drop Counters */
+       u64 rx_pkt_drop_phy;
+       u64 tx_pkt_drop_phy;
+
+       /* Per TC traffic Counters */
+       u64 rx_pkt_tc0_phy;
+       u64 tx_pkt_tc0_phy;
+       u64 rx_pkt_tc1_phy;
+       u64 tx_pkt_tc1_phy;
+       u64 rx_pkt_tc2_phy;
+       u64 tx_pkt_tc2_phy;
+       u64 rx_pkt_tc3_phy;
+       u64 tx_pkt_tc3_phy;
+       u64 rx_pkt_tc4_phy;
+       u64 tx_pkt_tc4_phy;
+       u64 rx_pkt_tc5_phy;
+       u64 tx_pkt_tc5_phy;
+       u64 rx_pkt_tc6_phy;
+       u64 tx_pkt_tc6_phy;
+       u64 rx_pkt_tc7_phy;
+       u64 tx_pkt_tc7_phy;
+
+       u64 rx_byte_tc0_phy;
+       u64 tx_byte_tc0_phy;
+       u64 rx_byte_tc1_phy;
+       u64 tx_byte_tc1_phy;
+       u64 rx_byte_tc2_phy;
+       u64 tx_byte_tc2_phy;
+       u64 rx_byte_tc3_phy;
+       u64 tx_byte_tc3_phy;
+       u64 rx_byte_tc4_phy;
+       u64 tx_byte_tc4_phy;
+       u64 rx_byte_tc5_phy;
+       u64 tx_byte_tc5_phy;
+       u64 rx_byte_tc6_phy;
+       u64 tx_byte_tc6_phy;
+       u64 rx_byte_tc7_phy;
+       u64 tx_byte_tc7_phy;
+
+       /* Per TC pause Counters */
+       u64 rx_pause_tc0_phy;
+       u64 tx_pause_tc0_phy;
+       u64 rx_pause_tc1_phy;
+       u64 tx_pause_tc1_phy;
+       u64 rx_pause_tc2_phy;
+       u64 tx_pause_tc2_phy;
+       u64 rx_pause_tc3_phy;
+       u64 tx_pause_tc3_phy;
+       u64 rx_pause_tc4_phy;
+       u64 tx_pause_tc4_phy;
+       u64 rx_pause_tc5_phy;
+       u64 tx_pause_tc5_phy;
+       u64 rx_pause_tc6_phy;
+       u64 tx_pause_tc6_phy;
+       u64 rx_pause_tc7_phy;
+       u64 tx_pause_tc7_phy;
+};
+
 struct mana_context {
        struct gdma_dev *gdma_dev;
 
@@ -474,6 +533,8 @@ struct mana_port_context {
 
        struct mana_ethtool_stats eth_stats;
 
+       struct mana_ethtool_phy_stats phy_stats;
+
        /* Debugfs */
        struct dentry *mana_port_debugfs;
 };
@@ -501,6 +562,7 @@ struct bpf_prog *mana_xdp_get(struct mana_port_context *apc);
 void mana_chn_setxdp(struct mana_port_context *apc, struct bpf_prog *prog);
 int mana_bpf(struct net_device *ndev, struct netdev_bpf *bpf);
 void mana_query_gf_stats(struct mana_port_context *apc);
+void mana_query_phy_stats(struct mana_port_context *apc);
 int mana_pre_alloc_rxbufs(struct mana_port_context *apc, int mtu, int num_queues);
 void mana_pre_dealloc_rxbufs(struct mana_port_context *apc);
 
@@ -527,6 +589,7 @@ enum mana_command_code {
        MANA_FENCE_RQ           = 0x20006,
        MANA_CONFIG_VPORT_RX    = 0x20007,
        MANA_QUERY_VPORT_CONFIG = 0x20008,
+       MANA_QUERY_PHY_STAT     = 0x2000c,
 
        /* Privileged commands for the PF mode */
        MANA_REGISTER_FILTER    = 0x28000,
@@ -689,6 +752,74 @@ struct mana_query_gf_stat_resp {
        u64 tx_err_gdma;
 }; /* HW DATA */
 
+/* Query phy stats */
+struct mana_query_phy_stat_req {
+       struct gdma_req_hdr hdr;
+       u64 req_stats;
+}; /* HW DATA */
+
+struct mana_query_phy_stat_resp {
+       struct gdma_resp_hdr hdr;
+       u64 reported_stats;
+
+       /* Aggregate Drop Counters */
+       u64 rx_pkt_drop_phy;
+       u64 tx_pkt_drop_phy;
+
+       /* Per TC(Traffic class) traffic Counters */
+       u64 rx_pkt_tc0_phy;
+       u64 tx_pkt_tc0_phy;
+       u64 rx_pkt_tc1_phy;
+       u64 tx_pkt_tc1_phy;
+       u64 rx_pkt_tc2_phy;
+       u64 tx_pkt_tc2_phy;
+       u64 rx_pkt_tc3_phy;
+       u64 tx_pkt_tc3_phy;
+       u64 rx_pkt_tc4_phy;
+       u64 tx_pkt_tc4_phy;
+       u64 rx_pkt_tc5_phy;
+       u64 tx_pkt_tc5_phy;
+       u64 rx_pkt_tc6_phy;
+       u64 tx_pkt_tc6_phy;
+       u64 rx_pkt_tc7_phy;
+       u64 tx_pkt_tc7_phy;
+
+       u64 rx_byte_tc0_phy;
+       u64 tx_byte_tc0_phy;
+       u64 rx_byte_tc1_phy;
+       u64 tx_byte_tc1_phy;
+       u64 rx_byte_tc2_phy;
+       u64 tx_byte_tc2_phy;
+       u64 rx_byte_tc3_phy;
+       u64 tx_byte_tc3_phy;
+       u64 rx_byte_tc4_phy;
+       u64 tx_byte_tc4_phy;
+       u64 rx_byte_tc5_phy;
+       u64 tx_byte_tc5_phy;
+       u64 rx_byte_tc6_phy;
+       u64 tx_byte_tc6_phy;
+       u64 rx_byte_tc7_phy;
+       u64 tx_byte_tc7_phy;
+
+       /* Per TC(Traffic Class) pause Counters */
+       u64 rx_pause_tc0_phy;
+       u64 tx_pause_tc0_phy;
+       u64 rx_pause_tc1_phy;
+       u64 tx_pause_tc1_phy;
+       u64 rx_pause_tc2_phy;
+       u64 tx_pause_tc2_phy;
+       u64 rx_pause_tc3_phy;
+       u64 tx_pause_tc3_phy;
+       u64 rx_pause_tc4_phy;
+       u64 tx_pause_tc4_phy;
+       u64 rx_pause_tc5_phy;
+       u64 tx_pause_tc5_phy;
+       u64 rx_pause_tc6_phy;
+       u64 tx_pause_tc6_phy;
+       u64 rx_pause_tc7_phy;
+       u64 tx_pause_tc7_phy;
+}; /* HW DATA */
+
 /* Configure vPort Rx Steering */
 struct mana_cfg_rx_steer_req_v2 {
        struct gdma_req_hdr hdr;