[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-ID: <18167.56026.110424.687062@robur.slu.se>
Date: Mon, 24 Sep 2007 17:42:18 +0200
From: Robert Olsson <Robert.Olsson@...a.slu.se>
To: Steve Wise <swise@...ngridcomputing.com>
Cc: Ben Greear <greearb@...delatech.com>, hadi@...erus.ca,
Evgeniy Polyakov <johnpol@....mipt.ru>, netdev@...r.kernel.org,
Robert Olsson <Robert.Olsson@...a.slu.se>
Subject: Re: pktgen question
Hi,
Steve Wise writes:
> I think pktgen should be cloning the skbs using skb_clone(). Then it
> will work for all devices, eh?
pktgen assumes for "fastpath" sending exclusive ownership of
the skb. And does a skb_get to avoid final skb destruction so
the same skb can be sent over and over. The idea is to avoid
memory allocation and keep things in cache to give very high
packet rates with identical packets.
I
But if you need to alter the packet then the skb_get trick can't
be done. And you have to turn off "fastpath" with clone_skb
> Perf-wise, you could clone the skbs up front, then deliver them to the
> nic in a tight loop. This would mitigate the added overhead introduced
> by calling skb_clone() in the loop doing transmits...
Sure it's can be done. It could replay sequences etc but it will not
beat the skb_get trick in sending identical packets. It has been
proposed before but I've avoided such efforts to keep things relatively
small and simple. Really pktgen should be reworked to have s small
skim in kernel and move the rest of the stuff to userland.
Cheers.
--ro
-
To unsubscribe from this list: send the line "unsubscribe netdev" in
the body of a message to majordomo@...r.kernel.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
Powered by blists - more mailing lists