Re: [dpdk-dev] [RFC 0/4] Include resources in tests

[Bruce Richardson <bruce.richardson@intel.com>]

On Thu, May 05, 2016 at 07:03:00PM +0200, Jan Viktorin wrote:
> On Thu, 5 May 2016 14:29:30 +0100
> Bruce Richardson <bruce.richardson@intel.com> wrote:
> 
> > 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. :-)
> 
> Hello, well, the patch set should be independent on that cleanup, so I take it as a
> bug report. There should be no dependency, sorry. I'll fix it and repost.
> 
> Thanks
> Jan
> 
It's the RTE_INIT macro is being missed otherwise.
I'm trying out this new infrastructure by reworking the lpm tests to use this
for the big routing table data. I'll let you know any feedback when I'm done.

	/Bruce