mbox series

[v5,0/4] test/lpm: fix rcu qsbr perf test

Message ID 20201104185858.18397-1-dharmik.thakkar@arm.com (mailing list archive)
Headers show
Series test/lpm: fix rcu qsbr perf test | expand

Message

Dharmik Thakkar Nov. 4, 2020, 6:58 p.m. UTC
Fix LPM adds, LPM deletes, and cycle calculation.
Return error if LPM add/delete fails in multi-writer test.
Return error if single or multi writer test fails
Remove redundant error checking for readers.
Combine single and multi threaded test cases to avoid code duplication.
---
v5:
 - Update logic for routes inserted by each writer in multi writer
   test to avoid possibility of routes getting skipped

v4:
 - Return error if rcu qsbr test fails
 - Improve multi writer test to enable more than 2 writers

v3:
 - Add 'goto error'
 - Remove unnecessary if statement

v2:
 - Add more details about the fix to the commit message
 - Replace hard coded values with an enum
 - Remove lock acquire/release for single writer

Dharmik Thakkar (4):
  test/lpm: fix cycle calculation in rcu qsbr perf
  test/lpm: return error on failure in rcu qsbr perf
  test/lpm: remove error checking in rcu qsbr perf
  test/lpm: avoid code duplication in rcu qsbr perf

 app/test/test_lpm_perf.c | 380 +++++++++------------------------------
 1 file changed, 88 insertions(+), 292 deletions(-)

Comments

David Marchand Nov. 5, 2020, 3:58 p.m. UTC | #1
On Wed, Nov 4, 2020 at 7:59 PM Dharmik Thakkar <dharmik.thakkar@arm.com> wrote:
>
> Fix LPM adds, LPM deletes, and cycle calculation.
> Return error if LPM add/delete fails in multi-writer test.
> Return error if single or multi writer test fails
> Remove redundant error checking for readers.
> Combine single and multi threaded test cases to avoid code duplication.

Series applied, thanks Dharmik.