Re: kernel BUG at drivers/net/wireless/iwlwifi/iwl3945-base.c:3127!

[Samuel Ortiz <samuel@sortiz.org>]

Hi Jason,

On Mon, Jan 26, 2009 at 10:35:39PM -0500, Jason Andryuk wrote:
> On Mon, Jan 26, 2009 at 10:13 PM, Jason Andryuk <jandryuk@gmail.com> wrote:
> > I was away from the computer, so I would not have been able to test
> > anything anyway.
> >
> > I tested  wireless-testings of  master-2009-01-23 and had the same
> > inability to authenticate.  The WARNING (previous BUG) was triggered.
> > Below are the logs.
> >
> > I see wireless-testing merged some of your commits, so I'll go try those out.
> >
> > Jason
> 
> Same result: no authentication.  The "wrong command queue 0, sequence
> 0x0 readp=9 writep=9" message came twice right after Network Manager
> gave up and displayed the status of the interface as disconnected.
> 
> Is the "Microcode SW error detected. Restarting 0x82000008." message
> of concern?  I have seen that in all the traces.
No, that's just a consequence of the bug, not the bug itself.
Would you mind applying this patch on top of your latest wireless-testing tree
and testing 3945 with it ? Thanks for your patience.

---
 drivers/net/wireless/iwlwifi/iwl-rx.c |    8 --------
 1 file changed, 8 deletions(-)

Index: wireless-testing/drivers/net/wireless/iwlwifi/iwl-rx.c
===================================================================
--- wireless-testing.orig/drivers/net/wireless/iwlwifi/iwl-rx.c	2009-01-27 17:13:46.000000000 +0100
+++ wireless-testing/drivers/net/wireless/iwlwifi/iwl-rx.c	2009-01-27 17:15:46.000000000 +0100
@@ -346,11 +346,6 @@ int iwl_rx_queue_alloc(struct iwl_priv *
 	if (!rxq->bd)
 		goto err_bd;
 
-	rxq->rb_stts = pci_alloc_consistent(dev, sizeof(struct iwl_rb_status),
-					&rxq->rb_stts_dma);
-	if (!rxq->rb_stts)
-		goto err_rb;
-
 	/* Fill the rx_used queue with _all_ of the Rx buffers */
 	for (i = 0; i < RX_FREE_BUFFERS + RX_QUEUE_SIZE; i++)
 		list_add_tail(&rxq->pool[i].list, &rxq->rx_used);
@@ -362,9 +357,6 @@ int iwl_rx_queue_alloc(struct iwl_priv *
 	rxq->need_update = 0;
 	return 0;
 
-err_rb:
-	pci_free_consistent(priv->pci_dev, 4 * RX_QUEUE_SIZE, rxq->bd,
-			    rxq->dma_addr);
 err_bd:
 	return -ENOMEM;
 }
-- 
Intel Open Source Technology Centre
http://oss.intel.com/