Bob Copeland wrote: > On Thu, Jul 23, 2009 at 6:45 PM, Philip A. > Prindeville<philipp_subx@xxxxxxxxxxxxxxxxxxxxx> wrote: >> Bob Copeland wrote: >>> On Thu, Jul 23, 2009 at 5:53 PM, Philip A. >>> Prindeville<philipp_subx@xxxxxxxxxxxxxxxxxxxxx> wrote: >>>>> Getting closer :-) >>>>> >>>>> ath5k_tx_processq(sc, &sc->txqs[i]); >>>>> >>>> Didn't work with rc4: >>> What did you apply against? It's supposed to be against >>> wireless-testing (or, at worst, latest compat-wireless). >>> >>>> BUG: unable to handle kernel NULL pointer dereference at 00000000 >>>> IP: [<d0935658>] :ath5k:ath5k_tasklet_tx+0x5e/0x27b > >> This was against compat-wireless-2.6.31-rc4. >> >> Alas, I already blew away the build directory... > > I think you need to use the unstable compat-wireless snapshots; > I have a feeling that the above doesn't have all of the tx queues. > > Check out ath5k.h, the txq array in struct ath5k_softc should have > NUM_TX_QUEUES entries, an earlier version only had 2. > > Although your compiler generates rather different code from mine > (different config or whatever), it looks like the oops happened > in the list_for_each_entry(... txq->q ...) -- but I don't see how > that can happen in my copy, since txq[i].q is initialized at the > same time as txq[i].setup. > Give me a shout when this patch makes its way into compat-wireless-2.6.31-rcN and I'll try again with that. -Philip -- To unsubscribe from this list: send the line "unsubscribe linux-wireless" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html
