[dpdk-dev] [PATCH v5 4/7] app/bbdev: define wait for offload
Tom Rix
trix at redhat.com
Mon Oct 26 14:33:28 CET 2020
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.chautru at intel.com>
> Acked-by: Liu Tianjiao <tianjiao.liu at 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();
More information about the dev
mailing list