mbox series

[v2,0/5] test/meson: fix hash readwrite timeout failure

Message ID 20200203194912.4669-1-honnappa.nagarahalli@arm.com (mailing list archive)
Headers show
Series test/meson: fix hash readwrite timeout failure | expand

Message

Honnappa Nagarahalli Feb. 3, 2020, 7:49 p.m. UTC
V2 changes:
- Remove duplicated code in the existing patch
- Add lock-free option to hash read-write functional test cases
- Move existing loack-free test cases to performance tests
- A typo fix in the comments for lock-free extendable table feature

V1 changes:
- hash_readwrite and hash_readwrite lockfree meson test was
  taking longer time to complete. The test always get TIMEOUT.
- hash readwrtie test is split into functional and perf tests 
  and moved to dpdk fast and perf testsuite accordingly.
- hash readwrite lockfree is moved to dpdk perf testsuite.

Amit Gupta (1):
  test/meson: hash test split into shorter subtests

Honnappa Nagarahalli (4):
  test/hash: remove duplicated test code
  test/hash: add lock free reader writer functional tests
  test/hash: move reader writer lock free tests to perf tests
  hash: correct lock free extendable table support

 app/test/Makefile                             |   2 +-
 app/test/autotest_data.py                     |  14 ++-
 app/test/meson.build                          |   7 +-
 app/test/test_hash_readwrite.c                | 117 +++++++++++++-----
 ...ite_lf.c => test_hash_readwrite_lf_perf.c} |   7 +-
 lib/librte_hash/rte_hash.h                    |   2 -
 6 files changed, 107 insertions(+), 42 deletions(-)
 rename app/test/{test_hash_readwrite_lf.c => test_hash_readwrite_lf_perf.c} (99%)

Comments

David Marchand Feb. 5, 2020, 6:41 p.m. UTC | #1
On Mon, Feb 3, 2020 at 8:49 PM Honnappa Nagarahalli
<honnappa.nagarahalli@arm.com> wrote:
>
> V2 changes:
> - Remove duplicated code in the existing patch
> - Add lock-free option to hash read-write functional test cases
> - Move existing loack-free test cases to performance tests
> - A typo fix in the comments for lock-free extendable table feature
>
> V1 changes:
> - hash_readwrite and hash_readwrite lockfree meson test was
>   taking longer time to complete. The test always get TIMEOUT.
> - hash readwrtie test is split into functional and perf tests
>   and moved to dpdk fast and perf testsuite accordingly.
> - hash readwrite lockfree is moved to dpdk perf testsuite.
>
> Amit Gupta (1):
>   test/meson: hash test split into shorter subtests
>
> Honnappa Nagarahalli (4):
>   test/hash: remove duplicated test code
>   test/hash: add lock free reader writer functional tests
>   test/hash: move reader writer lock free tests to perf tests
>   hash: correct lock free extendable table support
>
>  app/test/Makefile                             |   2 +-
>  app/test/autotest_data.py                     |  14 ++-
>  app/test/meson.build                          |   7 +-
>  app/test/test_hash_readwrite.c                | 117 +++++++++++++-----
>  ...ite_lf.c => test_hash_readwrite_lf_perf.c} |   7 +-
>  lib/librte_hash/rte_hash.h                    |   2 -
>  6 files changed, 107 insertions(+), 42 deletions(-)
>  rename app/test/{test_hash_readwrite_lf.c => test_hash_readwrite_lf_perf.c} (99%)

Squashed patch 2 in patch 1.
Series applied, thanks.