netdev.vger.kernel.org archive mirror
 help / color / mirror / Atom feed
From: David Miller <davem@davemloft.net>
To: herbert@gondor.apana.org.au
Cc: bhutchings@solarflare.com, buytenh@wantstofly.org,
	netdev@vger.kernel.org, akarkare@marvell.com, nico@cam.org,
	dale@farnsworth.org
Subject: Re: using software TSO on non-TSO capable netdevices
Date: Sun, 03 Aug 2008 01:19:45 -0700 (PDT)	[thread overview]
Message-ID: <20080803.011945.249059359.davem@davemloft.net> (raw)
In-Reply-To: <20080731132735.GA9085@gondor.apana.org.au>

From: Herbert Xu <herbert@gondor.apana.org.au>
Date: Thu, 31 Jul 2008 21:27:35 +0800

> On Thu, Jul 31, 2008 at 02:19:30PM +0100, Ben Hutchings wrote:
> >
> > You might want to think about providing a way for soft-GSO to generate
> > more lightweight structures than skbs.  The overhead for skb allocation
> > becomes quite significant beyond 1 Gbit/s, which is why we added the soft-
> > TSO implementation in sfc using per-interface pools of header buffers.  I
> > would guess niu would benefit from this sort of approach, though it looks
> > like all the other 10G NICs do TSO in hardware/firmware.
> 
> We could always provide a library that makes it easier for the
> drivers to do TSO in software without allocating skb's.

I took a brief look into this, and yes NIU would benefit a lot from
what the sfc driver is doing and using sw GSO in general.

I think that, in order to work out, the driver has to provide a pool
of DMA buffers to use in some generic fashion.

It seems likely that it's best to give the driver the largest amount
of flexibility wrt. the DMA bits.  There are two reasonable ways for
them to implement a header buffer pool:

1) A big coherent DMA block that gets chopped up into fixed size pieces.

2) A free list of kmalloc() buffers that get DMA mapped dynamically
   (because such dynamic DMA mappings transfer faster than coherent ones
   on some systems).

But anyways, we don't want to be in the business of enforcing one way
or the other in whatever interface we come up with.

So likely what we'll do is have the driver say it can do hw TSO and
then at ->hard_start_xmit() time it calls into the sw GSO engine,
passing header buffers in along the way.

I would start hacking on this beast but I haven't yet come up with
a clean way to share a lot of code with the existing sw GSO engine.
That's the key to implementing this properly.

  reply	other threads:[~2008-08-03  8:19 UTC|newest]

Thread overview: 35+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2008-07-30 23:50 using software TSO on non-TSO capable netdevices Lennert Buytenhek
2008-07-30 23:56 ` David Miller
2008-07-31  0:41   ` Lennert Buytenhek
2008-07-31  1:10     ` David Miller
2008-07-31  1:45       ` Lennert Buytenhek
2008-07-31  3:54         ` Herbert Xu
2008-07-31  9:45           ` Lennert Buytenhek
2008-07-31 10:55             ` Herbert Xu
2008-07-31 12:37               ` Lennert Buytenhek
2008-07-31 12:59                 ` Herbert Xu
2008-08-03  8:23                   ` David Miller
2008-07-31  7:34       ` Ilpo Järvinen
2008-07-31  9:50         ` Lennert Buytenhek
2008-07-31 10:27           ` Ilpo Järvinen
2008-07-31  2:29     ` Herbert Xu
2008-07-31  2:36       ` Lennert Buytenhek
2008-07-31  3:03         ` Herbert Xu
2008-07-31  6:55           ` Ilpo Järvinen
2008-07-31  9:39             ` Lennert Buytenhek
2008-07-31 10:14           ` Lennert Buytenhek
2008-07-31 10:16             ` David Miller
2008-07-31 12:25               ` Lennert Buytenhek
2008-07-31 12:35                 ` David Miller
2008-07-31 13:19                   ` Ben Hutchings
2008-07-31 13:27                     ` Herbert Xu
2008-08-03  8:19                       ` David Miller [this message]
2008-08-03  8:55                         ` Herbert Xu
2008-08-07  6:07                           ` David Miller
2008-08-07  6:15                             ` Herbert Xu
2008-09-12  4:08                               ` David Miller
2008-08-07 11:50                             ` Lennert Buytenhek
2008-08-07 20:32                             ` Rick Jones
2008-08-07 22:44                               ` David Miller
2008-07-31 17:00 ` Rick Jones
2008-07-31 17:45   ` Lennert Buytenhek

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=20080803.011945.249059359.davem@davemloft.net \
    --to=davem@davemloft.net \
    --cc=akarkare@marvell.com \
    --cc=bhutchings@solarflare.com \
    --cc=buytenh@wantstofly.org \
    --cc=dale@farnsworth.org \
    --cc=herbert@gondor.apana.org.au \
    --cc=netdev@vger.kernel.org \
    --cc=nico@cam.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).