bna: Async Mode Tx Rx Init Fix
authorRasesh Mody <rmody@brocade.com>
Tue, 30 Aug 2011 15:27:44 +0000 (15:27 +0000)
committerDavid S. Miller <davem@davemloft.net>
Thu, 15 Sep 2011 19:36:33 +0000 (15:36 -0400)
Change details:
 - Async mode of Tx/Rx queue initialization in BNAD from a task queue context
   runs into non-unique taskq allocation issues. Get rid of Tx/Rx
   initialization from task q context
 - In the attach function, wait for IOC enable, then do Tx/Rx queue
   initialization. Default BNA attributes are used when IOC enable from attach
   fails and values are set to:
   1 TxQ, 1 RxQ, 1 Unicast MAC, 1 RIT entry

Signed-off-by: Gurunatha Karaje <gkaraje@brocade.com>
Signed-off-by: Rasesh Mody <rmody@brocade.com>
Signed-off-by: David S. Miller <davem@davemloft.net>
drivers/net/ethernet/brocade/bna/bna_enet.c
drivers/net/ethernet/brocade/bna/bna_hw_defs.h
drivers/net/ethernet/brocade/bna/bna_types.h

index 68a275d66fcfbba67952e10064793e49e735846e..26f5c5abfd1f50aced1931148dbb66b3dd81b161 100644 (file)
@@ -167,13 +167,14 @@ bna_bfi_attr_get_rsp(struct bna_ioceth *ioceth,
         * Store only if not set earlier, since BNAD can override the HW
         * attributes
         */
-       if (!ioceth->attr.num_txq)
+       if (!ioceth->attr.fw_query_complete) {
                ioceth->attr.num_txq = ntohl(rsp->max_cfg);
-       if (!ioceth->attr.num_rxp)
                ioceth->attr.num_rxp = ntohl(rsp->max_cfg);
-       ioceth->attr.num_ucmac = ntohl(rsp->max_ucmac);
-       ioceth->attr.num_mcmac = BFI_ENET_MAX_MCAM;
-       ioceth->attr.max_rit_size = ntohl(rsp->rit_size);
+               ioceth->attr.num_ucmac = ntohl(rsp->max_ucmac);
+               ioceth->attr.num_mcmac = BFI_ENET_MAX_MCAM;
+               ioceth->attr.max_rit_size = ntohl(rsp->rit_size);
+               ioceth->attr.fw_query_complete = true;
+       }
 
        bfa_fsm_send_event(ioceth, IOCETH_E_ENET_ATTR_RESP);
 }
@@ -1693,6 +1694,16 @@ static struct bfa_ioc_cbfn bna_ioceth_cbfn = {
        bna_cb_ioceth_reset
 };
 
+static void bna_attr_init(struct bna_ioceth *ioceth)
+{
+       ioceth->attr.num_txq = BFI_ENET_DEF_TXQ;
+       ioceth->attr.num_rxp = BFI_ENET_DEF_RXP;
+       ioceth->attr.num_ucmac = BFI_ENET_DEF_UCAM;
+       ioceth->attr.num_mcmac = BFI_ENET_MAX_MCAM;
+       ioceth->attr.max_rit_size = BFI_ENET_DEF_RITSZ;
+       ioceth->attr.fw_query_complete = false;
+}
+
 static void
 bna_ioceth_init(struct bna_ioceth *ioceth, struct bna *bna,
                struct bna_res_info *res_info)
@@ -1738,6 +1749,8 @@ bna_ioceth_init(struct bna_ioceth *ioceth, struct bna *bna,
        ioceth->stop_cbfn = NULL;
        ioceth->stop_cbarg = NULL;
 
+       bna_attr_init(ioceth);
+
        bfa_fsm_set_state(ioceth, bna_ioceth_sm_stopped);
 }
 
@@ -2036,7 +2049,8 @@ bna_uninit(struct bna *bna)
 int
 bna_num_txq_set(struct bna *bna, int num_txq)
 {
-       if (num_txq > 0 && (num_txq <= bna->ioceth.attr.num_txq)) {
+       if (bna->ioceth.attr.fw_query_complete &&
+               (num_txq <= bna->ioceth.attr.num_txq)) {
                bna->ioceth.attr.num_txq = num_txq;
                return BNA_CB_SUCCESS;
        }
@@ -2047,7 +2061,8 @@ bna_num_txq_set(struct bna *bna, int num_txq)
 int
 bna_num_rxp_set(struct bna *bna, int num_rxp)
 {
-       if (num_rxp > 0 && (num_rxp <= bna->ioceth.attr.num_rxp)) {
+       if (bna->ioceth.attr.fw_query_complete &&
+               (num_rxp <= bna->ioceth.attr.num_rxp)) {
                bna->ioceth.attr.num_rxp = num_rxp;
                return BNA_CB_SUCCESS;
        }
index 7ecdca559c3528ecf8e842fce9621c9205b18c14..dde8a463b8d9828859e1f00ff699de70e05beb41 100644 (file)
  * SW imposed limits
  *
  */
+#define BFI_ENET_DEF_TXQ               1
+#define BFI_ENET_DEF_RXP               1
+#define BFI_ENET_DEF_UCAM              1
+#define BFI_ENET_DEF_RITSZ             1
 
 #define BFI_ENET_MAX_MCAM              256
 
index 59417b1f56a90e6d7d78bb36c4537ed8f67f32fe..242d7997ffb260983780dd195bb42d84f3fe2f40 100644 (file)
@@ -323,6 +323,7 @@ struct bna_qpt {
 };
 
 struct bna_attr {
+       bool                    fw_query_complete;
        int                     num_txq;
        int                     num_rxp;
        int                     num_ucmac;