Search Linux Wireless

Re: [PATCH] ath9k: Check for NULL sta in ath_tx_start

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



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.

Vasanth

> 
> Thanks,
> Ben
> 
> >
> > Vasanth
> 
> 
> -- 
> Ben Greear <greearb@xxxxxxxxxxxxxxx>
> Candela Technologies Inc  http://www.candelatech.com
--
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


[Index of Archives]     [Linux Host AP]     [ATH6KL]     [Linux Bluetooth]     [Linux Netdev]     [Kernel Newbies]     [Linux Kernel]     [IDE]     [Security]     [Git]     [Netfilter]     [Bugtraq]     [Yosemite News]     [MIPS Linux]     [ARM Linux]     [Linux Security]     [Linux RAID]     [Linux ATA RAID]     [Samba]     [Device Mapper]
  Powered by Linux