[PATCH] staging: brcm80211: remove assert to avoid panic on 2.6.37 kernel
Greg KH
greg at kroah.com
Thu Jan 20 12:17:22 PST 2011
On Wed, Jan 19, 2011 at 02:19:02PM +0100, Arend van Spriel wrote:
> The driver assumed it would receive skb packets from MAC80211 which
> are not cloned. To guard this assumption an assert was placed in
> the transmit routine. As of kernel 2.6.37 it turns out MAC80211 does
> pass skb packets that are cloned. The assert is also not needed as
> it does not lead to a failure state in our driver when the packet is
> cloned. Therefore the assert can safely be removed.
>
> > commit f8a0a781488ec7288d1049e5d2022850aa98f7b6
> > Author: Felix Fietkau <nbd at openwrt.org>
> > Date: Sat Dec 18 19:30:50 2010 +0100
> >
> > mac80211: fix potentially redundant skb data copying
> >
> > When an skb is shared, it needs to be duplicated, along with its data buffer
> > If the skb does not have enough headroom, using skb_copy might cause the dat
> > buffer to be copied twice (once by skb_copy and once by pskb_expand_head).
> > Fix this by using skb_clone initially and letting ieee80211_skb_resize sort
> > out the rest.
> >
> > Signed-off-by: Felix Fietkau <nbd at openwrt.org>
> > Signed-off-by: John W. Linville <linville at tuxdriver.com>
> >
>
> Acked-by: Brett Rudley <brudley at broadcom.com>
> Signed-off-by: Arend van Spriel <arend at broadcom.com>
> ---
> drivers/staging/brcm80211/brcmsmac/wlc_mac80211.c | 1 -
This patch doesn't apply at all to Linus's tree, which is where it needs
to go right now, right?
Care to provide a version that I can apply?
thanks,
greg k-h
More information about the devel
mailing list