[PATCH v1 12/13] test/bbdev: remove check for invalid opaque data
Maxime Coquelin
maxime.coquelin at redhat.com
Tue Jan 31 13:33:56 CET 2023
On 1/17/23 17:50, Hernan Vargas wrote:
> Assert also if the opaque date is invalid.
data*
>
> Signed-off-by: Hernan Vargas <hernan.vargas at intel.com>
> ---
> app/test-bbdev/test_bbdev_perf.c | 8 +++-----
> 1 file changed, 3 insertions(+), 5 deletions(-)
>
> diff --git a/app/test-bbdev/test_bbdev_perf.c b/app/test-bbdev/test_bbdev_perf.c
> index 0930786fda..81bf2c8b60 100644
> --- a/app/test-bbdev/test_bbdev_perf.c
> +++ b/app/test-bbdev/test_bbdev_perf.c
> @@ -79,7 +79,6 @@
>
> #define SYNC_WAIT 0
> #define SYNC_START 1
> -#define INVALID_OPAQUE -1
>
> #define INVALID_QUEUE_ID -1
> /* Increment for next code block in external HARQ memory */
> @@ -1998,10 +1997,9 @@ check_enc_status_and_ordering(struct rte_bbdev_enc_op *op,
> "op_status (%d) != expected_status (%d)",
> op->status, expected_status);
>
> - if (op->opaque_data != (void *)(uintptr_t)INVALID_OPAQUE)
> - TEST_ASSERT((void *)(uintptr_t)order_idx == op->opaque_data,
> - "Ordering error, expected %p, got %p",
> - (void *)(uintptr_t)order_idx, op->opaque_data);
> + TEST_ASSERT((void *)(uintptr_t)order_idx == op->opaque_data,
> + "Ordering error, expected %p, got %p",
> + (void *)(uintptr_t)order_idx, op->opaque_data);
>
> return TEST_SUCCESS;
> }
It is a fix, isn't it?
Maxime
More information about the dev
mailing list