New version of the NETIF_F_LLTX for network devices patch. This allows network drivers to set the NETIF_F_LLTX flag and then do their own locking in start_queue_xmit. This lowers locking overhead in
This missed the current round of merges, I'll try to get it in by the end of the week. I'm probably going to put the infrastructure patch in first, then the tg3/e1000 bits. Can you resend me the e100
Are you reinventing the rules or changing them? hard_start_xmit() return codes are intepreted as follows: 0: typically means the packet was put in the ring. It is being abused by a few drivers to mea
I'm just offering the driver an additional choice . Actually when you return 1 then the kernel prints an ugly message and it is considered a bug. Here -1 is legal. For safety -1 should be only used f
1 is legal in contexts where queueing occurs. See for example qdisc_restart(). It's only illegal here because this is the direct xmit path without queueing. Cheers, -- Visit Openswan at http://www.op
I agree my sentence was misleading. Basically the distinction here is: 1 is for flow control -1 is for lock contention I think it's better to separate them because it minimizes the risk of breaking o
Is this an effect of your code? This is not so in existing code. Dont see how its going to break old drivers since they wont be making this kind of call anyways. In both cases it is semantically a fl
We are merely moving the sch_generic.c locking logic into the drivers. The behavior is entirely equivalent except that one level of unnecessary locking has been removed. I think his change is valid,