From mboxrd@z Thu Jan 1 00:00:00 1970 From: Bruce Richardson Subject: Re: [RFC 0/4] Include resources in tests Date: Thu, 5 May 2016 14:29:30 +0100 Message-ID: <20160505132930.GB14228@bricha3-MOBL3> References: <1461935496-20367-1-git-send-email-viktorin@rehivetech.com> Mime-Version: 1.0 Content-Type: text/plain; charset=us-ascii Cc: dev@dpdk.org, Thomas Monjalon , David Marchand To: Jan Viktorin Return-path: Received: from mga02.intel.com (mga02.intel.com [134.134.136.20]) by dpdk.org (Postfix) with ESMTP id 8F87D37B8 for ; Thu, 5 May 2016 15:29:42 +0200 (CEST) Content-Disposition: inline In-Reply-To: <1461935496-20367-1-git-send-email-viktorin@rehivetech.com> List-Id: patches and discussions about DPDK List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: dev-bounces@dpdk.org Sender: "dev" On Fri, Apr 29, 2016 at 03:11:32PM +0200, Jan Viktorin wrote: > Hello, > > this patch set introduces a mechanism to include a resource (in general a blob) > into the test binary. This allows to make tests less dependent on the target > testing environment. The first use case is testing of PCI bus scan by changing > the hard-coded path (/sys/bus/pci/devices) to something different and provide > a fake tree of devices with the test. It can help with testing of device-tree > parsing as I've proposed in [1] where such mechanism was missing at that time. > I'd like to use such framework for the SoC infra testing as well. > > The patch set introduces a struct resource into the app/test. The resource is > generic to include any kind of binary data. The binary data can be created in > C or linked as an object file (created by objcopy). I am not sure where to > place the objcopy logic and how to perform guessing of the objcopy arguments > as they are pretty non-standard. > > To include a complex resource (a file hierarchy), the last patch implements > an archive extraction logic. So, it is possible to include a tar archive and > unpack it before a test starts. Any ideas how to do this in a better way are > welcome. > > [1] http://comments.gmane.org/gmane.comp.networking.dpdk.devel/36545 > > Regards > Jan Viktorin > BTW: It looks like your patch has a dependency on the 17-patch dev cleanup set from David, [and now on the header include bug fix I submitted yesterday too], so you should probably note that in any future revs of the patch you do. Save some head-scratching from those of us testing it out. :-) Thanks, /Bruce