[dpdk-dev] [PATCH v2 6/6] bbdev: reduce warning level for one scenario
Tom Rix
trix at redhat.com
Wed Sep 1 16:29:55 CEST 2021
On 8/19/21 2:10 PM, Nicolas Chautru wrote:
> Queue setup may genuinely fail when adding incremental queues
> for a given priority level. In that case application would
> attempt to configure a queue at a different priority level.
> Not an actual error.
>
> Signed-off-by: Nicolas Chautru <nicolas.chautru at intel.com>
> ---
> lib/bbdev/rte_bbdev.c | 7 ++++---
> 1 file changed, 4 insertions(+), 3 deletions(-)
>
> diff --git a/lib/bbdev/rte_bbdev.c b/lib/bbdev/rte_bbdev.c
> index fc37236..defddcf 100644
> --- a/lib/bbdev/rte_bbdev.c
> +++ b/lib/bbdev/rte_bbdev.c
> @@ -528,9 +528,10 @@ struct rte_bbdev *
> ret = dev->dev_ops->queue_setup(dev, queue_id, (conf != NULL) ?
> conf : &dev_info.default_queue_conf);
> if (ret < 0) {
> - rte_bbdev_log(ERR,
> - "Device %u queue %u setup failed", dev_id,
> - queue_id);
> + /* This may happen when trying different priority levels */
> + rte_bbdev_log(INFO,
> + "Device %u queue %u setup failed",
> + dev_id, queue_id);
Earlier code tears down the exiting plumbing.
If it is ok to fail, should this block move earlier before the teardown ?
Tom
> return ret;
> }
>
More information about the dev
mailing list