Sfoglia il codice sorgente

iwlwifi: pcie: don't init a Tx queue with an SSN > size of the queue

The TVQM tells us the initial write pointer for a queue,
but that write pointer is in WiFi sequence number unit
and not in TFD index unit. Which means that the write
pointer in the TVQM's response can be bigger than the
Tx queue ring size.

Fix that by modulo'ing the write pointer from the TVQM
with the Tx queue size.

Fixes: 66128fa08806 ("iwlwifi: move to TVQM mode")
Signed-off-by: Emmanuel Grumbach <emmanuel.grumbach@intel.com>
Signed-off-by: Luca Coelho <luciano.coelho@intel.com>
Emmanuel Grumbach 8 anni fa
parent
commit
39fff75993
1 ha cambiato i file con 5 aggiunte e 2 eliminazioni
  1. 5 2
      drivers/net/wireless/intel/iwlwifi/pcie/tx-gen2.c

+ 5 - 2
drivers/net/wireless/intel/iwlwifi/pcie/tx-gen2.c

@@ -1033,6 +1033,7 @@ int iwl_trans_pcie_dyn_txq_alloc(struct iwl_trans *trans,
 		.flags = CMD_WANT_SKB,
 		.flags = CMD_WANT_SKB,
 	};
 	};
 	int ret, qid;
 	int ret, qid;
+	u32 wr_ptr;
 
 
 	txq = kzalloc(sizeof(*txq), GFP_KERNEL);
 	txq = kzalloc(sizeof(*txq), GFP_KERNEL);
 	if (!txq)
 	if (!txq)
@@ -1073,6 +1074,7 @@ int iwl_trans_pcie_dyn_txq_alloc(struct iwl_trans *trans,
 
 
 	rsp = (void *)hcmd.resp_pkt->data;
 	rsp = (void *)hcmd.resp_pkt->data;
 	qid = le16_to_cpu(rsp->queue_number);
 	qid = le16_to_cpu(rsp->queue_number);
+	wr_ptr = le16_to_cpu(rsp->write_pointer);
 
 
 	if (qid >= ARRAY_SIZE(trans_pcie->txq)) {
 	if (qid >= ARRAY_SIZE(trans_pcie->txq)) {
 		WARN_ONCE(1, "queue index %d unsupported", qid);
 		WARN_ONCE(1, "queue index %d unsupported", qid);
@@ -1088,10 +1090,11 @@ int iwl_trans_pcie_dyn_txq_alloc(struct iwl_trans *trans,
 
 
 	txq->id = qid;
 	txq->id = qid;
 	trans_pcie->txq[qid] = txq;
 	trans_pcie->txq[qid] = txq;
+	wr_ptr &= (TFD_QUEUE_SIZE_MAX - 1);
 
 
 	/* Place first TFD at index corresponding to start sequence number */
 	/* Place first TFD at index corresponding to start sequence number */
-	txq->read_ptr = le16_to_cpu(rsp->write_pointer);
-	txq->write_ptr = le16_to_cpu(rsp->write_pointer);
+	txq->read_ptr = wr_ptr;
+	txq->write_ptr = wr_ptr;
 	iwl_write_direct32(trans, HBUS_TARG_WRPTR,
 	iwl_write_direct32(trans, HBUS_TARG_WRPTR,
 			   (txq->write_ptr) | (qid << 16));
 			   (txq->write_ptr) | (qid << 16));
 	IWL_DEBUG_TX_QUEUES(trans, "Activate queue %d\n", qid);
 	IWL_DEBUG_TX_QUEUES(trans, "Activate queue %d\n", qid);