Re: [RFC PATCH v2 4/4] mac80211: add beacon filtering support

[Kalle Valo <kalle.valo@nokia.com>]

Johannes Berg wrote:
> On Sat, 2009-03-14 at 19:14 +0200, Kalle Valo wrote:
> 
>> +void ieee80211_beacon_loss(struct ieee80211_hw *hw)
>> +{
>> +	struct ieee80211_local *local = hw_to_local(hw);
>> +	struct ieee80211_sub_if_data *sdata;
>> +
>> +	rcu_read_lock();
>> +	list_for_each_entry(sdata, &local->interfaces, list) {
>> +		if (sdata->vif.type != NL80211_IFTYPE_STATION)
>> +			continue;
>> +
>> +		queue_work(local->hw.workqueue,
>> +			   &sdata->u.mgd.beacon_loss_work);
>> +	}
>> +	rcu_read_unlock();
>> +}
>> +EXPORT_SYMBOL(ieee80211_beacon_loss);
> 
> Shouldn't the driver just pass the relevant vif pointer?

Nice, that way the implementation would be a lot simpler and consistent 
with the rest of the driver interface. Yes, I'll change it to use vif 
pointer.

Kalle