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 EF7D9A034F for ; Wed, 10 Nov 2021 07:44:12 +0100 (CET) Received: from [217.70.189.124] (localhost [127.0.0.1]) by mails.dpdk.org (Postfix) with ESMTP id E9BEA40E03; Wed, 10 Nov 2021 07:44:12 +0100 (CET) Received: from NAM10-DM6-obe.outbound.protection.outlook.com (mail-dm6nam10on2086.outbound.protection.outlook.com [40.107.93.86]) by mails.dpdk.org (Postfix) with ESMTP id BDC7B4014D for ; Wed, 10 Nov 2021 07:44:11 +0100 (CET) ARC-Seal: i=1; a=rsa-sha256; s=arcselector9901; d=microsoft.com; cv=none; b=Pa8PpdR3CnEM2O51uSfu+kn1Q2sTb6DM73AYYD6kQ7V+LEo2NY4ImkWAKLEu7UHCjzL64BVIeB5NqAMt52YPlIpr8b0HiGA0ttKLaiovszE0G/MzPaw5pnhHreciZ41ijprHoD5+cwlV9UQKil+oh5Sft3qcFHKpN/OqK9rPw3qKoxrLg2v0Bb/ZgWu74CACM7YRJpNXQ19k0V9PDQm8gs6pOjEFMCUBzNu3UuRch6KB+xXobEj0gzmcZ4Ohlawo/+DOWvS6ozi8QyomSvnvp02HBGpVR0FPNNLFlASKWGoP0RSSgswksAf6HMDhmJsoPR9REFa11pU2IvDhBHjZgA== ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=microsoft.com; s=arcselector9901; h=From:Date:Subject:Message-ID:Content-Type:MIME-Version:X-MS-Exchange-AntiSpam-MessageData-ChunkCount:X-MS-Exchange-AntiSpam-MessageData-0:X-MS-Exchange-AntiSpam-MessageData-1; bh=K1YypDRxdTvSJHlfwMq0O5EkTCnrjnrrmyDjJzmDpIs=; b=UFy7oCQFGA0469EOZ0r/N68pEzDvOnWYrDattwF7pfEb/tkT70FDUlKnOfw7J9REyp+vIxe9tC7CynUvxYa4BtceTwZnM/BWNdozsP0fCHh1fHHIAZX7ljXNlr/yP6amivlxIncaPXr+RMFXrALzuf4Gg0N1RsCWY6VMSoediOIAS6v9+0M/2ow9V0t2IEEHXAqd4Aj2iwNGnwJ6tSos1Cxm60aocR6K81b3aiImuDvcsE1qbKBMtjnrbRGS53hIYQYsR/o4X861d8Nw4WLaMs6knRLjkFlvxemaArnUzWZ1HxVehEfUX4sIeIcVZ5JFcvjnUDbBJC96xc60bW5VNA== ARC-Authentication-Results: i=1; mx.microsoft.com 1; spf=pass (sender ip is 216.228.112.34) smtp.rcpttodomain=xilinx.com smtp.mailfrom=nvidia.com; dmarc=pass (p=quarantine sp=quarantine pct=100) action=none header.from=nvidia.com; dkim=none (message not signed); arc=none DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=Nvidia.com; s=selector2; h=From:Date:Subject:Message-ID:Content-Type:MIME-Version:X-MS-Exchange-SenderADCheck; bh=K1YypDRxdTvSJHlfwMq0O5EkTCnrjnrrmyDjJzmDpIs=; b=Rug7H5cGp9lOcZrw8/sHGk+7caYSezgQKbgDPxBLyMmCkO54VAbAKUQiVjLmpikhIpntc1ScoB6KBLAlppARN1BITir7u2DCritU63udVHd1vpIQ4lGURLKjGb2qDyy0EEPuRpukZLCqQi1R4x5wi22LlAcge7L6A6Eh42OirIchZORREEzwDSE4k1AhOOrjhWFsiKfcS9xh9DRlgfYkpHU/WFl8ORBe2Cm0QdNb3fwdThJ+zzwPubLqPEDhsm0MucoATb8VrHwDs/xRzicNLPZI9gNpKLrj+yQRTRXKpUtAXLVdapPo59iJxWftLBUTT4uo3k1vgJDjUt3UAE92yA== Received: from DM6PR13CA0069.namprd13.prod.outlook.com (2603:10b6:5:134::46) by DM8PR12MB5445.namprd12.prod.outlook.com (2603:10b6:8:24::7) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384) id 15.20.4669.16; Wed, 10 Nov 2021 06:44:10 +0000 Received: from DM6NAM11FT021.eop-nam11.prod.protection.outlook.com (2603:10b6:5:134:cafe::c3) by DM6PR13CA0069.outlook.office365.com (2603:10b6:5:134::46) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384) id 15.20.4690.5 via Frontend Transport; Wed, 10 Nov 2021 06:44:10 +0000 X-MS-Exchange-Authentication-Results: spf=pass (sender IP is 216.228.112.34) smtp.mailfrom=nvidia.com; xilinx.com; dkim=none (message not signed) header.d=none;xilinx.com; dmarc=pass action=none header.from=nvidia.com; Received-SPF: Pass (protection.outlook.com: domain of nvidia.com designates 216.228.112.34 as permitted sender) receiver=protection.outlook.com; client-ip=216.228.112.34; helo=mail.nvidia.com; Received: from mail.nvidia.com (216.228.112.34) by DM6NAM11FT021.mail.protection.outlook.com (10.13.173.76) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_256_CBC_SHA384) id 15.20.4690.15 via Frontend Transport; Wed, 10 Nov 2021 06:44:09 +0000 Received: from nvidia.com (172.20.187.6) by HQMAIL107.nvidia.com (172.20.187.13) with Microsoft SMTP Server (TLS) id 15.0.1497.18; Wed, 10 Nov 2021 06:44:06 +0000 From: Xueming Li To: Ivan Ilchenko CC: Luca Boccassi , Andrew Rybchenko , Andy Moreton , Ferruh Yigit , dpdk stable Date: Wed, 10 Nov 2021 14:29:50 +0800 Message-ID: <20211110063216.2744012-107-xuemingl@nvidia.com> X-Mailer: git-send-email 2.33.0 In-Reply-To: <20211110063216.2744012-1-xuemingl@nvidia.com> References: <20211110063216.2744012-1-xuemingl@nvidia.com> MIME-Version: 1.0 Content-Transfer-Encoding: 8bit Content-Type: text/plain X-Originating-IP: [172.20.187.6] X-ClientProxiedBy: HQMAIL101.nvidia.com (172.20.187.10) To HQMAIL107.nvidia.com (172.20.187.13) X-EOPAttributedMessage: 0 X-MS-PublicTrafficType: Email X-MS-Office365-Filtering-Correlation-Id: d007e189-291f-4a7b-c9ee-08d9a415800f X-MS-TrafficTypeDiagnostic: DM8PR12MB5445: X-Microsoft-Antispam-PRVS: X-MLNXRule-Acc-MList-2: Rule triggered X-MS-Oob-TLC-OOBClassifiers: OLM:10000; X-MS-Exchange-SenderADCheck: 1 X-MS-Exchange-AntiSpam-Relay: 0 X-Microsoft-Antispam: BCL:0; X-Microsoft-Antispam-Message-Info: MEVwThPQ8/SmGpTt6hXDFU4rQnkh6pKiaDfvzO2WkiDHcYzfPgmbnNZ4COy4dLnD4c5IJkawxzasWELVxAlxVIduGcYuTGyNeBxxVwLcMGXSf86hzVytzJ96v9kecA396j747WQsype015WJ/zkC+Z8J5pl/H+7JBU+5LDkz679/DMrbJMo7lUjfTsRZuMGaHG7auSNutQcwHy+RuvMaYXfqlJnqp3nFv3pB+deg1j4etNiccHx4YoFMCaU5jC6bFmEeyN6uXzHuEOkw5HOk9hf3TAPAJelJmWMJSSI/spHVahpb2jRA/1HgeMaFNtBG90z+tDJWV4jUBVbffgxS12XUZHEzq359hULA6t7tGhv1eUCjSJNnUdKts6q+ZwN19dbDPVDlUoCnnMipbxmIMvIfR1n7D9QRNVYMzqG2lLgh7Zi5fwVXWZwrXLx/STQACdWq2yZ06868vTUE5m7Dno17wPVJ1oSKKvENwzIwTF8kokDNkT6VHaGpQuiew8xsgTUWtlWECHCiMve2s3q9T9lM6V88eLtps/UGOhwufFxeJv1TjJD0/HzO6VscCsckjrG8Q3ex/en0Yj+N3gD4LFWkX+E/9mhaUek3CHd5kn39gwI9vkmPZRHviLpIInvuZ5bKBeVaorEIaXz6lGj+0M4xMRoG1sm0E6daayuhuP82jqPrtfTDNaUVzxsJJBLi62mz+8te8k3N2pKzPEW46wphyYkgDVePFyZMC79SWDbz1vsZHMJPgi2VpVMyKpu7BGtwMV/gw/XZZAiKiZos6aXPUKARkcNflLniY/mFhst7wdHvFqR2h6D5EnH5+nZ1eZGTmDEcOJg6Udm93TpmVQ== X-Forefront-Antispam-Report: CIP:216.228.112.34; CTRY:US; LANG:en; SCL:1; SRV:; IPV:NLI; SFV:NSPM; H:mail.nvidia.com; PTR:schybrid03.nvidia.com; CAT:NONE; SFS:(4636009)(36840700001)(46966006)(426003)(966005)(54906003)(1076003)(316002)(6916009)(36906005)(16526019)(70206006)(26005)(53546011)(186003)(8676002)(82310400003)(2906002)(508600001)(8936002)(4326008)(5660300002)(7696005)(83380400001)(70586007)(2616005)(356005)(55016002)(336012)(86362001)(6286002)(36860700001)(36756003)(47076005)(4001150100001)(6666004)(7636003); DIR:OUT; SFP:1101; X-OriginatorOrg: Nvidia.com X-MS-Exchange-CrossTenant-OriginalArrivalTime: 10 Nov 2021 06:44:09.8353 (UTC) X-MS-Exchange-CrossTenant-Network-Message-Id: d007e189-291f-4a7b-c9ee-08d9a415800f X-MS-Exchange-CrossTenant-Id: 43083d15-7273-40c1-b7db-39efd9ccc17a X-MS-Exchange-CrossTenant-OriginalAttributedTenantConnectingIp: TenantId=43083d15-7273-40c1-b7db-39efd9ccc17a; Ip=[216.228.112.34]; Helo=[mail.nvidia.com] X-MS-Exchange-CrossTenant-AuthSource: DM6NAM11FT021.eop-nam11.prod.protection.outlook.com X-MS-Exchange-CrossTenant-AuthAs: Anonymous X-MS-Exchange-CrossTenant-FromEntityHeader: HybridOnPrem X-MS-Exchange-Transport-CrossTenantHeadersStamped: DM8PR12MB5445 Subject: [dpdk-stable] patch 'ethdev: fix xstats by ID API documentation' has been queued to stable release 20.11.4 X-BeenThere: stable@dpdk.org X-Mailman-Version: 2.1.29 Precedence: list List-Id: patches for DPDK stable branches List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: stable-bounces@dpdk.org Sender: "stable" Hi, FYI, your patch has been queued to stable release 20.11.4 Note it hasn't been pushed to http://dpdk.org/browse/dpdk-stable yet. It will be pushed if I get no objections before 11/12/21. So please shout if anyone has objections. Also note that after the patch there's a diff of the upstream commit vs the patch applied to the branch. This will indicate if there was any rebasing needed to apply to the stable branch. If there were code changes for rebasing (ie: not only metadata diffs), please double check that the rebase was correctly done. Queued patches are on a temporary branch at: https://github.com/steevenlee/dpdk This queued commit can be viewed at: https://github.com/steevenlee/dpdk/commit/8f825b3b132ba3f0f0a30ff8e6c135af3495320a Thanks. Xueming Li --- >From 8f825b3b132ba3f0f0a30ff8e6c135af3495320a Mon Sep 17 00:00:00 2001 From: Ivan Ilchenko Date: Fri, 1 Oct 2021 12:07:19 +0300 Subject: [PATCH] ethdev: fix xstats by ID API documentation Cc: Xueming Li [ upstream commit bc5112ca599c1e565c0f2d3dc3de0f73381a8c05 ] Document valid combinations of input arguments in accordance with current implementation in ethdev. Fixes: 79c913a42f0e ("ethdev: retrieve xstats by ID") Signed-off-by: Ivan Ilchenko Signed-off-by: Andrew Rybchenko Reviewed-by: Andy Moreton Reviewed-by: Ferruh Yigit --- lib/librte_ethdev/rte_ethdev.h | 35 ++++++++++++++++++---------------- 1 file changed, 19 insertions(+), 16 deletions(-) diff --git a/lib/librte_ethdev/rte_ethdev.h b/lib/librte_ethdev/rte_ethdev.h index de577ca075..57a2edf363 100644 --- a/lib/librte_ethdev/rte_ethdev.h +++ b/lib/librte_ethdev/rte_ethdev.h @@ -2835,21 +2835,23 @@ int rte_eth_xstats_get(uint16_t port_id, struct rte_eth_xstat *xstats, * @param port_id * The port identifier of the Ethernet device. * @param xstats_names - * An rte_eth_xstat_name array of at least *size* elements to - * be filled. If set to NULL, the function returns the required number - * of elements. - * @param ids - * IDs array given by app to retrieve specific statistics + * Array to be filled in with names of requested device statistics. + * Must not be NULL if @p ids are specified (not NULL). * @param size - * The size of the xstats_names array (number of elements). + * Number of elements in @p xstats_names array (if not NULL) and in + * @p ids array (if not NULL). Must be 0 if both array pointers are NULL. + * @param ids + * IDs array given by app to retrieve specific statistics. May be NULL to + * retrieve names of all available statistics or, if @p xstats_names is + * NULL as well, just the number of available statistics. * @return * - A positive value lower or equal to size: success. The return value * is the number of entries filled in the stats table. - * - A positive value higher than size: error, the given statistics table + * - A positive value higher than size: success. The given statistics table * is too small. The return value corresponds to the size that should * be given to succeed. The entries in the table are not valid and * shall not be used by the caller. - * - A negative value on error (invalid port id). + * - A negative value on error. */ int rte_eth_xstats_get_names_by_id(uint16_t port_id, @@ -2862,22 +2864,23 @@ rte_eth_xstats_get_names_by_id(uint16_t port_id, * @param port_id * The port identifier of the Ethernet device. * @param ids - * A pointer to an ids array passed by application. This tells which - * statistics values function should retrieve. This parameter - * can be set to NULL if size is 0. In this case function will retrieve - * all available statistics. + * IDs array given by app to retrieve specific statistics. May be NULL to + * retrieve all available statistics or, if @p values is NULL as well, + * just the number of available statistics. * @param values - * A pointer to a table to be filled with device statistics values. + * Array to be filled in with requested device statistics. + * Must not be NULL if ids are specified (not NULL). * @param size - * The size of the ids array (number of elements). + * Number of elements in @p values array (if not NULL) and in @p ids + * array (if not NULL). Must be 0 if both array pointers are NULL. * @return * - A positive value lower or equal to size: success. The return value * is the number of entries filled in the stats table. - * - A positive value higher than size: error, the given statistics table + * - A positive value higher than size: success: The given statistics table * is too small. The return value corresponds to the size that should * be given to succeed. The entries in the table are not valid and * shall not be used by the caller. - * - A negative value on error (invalid port id). + * - A negative value on error. */ int rte_eth_xstats_get_by_id(uint16_t port_id, const uint64_t *ids, uint64_t *values, unsigned int size); -- 2.33.0 --- Diff of the applied patch vs upstream commit (please double-check if non-empty: --- --- - 2021-11-10 14:17:06.696319804 +0800 +++ 0106-ethdev-fix-xstats-by-ID-API-documentation.patch 2021-11-10 14:17:01.867412785 +0800 @@ -1 +1 @@ -From bc5112ca599c1e565c0f2d3dc3de0f73381a8c05 Mon Sep 17 00:00:00 2001 +From 8f825b3b132ba3f0f0a30ff8e6c135af3495320a Mon Sep 17 00:00:00 2001 @@ -4,0 +5,3 @@ +Cc: Xueming Li + +[ upstream commit bc5112ca599c1e565c0f2d3dc3de0f73381a8c05 ] @@ -10 +12,0 @@ -Cc: stable@dpdk.org @@ -17 +19 @@ - lib/ethdev/rte_ethdev.h | 35 +++++++++++++++++++---------------- + lib/librte_ethdev/rte_ethdev.h | 35 ++++++++++++++++++---------------- @@ -20,5 +22,5 @@ -diff --git a/lib/ethdev/rte_ethdev.h b/lib/ethdev/rte_ethdev.h -index afdc53b674..2ad2bf2019 100644 ---- a/lib/ethdev/rte_ethdev.h -+++ b/lib/ethdev/rte_ethdev.h -@@ -2896,21 +2896,23 @@ int rte_eth_xstats_get(uint16_t port_id, struct rte_eth_xstat *xstats, +diff --git a/lib/librte_ethdev/rte_ethdev.h b/lib/librte_ethdev/rte_ethdev.h +index de577ca075..57a2edf363 100644 +--- a/lib/librte_ethdev/rte_ethdev.h ++++ b/lib/librte_ethdev/rte_ethdev.h +@@ -2835,21 +2835,23 @@ int rte_eth_xstats_get(uint16_t port_id, struct rte_eth_xstat *xstats, @@ -56 +58 @@ -@@ -2923,22 +2925,23 @@ rte_eth_xstats_get_names_by_id(uint16_t port_id, +@@ -2862,22 +2864,23 @@ rte_eth_xstats_get_names_by_id(uint16_t port_id,