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 at 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
