From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from wes1-so1.wedos.net (wes1-so1.wedos.net [46.28.106.15]) by dpdk.org (Postfix) with ESMTP id BE22F6A94 for ; Thu, 12 May 2016 23:00:41 +0200 (CEST) Received: from jvn (dynamic-109-81-211-56.ipv4.broadband.iol.cz [109.81.211.56]) by wes1-so1.wedos.net (Postfix) with ESMTPSA id 3r5QPP1cdyz4qg; Thu, 12 May 2016 23:00:41 +0200 (CEST) Date: Thu, 12 May 2016 23:00:44 +0200 From: Jan Viktorin To: Thomas Monjalon Cc: dev@dpdk.org, David Marchand , Bruce Richardson Message-ID: <20160512230044.654b1588@jvn> In-Reply-To: <1659775.2ddTAuyrkI@xps13> References: <1462904011-29838-1-git-send-email-viktorin@rehivetech.com> <2242767.BO2sf06KAx@xps13> <20160512175307.0573297e@pcviktorin.fit.vutbr.cz> <1659775.2ddTAuyrkI@xps13> Organization: RehiveTech X-Mailer: Claws Mail 3.13.0 (GTK+ 2.24.28; x86_64-unknown-linux-gnu) MIME-Version: 1.0 Content-Type: text/plain; charset=US-ASCII Content-Transfer-Encoding: 7bit Subject: Re: [dpdk-dev] [PATCH v2 06/11] app/test: use linked list to store PCI drivers X-BeenThere: dev@dpdk.org X-Mailman-Version: 2.1.15 Precedence: list List-Id: patches and discussions about DPDK List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , X-List-Received-Date: Thu, 12 May 2016 21:00:41 -0000 On Thu, 12 May 2016 18:08:16 +0200 Thomas Monjalon wrote: > 2016-05-12 17:53, Jan Viktorin: > > On Thu, 12 May 2016 17:31:28 +0200 > > Thomas Monjalon wrote: > > > > > 2016-05-10 20:13, Jan Viktorin: > > > > The test unregisters all real drivers before starting into an array. This > > > > inflexiable as we can use a linked list for this purpose. > > > > > > I don't understand this. Maybe some words are missing. > > > > Better? > > > > The test unregisters all real drivers before starting (stored into an array). > > This is inflexiable (due to its fixed size) and we can use a linked list for > > this purpose. > > Better with a past tense? "The test was unregistering..." No. This patch does not change the semantics (well, it does, because the list of drivers become global but that's all). The test still unregisters the drivers. That is the important fact to consider. Perhaps: The test unregisters all drivers before start. The drivers were stored into a fixed-sized array. This is inflexible. This patch change this to utilize a linked list for the same purpose. > > > > > +/* real drivers (not used for testing) */ > > > > > > What do mean by "not used for testing"? > > > > The test now avoids the DPDK builtin drivers. It only considers its > > internal fake drivers my_driver and my_driver2. So the real drivers > > are temporarily store into the real_pci_driver_list and returned back > > after the test finishes. > > Maybe adding "Save" or "Backup" would make it clear. Ok, makes sense. > > > It is the linked list mentioned in the commit log. It replaces the > > original fixed-size array. > > > > (For drivers, it does not matter that much. But for devices, I think, > > it is not a good practice to consider them in autotests. Every PC > > where you execute the tests have different set of PCI devices.) > > Yes Thanks for comments! Jan -- Jan Viktorin E-mail: Viktorin@RehiveTech.com System Architect Web: www.RehiveTech.com RehiveTech Brno, Czech Republic