| To: | Patrick McHardy <kaber@xxxxxxxxx> |
|---|---|
| Subject: | Re: 2.6: QoS scheduling not working with IP-over-IP |
| From: | "David S. Miller" <davem@xxxxxxxxxx> |
| Date: | Fri, 13 Feb 2004 21:36:06 -0800 |
| Cc: | hadi@xxxxxxxxxx, qnex@xxxxxxxxxxxxxxxxxxx, netdev@xxxxxxxxxxx, shemminger@xxxxxxxx |
| In-reply-to: | <402D54D6.5070708@xxxxxxxxx> |
| References: | <Pine.LNX.4.44.0402101324350.810-100000@xxxxxxxxxxxxxxxxxxx> <20040211200549.736fa8b3.davem@xxxxxxxxxx> <1076561489.1032.65.camel@xxxxxxxxxxxxxxxx> <1076561998.1035.72.camel@xxxxxxxxxxxxxxxx> <1076562282.1033.76.camel@xxxxxxxxxxxxxxxx> <20040211211536.23e97997.davem@xxxxxxxxxx> <1076563502.1031.85.camel@xxxxxxxxxxxxxxxx> <1076564638.1033.91.camel@xxxxxxxxxxxxxxxx> <20040211215142.7f817513.davem@xxxxxxxxxx> <1076566176.1033.94.camel@xxxxxxxxxxxxxxxx> <20040211222259.776ad818.davem@xxxxxxxxxx> <402D54D6.5070708@xxxxxxxxx> |
| Sender: | netdev-bounce@xxxxxxxxxxx |
On Fri, 13 Feb 2004 23:51:02 +0100 Patrick McHardy <kaber@xxxxxxxxx> wrote: > while looking for other devices that may be affected I noticed > this fix changes behaviour in dev_activate. dev_activate creates > pfifo_fast queues for devices with tx_queue_len != 0 and noqueue > qdiscs otherwise. Thanks for spotting this. > Although it is uncomfortable for some users, I don't know if there > is anything to fix at all. Work-conserving qdiscs are useless for > virtual devices, as you say there is maximum one packet queued at > any time. For non-work-conserving qdiscs with default pfifo qdiscs > as leaves, I'd say it is simply a configuration error not to increase > tx_queue_len. Even with a length of 1 (or a off-by-one), you won't get > good results, except when the traffic is already well-formed. All that > is changed is that nonsensical configurations stop working. Another way to view this is that configurations that, while nonsensical, used to send packets properly no longer do. However, I do agree with you, and we should not promote this kind of crap. I'm going to revert the changeset that set tx_queue_len to 1 in the tunnels. Thanks again Patrick. |
| Previous by Date: | Re: [IPSEC] Move hardware headers for decaped packets, Herbert Xu |
|---|---|
| Next by Date: | Re: [BK PATCH] 2.6 SCTP updates, David S. Miller |
| Previous by Thread: | Re: 2.6: QoS scheduling not working with IP-over-IP, David S. Miller |
| Next by Thread: | Re: 2.6: QoS scheduling not working with IP-over-IP, jamal |
| Indexes: | [Date] [Thread] [Top] [All Lists] |