]> git.proxmox.com Git - mirror_ubuntu-disco-kernel.git/commitdiff
mac80211: clarify alignment comment
authorJohannes Berg <johannes.berg@intel.com>
Wed, 20 Feb 2013 20:44:12 +0000 (21:44 +0100)
committerJohannes Berg <johannes.berg@intel.com>
Wed, 6 Mar 2013 15:35:41 +0000 (16:35 +0100)
The comment says something about __skb_push(), but that
isn't even called in the code any more. Looking at the
git history, that comment never even made sense when it
was still called, so just replace that part to note it
still works even when align isn't 0 or 2.

Reported-by: Eric Dumazet <eric.dumazet@gmail.com>
Signed-off-by: Johannes Berg <johannes.berg@intel.com>
net/mac80211/rx.c

index bb73ed2d20b90e8ece75bdd41a331fecb9c9d7ac..acf006f2d61a0a8054e846150ff8c02b2e1c608e 100644 (file)
@@ -1894,8 +1894,10 @@ ieee80211_deliver_skb(struct ieee80211_rx_data *rx)
                 * 'align' will only take the values 0 or 2 here
                 * since all frames are required to be aligned
                 * to 2-byte boundaries when being passed to
-                * mac80211. That also explains the __skb_push()
-                * below.
+                * mac80211; the code here works just as well if
+                * that isn't true, but mac80211 assumes it can
+                * access fields as 2-byte aligned (e.g. for
+                * compare_ether_addr)
                 */
                align = ((unsigned long)(skb->data + sizeof(struct ethhdr))) & 3;
                if (align) {