On Wed, 2010-05-26 at 16:13 +0300, Juuso Oikarinen wrote: > > > diff --git a/net/mac80211/ibss.c b/net/mac80211/ibss.c > > > index b2cc1fd..0b8360c 100644 > > > --- a/net/mac80211/ibss.c > > > +++ b/net/mac80211/ibss.c > > > @@ -529,7 +529,8 @@ static void ieee80211_sta_create_ibss(struct ieee80211_sub_if_data *sdata) > > > sdata->drop_unencrypted = 0; > > > > > > __ieee80211_sta_join_ibss(sdata, bssid, sdata->vif.bss_conf.beacon_int, > > > - ifibss->channel, 3, /* first two are basic */ > > > + ifibss->channel, > > > + 15, /* 1, 2, 5.5 and 11 are basic */ > > > > It's still wrong for 5 GHz. > > Yes, it is as wrong for 5GHz as it was before the change. No it wasn't wrong before the change, and only the comment is now wrong ... before the change, 6 and 9 mbps were basic for 5ghz, now it would be 5,9,12 and 18. Neither set actually overlaps with the mandatory rates, which is a bug I guess? > > We've had this discussion before. Making more rates required will break > > compatibility with 11b devices. > > I don't understand this. AFAIK also in 11b all these rates are > mandatory. How come this breaks 11b? No, only 1mbit is really mandatory in 11b. So I guess our default should be just "1" rather than "3" or something like that. > This change is for creating IBSS, not joining. In case of joining, we > still adhere to the basic rates set by the creator, or that is what I > understand based on the code. > > > What should be done here is add a basic rates parameter to the IBSS join > > nl80211 command so that you can decide at runtime which rates to use as > > basic. > > Yeah, this is a solution needed latest when 5GHz support is implemented. Well this hopefully doesn't come as a surprise to you, but mac80211 does support 5 GHz operation :) 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
