On Sun, 2009-02-22 at 00:18 +0100, Alina Friedrichsen wrote: > Give slow hardware some time to do the TSF sync, to not run into an > IBSS merging endless loop in some rarely situations. > > Version 2 adds a comment. > > Version 3: I had forgotten to change the bracket. > > diff --git a/net/mac80211/ibss.c b/net/mac80211/ibss.c > index 1bbfc70..ea737bf 100644 > --- a/net/mac80211/ibss.c > +++ b/net/mac80211/ibss.c > @@ -332,6 +332,10 @@ static void ieee80211_rx_bss_info(struct ieee80211_sub_if_data *sdata, > jiffies); > #endif > > + /* give slow hardware some time to do the TSF sync */ > + if (rx_timestamp < 0x400000) > + goto put_bss; Please elaborate on what this does. This uses the assumption that reset_tsf() makes it start at 0, and that then the hardware won't sync up until N seconds etc. I'd like that to be spelt out here, rather than having to think about this when reading the comment. johannes
Attachment:
signature.asc
Description: This is a digitally signed message part