[dts] [PATCH 0/4] compile dpdk different config with different test suite by dts framework befor run test suite
Xu, HuilongX
huilongx.xu at intel.com
Sat Apr 1 10:31:53 CEST 2017
See my comments
> -----Original Message-----
> From: Liu, Yong
> Sent: Saturday, April 01, 2017 4:21 PM
> To: Xu, HuilongX; dts at dpdk.org
> Cc: Xu, HuilongX
> Subject: RE: [dts] [PATCH 0/4] compile dpdk different config with different
> test suite by dts framework befor run test suite
>
> Thanks huilong. The overall idea look good for me. Just some comments as
> below:
>
> 1. Configuration dependency should be based on test case not suite.
> Configuration requirements maybe changed even in one suite.
If config dependency test case, must get all case name, but if case name same, maybe have issue.
> 2. No need to compile all compile sessions. If cases not in execution list, we
> can skip it. It will save more time :)
The config is global, if test suite not in exection.cfg, the compile session will skip.
3. Assume that suite can remove all dpdk
> build related code, cause of framework can make case not aware of
> configuration dependencies.
Maybe can savel compile switch in target result, but it only for check. Because, when we development a case.
We should know compile switch config and compile.cfg is static and global. So not need know dependencies.
>
> Thanks,
> Marvin
>
> > -----Original Message-----
> > From: dts [mailto:dts-bounces at dpdk.org] On Behalf Of xu,huilong
> > Sent: Saturday, April 01, 2017 3:42 PM
> > To: dts at dpdk.org
> > Cc: Xu, HuilongX <huilongx.xu at intel.com>
> > Subject: [dts] [PATCH 0/4] compile dpdk different config with
> > different test suite by dts framework befor run test suite
> >
> > we should compile dpdk many times for different test suite.
> > if we exec a test suite must update dpdk compile suite or change dpdk
> > code, we must cpmpile dpdk two times at least.
> > So this change for read exec test_suite list and compile dpdk befor
> > run test suite.
> >
> > update list:
> > 1. conf/common_base
> > this is dpdk all compile switch from dpdk source code. it for
> > check compile.cfg switch vaild.
> > 2. conf/compile.cfg
> > compile config template, detaile config please see comments in
> > file 3. framework/config.py
> > parse compile.cfg, detaile info please see comments in commit 4.
> > framework/project_dpdk.py
> > a)define interface for test suit:
> > reset_compile_target()
> > b) read compile conf and get test suite list, then update compile
> > switch and compile dpdk 5. framework/dts.py
> > set test suite list
> > 6. tests/TestSuite_coremask.py and tests/TestSuite_queue_start_stop.py
> > example for use compile config.
> > Now old interface for compile dpdk is vaild, so other case will also
> > run pass.
> >
> > xu,huilong (4):
> > add dpdk all compile switch
> > add compile config file template and parse compile config class
> > read compile switch config and compile dpdk before test
> > add example for test suite
> >
> > conf/common_base | 632
> > ++++++++++++++++++++++++++++++++++++
> > conf/compile.cfg | 30 ++
> > framework/config.py | 21 +-
> > framework/dts.py | 2 +-
> > framework/project_dpdk.py | 103 +++++-
> > tests/TestSuite_coremask.py | 12 +-
> > tests/TestSuite_queue_start_stop.py | 28 +-
> > 7 files changed, 789 insertions(+), 39 deletions(-) create mode
> > 100644 conf/common_base create mode 100644 conf/compile.cfg
> >
> > --
> > 1.9.3
More information about the dts
mailing list