since V4:
*Added condition in platform/linux-generic/Makefile.am to avoid building
these tests when not required.
*pktio_run prefixes $PATH (rather than suffix) to avoid possible
wrong pktio_main selection
All Patched unchanged except patch7 (validation: moving pktio
plt specific to platform).
----------
This series of patches comes following the request from Stuart to
see the effect of the new test structure on module pktio rather than on a
simpler module, such as 'random'.
Please, review carefully, keeping in mind that many of you have a better and
larger view on the whole environment... I can miss things :-) ...
Be also aware that after applying these patches, you end up with a mixed
environment: pktio will be ran from the platform side, while all other tests
will still be ran from the validation side: as a consequence, you will notice
that:
- the make check "grand total" (19 PASS) is now split 18 + 1: this split will
remain untill all tests are "moved" to the platform side.
- despite the creation of pktio own Makefile.am, a lot of complexity remains
in the validation Makefile.am: the complexity of the validation/Makefile.am
will reduce as tests gets moved, resulting in a simple list of modules.
- the tests results will spread between the validation and the platform sides.
I expect this to remain (depending on whether the test was platform dependent,
or not).
This approach also requires to build the things in the following order:
1) ODP (i.e: platform/<platform>/ things)
2) the platform agnostic tests (i.e: validation/test/ things), using ODP.
3) the platform dependent tests (platform/<platform>/test)
This changed is introduced by patch "0005-validation-changing-build-order.patch"
which is a hack to workaround what seems to be an autotools limitation:
"SUBDIRS = @MACRO@" works, while
"SUBDIRS = xxx/@MACRO@/yyy" does not work.
This is hopefully a temporary fix as I assume that this directory structure
will change: Having the ODP code in "platform/<platform>/*" and the tests in
"platform/<platform>/tests" seems to imply that tests are a subpart of ODP,
which is not true.
When this patch is accepted, other modeles conversion will follows.
Christophe Milard (8):
validation: preparing for main in tests
validation: own main in odp_pktio.c
validation: renaming in odp_pktio.c
validation: cosmetic fixes in odp_pktio.c
validation: creating own dir and lib for pktio
validation: changing build order
validation: moving pktio plt specific to platform
validation: removing obsolete environment variable
Makefile.am | 9 ++-
configure.ac | 6 +-
platform/Makefile.am | 1 -
platform/linux-generic/Makefile.am | 2 -
platform/linux-generic/test/.gitignore | 2 +
platform/linux-generic/test/Makefile.am | 14 +++-
platform/linux-generic/test/pktio/.gitignore | 2 +
platform/linux-generic/test/pktio/Makefile.am | 2 +
platform/linux-generic/test/{ => pktio}/pktio_env | 0
.../linux-generic/test/pktio/pktio_run | 51 ++++++-------
test/Makefile.inc | 7 +-
test/performance/odp_l2fwd_run | 4 +-
test/validation/.gitignore | 1 -
test/validation/Makefile.am | 51 ++++++-------
test/validation/Makefile.inc | 7 ++
test/validation/common/.gitignore | 2 +
test/validation/common/Makefile.am | 10 +++
test/validation/common/odp_cunit_common.c | 18 +++--
test/validation/common/odp_cunit_common.h | 3 +
test/validation/pktio/.gitignore | 2 +
test/validation/pktio/Makefile.am | 8 ++
test/validation/{odp_pktio.c => pktio/pktio.c} | 89 +++++++++++-----------
test/validation/pktio/pktio.h | 7 ++
test/validation/pktio/pktio_main.c | 12 +++
24 files changed, 193 insertions(+), 117 deletions(-)
delete mode 100644 platform/Makefile.am
create mode 100644 platform/linux-generic/test/.gitignore
create mode 100644 platform/linux-generic/test/pktio/.gitignore
create mode 100644 platform/linux-generic/test/pktio/Makefile.am
rename platform/linux-generic/test/{ => pktio}/pktio_env (100%)
rename test/validation/odp_pktio_run =>
platform/linux-generic/test/pktio/pktio_run (52%)
create mode 100644 test/validation/Makefile.inc
create mode 100644 test/validation/common/.gitignore
create mode 100644 test/validation/common/Makefile.am
create mode 100644 test/validation/pktio/.gitignore
create mode 100644 test/validation/pktio/Makefile.am
rename test/validation/{odp_pktio.c => pktio/pktio.c} (90%)
create mode 100644 test/validation/pktio/pktio.h
create mode 100644 test/validation/pktio/pktio_main.c
--
1.9.1
_______________________________________________
lng-odp mailing list
lng-odp@lists.linaro.org
https://lists.linaro.org/mailman/listinfo/lng-odp
