mbox series

[v4,0/6] hash: improve multiple places

Message ID 1540547626-197189-1-git-send-email-yipeng1.wang@intel.com (mailing list archive)
Headers show
Series hash: improve multiple places | expand

Message

Wang, Yipeng1 Oct. 26, 2018, 9:53 a.m. UTC
This patch set depends on Honnappa's patch set:
http://patchwork.dpdk.org/cover/47268/

This patch set fixes/improves a couple of places mostly
on unit tests:

commit 1: remove unnecessary code in hash library.
commit 2: use jhash in multiwriter unit test.
commit 3: cover more test case in unit test.
commit 4: improve readwrite test to consider the extendable table.
commit 5: remove scaling unit test.
commit 6: add readwrite test to meson and autotest file.

v3->v4:
* Fix memory leak in commit 3 (Honnappa)

v2->v3:
* Split commit 4 to commit 3 and 4 (Honnappa)
* Remove hard coded macro (Bruce)
* Add commit 6 fix to add readwrite test to autotest files.
* Remove unnecessary header in commit 1 (Mattias)

V1->V2:
* In commit 2 change use_jhash to a macro instead of a hard coded
local variable (Bruce).
* Add commit 4 to remove scaling unit test (Bruce).

Signed-off-by: Yipeng Wang <yipeng1.wang@intel.com>

Yipeng Wang (6):
  hash: fix unnecessary pause
  test/hash: change multiwriter test to use jhash
  test/hash: test more corner cases in unit test
  test/hash: add readwrite test for ext table
  test/hash: remove hash scaling unit test
  test/hash: fix to add read-write test to autotest

 lib/librte_hash/rte_cuckoo_hash.c |   5 +-
 test/test/Makefile                |   1 -
 test/test/autotest_data.py        |  12 +--
 test/test/meson.build             |   3 +-
 test/test/test_hash_multiwriter.c |   3 +-
 test/test/test_hash_readwrite.c   |  74 ++++++++++++---
 test/test/test_hash_scaling.c     | 191 --------------------------------------
 7 files changed, 71 insertions(+), 218 deletions(-)
 delete mode 100644 test/test/test_hash_scaling.c

Comments

Thomas Monjalon Oct. 26, 2018, 8:15 p.m. UTC | #1
> Yipeng Wang (6):
>   hash: fix unnecessary pause
>   test/hash: change multiwriter test to use jhash
>   test/hash: test more corner cases in unit test
>   test/hash: add readwrite test for ext table
>   test/hash: remove hash scaling unit test
>   test/hash: fix to add read-write test to autotest

Applied, thanks