Skip to contentSkip to conversation

qlcnic: Enhance logic to calculate msix vectors.

DISCUSSION Dan Carpenter 2 participants 2 messages Jan 28, 2014 Kernel janitors

lore
Share & tools
lore
#0 lore raw

Hello Himanshu Madhani,

The patch cb9327d567be: "qlcnic: Enhance logic to calculate msix vectors." from Jan 23, 2014, leads to the following static checker warning:

drivers/net/ethernet/qlogic/qlcnic/qlcnic_main.c:678 qlcnic_setup_tss_rss_intr() info: ignoring unreachable code."

drivers/net/ethernet/qlogic/qlcnic/qlcnic_main.c
669 if (qlcnic_83xx_check(adapter))
670 num_msix += 1;
671
672 netdev_info(adapter->netdev,

   673                              "Restoring %d Tx, %d SDS rings for total %d vectors.\n",
   674                              adapter->drv_tx_rings, adapter->drv_sds_rings,
   675                              num_msix);
   676                  goto restore;
   677  
   678                  err = -EIO;
                        ^^^^^^^^^^

Should this be in front of the goto?

679 }
680
681 return err;
682 }

regards,
dan carpenter

#1 lore raw
Quoted 14 lines · Dan Carpenter

Hello Himanshu Madhani,

The patch cb9327d567be: "qlcnic: Enhance logic to calculate msix vectors." from Jan 23, 2014, leads to the following static checker warning:

drivers/net/ethernet/qlogic/qlcnic/qlcnic_main.c:678 qlcnic_setup_tss_rss_intr() info: ignoring unreachable code."

drivers/net/ethernet/qlogic/qlcnic/qlcnic_main.c
669 if (qlcnic_83xx_check(adapter))
670 num_msix += 1;
671
672 netdev_info(adapter->netdev,

Quoted 9 lines
  673                              "Restoring %d Tx, %d SDS rings for total %d vectors.\n",
  674                              adapter->drv_tx_rings, adapter->drv_sds_rings,
  675                              num_msix);
  676                  goto restore;
  677  
  678                  err = -EIO;
                       ^^^^^^^^^^

Should this be in front of the goto?

This needs to be removed. We will send a patch to fix this and add you as reporter of this problem.

Quoted 9 lines

679 }
680
681 return err;
682 }

regards,
dan carpenter

Thanks,
-Himanshu