From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from mails.dpdk.org (mails.dpdk.org [217.70.189.124]) by inbox.dpdk.org (Postfix) with ESMTP id 696AE46868; Fri, 6 Jun 2025 19:09:46 +0200 (CEST) Received: from mails.dpdk.org (localhost [127.0.0.1]) by mails.dpdk.org (Postfix) with ESMTP id 8D75B40B97; Fri, 6 Jun 2025 19:09:34 +0200 (CEST) Received: from mgamail.intel.com (mgamail.intel.com [198.175.65.10]) by mails.dpdk.org (Postfix) with ESMTP id D4FAB40B91 for ; Fri, 6 Jun 2025 19:09:32 +0200 (CEST) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=intel.com; i=@intel.com; q=dns/txt; s=Intel; t=1749229773; x=1780765773; h=from:to:cc:subject:date:message-id:in-reply-to: references:mime-version:content-transfer-encoding; bh=bkz9tlymLlmrAwDuhkcRWJw1kwySph8rHoseL3jHbBE=; b=H740dswohUvAQR6BsPFCfpoqXP2XD9x9LxBlmBh1QyFxvQ4IHWs15RBI J3b2GZGAizVSU35i5WuZVM4QFbwvOtniz8N5XhqYwT3jJM5zDezPiDmr6 9qe++asTk+NkyDJnTk727OxsfiNvPy8SWKxJMFbIzQu9k2ca72bS/24iW oj2n1aC0NjdCk790CqbZXpLnHkUMhCtGaiQALv++Xfo/w2+MzIskzl6YX 7yONN/QHNxAbWNUotsF3fKA6HYktKhIlCDErFSTTK46fJe4fQRWwuZub2 chTEMmB9GICXzTrYc1Y9V9gGC3o86ZFPvzaokUTo/yHPsodHF9G6Azaul w==; X-CSE-ConnectionGUID: mXlMO813R9SPJ+k+O9KPkg== X-CSE-MsgGUID: RqaVWiWCSaKBVZB4q4Jglw== X-IronPort-AV: E=McAfee;i="6800,10657,11456"; a="68828381" X-IronPort-AV: E=Sophos;i="6.16,215,1744095600"; d="scan'208";a="68828381" Received: from orviesa010.jf.intel.com ([10.64.159.150]) by orvoesa102.jf.intel.com with ESMTP/TLS/ECDHE-RSA-AES256-GCM-SHA384; 06 Jun 2025 10:09:32 -0700 X-CSE-ConnectionGUID: y6CV2rLKQvyt9uwnVteDcQ== X-CSE-MsgGUID: V9P/bctBRVexCTesPXcFJQ== X-ExtLoop1: 1 X-IronPort-AV: E=Sophos;i="6.16,215,1744095600"; d="scan'208";a="145808071" Received: from silpixa00401119.ir.intel.com ([10.55.129.167]) by orviesa010.jf.intel.com with ESMTP; 06 Jun 2025 10:09:31 -0700 From: Anatoly Burakov To: dev@dpdk.org, Vladimir Medvedkin , Ian Stokes Cc: bruce.richardson@intel.com Subject: [PATCH v5 02/34] net/iavf: make IPsec stats dynamically allocated Date: Fri, 6 Jun 2025 18:08:41 +0100 Message-ID: <65bf931be8b0a57bc422f38d3cfb940fcf204e9b.1749229650.git.anatoly.burakov@intel.com> X-Mailer: git-send-email 2.47.1 In-Reply-To: References: MIME-Version: 1.0 Content-Transfer-Encoding: 8bit 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 Currently, the stats structure is directly embedded in the queue structure. We're about to move iavf driver to a common Rx queue structure, so we can't have driver-specific structures that aren't pointers, inside the common queue structure. To prepare, we replace direct embedding into the queue structure with a pointer to the stats structure. Signed-off-by: Anatoly Burakov Acked-by: Bruce Richardson --- drivers/net/intel/iavf/iavf_ethdev.c | 2 +- drivers/net/intel/iavf/iavf_rxtx.c | 21 ++++++++++++++++++--- drivers/net/intel/iavf/iavf_rxtx.h | 2 +- 3 files changed, 20 insertions(+), 5 deletions(-) diff --git a/drivers/net/intel/iavf/iavf_ethdev.c b/drivers/net/intel/iavf/iavf_ethdev.c index b3dacbef84..5babd587b3 100644 --- a/drivers/net/intel/iavf/iavf_ethdev.c +++ b/drivers/net/intel/iavf/iavf_ethdev.c @@ -1870,7 +1870,7 @@ iavf_dev_update_ipsec_xstats(struct rte_eth_dev *ethdev, struct iavf_rx_queue *rxq; struct iavf_ipsec_crypto_stats *stats; rxq = (struct iavf_rx_queue *)ethdev->data->rx_queues[idx]; - stats = &rxq->stats.ipsec_crypto; + stats = &rxq->stats->ipsec_crypto; ips->icount += stats->icount; ips->ibytes += stats->ibytes; ips->ierrors.count += stats->ierrors.count; diff --git a/drivers/net/intel/iavf/iavf_rxtx.c b/drivers/net/intel/iavf/iavf_rxtx.c index 5411eb6897..d23d2df807 100644 --- a/drivers/net/intel/iavf/iavf_rxtx.c +++ b/drivers/net/intel/iavf/iavf_rxtx.c @@ -619,6 +619,18 @@ iavf_dev_rx_queue_setup(struct rte_eth_dev *dev, uint16_t queue_idx, return -ENOMEM; } + /* Allocate stats */ + rxq->stats = rte_zmalloc_socket("iavf rxq stats", + sizeof(struct iavf_rx_queue_stats), + RTE_CACHE_LINE_SIZE, + socket_id); + if (!rxq->stats) { + PMD_INIT_LOG(ERR, "Failed to allocate memory for " + "rx queue stats"); + rte_free(rxq); + return -ENOMEM; + } + if (vf->vf_res->vf_cap_flags & VIRTCHNL_VF_OFFLOAD_RX_FLEX_DESC) { proto_xtr = vf->proto_xtr ? vf->proto_xtr[queue_idx] : IAVF_PROTO_XTR_NONE; @@ -677,6 +689,7 @@ iavf_dev_rx_queue_setup(struct rte_eth_dev *dev, uint16_t queue_idx, socket_id); if (!rxq->sw_ring) { PMD_INIT_LOG(ERR, "Failed to allocate memory for SW ring"); + rte_free(rxq->stats); rte_free(rxq); return -ENOMEM; } @@ -693,6 +706,7 @@ iavf_dev_rx_queue_setup(struct rte_eth_dev *dev, uint16_t queue_idx, if (!mz) { PMD_INIT_LOG(ERR, "Failed to reserve DMA memory for RX"); rte_free(rxq->sw_ring); + rte_free(rxq->stats); rte_free(rxq); return -ENOMEM; } @@ -1054,6 +1068,7 @@ iavf_dev_rx_queue_release(struct rte_eth_dev *dev, uint16_t qid) iavf_rxq_release_mbufs_ops[q->rel_mbufs_type].release_mbufs(q); rte_free(q->sw_ring); rte_memzone_free(q->mz); + rte_free(q->stats); rte_free(q); } @@ -1581,7 +1596,7 @@ iavf_recv_pkts_flex_rxd(void *rx_queue, rte_le_to_cpu_16(rxd.wb.ptype_flex_flags0)]; iavf_flex_rxd_to_vlan_tci(rxm, &rxd); iavf_flex_rxd_to_ipsec_crypto_status(rxm, &rxd, - &rxq->stats.ipsec_crypto); + &rxq->stats->ipsec_crypto); rxd_to_pkt_fields_ops[rxq->rxdid](rxq, rxm, &rxd); pkt_flags = iavf_flex_rxd_error_to_pkt_flags(rx_stat_err0); @@ -1750,7 +1765,7 @@ iavf_recv_scattered_pkts_flex_rxd(void *rx_queue, struct rte_mbuf **rx_pkts, rte_le_to_cpu_16(rxd.wb.ptype_flex_flags0)]; iavf_flex_rxd_to_vlan_tci(first_seg, &rxd); iavf_flex_rxd_to_ipsec_crypto_status(first_seg, &rxd, - &rxq->stats.ipsec_crypto); + &rxq->stats->ipsec_crypto); rxd_to_pkt_fields_ops[rxq->rxdid](rxq, first_seg, &rxd); pkt_flags = iavf_flex_rxd_error_to_pkt_flags(rx_stat_err0); @@ -2034,7 +2049,7 @@ iavf_rx_scan_hw_ring_flex_rxd(struct iavf_rx_queue *rxq, rte_le_to_cpu_16(rxdp[j].wb.ptype_flex_flags0)]; iavf_flex_rxd_to_vlan_tci(mb, &rxdp[j]); iavf_flex_rxd_to_ipsec_crypto_status(mb, &rxdp[j], - &rxq->stats.ipsec_crypto); + &rxq->stats->ipsec_crypto); rxd_to_pkt_fields_ops[rxq->rxdid](rxq, mb, &rxdp[j]); stat_err0 = rte_le_to_cpu_16(rxdp[j].wb.status_error0); pkt_flags = iavf_flex_rxd_error_to_pkt_flags(stat_err0); diff --git a/drivers/net/intel/iavf/iavf_rxtx.h b/drivers/net/intel/iavf/iavf_rxtx.h index 0b5d67e718..62b5a67c84 100644 --- a/drivers/net/intel/iavf/iavf_rxtx.h +++ b/drivers/net/intel/iavf/iavf_rxtx.h @@ -268,7 +268,7 @@ struct iavf_rx_queue { uint8_t proto_xtr; /* protocol extraction type */ uint64_t xtr_ol_flag; /* flexible descriptor metadata extraction offload flag */ - struct iavf_rx_queue_stats stats; + struct iavf_rx_queue_stats *stats; uint64_t offloads; uint64_t phc_time; uint64_t hw_time_update; -- 2.47.1