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 <[email protected]> > > 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 <[email protected]> > > Signed-off-by: John W. Linville <[email protected]> > > > > Acked-by: Brett Rudley <[email protected]> > Signed-off-by: Arend van Spriel <[email protected]> > --- > 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 _______________________________________________ devel mailing list [email protected] http://driverdev.linuxdriverproject.org/mailman/listinfo/devel
