Commit c710245c authored by Casey Leedom's avatar Casey Leedom Committed by David S. Miller

cxgb4vf: Ingress Queue Entry Size needs to be 64 bytes

Was using L1_CACHE_BYTES for the Ingress Queue Entry Size but it really
needs to be 64 bytes in order to support the largest message sizes.
Signed-off-by: default avatarCasey Leedom <leedom@chelsio.com>
Signed-off-by: default avatarDavid S. Miller <davem@davemloft.net>
parent 377ecca9
...@@ -2269,6 +2269,7 @@ static void __devinit cfg_queues(struct adapter *adapter) ...@@ -2269,6 +2269,7 @@ static void __devinit cfg_queues(struct adapter *adapter)
{ {
struct sge *s = &adapter->sge; struct sge *s = &adapter->sge;
int q10g, n10g, qidx, pidx, qs; int q10g, n10g, qidx, pidx, qs;
size_t iqe_size;
/* /*
* We should not be called till we know how many Queue Sets we can * We should not be called till we know how many Queue Sets we can
...@@ -2312,6 +2313,13 @@ static void __devinit cfg_queues(struct adapter *adapter) ...@@ -2312,6 +2313,13 @@ static void __devinit cfg_queues(struct adapter *adapter)
} }
s->ethqsets = qidx; s->ethqsets = qidx;
/*
* The Ingress Queue Entry Size for our various Response Queues needs
* to be big enough to accommodate the largest message we can receive
* from the chip/firmware; which is 64 bytes ...
*/
iqe_size = 64;
/* /*
* Set up default Queue Set parameters ... Start off with the * Set up default Queue Set parameters ... Start off with the
* shortest interrupt holdoff timer. * shortest interrupt holdoff timer.
...@@ -2320,7 +2328,7 @@ static void __devinit cfg_queues(struct adapter *adapter) ...@@ -2320,7 +2328,7 @@ static void __devinit cfg_queues(struct adapter *adapter)
struct sge_eth_rxq *rxq = &s->ethrxq[qs]; struct sge_eth_rxq *rxq = &s->ethrxq[qs];
struct sge_eth_txq *txq = &s->ethtxq[qs]; struct sge_eth_txq *txq = &s->ethtxq[qs];
init_rspq(&rxq->rspq, 0, 0, 1024, L1_CACHE_BYTES); init_rspq(&rxq->rspq, 0, 0, 1024, iqe_size);
rxq->fl.size = 72; rxq->fl.size = 72;
txq->q.size = 1024; txq->q.size = 1024;
} }
...@@ -2329,8 +2337,7 @@ static void __devinit cfg_queues(struct adapter *adapter) ...@@ -2329,8 +2337,7 @@ static void __devinit cfg_queues(struct adapter *adapter)
* The firmware event queue is used for link state changes and * The firmware event queue is used for link state changes and
* notifications of TX DMA completions. * notifications of TX DMA completions.
*/ */
init_rspq(&s->fw_evtq, SGE_TIMER_RSTRT_CNTR, 0, 512, init_rspq(&s->fw_evtq, SGE_TIMER_RSTRT_CNTR, 0, 512, iqe_size);
L1_CACHE_BYTES);
/* /*
* The forwarded interrupt queue is used when we're in MSI interrupt * The forwarded interrupt queue is used when we're in MSI interrupt
...@@ -2346,7 +2353,7 @@ static void __devinit cfg_queues(struct adapter *adapter) ...@@ -2346,7 +2353,7 @@ static void __devinit cfg_queues(struct adapter *adapter)
* any time ... * any time ...
*/ */
init_rspq(&s->intrq, SGE_TIMER_RSTRT_CNTR, 0, MSIX_ENTRIES + 1, init_rspq(&s->intrq, SGE_TIMER_RSTRT_CNTR, 0, MSIX_ENTRIES + 1,
L1_CACHE_BYTES); iqe_size);
} }
/* /*
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment