qlcnic: Enhance logic to calculate msix vectors.
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
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:678qlcnic_setup_tss_rss_intr()info: ignoring unreachable code."
drivers/net/ethernet/qlogic/qlcnic/qlcnic_main.c
669 if (qlcnic_83xx_check(adapter))
670num_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