netdev.vger.kernel.org archive mirror
 help / color / mirror / Atom feed
From: "David S. Miller" <davem@redhat.com>
To: hadi@cyberus.ca
Cc: qnex@atlantis.knm.org.pl, netdev@oss.sgi.com, shemminger@osdl.org
Subject: Re: 2.6: QoS scheduling not working with IP-over-IP
Date: Wed, 11 Feb 2004 22:22:59 -0800	[thread overview]
Message-ID: <20040211222259.776ad818.davem@redhat.com> (raw)
In-Reply-To: <1076566176.1033.94.camel@jzny.localdomain>

On 12 Feb 2004 01:09:36 -0500
jamal <hadi@cyberus.ca> wrote:

> On Thu, 2004-02-12 at 00:51, David S. Miller wrote:
> 
> >  Show me a code path where more than one packet
> > can occupy the queue of one of these tunnel devices? :-)
> 
> ok, makes sense ;->

Great, here is the patch I am testing now.

BTW, you missed fixing up the xxx0 device in each of these things.

===== net/ipv4/ip_gre.c 1.34 vs edited =====
--- 1.34/net/ipv4/ip_gre.c	Tue Oct 28 01:13:14 2003
+++ edited/net/ipv4/ip_gre.c	Wed Feb 11 22:17:28 2004
@@ -276,6 +276,7 @@
 	  return NULL;
 
 	dev->init = ipgre_tunnel_init;
+	dev->tx_queue_len = 1;
 	nt = dev->priv;
 	nt->parms = *parms;
 
@@ -1269,6 +1270,7 @@
 	}
 
 	ipgre_fb_tunnel_dev->init = ipgre_fb_tunnel_init;
+	ipgre_fb_tunnel_dev->tx_queue_len = 1;
 
 	if ((err = register_netdev(ipgre_fb_tunnel_dev)))
 		goto fail;
===== net/ipv4/ipip.c 1.37 vs edited =====
--- 1.37/net/ipv4/ipip.c	Sun Oct 19 00:12:04 2003
+++ edited/net/ipv4/ipip.c	Wed Feb 11 22:17:46 2004
@@ -246,6 +246,7 @@
 	nt = dev->priv;
 	SET_MODULE_OWNER(dev);
 	dev->init = ipip_tunnel_init;
+	dev->tx_queue_len = 1;
 	dev->destructor = free_netdev;
 	nt->parms = *parms;
 
@@ -892,6 +893,7 @@
 	}
 
 	ipip_fb_tunnel_dev->init = ipip_fb_tunnel_init;
+	ipip_fb_tunnel_dev->tx_queue_len = 1;
 
 	if ((err = register_netdev(ipip_fb_tunnel_dev)))
 	    goto fail;
===== net/ipv6/sit.c 1.33 vs edited =====
--- 1.33/net/ipv6/sit.c	Mon Jan 19 21:23:19 2004
+++ edited/net/ipv6/sit.c	Wed Feb 11 22:18:05 2004
@@ -184,6 +184,7 @@
 
 	nt = dev->priv;
 	dev->init = ipip6_tunnel_init;
+	dev->tx_queue_len = 1;
 	nt->parms = *parms;
 
 	if (register_netdevice(dev) < 0) {
@@ -832,6 +833,7 @@
 	}
 
 	ipip6_fb_tunnel_dev->init = ipip6_fb_tunnel_init;
+	ipip6_fb_tunnel_dev->tx_queue_len = 1;
 
 	if ((err =  register_netdev(ipip6_fb_tunnel_dev)))
 		goto fail;

  reply	other threads:[~2004-02-12  6:22 UTC|newest]

Thread overview: 19+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
     [not found] <Pine.LNX.4.44.0402101324350.810-100000@atlantis.knm.org.pl>
2004-02-12  4:05 ` 2.6: QoS scheduling not working with IP-over-IP David S. Miller
2004-02-12  4:51   ` jamal
2004-02-12  4:59     ` jamal
2004-02-12  5:04       ` jamal
2004-02-12  5:15         ` David S. Miller
2004-02-12  5:25           ` jamal
2004-02-12  5:43             ` jamal
2004-02-12  5:51               ` David S. Miller
2004-02-12  6:09                 ` jamal
2004-02-12  6:22                   ` David S. Miller [this message]
     [not found]                     ` <402D54D6.5070708@trash.net>
2004-02-14  5:36                       ` David S. Miller
2004-02-14 12:45                         ` jamal
2004-02-14 13:06                           ` jamal
2004-02-14 13:56                             ` jamal
2004-02-14 20:49                               ` David S. Miller
2004-02-15 10:30                                 ` Patrick McHardy
2004-02-18  3:10                                   ` David S. Miller
2004-02-18 11:21                                     ` Patrick McHardy
2004-02-18 21:22                                       ` David S. Miller

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=20040211222259.776ad818.davem@redhat.com \
    --to=davem@redhat.com \
    --cc=hadi@cyberus.ca \
    --cc=netdev@oss.sgi.com \
    --cc=qnex@atlantis.knm.org.pl \
    --cc=shemminger@osdl.org \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).