[PATCH] net/bnxt: check return code of bnxt_hwrm_vnic_rss_cfg in flow create
Kishore Padmanabha
kishore.padmanabha at broadcom.com
Mon Sep 21 20:00:11 CEST 2026
On Tue, Sep 15, 2026 at 4:50 AM Evgeny Sokolov <Evgeny.Sokolov at infotecs.ru>
wrote:
> From: Sokolov Evgeny <Evgeny.Sokolov at infotecs.ru>
>
> The return value of bnxt_hwrm_vnic_rss_cfg() was not checked in
> bnxt_flow.c. As a result, RSS configuration failures could be
> silently ignored and the flow setup could continue with an
> incorrect VNIC configuration.
>
> Check the return status and propagate the error to the caller.
>
> Fixes: adc0f81c655 ("net/bnxt: support RSS action")
> Cc: stable at dpdk.org
>
> Signed-off-by: Sokolov Evgeny <Evgeny.Sokolov at infotecs.ru>
> ---
> drivers/net/bnxt/bnxt_flow.c | 11 ++++++++++-
> 1 file changed, 10 insertions(+), 1 deletion(-)
>
> diff --git a/drivers/net/bnxt/bnxt_flow.c b/drivers/net/bnxt/bnxt_flow.c
> index a2e590540b..63cfa3a045 100644
> --- a/drivers/net/bnxt/bnxt_flow.c
> +++ b/drivers/net/bnxt/bnxt_flow.c
> @@ -1631,7 +1631,16 @@ bnxt_validate_and_parse_flow(struct rte_eth_dev
> *dev,
> rss->key,
> rss->key_len);
> }
> - bnxt_hwrm_vnic_rss_cfg(bp, vnic);
> + rc = bnxt_hwrm_vnic_rss_cfg(bp, vnic);
> + if (rc) {
> + rte_flow_error_set(error,
> + -rc,
> +
> RTE_FLOW_ERROR_TYPE_ACTION,
> + act,
> + "VNIC RSS configure
> failed");
> + rc = -rte_errno;
> + goto ret;
> + }
> } else {
> PMD_DRV_LOG_LINE(DEBUG, "No RSS config required");
> }
>
Acked-by: Kishore Padmanabha <kishore.padmanabha at broadcom.com>
> --
> 2.30.2
>
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mails.dpdk.org/archives/stable/attachments/20260921/ca2481d5/attachment.htm>
-------------- next part --------------
A non-text attachment was scrubbed...
Name: smime.p7s
Type: application/pkcs7-signature
Size: 5493 bytes
Desc: S/MIME Cryptographic Signature
URL: <http://mails.dpdk.org/archives/stable/attachments/20260921/ca2481d5/attachment.bin>
More information about the stable
mailing list