From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from wes1-so2.wedos.net (wes1-so2.wedos.net [46.28.106.16]) by dpdk.org (Postfix) with ESMTP id 0EE8C6A56 for ; Thu, 12 May 2016 17:54:56 +0200 (CEST) Received: from pcviktorin.fit.vutbr.cz (pcviktorin.fit.vutbr.cz [147.229.13.147]) by wes1-so2.wedos.net (Postfix) with ESMTPSA id 3r5Hcb5mbQzSt; Thu, 12 May 2016 17:54:55 +0200 (CEST) Date: Thu, 12 May 2016 17:53:07 +0200 From: Jan Viktorin To: Thomas Monjalon Cc: dev@dpdk.org, David Marchand , Bruce Richardson Message-ID: <20160512175307.0573297e@pcviktorin.fit.vutbr.cz> In-Reply-To: <2242767.BO2sf06KAx@xps13> References: <1462904011-29838-1-git-send-email-viktorin@rehivetech.com> <1462904011-29838-7-git-send-email-viktorin@rehivetech.com> <2242767.BO2sf06KAx@xps13> Organization: RehiveTech 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 15:54:56 -0000 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. > > > +/* 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. 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.) > > > +struct pci_driver_list real_pci_driver_list = > > + TAILQ_HEAD_INITIALIZER(real_pci_driver_list); > -- Jan Viktorin E-mail: Viktorin@RehiveTech.com System Architect Web: www.RehiveTech.com RehiveTech Brno, Czech Republic