From mboxrd@z Thu Jan  1 00:00:00 1970
Received: from mailman by lists.gnu.org with tmda-scanned (Exim 4.43)
	id 1O21nV-0001c0-J6
	for qemu-devel@nongnu.org; Wed, 14 Apr 2010 08:34:25 -0400
Received: from [140.186.70.92] (port=54596 helo=eggs.gnu.org)
	by lists.gnu.org with esmtp (Exim 4.43) id 1O21nS-0001be-Tt
	for qemu-devel@nongnu.org; Wed, 14 Apr 2010 08:34:23 -0400
Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.69)
	(envelope-from <davem@davemloft.net>) id 1O21nR-0000yp-BF
	for qemu-devel@nongnu.org; Wed, 14 Apr 2010 08:34:22 -0400
Received: from 74-93-104-97-washington.hfc.comcastbusiness.net
	([74.93.104.97]:55794 helo=sunset.davemloft.net)
	by eggs.gnu.org with esmtp (Exim 4.69)
	(envelope-from <davem@davemloft.net>) id 1O21nR-0000yY-5U
	for qemu-devel@nongnu.org; Wed, 14 Apr 2010 08:34:21 -0400
Date: Wed, 14 Apr 2010 04:55:06 -0700 (PDT)
Message-Id: <20100414.045506.52212061.davem@davemloft.net>
From: David Miller <davem@davemloft.net>
In-Reply-To: <20100414005822.GD18044@gondor.apana.org.au>
References: <20100413173919.GC26011@redhat.com>
	<1271183463.16881.545.camel@edumazet-laptop>
	<20100414005822.GD18044@gondor.apana.org.au>
Mime-Version: 1.0
Content-Type: Text/Plain; charset=us-ascii
Content-Transfer-Encoding: 7bit
Subject: [Qemu-devel] Re: [PATCH] tun: orphan an skb on tx
List-Id: qemu-devel.nongnu.org
List-Unsubscribe: <http://lists.nongnu.org/mailman/listinfo/qemu-devel>,
	<mailto:qemu-devel-request@nongnu.org?subject=unsubscribe>
List-Archive: <http://lists.gnu.org/pipermail/qemu-devel>
List-Post: <mailto:qemu-devel@nongnu.org>
List-Help: <mailto:qemu-devel-request@nongnu.org?subject=help>
List-Subscribe: <http://lists.nongnu.org/mailman/listinfo/qemu-devel>,
	<mailto:qemu-devel-request@nongnu.org?subject=subscribe>
To: herbert@gondor.apana.org.au
Cc: paul.moore@hp.com, David.Woodhouse@intel.com, eric.dumazet@gmail.com, mst@redhat.com, jan.kiszka@siemens.com, linux-kernel@vger.kernel.org, qemu-devel@nongnu.org, netdev@vger.kernel.org

From: Herbert Xu <herbert@gondor.apana.org.au>
Date: Wed, 14 Apr 2010 08:58:22 +0800

> On Tue, Apr 13, 2010 at 08:31:03PM +0200, Eric Dumazet wrote:
>>
>> Herbert Acked your patch, so I guess its OK, but I think it can be
>> dangerous.
> 
> The tun socket accounting was never designed to stop it from
> flooding another tun interface.  It's there to stop it from
> transmitting above a destination interface TX bandwidth and
> cause unnecessary packet drops.  It also limits the total amount
> of kernel memory that can be pinned down by a single tun interface.
> 
> In this case, all we're doing is shifting the accounting from the
> "hardware" queue to the qdisc queue.
> 
> So your ability to flood a tun interface is essentially unchanged.
> 
> BTW we do the same thing in a number of hardware drivers, as well
> as virtio-net.

Right.  Although this reminds me about the whole SKB
orphaning on xmit issue that keeps coming back to haunt
us.

If there weren't odd references to the SKB's socket in
the packet scheduler et al. we could just orphan these
things right upon entry to the qdisc and not have to
add hacks like this to every driver.

In fact... maybe we can just do it in dev_hard_queue_xmit()
since we are out of the qdisc at that point.... but I guess
there might be weird drivers that want the SKB socket in
their ->xmit routine...  Ho hum.

In any event that's net-next-2.6 exploratory material, and I've
applied this patch to net-2.6, thanks!