On Mon, 2018-02-19 at 10:02 +0100, Nicolas Cavallari wrote:
> On 16/02/2018 17:12, Luca Coelho wrote:
> > -   sta = sta_info_alloc(sdata, addr, GFP_ATOMIC);
> > -   if (!sta)
> > -           return;
> >  
> > -   /* make sure mandatory rates are always added */
> > -   sband = local->hw.wiphy->bands[band];
> > -   sta->sta.supp_rates[band] = supp_rates |
> > -                   ieee80211_mandatory_rates(sband,
> > scan_width);
> > -
> > -   spin_lock(&ifibss->incomplete_lock);
> > -   list_add(&sta->list, &ifibss->incomplete_stations);
> > -   spin_unlock(&ifibss->incomplete_lock);
> > -   ieee80211_queue_work(&local->hw, &sdata->work);
> > +   ieee80211_send_probe_req(sdata, sdata->vif.addr, addr,
> > +                            sdata->u.ibss.ssid, sdata-
> > >u.ibss.ssid_len,
> > +                            NULL, 0, (u32)-1, true, 0,
> > +                            chanctx_conf->def.chan, false);
> > +   rcu_read_unlock();
> >  }
> 
> Won't that spam the channel with probe request if the traffic from
> the
> other station is heavy ?
> 
> And, if the other station is running the same code and didn't know
> about us (e.g. at join time), won't it also spam the channel with
> probe requests after receiving our many probe requests ? (as well as
> the probe responses from both stations)
> 
> Obviously it should stop after receiving a probe response... But what
> if the link is badly asymmetric ?

Sari, can you please look into this?

--
Cheers,
Luca.

Reply via email to