On Tue, Dec 07, 2010 at 10:43:38PM +0530, Ben Greear wrote: > On 12/07/2010 12:31 AM, Vasanthakumar Thiagarajan wrote: > > On Tue, Dec 07, 2010 at 01:19:22PM +0530, Ben Greear wrote: > >> On 12/06/2010 11:42 PM, Vasanthakumar Thiagarajan wrote: > >>> On Tue, Dec 07, 2010 at 11:06:24AM +0530, Ben Greear wrote: > >>>> On 12/06/2010 09:21 PM, Luis R. Rodriguez wrote: > >>>>> On Mon, Dec 6, 2010 at 9:13 PM,<greearb@xxxxxxxxx> wrote: > >>>>>> From: Ben Greear<greearb@xxxxxxxxxxxxxxx> > >>>>>> > >>>>>> It can be NULL according to docs, and logging showed it > >>>>>> to be NULL in practice. > >>>>>> > >>>>>> Signed-off-by: Ben Greear<greearb@xxxxxxxxxxxxxxx> > >>>>> > >>>>> Does this fix an oops? If so can you explain and provide the trace and > >>>>> resubmit and cc stable in the commit log? > >>>> > >>>> I think it fixes the TID corruption I posted about earlier. It seems > >>>> so obvious though, that I'm curious why no one else sees problems, > >>>> and why I didn't see more crashes in my setup. > >>>> > >>>> (The paprd code appears to send with null STA, for instance, and my > >>>> printks showed lots of NULL stas in my 16-sta test setup, though I > >>>> don't think I'm using the paprd code path.) > >>> > >>> paprd is used only with>= AR9003. > >> > >> Whoever coded it up hopefully had that hardware...so why didn't > >> they see lots of crashes? > > > > I myself tested it lot of times, but did not see any crash, weird. > > Looks like the offending change when in recently (11/4/10, one of Felix's > patches). > > That is probably why no one else is hitting this yet, and it > isn't needed for stable I'm guessing.... Nice, care to send a patch?. Yeah, it is not needed for stable. Vasanth -- 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
