mac80211: Minor optimization in ieee80211_rx_h_data
authorHelmut Schaa <helmut.schaa@googlemail.com>
Tue, 30 Nov 2010 11:19:34 +0000 (12:19 +0100)
committerJohn W. Linville <linville@tuxdriver.com>
Tue, 30 Nov 2010 18:58:07 +0000 (13:58 -0500)
commit08ca944eb240b2299e743c76b43fbc7c2dd251de
tree9782965ad4754969685cfd440b81ded25d4d6023
parent20ed3166c84d145589a89d8cde12aa32cf2d17f4
mac80211: Minor optimization in ieee80211_rx_h_data

Remove a superfluous ieee80211_is_data check as that was checked a few
lines before already and we wont't get here for non-data frames at all.

Second, the frame was already converted to 802.3 header format and
reading the fc and addr1 fields was only possible because the 802.3
header is short enough and didn't overwrite the relevant parts of the
802.11 header. Make the code more obvious by checking the ethernet
header's h_dest field.

Furthermore reorder the conditions to reduce the number of checks
when dynamic powersave is not needed (AP mode for example).

Signed-off-by: Helmut Schaa <helmut.schaa@googlemail.com>
Reviewed-by: Johannes Berg <johannes@sipsolutions.net>
Signed-off-by: John W. Linville <linville@tuxdriver.com>
net/mac80211/rx.c