On Mon, 2012-03-05 at 13:17 -0800, Thomas Pedersen wrote: > Airtime link metric estimation was broken in HT mesh, use > cfg80211_calculate_bitrate to get the right rate value. > > Also factor out tx rate copying from sta_set_sinfo(). > > Signed-off-by: Thomas Pedersen <thomas@xxxxxxxxxxx> > --- > v2: > factor out rate copying (Johannes) > > v3: > just factor out tx rate copying (Johannes) > > net/mac80211/cfg.c | 22 ++++++++++++++-------- > net/mac80211/mesh_hwmp.c | 6 ++++-- > net/mac80211/sta_info.h | 2 ++ > 3 files changed, 20 insertions(+), 10 deletions(-) > > diff --git a/net/mac80211/cfg.c b/net/mac80211/cfg.c > index 6a77d4c..9e6accd 100644 > --- a/net/mac80211/cfg.c > +++ b/net/mac80211/cfg.c > @@ -336,6 +336,19 @@ static void rate_idx_to_bitrate(struct rate_info *rate, struct sta_info *sta, in > rate->mcs = idx; > } > > +void sta_set_rate_info(struct sta_info *sta, struct ieee80211_tx_rate rate, > + struct rate_info *rinfo) I think passing a struct by value is rather expensive? Also I think it should have "tx" somewhere in the name. johannes -- 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
