[v2] app/bbdev: fix wrong variable
Checks
Commit Message
The logging should show context by printing the two variable which
compared to each other. 'nb_harq_inputs', not 'nb_hard_outputs';
'nb_harq_outputs', not 'nb_hard_outputs'.
This patch corrected misused variable.
Fixes: d819c08327f3 ("app/bbdev: update for 5GNR")
Cc: stable@dpdk.org
Signed-off-by: Min Hu (Connor) <humin29@huawei.com>
Acked-by: Nicolas Chautru <Nicolas.chautru@intel.com>
---
app/test-bbdev/test_bbdev_perf.c | 4 ++--
1 file changed, 2 insertions(+), 2 deletions(-)
Comments
> The logging should show context by printing the two variable which
> compared to each other. 'nb_harq_inputs', not 'nb_hard_outputs';
> 'nb_harq_outputs', not 'nb_hard_outputs'.
>
> This patch corrected misused variable.
>
> Fixes: d819c08327f3 ("app/bbdev: update for 5GNR")
> Cc: stable@dpdk.org
>
> Signed-off-by: Min Hu (Connor) <humin29@huawei.com>
> Acked-by: Nicolas Chautru <Nicolas.chautru@intel.com>
> ---
Applied to dpdk-next-crypto
Thanks.
@@ -372,14 +372,14 @@ check_dev_cap(const struct rte_bbdev_info *dev_info)
if (nb_harq_inputs > cap->num_buffers_hard_out) {
printf(
"Too many HARQ inputs defined: %u, max: %u\n",
- nb_hard_outputs,
+ nb_harq_inputs,
cap->num_buffers_hard_out);
return TEST_FAILED;
}
if (nb_harq_outputs > cap->num_buffers_hard_out) {
printf(
"Too many HARQ outputs defined: %u, max: %u\n",
- nb_hard_outputs,
+ nb_harq_outputs,
cap->num_buffers_hard_out);
return TEST_FAILED;
}