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 8595B46810; Wed, 28 May 2025 10:24:00 +0200 (CEST) Received: from mails.dpdk.org (localhost [127.0.0.1]) by mails.dpdk.org (Postfix) with ESMTP id 221D7402A8; Wed, 28 May 2025 10:24:00 +0200 (CEST) Received: from frasgout.his.huawei.com (frasgout.his.huawei.com [185.176.79.56]) by mails.dpdk.org (Postfix) with ESMTP id 7F42F40279 for ; Wed, 28 May 2025 10:23:59 +0200 (CEST) Received: from mail.maildlp.com (unknown [172.18.186.31]) by frasgout.his.huawei.com (SkyGuard) with ESMTP id 4b6j9j5mVkz6K6Z4; Wed, 28 May 2025 16:20:25 +0800 (CST) Received: from frapeml100008.china.huawei.com (unknown [7.182.85.131]) by mail.maildlp.com (Postfix) with ESMTPS id AD40F140277; Wed, 28 May 2025 16:23:58 +0800 (CST) Received: from frapeml500007.china.huawei.com (7.182.85.172) by frapeml100008.china.huawei.com (7.182.85.131) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384) id 15.1.2507.39; Wed, 28 May 2025 10:23:58 +0200 Received: from frapeml500007.china.huawei.com ([7.182.85.172]) by frapeml500007.china.huawei.com ([7.182.85.172]) with mapi id 15.01.2507.039; Wed, 28 May 2025 10:23:58 +0200 From: Konstantin Ananyev To: Konstantin Ananyev , =?iso-8859-1?Q?Morten_Br=F8rup?= , "Stephen Hemminger" , "skori@marvell.com" CC: Shepard Siegel , Ed Czeck , John Miller , "Igor Russkikh" , Ajit Khaparde , Somnath Kotur , Nithin Dabilpuram , Kiran Kumar K , Satha Rao , Harman Kalra , Hemant Agrawal , Sachin Saxena , John Daley , Hyong Youb Kim , Ian Stokes , Bruce Richardson , Vladimir Medvedkin , Anatoly Burakov , Dariusz Sosnowski , Viacheslav Ovsiienko , Bing Zhao , Ori Kam , Suanming Mou , Matan Azrad , Long Li , Wei Hu , Chaoyong He , Jiawen Wu , Andrew Rybchenko , Jerin Jacob , Maciej Czekaj , "Jian Wang" , Maxime Coquelin , Chenbo Xia , Jochen Behrens , Thomas Monjalon , "Ferruh Yigit" , "dev@dpdk.org" Subject: RE: [PATCH v2 1/2] ethdev: remove unnecessary type conversion Thread-Topic: [PATCH v2 1/2] ethdev: remove unnecessary type conversion Thread-Index: AQHbw0/AjHqMNRGXNUegkZRPaA3HnbPmidiAgAAVuwCAASoo4IAAAtRw Date: Wed, 28 May 2025 08:23:58 +0000 Message-ID: References: <20250429181132.2544771-1-skori@marvell.com> <20250512150732.65743-1-skori@marvell.com> <20250527080701.712af797@hermes.local> <98CBD80474FA8B44BF855DF32C47DC35E9FC9B@smartserver.smartshare.dk> <211102eb85854ab2b64f34e72603587c@huawei.com> In-Reply-To: <211102eb85854ab2b64f34e72603587c@huawei.com> Accept-Language: en-GB, en-US Content-Language: en-US X-MS-Has-Attach: X-MS-TNEF-Correlator: x-originating-ip: [10.126.172.111] Content-Type: text/plain; charset="iso-8859-1" Content-Transfer-Encoding: quoted-printable MIME-Version: 1.0 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 > > > > > From: Stephen Hemminger [mailto:stephen@networkplumber.org] > > > Sent: Tuesday, 27 May 2025 17.07 > > > > > > On Mon, 12 May 2025 20:37:19 +0530 > > > wrote: > > > > > > > /**@{@name Rx hardware descriptor states > > > > diff --git a/lib/ethdev/rte_ethdev_core.h > > > b/lib/ethdev/rte_ethdev_core.h > > > > index e55fb42996..4ffae4921a 100644 > > > > --- a/lib/ethdev/rte_ethdev_core.h > > > > +++ b/lib/ethdev/rte_ethdev_core.h > > > > @@ -45,7 +45,7 @@ typedef uint16_t (*eth_tx_prep_t)(void *txq, > > > > > > > > > > > > /** @internal Get number of used descriptors on a receive queue. *= / > > > > -typedef uint32_t (*eth_rx_queue_count_t)(void *rxq); > > > > +typedef int (*eth_rx_queue_count_t)(void *rxq); > > > > > > > > /** @internal Check the status of a Rx descriptor */ > > > > typedef int (*eth_rx_descriptor_status_t)(void *rxq, uint16_t > > > offset); > > > > > > > > > This gets reported as ABI breakage. The change will have to wait unti= l > > > next LTS (25.11) > > > > The return type was weird (wrong) to begin with. > > When used, it gets cast to int: > > https://elixir.bootlin.com/dpdk/v25.03/source/lib/ethdev/rte_ethdev.h#L= 6404 >=20 > Personally, I don't see anything strange here. > devops rx_queue_count() supposed to return uint, because it should never = failed for > valid queue. > While rte_eth_rx_queue_count() itself can fail - wrong port/queue id, etc= . BTW, rx_queue_setup() accepts only uint16_t for number of rx descritoirs: int rte_eth_rx_queue_setup(uint16_t port_id, uint16_t rx_queue_id, uint16_t nb_rx_desc, unsigned int socket_id, const struct rte_eth_rxconf *rx_conf, struct rte_mempool *mb_pool); shouldn't dev->rx_queue_count() then also return uitn16_t (for consistency)= ? > > > > You are right that it formally changes the ABI, and we should go throug= h the LTS motions. > > But, for this change, I'd favor an exception. >=20 > Again, from my opinion, there is nothing that urgent/important why such c= hanges (if needed) > can't wait till next LTS. > For now, we can simply do type conversion explicitly at rte_eth_rx_queue_= count(). >=20 > > PS: As a consequence of this change, a patch to update the return type = of the callback in all the ethdev drivers should be provided. > > > > > > > > > > > [C] 'rte_eth_fp_ops rte_eth_fp_ops[32]' was changed at > > > rte_ethdev.c:47:1: > > > type of variable changed: > > > array element type 'struct rte_eth_fp_ops' changed: > > > type size hasn't changed > > > 1 data member change: > > > type of 'eth_rx_queue_count_t rx_queue_count' changed: > > > underlying type 'uint32_t (*)(void*)' changed: > > > in pointed to type 'function type uint32_t (void*)': > > > return type changed: > > > entity changed from 'typedef uint32_t' to 'int' > > > type size hasn't changed > > > type size hasn't changed