On 10/23/20 4:42 PM, Nicolas Chautru wrote: > Replacing magic number for default wait time for hw > offload. > > Signed-off-by: Nicolas Chautru <nicolas.chau...@intel.com> > Acked-by: Liu Tianjiao <tianjiao....@intel.com> > --- > app/test-bbdev/test_bbdev_perf.c | 9 +++++---- > 1 file changed, 5 insertions(+), 4 deletions(-) > > diff --git a/app/test-bbdev/test_bbdev_perf.c > b/app/test-bbdev/test_bbdev_perf.c > index f30cbdb..39f06db 100644 > --- a/app/test-bbdev/test_bbdev_perf.c > +++ b/app/test-bbdev/test_bbdev_perf.c > @@ -25,6 +25,7 @@ > > #define MAX_QUEUES RTE_MAX_LCORE > #define TEST_REPETITIONS 1000 > +#define WAIT_OFFLOAD_US 1000
Why wasn't 200 used ? Tom > > #ifdef RTE_LIBRTE_PMD_BBDEV_FPGA_LTE_FEC > #include <fpga_lte_fec.h> > @@ -4451,7 +4452,7 @@ typedef int (test_case_function)(struct active_device > *ad, > time_st->enq_acc_total_time += stats.acc_offload_cycles; > > /* give time for device to process ops */ > - rte_delay_us(200); > + rte_delay_us(WAIT_OFFLOAD_US); > > /* Start time meas for dequeue function offload latency */ > deq_start_time = rte_rdtsc_precise(); > @@ -4542,7 +4543,7 @@ typedef int (test_case_function)(struct active_device > *ad, > time_st->enq_acc_total_time += stats.acc_offload_cycles; > > /* give time for device to process ops */ > - rte_delay_us(200); > + rte_delay_us(WAIT_OFFLOAD_US); > > /* Start time meas for dequeue function offload latency */ > deq_start_time = rte_rdtsc_precise(); > @@ -4630,7 +4631,7 @@ typedef int (test_case_function)(struct active_device > *ad, > time_st->enq_acc_total_time += stats.acc_offload_cycles; > > /* give time for device to process ops */ > - rte_delay_us(200); > + rte_delay_us(WAIT_OFFLOAD_US); > > /* Start time meas for dequeue function offload latency */ > deq_start_time = rte_rdtsc_precise(); > @@ -4713,7 +4714,7 @@ typedef int (test_case_function)(struct active_device > *ad, > time_st->enq_acc_total_time += stats.acc_offload_cycles; > > /* give time for device to process ops */ > - rte_delay_us(200); > + rte_delay_us(WAIT_OFFLOAD_US); > > /* Start time meas for dequeue function offload latency */ > deq_start_time = rte_rdtsc_precise();
