-----BEGIN PGP SIGNED MESSAGE----- Hash: SHA1 Jouni Malinen a écrit : > On Mon, Dec 15, 2008 at 03:34:58PM +0100, Benoit Papillault wrote: >> This patch is close to the original code except that >> ieee80211_get_hdrlen_from_skb() has been replaced by >> ath5k_hw_get_hdrlen_from_skb() which is specific to Atheros hardware. The same >> probably applies to ath9k as well. > > Most of this is changes on how unspecified frames are handled (e.g., > frames that are not used in IEEE 802.11). What is the use case that > justifies this type of extra complexity to be added into the driver? > Please note that there are no guarantees that all hardware revisions > behave the same as far as undocumented functionality is concerned. The > only clear case when padding is required is data frames with non-empty > frame body. > Agreed. In fact, I wanted to know how the hardware behaves in the general case to better understand it in the particular case of 802.11 frames. To me, the bug we have seen when receiving a ACK frame in monitor mode was not because "ACK is a small control frame", it was because the header length we computed was smaller than what the hardware considered. Another point is that we have to deal with anything on the RX side, be it later classified as invalid or not. As far as the hardware is concerned, I'm not working for Atheros, but I bet that hardware guys did the same thing for all their chipsets. I could test it on AR5416 and AR9160 if needed. To reply to Bob as well, we need to compute the number of padded bytes + the position at which the padding occurs, so proper "header" length is required. Just my 2 cents. Benoit -----BEGIN PGP SIGNATURE----- Version: GnuPG v1.4.6 (GNU/Linux) Comment: Using GnuPG with Mozilla - http://enigmail.mozdev.org iD8DBQFJRsbZOR6EySwP7oIRArWVAJ0fjqY01Xi3qTH7DIJBE9D+c099pQCgp/u5 pHHZiKOzsU83CEecamnuZpY= =w10J -----END PGP SIGNATURE----- -- 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
