aboutsummaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorHelmut Schaa <helmut.schaa@googlemail.com>2011-03-03 13:40:33 -0500
committerJohn W. Linville <linville@tuxdriver.com>2011-03-04 14:06:47 -0500
commit5356d963304638735b4b53dfbded7a1c4ae0818d (patch)
tree2ad65243ee2dec57ca418facff48555ebd5d32c9
parentc262e08b79204f57aba1f180d6ebdb5ea9db01dd (diff)
rt2x00: Add unlikely macro to special case tx status handling
This special case shouldn't happen very often. Only if a frame that is not intended to be aggregated ends up in an AMPDU _and_ was intended to be sent at a different MCS rate as the aggregate. Hence, using unlikely is justified. Signed-off-by: Helmut Schaa <helmut.schaa@googlemail.com> Signed-off-by: Ivo van Doorn <IvDoorn@gmail.com> Signed-off-by: John W. Linville <linville@tuxdriver.com>
-rw-r--r--drivers/net/wireless/rt2x00/rt2800lib.c2
1 files changed, 1 insertions, 1 deletions
diff --git a/drivers/net/wireless/rt2x00/rt2800lib.c b/drivers/net/wireless/rt2x00/rt2800lib.c
index 76ced6dcee05..ad90c86810d7 100644
--- a/drivers/net/wireless/rt2x00/rt2800lib.c
+++ b/drivers/net/wireless/rt2x00/rt2800lib.c
@@ -681,7 +681,7 @@ void rt2800_txdone_entry(struct queue_entry *entry, u32 status)
681 * confuse the rate control algortihm by providing clearly wrong 681 * confuse the rate control algortihm by providing clearly wrong
682 * data. 682 * data.
683 */ 683 */
684 if (aggr == 1 && ampdu == 0 && real_mcs != mcs) { 684 if (unlikely(aggr == 1 && ampdu == 0 && real_mcs != mcs)) {
685 skbdesc->tx_rate_idx = real_mcs; 685 skbdesc->tx_rate_idx = real_mcs;
686 mcs = real_mcs; 686 mcs = real_mcs;
687 } 687 }