Search Linux Wireless

Re: [PATCH] d80211: respect extra_tx_headroom

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

 



On Mon, 5 Feb 2007 16:32:24 +0100, Ivo van Doorn wrote:
> When a driver requested additional header room
> through the extra_tx_headroom field, the stack
> should respect that and make sure that all frames
> that are being send to the stack actually have
> that extra header room.
> 
> [...]
> --- dscape/net/d80211/ieee80211_sta.c	2007-02-05 16:18:41.000000000 +0100
> [...]
> @@ -2007,6 +2017,8 @@
>  		if (!skb)
>  			break;
>  
> +		skb_reserve(skb, local->hw.extra_tx_headroom);
> +
>  		mgmt = (struct ieee80211_mgmt *)
>  			skb_put(skb, 24 + sizeof(mgmt->u.beacon));
>  		memset(mgmt, 0, 24 + sizeof(mgmt->u.beacon));

Please enlarge that dev_alloc_skb(400) few lines above by
extra_tx_headroom as well.

Btw, it would be nice if you could use -p option for diff as it makes
reviewing easier for me.

Thanks,

 Jiri

-- 
Jiri Benc
SUSE Labs
-
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