From: Joseph Gasparakis <joseph.gasparakis@intel.com>
To: Or Gerlitz <or.gerlitz@gmail.com>
Cc: Joseph Gasparakis <joseph.gasparakis@intel.com>,
"Kirsher, Jeffrey T" <jeffrey.t.kirsher@intel.com>,
David Miller <davem@davemloft.net>,
"netdev@vger.kernel.org" <netdev@vger.kernel.org>,
"gospo@redhat.com" <gospo@redhat.com>,
"sassmann@redhat.com" <sassmann@redhat.com>,
"Brandeburg, Jesse" <jesse.brandeburg@intel.com>
Subject: Re: [net-next v4 07/16] i40e: Rx checksum offload for VXLAN
Date: Fri, 3 Jan 2014 11:07:39 -0800 (PST) [thread overview]
Message-ID: <alpine.LFD.2.03.1401031106520.6092@intel.com> (raw)
In-Reply-To: <CAJZOPZK_b=Ve1YXsHfZjyeOQiMxj_jAKscmgZT_AKCEK1+V1UA@mail.gmail.com>
On Thu, 2 Jan 2014, Or Gerlitz wrote:
> On Thu, Jan 2, 2014 at 10:57 PM, Joseph Gasparakis
> <joseph.gasparakis@intel.com> wrote:
>
> >
> > > The comment says that the driver has to compute the udp checksum
> > > **if** the packet has an outer udp checksum,
> > > but the driver call to issue checksumming is done unconditionally
> >
> >
> > This code runs when the right bits are set in the rx hardware descriptor
> > field (rx_ptype and rx_status), which implies the hardware has detected
> > ipv4 tunneling and there is non zero UDP csum.
>
>
> Got it -- thanks for explaining, also for the pointer on where
> skb->encapsulation is set
>
Any time Or. And thanks for your reviews.
next prev parent reply other threads:[~2014-01-03 18:49 UTC|newest]
Thread overview: 24+ messages / expand[flat|nested] mbox.gz Atom feed top
2013-12-28 3:22 [net-next v4 00/16][pull request] Intel Wired LAN Driver Updates Jeff Kirsher
2013-12-28 3:22 ` [net-next v4 01/16] i40e: using for_each_set_bit to simplify the code Jeff Kirsher
2013-12-28 3:22 ` [net-next v4 02/16] i40e: Suppress HMC error to Interrupt message level Jeff Kirsher
2013-12-28 6:11 ` David Miller
2013-12-28 3:22 ` [net-next v4 03/16] i40e: Populate and check pci bus speed and width Jeff Kirsher
2013-12-28 6:10 ` David Miller
2013-12-28 3:22 ` [net-next v4 04/16] i40e: add wake-on-lan support Jeff Kirsher
2013-12-28 3:22 ` [net-next v4 05/16] i40e: fix curly brace use and return type Jeff Kirsher
2013-12-28 3:22 ` [net-next v4 06/16] i40e: Implementation of VXLAN ndo's Jeff Kirsher
2013-12-28 6:17 ` David Miller
2013-12-28 3:22 ` [net-next v4 07/16] i40e: Rx checksum offload for VXLAN Jeff Kirsher
2014-01-02 10:20 ` Or Gerlitz
2014-01-02 20:57 ` Joseph Gasparakis
2014-01-02 21:31 ` Or Gerlitz
2014-01-03 19:07 ` Joseph Gasparakis [this message]
2013-12-28 3:22 ` [net-next v4 08/16] i40e: move i40e_reset_vf Jeff Kirsher
2013-12-28 3:22 ` [net-next v4 09/16] i40e: refactor VF reset flow Jeff Kirsher
2013-12-28 3:22 ` [net-next v4 10/16] i40e: remove redundant code Jeff Kirsher
2013-12-28 3:22 ` [net-next v4 11/16] i40e: remove chatty log messages Jeff Kirsher
2013-12-28 3:22 ` [net-next v4 12/16] i40e: fix error return Jeff Kirsher
2013-12-28 3:22 ` [net-next v4 13/16] i40e: be more informative Jeff Kirsher
2013-12-28 3:22 ` [net-next v4 14/16] i40e: make a define from a large constant Jeff Kirsher
2013-12-28 3:22 ` [net-next v4 15/16] i40e: update led set args Jeff Kirsher
2013-12-28 3:22 ` [net-next v4 16/16] i40e: report VF MAC addresses correctly Jeff Kirsher
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=alpine.LFD.2.03.1401031106520.6092@intel.com \
--to=joseph.gasparakis@intel.com \
--cc=davem@davemloft.net \
--cc=gospo@redhat.com \
--cc=jeffrey.t.kirsher@intel.com \
--cc=jesse.brandeburg@intel.com \
--cc=netdev@vger.kernel.org \
--cc=or.gerlitz@gmail.com \
--cc=sassmann@redhat.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 an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.