Re: [dpdk-dev] [PATCH v2 5/6] test/lpm: reset total time

[Medvedkin, Vladimir]

Hi Ruifeng,

Thanks for this bug fix.

I think it should be sent separately from this RCU related patch series.

On 06/09/2019 10:45, Ruifeng Wang wrote:

From: Honnappa Nagarahalli <honnappa.nagaraha...@arm.com>

total_time needs to be reset to measure the cycles for delete API.

Fixes: af75078fece3 ("first public release")
Cc: sta...@dpdk.org

Signed-off-by: Honnappa Nagarahalli <honnappa.nagaraha...@arm.com>
Reviewed-by: Gavin Hu <gavin...@arm.com>
Reviewed-by: Ruifeng Wang <ruifeng.w...@arm.com>
---
  app/test/test_lpm_perf.c | 4 ++--
  1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/app/test/test_lpm_perf.c b/app/test/test_lpm_perf.c
index 77eea66ad..a2578fe90 100644
--- a/app/test/test_lpm_perf.c
+++ b/app/test/test_lpm_perf.c
@@ -460,7 +460,7 @@ test_lpm_perf(void)
                        (double)total_time / ((double)ITERATIONS * BATCH_SIZE),
                        (count * 100.0) / (double)(ITERATIONS * BATCH_SIZE));

- /* Delete */

+       /* Measure Delete */
        status = 0;
        begin = rte_rdtsc();

@@ -470,7 +470,7 @@ test_lpm_perf(void)

                                large_route_table[i].depth);
        }

- total_time += rte_rdtsc() - begin;

+       total_time = rte_rdtsc() - begin;

printf("Average LPM Delete: %g cycles\n",

                        (double)total_time / NUM_ROUTE_ENTRIES);

--
Regards,
Vladimir