> -----Original Message-----
> From: Honnappa Nagarahalli <[email protected]>
> Sent: Thursday, September 12, 2019 8:31 PM
> To: Wang, Yipeng1 <[email protected]>; Amit Gupta
> <[email protected]>; Gobriel, Sameh <[email protected]>;
> Dharmik Thakkar <[email protected]>; Ruifeng Wang (Arm
> Technology China) <[email protected]>
> Cc: [email protected]; nd <[email protected]>; Honnappa Nagarahalli
> <[email protected]>; nd <[email protected]>
> Subject: [EXT] RE: [PATCH 2/2] test/meson: hash lf test moved to dpdk perf
> testsuite
>
> External Email
>
> ----------------------------------------------------------------------
> <snip>
>
> > >Subject: [PATCH 2/2] test/meson: hash lf test moved to dpdk perf
> > >testsuite
> > >
> > >From: Amit Gupta <[email protected]>
> > >
> > >hash_readwrite_lf test always getting TIMEOUT as required time to
> > >finish this test was much longer compare to time required for fast
> > >tests(10s). Hence, the test is being moved to perf test category for
> > >its execution to complete.
> > >
> > >Signed-off-by: Amit Gupta <[email protected]>
> > >---
> > > app/test/meson.build | 2 +-
> > > 1 file changed, 1 insertion(+), 1 deletion(-)
> > >
> > >diff --git a/app/test/meson.build b/app/test/meson.build index
> > >94fd9f8..34141c5 100644
> > >--- a/app/test/meson.build
> > >+++ b/app/test/meson.build
> > >@@ -220,7 +220,6 @@ fast_test_names = [
> > > 'eventdev_common_autotest',
> > > 'fbarray_autotest',
> > > 'hash_readwrite_func_autotest',
> > >- 'hash_readwrite_lf_autotest',
> > > 'ipsec_autotest',
> > > 'kni_autotest',
> > > 'kvargs_autotest',
> > >@@ -263,6 +262,7 @@ perf_test_names = [
> > > 'stack_lf_perf_autotest',
> > > 'rand_perf_autotest',
> > > 'hash_readwrite_perf_autotest',
> > >+ 'hash_readwrite_lf_autotest',
> > > ]
> > >
> > > driver_test_names = [
> > >--
> > >1.8.3.1
> > [Wang, Yipeng]
> > I believe the lf_autotest includes functional test as well which is
> > critical for testing the lock free implementation on non-TSO machine.
> > Do you think it is possible to also separate this test?
> > I also include the ARM folks for opinions.
> Thanks Yipeng. We combined the functional testing with performance
> testing. So, it is all performance tests. From that perspective, I suggest
> changing 'hash_readwrite_lf_autotest' to
> 'hash_readwrite_lf_perf_autotest'.
Agreed, will update in v2!
>
> >
> > Thanks!
> > Yipeng
