Search Linux Wireless

Re: rt61pci: oops in rt2x00lib_txdone(), entry->skb is NULL

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

 



Hi Pavel,

Am Mittwoch 08 September 2010 schrieb Pavel Roskin:
> wireless-testing.git as of yesterday has a problem with rt61pci.  After
> a minute of web browsing, the kernel reports an oops in
> rt2x00lib_txdone() and no more traffic goes through.  That system used
> to run a month old wireless-testing.git identified as 2.6.35-rc6-wl, and
> it didn't have that problem.
> 
> The oops happens on this line:
> enum data_queue_qid qid = skb_get_queue_mapping(entry->skb);
> 
> When it happens, entry->skb is NULL.  I cannot reproduce the problem by
> ping, even by flood ping, but using a web browser on a moderately
> complex site (e.g. cnn.com) triggers the oops reliably within a minute
> or so.
> 
> The architecture is x86_64, it's a multicore system, SMP is enabled in
> the kernel.
> 
> If there are no good ideas what it might be, I'll try to find time to
> bisect the bug.

Seems like this was introduced by "rt2x00: Add helper function for
reporting tx status". At least the last hunk looks suspicious.

Pavel, could you please try this patch?

Thanks,
Helmut

diff --git a/drivers/net/wireless/rt2x00/rt61pci.c b/drivers/net/wireless/rt2x00/rt61pci.c
index 8b67cfd..80f80a0 100644
--- a/drivers/net/wireless/rt2x00/rt61pci.c
+++ b/drivers/net/wireless/rt2x00/rt61pci.c
@@ -2106,7 +2106,7 @@ static void rt61pci_txdone(struct rt2x00_dev *rt2x00dev)
 				"TX status report missed for entry %d\n",
 				entry_done->entry_idx);
 
-			rt2x00lib_txdone_noinfo(entry, TXDONE_UNKNOWN);
+			rt2x00lib_txdone_noinfo(entry_done, TXDONE_UNKNOWN);
 			entry_done = rt2x00queue_get_entry(queue, Q_INDEX_DONE);
 		}
 
--
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