Re: [PATCH V2] mac80211: Revise,pending queue depth in ieee80211_local data structure

[Johannes Berg <johannes@sipsolutions.net>]

On Fri, 19 Mar 2010 10:33:47 +0100, "lorenzo.bianconi83@gmail.com"
>>  	if (!skb_queue_empty(&local->pending[queue]))
>>  		tasklet_schedule(&local->tx_pending_tasklet);
>> -
>> -	rcu_read_lock();
>> -	list_for_each_entry_rcu(sdata, &local->interfaces, list)
>> -		netif_tx_wake_queue(netdev_get_tx_queue(sdata->dev, queue));
>> -	rcu_read_unlock();

That's obviously wrong, need to move the code to the else branch of the
above if instead.

> I tested the patch on kernel 2.6.32.7 with compat-wireless-2010-03-03
but
> it seems that the problem is not solved. If I set the lowest priority
queue
> (Backgreound), the system will crash for an out of memory panic. During
the
> tests I carried out, I transmit 50Mbps UDP traffic.

How are you generating traffic? I just dumped like 2Gbps traffic at it
and everything works just fine. I verified queues are stopped and started
properly.

johannes