patches for DPDK stable branches
 help / color / mirror / Atom feed
From: Christian Ehrhardt <christian.ehrhardt@canonical.com>
To: Feifei Wang <feifei.wang2@arm.com>
Cc: ruifeng.wang@arm.com, dpdk stable <stable@dpdk.org>, nd@arm.com
Subject: Re: [dpdk-stable] [PATCH 19.11 0/4] Enable Checksum Offloading for NEON vector
Date: Mon, 22 Feb 2021 14:21:35 +0100	[thread overview]
Message-ID: <CAATJJ0LtD17aEPHLCiXhnTyZH_pYJ2PEDLXg3QC3DJXwwCj_fw@mail.gmail.com> (raw)
In-Reply-To: <20210219062041.2015421-1-feifei.wang2@arm.com>

On Fri, Feb 19, 2021 at 7:20 AM Feifei Wang <feifei.wang2@arm.com> wrote:
>
> This patch series are mainly to enable checksum offloading for IXGBE NEON
> vector PMD, including good and bad checksum flags. In the meanwhile, the
> first patch enable VLAN stripping flag for Arm.
> Following are the test results for the patches:
>
> NICs: 82599(igb)
> Driver: ixgbe(vector)
> $:./app/dpdk-testpmd -c 0x3 -w 0002:f9:00.0 -- -i --port-topology=chained
> test-pmd> set fwd rxonly
> test-pmd> set verbose 1
> test-pmd> start
>
> With Patch a:
> enable vlan stripping:
> src=00:00:00:00:00:02 - dst=00:00:00:00:00:01 - type=0x0800 - length=70 - nb_segs=1 - VLAN tci=0x1
> ol_flags: PKT_RX_VLAN PKT_RX_VLAN_STRIPPED
>
> With Patch b:
> Packet: IPv4_checksum = 0xee && UDP_checksum = 0xee
> src=00:00:00:00:00:02 - dst=00:00:00:00:00:01 - type=0x0800 - length=70 - nb_segs=1
> ol_flags: PKT_RX_L4_CKSUM_BAD PKT_RX_IP_CKSUM_BAD
>
> With Patch c:
> Packet: IPv4_checksum = correct value && UDP_checksum = correct value
> ol_flags: PKT_RX_L4_CKSUM_GOOD PKT_RX_IP_CKSUM_GOOD
>
> Feifei Wang (4):
>   a). net/ixgbe: add new flag of stripped VLAN for NEON vector
>   b). net/ixgbe: support bad checksum flag for NEON vector
>   c). net/ixgbe: support good checksum flag for NEON vector
>   d). net/ixgbe: enable IXGBE NEON vector PMD when CHECKSUM need to checksum

Applied, thank you!

>  drivers/net/ixgbe/ixgbe_rxtx_vec_neon.c | 94 ++++++++++++++++++-------
>  1 file changed, 68 insertions(+), 26 deletions(-)
>
> --
> 2.25.1
>


-- 
Christian Ehrhardt
Staff Engineer, Ubuntu Server
Canonical Ltd

      parent reply	other threads:[~2021-02-22 13:22 UTC|newest]

Thread overview: 6+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2021-02-19  6:20 Feifei Wang
2021-02-19  6:20 ` [dpdk-stable] [PATCH 19.11 1/4] net/ixgbe: add new flag of stripped VLAN for NEON Feifei Wang
2021-02-19  6:20 ` [dpdk-stable] [PATCH 19.11 2/4] net/ixgbe: support bad checksum flag " Feifei Wang
2021-02-19  6:20 ` [dpdk-stable] [PATCH 19.11 3/4] net/ixgbe: support good " Feifei Wang
2021-02-19  6:20 ` [dpdk-stable] [PATCH 19.11 4/4] net/ixgbe: enable IXGBE NEON vector PMD when CHECKSUM need to checksum Feifei Wang
2021-02-22 13:21 ` Christian Ehrhardt [this message]

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=CAATJJ0LtD17aEPHLCiXhnTyZH_pYJ2PEDLXg3QC3DJXwwCj_fw@mail.gmail.com \
    --to=christian.ehrhardt@canonical.com \
    --cc=feifei.wang2@arm.com \
    --cc=nd@arm.com \
    --cc=ruifeng.wang@arm.com \
    --cc=stable@dpdk.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).