> -----Original Message-----
> From: Aaron Conole [mailto:[email protected]]
> Sent: Friday, September 13, 2019 7:41 AM
> To: [email protected]
> Cc: Wang, Yipeng1 <[email protected]>; Gobriel, Sameh
> <[email protected]>; Richardson, Bruce
> <[email protected]>; De Lara Guarch, Pablo
> <[email protected]>; [email protected]; Honnappa Nagarahalli
> <[email protected]>
> Subject: Re: [dpdk-dev] [PATCH v2 1/1] test/meson: hash lf test moved to
> dpdk perf testsuite
>
> <[email protected]> writes:
>
> > 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 renamed moved to perf test
> > category for its execution to complete.
> >
> > Signed-off-by: Amit Gupta <[email protected]>
> > ---
>
> Okay. I'll note that we pass the '-t 3' flag, so it is actually timing out
> with 30s
> instead of the default 10. We do this because occasionally the lpm6 and table
> tests would also exceed the 10s timeout in the travis environment. I agree,
> it's better to pull the perf part of tests out.
>
> I think there isn't any additional functional test in this readwrite - is
> that so?
> If it is, then we need to also prioritize adding back in some of the
> functional
> testing. Maybe I misread the lf_autotest, though.
>
[Wang, Yipeng]
Yes that is my concern too, if we just move all the lock-free test into perf
test then we miss
the functional test.
Would any of you like to consider adding a small functional test into the
readwrite or readwrite_lf_functional?
That would be great :)
