From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: X-Spam-Checker-Version: SpamAssassin 3.4.0 (2014-02-07) on aws-us-west-2-korg-lkml-1.web.codeaurora.org Received: from mails.dpdk.org (mails.dpdk.org [217.70.189.124]) by smtp.lore.kernel.org (Postfix) with ESMTP id DF031C5AC82 for ; Mon, 10 Aug 2026 10:49:11 +0000 (UTC) Received: from mails.dpdk.org (localhost [127.0.0.1]) by mails.dpdk.org (Postfix) with ESMTP id D8FCB40E8A; Mon, 10 Aug 2026 12:48:30 +0200 (CEST) Received: from inva020.nxp.com (inva020.nxp.com [92.121.34.13]) by mails.dpdk.org (Postfix) with ESMTP id 7914F40671 for ; Mon, 10 Aug 2026 12:48:20 +0200 (CEST) Received: from inva020.nxp.com (localhost [127.0.0.1]) by inva020.eu-rdc02.nxp.com (Postfix) with ESMTP id 59D0D1A0017; Mon, 10 Aug 2026 12:48:20 +0200 (CEST) Received: from aprdc01srsp001v.ap-rdc01.nxp.com (aprdc01srsp001v.ap-rdc01.nxp.com [165.114.16.16]) by inva020.eu-rdc02.nxp.com (Postfix) with ESMTP id 212721A0011; Mon, 10 Aug 2026 12:48:20 +0200 (CEST) Received: from lsv03457.swis.in-blr01.nxp.com (lsv03457.swis.in-blr01.nxp.com [92.120.147.250]) by aprdc01srsp001v.ap-rdc01.nxp.com (Postfix) with ESMTP id A38681800088; Mon, 10 Aug 2026 18:48:19 +0800 (+08) From: Gagandeep Singh To: dev@dpdk.org Cc: hemant.agrawal@nxp.com, Gagandeep Singh Subject: [PATCH v5 09/14] net/enetc: support stats reset for VF Date: Mon, 10 Aug 2026 16:18:10 +0530 Message-Id: <20260810104815.2827703-10-g.singh@nxp.com> X-Mailer: git-send-email 2.25.1 In-Reply-To: <20260810104815.2827703-1-g.singh@nxp.com> References: <20260807112623.1402493-1-g.singh@nxp.com> <20260810104815.2827703-1-g.singh@nxp.com> MIME-Version: 1.0 Content-Transfer-Encoding: 8bit X-Virus-Scanned: ClamAV using ClamSMTP X-BeenThere: dev@dpdk.org X-Mailman-Version: 2.1.29 Precedence: list List-Id: DPDK patches and discussions List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: dev-bounces@dpdk.org The NETC SI-level hardware counters (SIROCT0, SIRFRM0, SITOCT0, SITFRM0, SITDFCR) are read-only for a VF and cannot be directly zeroed. No VSI-PSI command class exists for stats reset, and using FLR/soft-reset to clear counters is not reliable due to a known hardware erratum on some NETC platforms. Implement stats_reset for the enetc4 VF PMD using a software snapshot/delta approach: on stats_reset, the current HW counter values are captured as a baseline in a new per-device enetc4_vf_stats_saved struct. stats_get then reports the delta (current_hw_value - saved_baseline), so counters appear to start from zero after each reset call. Per-ring software Rx error accumulators (ierrors) are zeroed directly on reset. Register the stats_reset callback in both VF ops tables (enetc4_vf_ops and enetc4_vf_ops_no_vsi_m) so the feature is available regardless of whether the VSI messaging path is enabled. Signed-off-by: Gagandeep Singh --- doc/guides/rel_notes/release_26_11.rst | 1 + drivers/net/enetc/enetc.h | 17 +++++++ drivers/net/enetc/enetc4_vf.c | 61 +++++++++++++++++++++++--- 3 files changed, 74 insertions(+), 5 deletions(-) diff --git a/doc/guides/rel_notes/release_26_11.rst b/doc/guides/rel_notes/release_26_11.rst index 1765adc1bc..a42577c681 100644 --- a/doc/guides/rel_notes/release_26_11.rst +++ b/doc/guides/rel_notes/release_26_11.rst @@ -70,6 +70,7 @@ New Features * Added register dump support for ENETC4 PF and VF. * Added ring parameters support for the ENETC4 VF (rxq_info_get / txq_info_get). * Refreshed VF link speed on the link-up interrupt in the ENETC4 VF driver. + * Added stats reset for the ENETC4 VF using a software snapshot/delta approach. Removed Items ------------- diff --git a/drivers/net/enetc/enetc.h b/drivers/net/enetc/enetc.h index 6ff438eff6..3cddc4873f 100644 --- a/drivers/net/enetc/enetc.h +++ b/drivers/net/enetc/enetc.h @@ -104,6 +104,21 @@ struct enetc_bdr { uint8_t rsc_enable; }; +/* + * Saved SI counter baseline for VF stats reset. Since the SI-level + * hardware counters (SIROCT0, SIRFRM0, SITOCT0, SITFRM0, SITDFCR) + * are read-only for a VF and cannot be directly zeroed, stats_reset + * captures the current counter values as a baseline. stats_get then + * reports the delta: current_hw_value - baseline. + */ +struct enetc4_vf_stats_saved { + uint64_t ipackets; + uint64_t opackets; + uint64_t ibytes; + uint64_t obytes; + uint64_t oerrors; +}; + struct enetc_eth_hw { struct rte_eth_dev *ndev; struct enetc_hw hw; @@ -123,6 +138,8 @@ struct enetc_eth_hw { * for PF kernel versions before 6.18.37. Set via vf_link_legacy devarg. */ uint8_t vf_link_legacy; + /* Baseline snapshot for VF stats reset (software delta approach). */ + struct enetc4_vf_stats_saved vf_stats_saved; }; /* diff --git a/drivers/net/enetc/enetc4_vf.c b/drivers/net/enetc/enetc4_vf.c index dae6e4c772..1399bfad31 100644 --- a/drivers/net/enetc/enetc4_vf.c +++ b/drivers/net/enetc/enetc4_vf.c @@ -224,15 +224,64 @@ enetc4_vf_stats_get(struct rte_eth_dev *dev, struct rte_eth_stats *stats, uint8_t i; PMD_INIT_FUNC_TRACE(); - stats->ipackets = enetc4_rd(enetc_hw, ENETC4_SIRFRM0); - stats->opackets = enetc4_rd(enetc_hw, ENETC4_SITFRM0); - stats->ibytes = enetc4_rd(enetc_hw, ENETC4_SIROCT0); - stats->obytes = enetc4_rd(enetc_hw, ENETC4_SITOCT0); - stats->oerrors = enetc4_rd(enetc_hw, ENETC4_SITDFCR); + + /* + * The SI-level counters are read-only for a VF; they cannot be + * zeroed directly. Instead, stats_reset captures a baseline + * snapshot, and stats_get reports the delta so that the reported + * values appear to start from zero after each reset call. + */ + stats->ipackets = enetc4_rd(enetc_hw, ENETC4_SIRFRM0) - + hw->vf_stats_saved.ipackets; + stats->opackets = enetc4_rd(enetc_hw, ENETC4_SITFRM0) - + hw->vf_stats_saved.opackets; + stats->ibytes = enetc4_rd(enetc_hw, ENETC4_SIROCT0) - + hw->vf_stats_saved.ibytes; + stats->obytes = enetc4_rd(enetc_hw, ENETC4_SITOCT0) - + hw->vf_stats_saved.obytes; + stats->oerrors = enetc4_rd(enetc_hw, ENETC4_SITDFCR) - + hw->vf_stats_saved.oerrors; + for (i = 0; i < dev->data->nb_rx_queues; i++) { rx_ring = dev->data->rx_queues[i]; stats->ierrors += rx_ring->ierrors; } + + return 0; +} + +/* + * Reset VF statistics by capturing a new baseline snapshot of the + * SI-level hardware counters. Because those counters are read-only + * for a VF (hardware erratum prevents reliable clear via FLR/soft + * reset too), the driver uses a software delta approach: every + * stats_get call reports current_hw_value - saved_baseline. + */ +static int +enetc4_vf_stats_reset(struct rte_eth_dev *dev) +{ + struct enetc_eth_hw *hw = + ENETC_DEV_PRIVATE_TO_HW(dev->data->dev_private); + struct enetc_hw *enetc_hw = &hw->hw; + struct enetc_bdr *rx_ring; + uint8_t i; + + PMD_INIT_FUNC_TRACE(); + + /* Snapshot current HW SI counter values as the new zero baseline. */ + hw->vf_stats_saved.ipackets = enetc4_rd(enetc_hw, ENETC4_SIRFRM0); + hw->vf_stats_saved.opackets = enetc4_rd(enetc_hw, ENETC4_SITFRM0); + hw->vf_stats_saved.ibytes = enetc4_rd(enetc_hw, ENETC4_SIROCT0); + hw->vf_stats_saved.obytes = enetc4_rd(enetc_hw, ENETC4_SITOCT0); + hw->vf_stats_saved.oerrors = enetc4_rd(enetc_hw, ENETC4_SITDFCR); + + /* Reset the per-ring software Rx error accumulators. */ + for (i = 0; i < dev->data->nb_rx_queues; i++) { + rx_ring = dev->data->rx_queues[i]; + if (rx_ring) + rx_ring->ierrors = 0; + } + return 0; } @@ -1544,6 +1593,7 @@ static const struct eth_dev_ops enetc4_vf_ops_no_vsi_m = { .dev_stop = enetc4_vf_dev_stop, .dev_close = enetc4_dev_close, .stats_get = enetc4_vf_stats_get, + .stats_reset = enetc4_vf_stats_reset, .dev_infos_get = enetc4_vf_dev_infos_get, .get_reg = enetc4_vf_get_regs, .mtu_set = enetc4_vf_mtu_set, @@ -1567,6 +1617,7 @@ static const struct eth_dev_ops enetc4_vf_ops = { .dev_stop = enetc4_vf_dev_stop, .dev_close = enetc4_dev_close, .stats_get = enetc4_vf_stats_get, + .stats_reset = enetc4_vf_stats_reset, .dev_infos_get = enetc4_vf_dev_infos_get, .fw_version_get = enetc4_vf_fw_version_get, .get_reg = enetc4_vf_get_regs, -- 2.25.1