> Well Ralink doesn't seem to add this padding since this bug appeared, > remember all bytes from the DMA was copied to the skb buffer so if there > was any padding included it would have been copied as well. ;) Not necessarily, Broadcom hardware adds the padding in front of the 802.11 header so if you'd start copying with the 802.11 header you'd run into the same thing. A quick look at the rt2x00pci.c file doesn't suggest that there's anything variable about the RX header though so I guess that indeed this may be a problem. > Anyway, I have worked on a fix for the padding and I'll commit it to > rt2x00.git first to see if anybody reports any problems with it before > sending it to wireless-dev. Great. I just posted a similar fix in the other thread for zd1211, does that look similar to yours as well? Should we have a static inline with this code "ieee80211_needs_padding()" or something? johannes
Attachment:
signature.asc
Description: This is a digitally signed message part