netdev
[Top] [All Lists]

Re: simple change to qdisc_restart()

To: Eric Lemoine <Eric.Lemoine@xxxxxxx>
Subject: Re: simple change to qdisc_restart()
From: Robert Olsson <Robert.Olsson@xxxxxxxxxxx>
Date: Tue, 20 May 2003 12:36:43 +0200
Cc: "David S. Miller" <davem@xxxxxxxxxx>, netdev@xxxxxxxxxxx
In-reply-to: <20030520085724.GD978@udine>
References: <20030520082217.GC978@udine> <20030520.012824.85398613.davem@xxxxxxxxxx> <20030520085724.GD978@udine>
Sender: netdev-bounce@xxxxxxxxxxx
Eric Lemoine writes:
 > >    From: Eric Lemoine <Eric.Lemoine@xxxxxxx>
 > >    Date: Tue, 20 May 2003 10:22:17 +0200
 > >    
 > >    Any comments regarding the following patch?

 I think it will make any use of "raw" dev->hard_start_xmit" impossible. 
 Which is what pktgen uses.

 > > I understand why it is valid, etc., but why do we even want to do
 > > this?  It is not like this dead-loop detection stuff is a hot-path or
 > > anything like that.
 > 
 > I've implemented a prototype that uses per-CPU kernel threads for
 > processing packets coming in from a single interface. The idea is to
 > apply multiple CPUs to a single network interface to be able to have
 > multiple CPUs simultaneously pumping data into the network. So in my
 > case, I have lots of cpu_collisions and running the tx softirq to do
 > nothing may lower the performances. Anyway, even though my patch may
 > help me, it may indeed be irrelevant to the stock kernel.

 Sounds like a project at least having packet reordering and cache bouncing 
 in mind.
 

 Cheers.
                                                --ro
 

<Prev in Thread] Current Thread [Next in Thread>