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

[Maxime Coquelin]

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.var...@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.coque...@redhat.com>

Thanks,
Maxime