From patchwork Wed Sep 23 12:59:59 2020 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: "humin (Q)" X-Patchwork-Id: 78557 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 B11F2A04B1; Wed, 23 Sep 2020 15:02:35 +0200 (CEST) Received: from [92.243.14.124] (localhost [127.0.0.1]) by dpdk.org (Postfix) with ESMTP id D96221D41E; Wed, 23 Sep 2020 15:02:34 +0200 (CEST) Received: from huawei.com (szxga05-in.huawei.com [45.249.212.191]) by dpdk.org (Postfix) with ESMTP id CF18B1D41E for ; Wed, 23 Sep 2020 15:02:31 +0200 (CEST) Received: from DGGEMS411-HUB.china.huawei.com (unknown [172.30.72.59]) by Forcepoint Email with ESMTP id 8ADCCF0AE455D46CDC5E; Wed, 23 Sep 2020 21:02:30 +0800 (CST) Received: from localhost.localdomain (10.69.192.56) by DGGEMS411-HUB.china.huawei.com (10.3.19.211) with Microsoft SMTP Server id 14.3.487.0; Wed, 23 Sep 2020 21:02:29 +0800 From: "Min Hu (Connor)" To: CC: , , , Date: Wed, 23 Sep 2020 20:59:59 +0800 Message-ID: <1600866002-35908-2-git-send-email-humin29@huawei.com> X-Mailer: git-send-email 2.7.4 In-Reply-To: <1600866002-35908-1-git-send-email-humin29@huawei.com> References: <1599219135-53194-2-git-send-email-humin29@huawei.com> <1600866002-35908-1-git-send-email-humin29@huawei.com> MIME-Version: 1.0 X-Originating-IP: [10.69.192.56] X-CFilter-Loop: Reflected Subject: [dpdk-dev] [PATCH V3 1/4] dpdk: fix compiling errors for per-queue statistics 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" From: Huisong Li Currently, only statistics of rx/tx queues with queue_id less than RTE_ETHDEV_QUEUE_STAT_CNTRS can be displayed. If there is a certain application scenario that it needs to use 256 or more than 256 queues and display all statistics of rx/tx queue. At this moment, we have to change the macro to be equaled to the queue number. However, modifying the macro to be greater than 256 will trigger many errors and warnings from test-pmd, PMD driver and librte_ethdev during compiling dpdk project. But it is possible and permitted that rx/tx queue number is greater than 256 and all statistics of rx/tx queue need to be displayed. In addition, the data type of rx/tx queue number in rte_eth_dev_configure API is 'uint16_t'. So It is unreasonable to use the 'uint8_t' type for variables that control which per-queue statistics can be displayed. Fixes: ed30d9b691b2 ("app/testpmd: add stats per queue") Fixes: 09c7e63a71f9 ("net/memif: introduce memory interface PMD") Fixes: abf7275bbaa2 ("ixgbe: move to drivers/net/") Fixes: e6defdfddc3b ("net/igc: enable statistics") Fixes: 2265e4b4e84b ("net/octeontx2: add basic stats operation") Fixes: 6c3169a3dc04 ("virtio: move to drivers/net/") Signed-off-by: Huisong Li Signed-off-by: Min Hu (Connor) Reviewed-by: Wei Hu (Xavier) Reviewed-by: Dongdong Liu --- v2->v3: change 'uint8_t i' to 'uint16_t i' in nic_stats_display function --- app/proc-info/main.c | 2 +- app/test-pmd/cmdline.c | 2 +- app/test-pmd/config.c | 4 ++-- app/test-pmd/testpmd.c | 2 +- app/test-pmd/testpmd.h | 5 +++-- drivers/net/igc/igc_ethdev.c | 4 ++-- drivers/net/ixgbe/ixgbe_ethdev.c | 4 ++-- drivers/net/memif/rte_eth_memif.c | 2 +- drivers/net/octeontx2/otx2_ethdev.h | 2 +- drivers/net/octeontx2/otx2_stats.c | 2 +- drivers/net/virtio/virtio_ethdev.c | 4 ++-- lib/librte_ethdev/rte_ethdev.c | 6 +++--- lib/librte_ethdev/rte_ethdev.h | 4 ++-- lib/librte_ethdev/rte_ethdev_driver.h | 2 +- 14 files changed, 23 insertions(+), 22 deletions(-) diff --git a/app/proc-info/main.c b/app/proc-info/main.c index 64fb83b..26d9355 100644 --- a/app/proc-info/main.c +++ b/app/proc-info/main.c @@ -348,7 +348,7 @@ static void nic_stats_display(uint16_t port_id) { struct rte_eth_stats stats; - uint8_t i; + uint16_t i; static const char *nic_stats_border = "########################"; diff --git a/app/test-pmd/cmdline.c b/app/test-pmd/cmdline.c index 5f93409..200b8a0 100644 --- a/app/test-pmd/cmdline.c +++ b/app/test-pmd/cmdline.c @@ -8267,7 +8267,7 @@ struct cmd_set_qmap_result { cmdline_fixed_string_t what; portid_t port_id; uint16_t queue_id; - uint8_t map_value; + uint16_t map_value; }; static void diff --git a/app/test-pmd/config.c b/app/test-pmd/config.c index 2d9a456..f23b193 100644 --- a/app/test-pmd/config.c +++ b/app/test-pmd/config.c @@ -160,7 +160,7 @@ nic_stats_display(portid_t port_id) uint64_t mpps_rx, mpps_tx, mbps_rx, mbps_tx; struct rte_eth_stats stats; struct rte_port *port = &ports[port_id]; - uint8_t i; + uint16_t i; static const char *nic_stats_border = "########################"; @@ -3625,7 +3625,7 @@ tx_vlan_pvid_set(portid_t port_id, uint16_t vlan_id, int on) } void -set_qmap(portid_t port_id, uint8_t is_rx, uint16_t queue_id, uint8_t map_value) +set_qmap(portid_t port_id, uint8_t is_rx, uint16_t queue_id, uint16_t map_value) { uint16_t i; uint8_t existing_mapping_found = 0; diff --git a/app/test-pmd/testpmd.c b/app/test-pmd/testpmd.c index fe6450c..4b26c5c 100644 --- a/app/test-pmd/testpmd.c +++ b/app/test-pmd/testpmd.c @@ -1840,7 +1840,7 @@ fwd_stats_display(void) fwd_cycles += fs->core_cycles; } for (i = 0; i < cur_fwd_config.nb_fwd_ports; i++) { - uint8_t j; + uint16_t j; pt_id = fwd_ports_ids[i]; port = &ports[pt_id]; diff --git a/app/test-pmd/testpmd.h b/app/test-pmd/testpmd.h index f139fe7..16aa81c 100644 --- a/app/test-pmd/testpmd.h +++ b/app/test-pmd/testpmd.h @@ -279,7 +279,7 @@ enum dcb_mode_enable struct queue_stats_mappings { portid_t port_id; uint16_t queue_id; - uint8_t stats_counter_id; + uint16_t stats_counter_id; } __rte_cache_aligned; extern struct queue_stats_mappings tx_queue_stats_mappings_array[]; @@ -792,7 +792,8 @@ void tx_qinq_set(portid_t port_id, uint16_t vlan_id, uint16_t vlan_id_outer); void tx_vlan_reset(portid_t port_id); void tx_vlan_pvid_set(portid_t port_id, uint16_t vlan_id, int on); -void set_qmap(portid_t port_id, uint8_t is_rx, uint16_t queue_id, uint8_t map_value); +void set_qmap(portid_t port_id, uint8_t is_rx, uint16_t queue_id, + uint16_t map_value); void set_xstats_hide_zero(uint8_t on_off); diff --git a/drivers/net/igc/igc_ethdev.c b/drivers/net/igc/igc_ethdev.c index 810568b..875874e 100644 --- a/drivers/net/igc/igc_ethdev.c +++ b/drivers/net/igc/igc_ethdev.c @@ -221,7 +221,7 @@ static int eth_igc_xstats_get_names_by_id(struct rte_eth_dev *dev, static int eth_igc_xstats_reset(struct rte_eth_dev *dev); static int eth_igc_queue_stats_mapping_set(struct rte_eth_dev *dev, - uint16_t queue_id, uint8_t stat_idx, uint8_t is_rx); + uint16_t queue_id, uint16_t stat_idx, uint8_t is_rx); static int eth_igc_rx_queue_intr_disable(struct rte_eth_dev *dev, uint16_t queue_id); static int @@ -2075,7 +2075,7 @@ eth_igc_xstats_get_by_id(struct rte_eth_dev *dev, const uint64_t *ids, static int eth_igc_queue_stats_mapping_set(struct rte_eth_dev *dev, - uint16_t queue_id, uint8_t stat_idx, uint8_t is_rx) + uint16_t queue_id, uint16_t stat_idx, uint8_t is_rx) { struct igc_adapter *igc = IGC_DEV_PRIVATE(dev); diff --git a/drivers/net/ixgbe/ixgbe_ethdev.c b/drivers/net/ixgbe/ixgbe_ethdev.c index 0f065bb..9850488 100644 --- a/drivers/net/ixgbe/ixgbe_ethdev.c +++ b/drivers/net/ixgbe/ixgbe_ethdev.c @@ -178,7 +178,7 @@ static int ixgbe_dev_xstats_get_names_by_id( unsigned int limit); static int ixgbe_dev_queue_stats_mapping_set(struct rte_eth_dev *eth_dev, uint16_t queue_id, - uint8_t stat_idx, + uint16_t stat_idx, uint8_t is_rx); static int ixgbe_fw_version_get(struct rte_eth_dev *dev, char *fw_version, size_t fw_size); @@ -890,7 +890,7 @@ ixgbe_reset_qstat_mappings(struct ixgbe_hw *hw) static int ixgbe_dev_queue_stats_mapping_set(struct rte_eth_dev *eth_dev, uint16_t queue_id, - uint8_t stat_idx, + uint16_t stat_idx, uint8_t is_rx) { #define QSM_REG_NB_BITS_PER_QMAP_FIELD 8 diff --git a/drivers/net/memif/rte_eth_memif.c b/drivers/net/memif/rte_eth_memif.c index a19c0f3..29cea93 100644 --- a/drivers/net/memif/rte_eth_memif.c +++ b/drivers/net/memif/rte_eth_memif.c @@ -1356,7 +1356,7 @@ memif_stats_get(struct rte_eth_dev *dev, struct rte_eth_stats *stats) struct pmd_internals *pmd = dev->data->dev_private; struct memif_queue *mq; int i; - uint8_t tmp, nq; + uint16_t tmp, nq; stats->ipackets = 0; stats->ibytes = 0; diff --git a/drivers/net/octeontx2/otx2_ethdev.h b/drivers/net/octeontx2/otx2_ethdev.h index a114112..0c761e3 100644 --- a/drivers/net/octeontx2/otx2_ethdev.h +++ b/drivers/net/octeontx2/otx2_ethdev.h @@ -476,7 +476,7 @@ int otx2_nix_dev_stats_get(struct rte_eth_dev *eth_dev, int otx2_nix_dev_stats_reset(struct rte_eth_dev *eth_dev); int otx2_nix_queue_stats_mapping(struct rte_eth_dev *dev, - uint16_t queue_id, uint8_t stat_idx, + uint16_t queue_id, uint16_t stat_idx, uint8_t is_rx); int otx2_nix_xstats_get(struct rte_eth_dev *eth_dev, struct rte_eth_xstat *xstats, unsigned int n); diff --git a/drivers/net/octeontx2/otx2_stats.c b/drivers/net/octeontx2/otx2_stats.c index 8aaf270..6efe122 100644 --- a/drivers/net/octeontx2/otx2_stats.c +++ b/drivers/net/octeontx2/otx2_stats.c @@ -145,7 +145,7 @@ otx2_nix_dev_stats_reset(struct rte_eth_dev *eth_dev) int otx2_nix_queue_stats_mapping(struct rte_eth_dev *eth_dev, uint16_t queue_id, - uint8_t stat_idx, uint8_t is_rx) + uint16_t stat_idx, uint8_t is_rx) { struct otx2_eth_dev *dev = otx2_eth_pmd_priv(eth_dev); diff --git a/drivers/net/virtio/virtio_ethdev.c b/drivers/net/virtio/virtio_ethdev.c index 013a290..9d26012 100644 --- a/drivers/net/virtio/virtio_ethdev.c +++ b/drivers/net/virtio/virtio_ethdev.c @@ -82,7 +82,7 @@ static int virtio_intr_disable(struct rte_eth_dev *dev); static int virtio_dev_queue_stats_mapping_set( struct rte_eth_dev *eth_dev, uint16_t queue_id, - uint8_t stat_idx, + uint16_t stat_idx, uint8_t is_rx); static void virtio_notify_peers(struct rte_eth_dev *dev); @@ -2648,7 +2648,7 @@ virtio_dev_info_get(struct rte_eth_dev *dev, struct rte_eth_dev_info *dev_info) */ static int virtio_dev_queue_stats_mapping_set(__rte_unused struct rte_eth_dev *eth_dev, -__rte_unused uint16_t queue_id, __rte_unused uint8_t stat_idx, +__rte_unused uint16_t queue_id, __rte_unused uint16_t stat_idx, __rte_unused uint8_t is_rx) { return 0; diff --git a/lib/librte_ethdev/rte_ethdev.c b/lib/librte_ethdev/rte_ethdev.c index dfe5c1b..749b372 100644 --- a/lib/librte_ethdev/rte_ethdev.c +++ b/lib/librte_ethdev/rte_ethdev.c @@ -2942,7 +2942,7 @@ rte_eth_xstats_reset(uint16_t port_id) } static int -set_queue_stats_mapping(uint16_t port_id, uint16_t queue_id, uint8_t stat_idx, +set_queue_stats_mapping(uint16_t port_id, uint16_t queue_id, uint16_t stat_idx, uint8_t is_rx) { struct rte_eth_dev *dev; @@ -2969,7 +2969,7 @@ set_queue_stats_mapping(uint16_t port_id, uint16_t queue_id, uint8_t stat_idx, int rte_eth_dev_set_tx_queue_stats_mapping(uint16_t port_id, uint16_t tx_queue_id, - uint8_t stat_idx) + uint16_t stat_idx) { return eth_err(port_id, set_queue_stats_mapping(port_id, tx_queue_id, stat_idx, STAT_QMAP_TX)); @@ -2978,7 +2978,7 @@ rte_eth_dev_set_tx_queue_stats_mapping(uint16_t port_id, uint16_t tx_queue_id, int rte_eth_dev_set_rx_queue_stats_mapping(uint16_t port_id, uint16_t rx_queue_id, - uint8_t stat_idx) + uint16_t stat_idx) { return eth_err(port_id, set_queue_stats_mapping(port_id, rx_queue_id, stat_idx, STAT_QMAP_RX)); diff --git a/lib/librte_ethdev/rte_ethdev.h b/lib/librte_ethdev/rte_ethdev.h index eeef52e..fb10836 100644 --- a/lib/librte_ethdev/rte_ethdev.h +++ b/lib/librte_ethdev/rte_ethdev.h @@ -2669,7 +2669,7 @@ int rte_eth_xstats_reset(uint16_t port_id); * Zero if successful. Non-zero otherwise. */ int rte_eth_dev_set_tx_queue_stats_mapping(uint16_t port_id, - uint16_t tx_queue_id, uint8_t stat_idx); + uint16_t tx_queue_id, uint16_t stat_idx); /** * Set a mapping for the specified receive queue to the specified per-queue @@ -2690,7 +2690,7 @@ int rte_eth_dev_set_tx_queue_stats_mapping(uint16_t port_id, */ int rte_eth_dev_set_rx_queue_stats_mapping(uint16_t port_id, uint16_t rx_queue_id, - uint8_t stat_idx); + uint16_t stat_idx); /** * Retrieve the Ethernet address of an Ethernet device. diff --git a/lib/librte_ethdev/rte_ethdev_driver.h b/lib/librte_ethdev/rte_ethdev_driver.h index 23cc1e0..c68d465 100644 --- a/lib/librte_ethdev/rte_ethdev_driver.h +++ b/lib/librte_ethdev/rte_ethdev_driver.h @@ -222,7 +222,7 @@ typedef int (*eth_xstats_get_names_by_id_t)(struct rte_eth_dev *dev, typedef int (*eth_queue_stats_mapping_set_t)(struct rte_eth_dev *dev, uint16_t queue_id, - uint8_t stat_idx, + uint16_t stat_idx, uint8_t is_rx); /**< @internal Set a queue statistics mapping for a tx/rx queue of an Ethernet device. */ From patchwork Wed Sep 23 13:00:00 2020 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: "humin (Q)" X-Patchwork-Id: 78559 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 2305AA04B1; Wed, 23 Sep 2020 15:02:55 +0200 (CEST) Received: from [92.243.14.124] (localhost [127.0.0.1]) by dpdk.org (Postfix) with ESMTP id 618351D665; Wed, 23 Sep 2020 15:02:38 +0200 (CEST) Received: from huawei.com (szxga05-in.huawei.com [45.249.212.191]) by dpdk.org (Postfix) with ESMTP id E620D1D53D for ; Wed, 23 Sep 2020 15:02:33 +0200 (CEST) Received: from DGGEMS411-HUB.china.huawei.com (unknown [172.30.72.59]) by Forcepoint Email with ESMTP id 86CAFCA1036DB84EA891; Wed, 23 Sep 2020 21:02:30 +0800 (CST) Received: from localhost.localdomain (10.69.192.56) by DGGEMS411-HUB.china.huawei.com (10.3.19.211) with Microsoft SMTP Server id 14.3.487.0; Wed, 23 Sep 2020 21:02:29 +0800 From: "Min Hu (Connor)" To: CC: , , , Date: Wed, 23 Sep 2020 21:00:00 +0800 Message-ID: <1600866002-35908-3-git-send-email-humin29@huawei.com> X-Mailer: git-send-email 2.7.4 In-Reply-To: <1600866002-35908-1-git-send-email-humin29@huawei.com> References: <1599219135-53194-2-git-send-email-humin29@huawei.com> <1600866002-35908-1-git-send-email-humin29@huawei.com> MIME-Version: 1.0 X-Originating-IP: [10.69.192.56] X-CFilter-Loop: Reflected Subject: [dpdk-dev] [PATCH V3 2/4] ethdev: change data type in TC rxq and TC txq 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" From: Huisong Li Currently, base and nb_queue in the tc_rxq and tc_txq information of queue and TC mapping on both TX and RX paths are uint8_t. However, these data will be truncated when queue number under a TC is greater than 256. So it is necessary for base and nb_queue to change from uint8_t to uint16_t. Fixes: 01eb53eefeb40e8 ("ethdev: rename folder to library name") Signed-off-by: Huisong Li Signed-off-by: Min Hu (Connor) Reviewed-by: Wei Hu (Xavier) Reviewed-by: Dongdong Liu --- lib/librte_ethdev/rte_ethdev.h | 8 ++++---- 1 file changed, 4 insertions(+), 4 deletions(-) diff --git a/lib/librte_ethdev/rte_ethdev.h b/lib/librte_ethdev/rte_ethdev.h index fb10836..6dfe666 100644 --- a/lib/librte_ethdev/rte_ethdev.h +++ b/lib/librte_ethdev/rte_ethdev.h @@ -1519,13 +1519,13 @@ struct rte_eth_xstat_name { struct rte_eth_dcb_tc_queue_mapping { /** rx queues assigned to tc per Pool */ struct { - uint8_t base; - uint8_t nb_queue; + uint16_t base; + uint16_t nb_queue; } tc_rxq[ETH_MAX_VMDQ_POOL][ETH_DCB_NUM_TCS]; /** rx queues assigned to tc per Pool */ struct { - uint8_t base; - uint8_t nb_queue; + uint16_t base; + uint16_t nb_queue; } tc_txq[ETH_MAX_VMDQ_POOL][ETH_DCB_NUM_TCS]; }; From patchwork Wed Sep 23 13:00:01 2020 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: "humin (Q)" X-Patchwork-Id: 78561 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 6C641A04B1; Wed, 23 Sep 2020 15:03:21 +0200 (CEST) Received: from [92.243.14.124] (localhost [127.0.0.1]) by dpdk.org (Postfix) with ESMTP id 798ED1DBA0; Wed, 23 Sep 2020 15:02:56 +0200 (CEST) Received: from huawei.com (szxga06-in.huawei.com [45.249.212.32]) by dpdk.org (Postfix) with ESMTP id CA66C1D65A for ; Wed, 23 Sep 2020 15:02:37 +0200 (CEST) Received: from DGGEMS411-HUB.china.huawei.com (unknown [172.30.72.59]) by Forcepoint Email with ESMTP id 951949BDFAB3F972C8E4; Wed, 23 Sep 2020 21:02:35 +0800 (CST) Received: from localhost.localdomain (10.69.192.56) by DGGEMS411-HUB.china.huawei.com (10.3.19.211) with Microsoft SMTP Server id 14.3.487.0; Wed, 23 Sep 2020 21:02:30 +0800 From: "Min Hu (Connor)" To: CC: , , , Date: Wed, 23 Sep 2020 21:00:01 +0800 Message-ID: <1600866002-35908-4-git-send-email-humin29@huawei.com> X-Mailer: git-send-email 2.7.4 In-Reply-To: <1600866002-35908-1-git-send-email-humin29@huawei.com> References: <1599219135-53194-2-git-send-email-humin29@huawei.com> <1600866002-35908-1-git-send-email-humin29@huawei.com> MIME-Version: 1.0 X-Originating-IP: [10.69.192.56] X-CFilter-Loop: Reflected Subject: [dpdk-dev] [PATCH V3 3/4] doc: announce modified field in DCB TC queue mapping 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" From: Huisong Li Field type of base and nb_queue will be modified from uint8_t to uint16_t in struct "rte_eth_dcb_tc_queue_mapping". As the data of uint8_t will be truncated when queue number under a TC is greater than 256. Signed-off-by: Huisong Li Signed-off-by: Min Hu (Connor) Reviewed-by: Wei Hu (Xavier) --- v2->v3: add announce modified field in DCB TC queue mapping. --- doc/guides/rel_notes/deprecation.rst | 5 +++++ doc/guides/rel_notes/release_20_11.rst | 4 ++++ 2 files changed, 9 insertions(+) diff --git a/doc/guides/rel_notes/deprecation.rst b/doc/guides/rel_notes/deprecation.rst index 444dcbe..cc744d8 100644 --- a/doc/guides/rel_notes/deprecation.rst +++ b/doc/guides/rel_notes/deprecation.rst @@ -134,6 +134,11 @@ Deprecation Notices and the related structures (``rte_fdir_*`` and ``rte_eth_fdir_*``), will be removed in DPDK 20.11. +* ethdev: Modified field type of base and nb_queue, from uint8_t to uint16_t in + struct ``rte_eth_dcb_tc_queue_mapping``. As the data of uint8_t will be + truncated when queue number under a TC is greater than 256. This will be + modified in DPDK 20.11. + * ethdev: The legacy L2 tunnel filtering API is deprecated as the rest of the legacy filtering API. The functions ``rte_eth_dev_l2_tunnel_eth_type_conf`` and diff --git a/doc/guides/rel_notes/release_20_11.rst b/doc/guides/rel_notes/release_20_11.rst index d4a66d0..3de2895 100644 --- a/doc/guides/rel_notes/release_20_11.rst +++ b/doc/guides/rel_notes/release_20_11.rst @@ -149,6 +149,10 @@ API Changes * bpf: ``RTE_BPF_XTYPE_NUM`` has been dropped from ``rte_bpf_xtype``. +* ethdev: Modified field type of base and nb_queue, from uint8_t to uint16_t in + struct ``rte_eth_dcb_tc_queue_mapping``. As the data of uint8_t will be + truncated when queue number under a TC is greater than 256. + ABI Changes ----------- From patchwork Wed Sep 23 13:00:02 2020 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: "humin (Q)" X-Patchwork-Id: 78560 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 258A9A04B5; Wed, 23 Sep 2020 15:03:10 +0200 (CEST) Received: from [92.243.14.124] (localhost [127.0.0.1]) by dpdk.org (Postfix) with ESMTP id 439331DB82; Wed, 23 Sep 2020 15:02:55 +0200 (CEST) Received: from huawei.com (szxga05-in.huawei.com [45.249.212.191]) by dpdk.org (Postfix) with ESMTP id 8F7691D63D for ; Wed, 23 Sep 2020 15:02:36 +0200 (CEST) Received: from DGGEMS411-HUB.china.huawei.com (unknown [172.30.72.58]) by Forcepoint Email with ESMTP id BE8D921CCB153DBE0710; Wed, 23 Sep 2020 21:02:35 +0800 (CST) Received: from localhost.localdomain (10.69.192.56) by DGGEMS411-HUB.china.huawei.com (10.3.19.211) with Microsoft SMTP Server id 14.3.487.0; Wed, 23 Sep 2020 21:02:30 +0800 From: "Min Hu (Connor)" To: CC: , , , Date: Wed, 23 Sep 2020 21:00:02 +0800 Message-ID: <1600866002-35908-5-git-send-email-humin29@huawei.com> X-Mailer: git-send-email 2.7.4 In-Reply-To: <1600866002-35908-1-git-send-email-humin29@huawei.com> References: <1599219135-53194-2-git-send-email-humin29@huawei.com> <1600866002-35908-1-git-send-email-humin29@huawei.com> MIME-Version: 1.0 X-Originating-IP: [10.69.192.56] X-CFilter-Loop: Reflected Subject: [dpdk-dev] [PATCH V3 4/4] doc: announce modified in queue stats mapping API 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" From: Huisong Li Data type of stat_idx will be modified, from uint8_t to uint16_t in "set_queue_stats_mapping", "rte_eth_dev_set_tx_queue_stats_mapping", "rte_eth_dev_set_rx_queue_stats_mapping" for meeting the needs use 256 or more than 256 queues and display all statistics of rx/tx queue. Signed-off-by: Huisong Li Reviewed-by: Min Hu (Connor) Reviewed-by: Wei Hu (Xavier) --- v2->v3: add announce modified queue_stats_mapping API. --- doc/guides/rel_notes/deprecation.rst | 6 ++++++ doc/guides/rel_notes/release_20_11.rst | 4 ++++ 2 files changed, 10 insertions(+) diff --git a/doc/guides/rel_notes/deprecation.rst b/doc/guides/rel_notes/deprecation.rst index cc744d8..82eb959 100644 --- a/doc/guides/rel_notes/deprecation.rst +++ b/doc/guides/rel_notes/deprecation.rst @@ -192,6 +192,12 @@ Deprecation Notices following the IPv6 header, as proposed in RFC https://mails.dpdk.org/archives/dev/2020-August/177257.html. +* ethdev: Data type of input parameter ``stat_idx`` in ``set_queue_stats_mapping``, + ``rte_eth_dev_set_tx_queue_stats_mapping`` and ``rte_eth_dev_set_rx_queue_stats_mapping`` + function will be change from uint8_t to uint16_t, which supports that the needs + use 256 or more than 256 queues and display all statistics of rx/tx queue. + The change is planned for 20.11. + * vhost: Vhost-user dequeue zero-copy support will be removed in 20.11. The only known user is OVS where the feature is still experimental, and has not received any update for 2.5 years. diff --git a/doc/guides/rel_notes/release_20_11.rst b/doc/guides/rel_notes/release_20_11.rst index 3de2895..6149199 100644 --- a/doc/guides/rel_notes/release_20_11.rst +++ b/doc/guides/rel_notes/release_20_11.rst @@ -153,6 +153,10 @@ API Changes struct ``rte_eth_dcb_tc_queue_mapping``. As the data of uint8_t will be truncated when queue number under a TC is greater than 256. +* ethdev: Data type of input parameter ``stat_idx`` in ``set_queue_stats_mapping``, + ``rte_eth_dev_set_tx_queue_stats_mapping`` and ``rte_eth_dev_set_rx_queue_stats_mapping`` + function will be change from uint8_t to uint16_t, which supports that the needs + use 256 or more than 256 queues and display all statistics of rx/tx queue. ABI Changes -----------