[PATCH v1 06/11] test/bbdev: assert failed test for queue configure

Maxime Coquelin maxime.coquelin at redhat.com
Tue Oct 17 21:43:05 CEST 2023


On 9/29/23 20:13, Hernan Vargas wrote:
> Stop test if rte_bbdev_queue_configure fails to configure queue.
> 
> Signed-off-by: Hernan Vargas <hernan.vargas at intel.com>
> ---
>   app/test-bbdev/test_bbdev.c | 3 ++-
>   1 file changed, 2 insertions(+), 1 deletion(-)
> 
> diff --git a/app/test-bbdev/test_bbdev.c b/app/test-bbdev/test_bbdev.c
> index 65805977aead..cf224dca5d04 100644
> --- a/app/test-bbdev/test_bbdev.c
> +++ b/app/test-bbdev/test_bbdev.c
> @@ -366,7 +366,8 @@ test_bbdev_configure_stop_queue(void)
>   	 * - queue should be started if deferred_start ==
>   	 */
>   	ts_params->qconf.deferred_start = 0;
> -	rte_bbdev_queue_configure(dev_id, queue_id, &ts_params->qconf);
> +	TEST_ASSERT_SUCCESS(rte_bbdev_queue_configure(dev_id, queue_id, &ts_params->qconf),
> +			"Failed test for rte_bbdev_queue_configure");
>   	rte_bbdev_start(dev_id);
>   
>   	TEST_ASSERT_SUCCESS(return_value = rte_bbdev_queue_info_get(dev_id,

If should be a fix IMO.
With fixes tag added and stable cc'ed:

Reviewed-by: Maxime Coquelin <maxime.coquelin at redhat.com>

Thanks,
Maxime



More information about the dev mailing list