Re: [PATCH] mac80211: rewrite fragmentation code

[Johannes Berg <johannes@sipsolutions.net>]

[-- Attachment #1: Type: text/plain, Size: 1132 bytes --]

On Wed, 2008-05-07 at 19:41 +0800, Herbert Xu wrote:
> On Wed, May 07, 2008 at 01:22:19PM +0200, Johannes Berg wrote:
> >
> > +	if (unlikely(netif_queue_stopped(local->mdev) ||
> > +		     __ieee80211_queue_stopped(local, control->queue)))
> > +		return NETDEV_TX_BUSY;
> 
> Returning busy should be avoided because not everything expects it.

Hmm. Why does busy exist then? Historical accident? I really really
don't want to keep all that logic that does "if device enables queue
then first check if maybe we have pending packets and try to send them
and then go back to the regular queues and see if ..."

> Besides, I'm not sure if this will even work if you're fiddling
> with skb->next.  Perhaps you can stash it in a pointer local to
> your device.

You're looking at an internal helper function that is never called with
skb->next assigned; if it returns BUSY for a fragmented skb then later
in __ieee80211_tx() I will store the fragmented frame away and return
OK.

But if I really need to avoid returning busy I need to hook into all the
queue management stuff which I'd like to avoid.

johannes

[-- Attachment #2: This is a digitally signed message part --]
[-- Type: application/pgp-signature, Size: 828 bytes --]