Hi Maxim, I am a bit confused about the path in your mail, but I assume you are trying to build out of tree and then run the pktio_run script. I assume you did not try with patch V3 either... At least, when I do this, I get:
(after building out of tree in /tmp/odp) erachmi@erachmi-VirtualBox:/tmp/odp$ /home/erachmi/linaro/ODP/odp/platform/linux-generic/test/pktio/pktio_run cannot find pktio_main: please set you PATH for it. If I do set my path, as required, export PATH=$PATH:/tmp/odp/test/validation/pktio/ Then It works. At least for me. The same situation existed before, which was solved by setting TEST_DIR. Hope that is the answer to your issue. If not, let me know. Christophe. On 8 June 2015 at 13:15, Maxim Uvarov <[email protected]> wrote: > We need that test case run also: > > mkidr /tmp/t11 > cd /tmp/t11 > ../../../...odp.git/configure --enable-test-vald --enable-test-perf > --enable-test-cpp --enable-debug CFLAGS="-O0 -g" --enable-debug > --enable-debug-print > make > > cd /tmp/t11/test/validation/pktio > > ../../../../../odp.git/platform/linux-generic/test/pktio/pktio_run > pktio_run: pktio_main: not found > > Thanks, > Maxim. > > > > On 06/04/15 21:06, Christophe Milard wrote: > >> since v2: >> -history rewritten regarding PRE_LDADD >> -missing .gitignore added >> -correction for finding pktio_main in different cases >> -patch 8 added, removing obsolete test environment variables in validation >> >> ---------- >> >> 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. >> >> thanks for your time. >> >> 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 | 9 ++- >> 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 | 49 ++++++------ >> 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, 186 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 >> >> >
_______________________________________________ lng-odp mailing list [email protected] https://lists.linaro.org/mailman/listinfo/lng-odp
