From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from dpdk.org (dpdk.org [92.243.14.124]) by inbox.dpdk.org (Postfix) with ESMTP id B8399A00C5; Thu, 11 Jun 2020 10:08:16 +0200 (CEST) Received: from [92.243.14.124] (localhost [127.0.0.1]) by dpdk.org (Postfix) with ESMTP id C5B3414583; Thu, 11 Jun 2020 10:08:15 +0200 (CEST) Received: from relay12.mail.gandi.net (relay12.mail.gandi.net [217.70.178.232]) by dpdk.org (Postfix) with ESMTP id B327614581 for ; Thu, 11 Jun 2020 10:08:14 +0200 (CEST) Received: from u256.net (lfbn-idf2-1-566-132.w86-246.abo.wanadoo.fr [86.246.31.132]) (Authenticated sender: grive@u256.net) by relay12.mail.gandi.net (Postfix) with ESMTPSA id 2FA72200018; Thu, 11 Jun 2020 08:08:13 +0000 (UTC) Date: Thu, 11 Jun 2020 10:08:08 +0200 From: =?utf-8?Q?Ga=C3=ABtan?= Rivet To: Maxime Coquelin Cc: dev@dpdk.org, david.marchand@redhat.com, wenzhuo.lu@intel.com, beilei.xing@intel.com, bernard.iremonger@intel.com Message-ID: <20200611080808.pvq3xf2ksdwzbsdy@u256.net> References: <20200608155330.163874-1-maxime.coquelin@redhat.com> <20200608155330.163874-2-maxime.coquelin@redhat.com> MIME-Version: 1.0 Content-Type: text/plain; charset=iso-8859-1 Content-Disposition: inline Content-Transfer-Encoding: 8bit In-Reply-To: <20200608155330.163874-2-maxime.coquelin@redhat.com> Subject: Re: [dpdk-dev] [PATCH 1/2] doc: announce rte_dev_probe() API change 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" On 08/06/20 17:53 +0200, Maxime Coquelin wrote: > In order to simplify the use of rte_dev_probe() and > rte_dev_remove() by applications, rte_dev_probe() will > return a reference on the rte_device stating DPDK v20.11. > > Signed-off-by: Maxime Coquelin > --- > doc/guides/rel_notes/deprecation.rst | 5 +++++ > 1 file changed, 5 insertions(+) > > diff --git a/doc/guides/rel_notes/deprecation.rst b/doc/guides/rel_notes/deprecation.rst > index 0bee924255..47151eac0b 100644 > --- a/doc/guides/rel_notes/deprecation.rst > +++ b/doc/guides/rel_notes/deprecation.rst > @@ -138,3 +138,8 @@ Deprecation Notices > driver probe scheme. The legacy virtio support will be available through > the existing VFIO/UIO based kernel driver scheme. > More details at https://patches.dpdk.org/patch/69351/ On the principle ok, formulation seems heavy though (but I'm not a native speaker so ymmv...): > + > +* eal: Change ``rte_dev_probe`` API to return a pointer on the probed > + rte_device or NULL instead of 0 or an error code in DPDK v20.11. The 'in DPDK v20.11' is confusing here (it could equally apply to first or second part of the sentence). Given the context it's obvious, but maybe: Change ``rte_dev_probe`` API in DPDK v20.11 to return a pointer on ... > + This > + change will help calling application in avoiding to iterate the devices > + list when willing to call rte_dev_remove() later. How about: This change will allow applications avoid iterating on devices after a probe to get access to the new rte_device. > -- > 2.26.2 > -- Gaëtan