Re: [lng-odp] [PATCHv4 7/8] validation: moving pktio plt specific to platform

2015-06-10 Thread Christophe Milard
On 10 June 2015 at 11:17, Maxim Uvarov wrote: > On 06/10/15 11:37, Christophe Milard wrote: > >> Hi, >> I would agree with you, but saddly pktio_env (which is defined under >> platform/linux-generic/test/pktio) is also used in the performance tests. >> pktio_env was before packed in the debian pa

Re: [lng-odp] [PATCHv4 7/8] validation: moving pktio plt specific to platform

2015-06-10 Thread Maxim Uvarov
On 06/10/15 11:37, Christophe Milard wrote: Hi, I would agree with you, but saddly pktio_env (which is defined under platform/linux-generic/test/pktio) is also used in the performance tests. pktio_env was before packed in the debian package, despite the fact that the tests were not. So, If we

Re: [lng-odp] [PATCHv4 7/8] validation: moving pktio plt specific to platform

2015-06-10 Thread Maxim Uvarov
Christophe, if we do not package other tests to deb then there is no need to package pktio tests also. In that patch you can extend if test_vald on next 2 lines: --- a/platform/linux-generic/test/Makefile.am +++ b/platform/linux-generic/test/Makefile.am @@ -2,7 +2,9 @@ TESTS_ENVIRONMENT = TEST_

Re: [lng-odp] [PATCHv4 7/8] validation: moving pktio plt specific to platform

2015-06-10 Thread Maxim Uvarov
On 06/09/15 14:44, Christophe Milard wrote: The platform specific scripts for pktio are now moved from the validation to the platform side. Pktio tests are now initiated from the platform side. Signed-off-by: Christophe Milard --- configure.ac | 1 + pl

Re: [lng-odp] [PATCHv4 7/8] validation: moving pktio plt specific to platform

2015-06-10 Thread Christophe Milard
Hi, I would agree with you, but saddly pktio_env (which is defined under platform/linux-generic/test/pktio) is also used in the performance tests. pktio_env was before packed in the debian package, despite the fact that the tests were not. So, If we really want to do something now, it would then b

Re: [lng-odp] [PATCHv4 7/8] validation: moving pktio plt specific to platform

2015-06-10 Thread Maxim Uvarov
Christophe, if we do not package other tests to deb then there is no need to package pktio tests also. In that patch you can extend if test_vald on next 2 lines: --- a/platform/linux-generic/test/Makefile.am +++ b/platform/linux-generic/test/Makefile.am @@ -2,7 +2,9 @@ TESTS_ENVIRONMENT = TEST_

[lng-odp] [PATCHv4 7/8] validation: moving pktio plt specific to platform

2015-06-09 Thread Christophe Milard
The platform specific scripts for pktio are now moved from the validation to the platform side. Pktio tests are now initiated from the platform side. Signed-off-by: Christophe Milard --- configure.ac | 1 + platform/linux-generic/test/.gitignore