Search Linux Wireless

Re: [RFC] mac80211: Print unknown packet type

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

 



On Wed, 2008-12-17 at 12:27 -0600, Larry Finger wrote:
> Johannes,
> 
> I'm trying to find a problem with p54usb that causes it to go belly-up under
> stress testing. In one of my runs, I hit the WARN_ON in
> ieee80211_tasklet_handler(). Unfortunately, I got no indication of what was in
> skb->pkt_type, just that it was not one of the expected types.

That really points to memory corruption somewhere.

> Do you think that adding the printk below would be worthwhile?
> 
> Thanks,
> 
> Larry
> ---
> 
> Index: wireless-testing/net/mac80211/main.c
> ===================================================================
> --- wireless-testing.orig/net/mac80211/main.c
> +++ wireless-testing/net/mac80211/main.c
> @@ -341,6 +341,8 @@ static void ieee80211_tasklet_handler(un
>  			dev_kfree_skb(skb);
>  			break ;
>  		default:
> +			printk(KERN_INFO "mac80211: Packet is of unknown type"
> +					 " %d\n", skb->pkt_type);
>  			WARN_ON(1);

Seems reasonable to me. I'd do

+	WARN(1, "mac80211: ...", skb->pkt_type);
-	WARN_ON(1);

instead.

johannes

--
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