An array of the size of total operations needed for the de/compression is reserved for ops while enqueueing, although only first burst_size entries of the array are used. Reduce the size of the array allocated. Fixes: b68a82425da4 ("app/compress-perf: add performance measurement") Cc: stable@dpdk.org Signed-off-by: Raja Zidane <rzidane@nvidia.com> --- Acked-by: Matan Azrad matan@nvidia.com app/test-compress-perf/comp_perf_test_cyclecount.c | 5 +++-- 1 file changed, 3 insertions(+), 2 deletions(-) diff --git a/app/test-compress-perf/comp_perf_test_cyclecount.c b/app/test-compress-perf/comp_perf_test_cyclecount.c index c215547291..a3f6404eb2 100644 --- a/app/test-compress-perf/comp_perf_test_cyclecount.c +++ b/app/test-compress-perf/comp_perf_test_cyclecount.c @@ -175,7 +175,8 @@ main_loop(struct cperf_cyclecount_ctx *ctx, enum rte_comp_xform_type type) /* one array for both enqueue and dequeue */ ops = rte_zmalloc_socket(NULL, - 2 * mem->total_bufs * sizeof(struct rte_comp_op *), + (test_data->burst_sz + mem->total_bufs) * + sizeof(struct rte_comp_op *), 0, rte_socket_id()); if (ops == NULL) { @@ -184,7 +185,7 @@ main_loop(struct cperf_cyclecount_ctx *ctx, enum rte_comp_xform_type type) return -1; } - deq_ops = &ops[mem->total_bufs]; + deq_ops = &ops[test_data->burst_sz]; if (type == RTE_COMP_COMPRESS) { xform = (struct rte_comp_xform) { -- 2.21.0
> -----Original Message-----
> From: Raja Zidane <rzidane@nvidia.com>
> Sent: Wednesday, February 23, 2022 1:33 PM
> To: dev@dpdk.org
> Cc: matan@nvidia.com; stable@dpdk.org
> Subject: [PATCH] app/compress-perf: optimize ops pool allocation
>
> An array of the size of total operations needed for the de/compression is
> reserved for ops while enqueueing, although only first burst_size entries
> of the array are used.
>
> Reduce the size of the array allocated.
>
> Fixes: b68a82425da4 ("app/compress-perf: add performance measurement")
> Cc: stable@dpdk.org
>
> Signed-off-by: Raja Zidane <rzidane@nvidia.com>
> ---
Acked-by: Fan Zhang <roy.fan.zhang@intel.com>
> -----Original Message-----
> From: Raja Zidane <rzidane@nvidia.com>
> Sent: Wednesday, February 23, 2022 1:33 PM
> To: dev@dpdk.org
> Cc: matan@nvidia.com; stable@dpdk.org
> Subject: [PATCH] app/compress-perf: optimize ops pool allocation
>
> An array of the size of total operations needed for the de/compression is
> reserved for ops while enqueueing, although only first burst_size entries
> of the array are used.
>
> Reduce the size of the array allocated.
>
> Fixes: b68a82425da4 ("app/compress-perf: add performance measurement")
> Cc: stable@dpdk.org
>
> Signed-off-by: Raja Zidane <rzidane@nvidia.com>
> ---
Acked-by: Fan Zhang <roy.fan.zhang@intel.com>
> An array of the size of total operations needed for the de/compression is
> reserved for ops while enqueueing, although only first burst_size entries
> of the array are used.
>
> Reduce the size of the array allocated.
>
> Fixes: b68a82425da4 ("app/compress-perf: add performance measurement")
> Cc: stable@dpdk.org
>
> Signed-off-by: Raja Zidane <rzidane@nvidia.com>
> ---
> Acked-by: Matan Azrad matan@nvidia.com
Please use proper syntax for acks
Applied to dpdk-next-crypto