From patchwork Tue Nov 24 17:36:28 2020 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: Ferruh Yigit X-Patchwork-Id: 84522 X-Patchwork-Delegate: ferruh.yigit@amd.com Return-Path: X-Original-To: patchwork@inbox.dpdk.org Delivered-To: patchwork@inbox.dpdk.org Received: from dpdk.org (dpdk.org [92.243.14.124]) by inbox.dpdk.org (Postfix) with ESMTP id DE245A04DB; Tue, 24 Nov 2020 18:36:39 +0100 (CET) Received: from [92.243.14.124] (localhost [127.0.0.1]) by dpdk.org (Postfix) with ESMTP id 7638CC914; Tue, 24 Nov 2020 18:36:37 +0100 (CET) Received: from mga11.intel.com (mga11.intel.com [192.55.52.93]) by dpdk.org (Postfix) with ESMTP id D45A2C90E for ; Tue, 24 Nov 2020 18:36:33 +0100 (CET) IronPort-SDR: LHSugZWHgAKQbuURFId7MANcbKuka0fWPgNuH1cyUd/KzM34+fHjGIzOKxNmNUOFRhMakOQmPr H9cIWDC3weQg== X-IronPort-AV: E=McAfee;i="6000,8403,9815"; a="168477623" X-IronPort-AV: E=Sophos;i="5.78,366,1599548400"; d="scan'208";a="168477623" X-Amp-Result: SKIPPED(no attachment in message) X-Amp-File-Uploaded: False Received: from fmsmga003.fm.intel.com ([10.253.24.29]) by fmsmga102.fm.intel.com with ESMTP/TLS/ECDHE-RSA-AES256-GCM-SHA384; 24 Nov 2020 09:36:32 -0800 IronPort-SDR: cVEmbNXHYEWWFIThx3WwJ+A9kxQfSC1h1L+xlvt4GZl5f7SkH+lYYxMNM/P/B90dwW9QXyzrlR s7377Vp5M57A== X-ExtLoop1: 1 X-IronPort-AV: E=Sophos;i="5.78,366,1599548400"; d="scan'208";a="370511462" Received: from silpixa00399752.ir.intel.com (HELO silpixa00399752.ger.corp.intel.com) ([10.237.222.180]) by FMSMGA003.fm.intel.com with ESMTP; 24 Nov 2020 09:36:30 -0800 From: Ferruh Yigit To: Ray Kinsella , Neil Horman Cc: Ferruh Yigit , dev@dpdk.org, Konstantin Ananyev , Thomas Monjalon , Andrew Rybchenko , Matan Azrad , Olivier Matz , Jerin Jacob Date: Tue, 24 Nov 2020 17:36:28 +0000 Message-Id: <20201124173628.1057348-1-ferruh.yigit@intel.com> X-Mailer: git-send-email 2.26.2 In-Reply-To: <20201020120305.1516513-1-ferruh.yigit@intel.com> References: <20201020120305.1516513-1-ferruh.yigit@intel.com> MIME-Version: 1.0 Subject: [dpdk-dev] [RFC v2] doc: announce max Rx packet len field deprecation X-BeenThere: dev@dpdk.org X-Mailman-Version: 2.1.15 Precedence: list List-Id: DPDK patches and discussions List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: dev-bounces@dpdk.org Sender: "dev" Signed-off-by: Ferruh Yigit Acked-by: Konstantin Ananyev Acked-by: Ajit Khaparde Acked-by: Andrew Rybchenko Acked-by: Thomas Monjalon --- Cc: Thomas Monjalon Cc: Andrew Rybchenko Cc: Konstantin Ananyev Cc: Matan Azrad Cc: Olivier Matz Cc: Jerin Jacob v2: * ``uint32_t mtu`` moved to ``struct rte_eth_conf`` * The "Driver is responsible from updating ``(struct rte_eth_dev)->data->mtu``" updated because ethdev layer also can do this. The intention there was both APIs should update the variable. Another open question is from Andrew, if we can remove the ``uint32_t max_rx_pkt_len`` completely from the ``rte_eth_dev_configure()``. This may force applications to have one more additional ``rte_eth_dev_set_mtu()`` call for device initialization, but if applications are OK with the default values most of times, agree that removing is easier solution, please comment. --- doc/guides/rel_notes/deprecation.rst | 25 +++++++++++++++++++++++++ 1 file changed, 25 insertions(+) diff --git a/doc/guides/rel_notes/deprecation.rst b/doc/guides/rel_notes/deprecation.rst index 96986fabd598..73e43581d615 100644 --- a/doc/guides/rel_notes/deprecation.rst +++ b/doc/guides/rel_notes/deprecation.rst @@ -88,6 +88,31 @@ Deprecation Notices will be limited to maximum 256 queues. Also compile time flag ``RTE_ETHDEV_QUEUE_STAT_CNTRS`` will be removed. +* ethdev: ``uint32_t max_rx_pkt_len`` field of ``struct rte_eth_rxmode``, will be + replaced by a new ``uint32_t mtu`` field of ``struct rte_eth_conf`` in v21.11. + The new ``mtu`` field will be used to configure the initial device MTU via + ``rte_eth_dev_configure()`` API. + Later MTU can be changed by ``rte_eth_dev_set_mtu()`` API as done now. + The existing ``(struct rte_eth_dev)->data->mtu`` variable will be used to store + the configured ``mtu`` value, + and this new ``(struct rte_eth_dev)->data->dev_conf.mtu`` variable will + be used to store the user configuration request. + Unlike ``max_rx_pkt_len``, which was valid only when ``JUMBO_FRAME`` enabled, + ``mtu`` field will be always valid. + When ``mtu`` config is not provided by the application, default ``RTE_ETHER_MTU`` + value will be used. + ``(struct rte_eth_dev)->data->mtu`` should be updated after MTU set successfully, + either by ``rte_eth_dev_configure()`` or ``rte_eth_dev_set_mtu()``. + + An application may need to configure device for a specific Rx packet size, like for + cases ``DEV_RX_OFFLOAD_SCATTER`` is not supported and device received packet size + can't be bigger than Rx buffer size. + To cover these cases an application needs to know the device packet overhead to be + able to calculate the ``mtu`` corresponding to a Rx buffer size, for this + ``(struct rte_eth_dev_info).max_rx_pktlen`` will be kept, + the device packet overhead can be calculated as: + ``(struct rte_eth_dev_info).max_rx_pktlen - (struct rte_eth_dev_info).max_mtu`` + * sched: To allow more traffic classes, flexible mapping of pipe queues to traffic classes, and subport level configuration of pipes and queues changes will be made to macros, data structures and API functions defined