DPDK patches and discussions
 help / color / mirror / Atom feed
From: Stephen Hemminger <stephen@networkplumber.org>
To: jigsaw <jigsaw@gmail.com>
Cc: "dev@dpdk.org" <dev@dpdk.org>
Subject: Re: [dpdk-dev] [PATCH] Request for comments on ixgbe TSO support
Date: Fri, 4 Oct 2013 11:23:02 -0700	[thread overview]
Message-ID: <20131004112302.50693f04@nehalam.linuxnetplumber.net> (raw)
In-Reply-To: <CAHVfvh7EtGcUB1NeH8H3VyHBJuSYA=AiyHuLAgs7L7NNJaUgsw@mail.gmail.com>

On Fri, 4 Oct 2013 20:54:31 +0300
jigsaw <jigsaw@gmail.com> wrote:

> Hi Stephen,
> 
> 
> >>This will work for local generated packets but overlapping existing field won't work well for forwarding.
> So adding a new mss field in mbuf could be the way out? or I
> misunderstand something.
> 
> >> What we want to be able to do is to take offload (jumbo) packets in with from virtio
> Sorry I don't understand why TSO is connected to virtio. Could you
> give more details here?
> Are you suggesting this TSO patch overlaps your work, or it should be
> based on your work?

I am working on a better virtio driver. Already have lots more features working,
and doing better offload support is planned.

TSO is a subset of the more generic segment offload (GSO) on Linux.
With virtio is possible to receive GSO packets as well as send them.
This feature is negotiated between guest and host.

The idea is that between guests they can exchange jumbo (64K) packets even with
a smaller MTU. This helps in many ways. One example is only a single
route lookup is needed.

Another issue is that the current DPDK model of offload flags for checksum is problematic.
It matches what is available in Intel hardware and is not easily generalizable to other
devices.

Current DPDK flag is checksum bad. I would like to change it to checksum known
good. Then drivers which dont' do checksum would leave it 0, but if receive
checksum is known good set it to 1.  Basically 1 means known good, and
0 means unknown (or bad).  Higher level software can then do sw checksum
if necessary.

  reply	other threads:[~2013-10-04 18:22 UTC|newest]

Thread overview: 10+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2013-10-04 17:06 Qinglai Xiao
2013-10-04 17:06 ` [dpdk-dev] [PATCH] ixgbe: TCP/UDP segment offload support on 82599 Qinglai Xiao
2013-10-04 17:40 ` [dpdk-dev] [PATCH] Request for comments on ixgbe TSO support Stephen Hemminger
2013-10-04 17:54   ` jigsaw
2013-10-04 18:23     ` Stephen Hemminger [this message]
2013-10-04 18:38       ` Venkatesan, Venky
2013-10-04 19:10         ` jigsaw
2013-10-04 21:19           ` Stephen Hemminger
2013-10-08  6:59           ` Zhu, Heqing
2013-10-08  7:56             ` jigsaw

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=20131004112302.50693f04@nehalam.linuxnetplumber.net \
    --to=stephen@networkplumber.org \
    --cc=dev@dpdk.org \
    --cc=jigsaw@gmail.com \
    /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).