On Tue, Sep 15, 2026 at 5:07 AM Evgeny Sokolov <[email protected]> wrote:
> From: Sokolov Evgeny <[email protected]> > > Since commit 51fafb89a9a ("net/bnxt: get rid of ff pools and use > VNIC info array"), VNIC objects are obtained from the VNIC info > array. > > The address of an array element cannot be NULL, making the NULL > check redundant. > > Remove the dead code. > > Found by InfoTeCS on behalf of Linux Verification Center (linuxtesting.org) > with SVACE. > > Signed-off-by: Sokolov Evgeny <[email protected]> > --- > drivers/net/bnxt/bnxt_rxq.c | 5 ----- > 1 file changed, 5 deletions(-) > > diff --git a/drivers/net/bnxt/bnxt_rxq.c b/drivers/net/bnxt/bnxt_rxq.c > index 023cb0e174..b7e16c8a14 100644 > --- a/drivers/net/bnxt/bnxt_rxq.c > +++ b/drivers/net/bnxt/bnxt_rxq.c > @@ -131,11 +131,6 @@ int bnxt_mq_rx_configure(struct bnxt *bp) > > for (i = 0; i < pools; i++) { > vnic = &bp->vnic_info[i]; > - if (!vnic) { > - PMD_DRV_LOG_LINE(ERR, "VNIC alloc failed"); > - rc = -ENOMEM; > - goto err_out; > - } > vnic->flags |= BNXT_VNIC_INFO_BCAST; > bp->nr_vnics++; > > -- Acked-by: Kishore Padmanabha <[email protected]> > > 2.30.2 > >
smime.p7s
Description: S/MIME Cryptographic Signature
