DPDK usage discussions
 help / color / mirror / Atom feed
From: "Gavin Hu (Arm Technology China)" <Gavin.Hu@arm.com>
To: "Zhang, Qi Z" <qi.z.zhang@intel.com>,
	"Richardson, Bruce" <bruce.richardson@intel.com>,
	"Ananyev, Konstantin" <konstantin.ananyev@intel.com>
Cc: "users@dpdk.org" <users@dpdk.org>,
	Honnappa Nagarahalli <Honnappa.Nagarahalli@arm.com>,
	"Phil Yang (Arm Technology China)" <Phil.Yang@arm.com>,
	"Ruifeng Wang (Arm Technology China)" <Ruifeng.Wang@arm.com>
Subject: Re: [dpdk-users] i40e vPMD fix out of order Rx read issue
Date: Wed, 14 Aug 2019 03:19:34 +0000	[thread overview]
Message-ID: <VI1PR08MB53762725D991500FD0414CDC8FAD0@VI1PR08MB5376.eurprd08.prod.outlook.com> (raw)
In-Reply-To: <039ED4275CED7440929022BC67E7061153D7485E@SHSMSX105.ccr.corp.intel.com>

Hi Qi,

> -----Original Message-----
> From: Zhang, Qi Z <qi.z.zhang@intel.com>
> Sent: Saturday, July 27, 2019 9:33 AM
> To: Gavin Hu (Arm Technology China) <Gavin.Hu@arm.com>; Richardson,
> Bruce <bruce.richardson@intel.com>; Ananyev, Konstantin
> <konstantin.ananyev@intel.com>
> Cc: users@dpdk.org; Honnappa Nagarahalli
> <Honnappa.Nagarahalli@arm.com>; Phil Yang (Arm Technology China)
> <Phil.Yang@arm.com>; Ruifeng Wang (Arm Technology China)
> <Ruifeng.Wang@arm.com>
> Subject: RE: i40e vPMD fix out of order Rx read issue
>
>
>
> > -----Original Message-----
> > From: Gavin Hu (Arm Technology China) [mailto:Gavin.Hu@arm.com]
> > Sent: Friday, July 26, 2019 6:01 PM
> > To: Zhang, Qi Z <qi.z.zhang@intel.com>; Richardson, Bruce
> > <bruce.richardson@intel.com>; Ananyev, Konstantin
> > <konstantin.ananyev@intel.com>
> > Cc: users@dpdk.org; Honnappa Nagarahalli
> > <Honnappa.Nagarahalli@arm.com>; Phil Yang (Arm Technology China)
> > <Phil.Yang@arm.com>; Ruifeng Wang (Arm Technology China)
> > <Ruifeng.Wang@arm.com>
> > Subject: RE: i40e vPMD fix out of order Rx read issue
> >
> > Hi Qi,
> >
> > Thanks for your explanation!
> > I did some testing and found the barriers caused a big drop in RFC2544 NDR
> > performance on aarch64, how about it on X86?
> For x86, I think we don't have performance drop, the memory barrier just
> change the compiler's behavior to avoid generate out of order read
> instructions, and x86 guarantee no out of read execution, so it does not add
> new instructions that cost CPU cycles.
>
> > Is it possible to count DD bits in a way of surviving across the out-of-order
> > descriptors reading?
>
> I think it is possible, but this will impact performance on x86,
> but for aarch64, you can try out to see if that benefit and do proper
> optimization on related vPMD implementation.
The patches for aarch64 are in community review, and performances gain were measured
both for ixgbe and i40e vPMD. Very appreciate if you can further review and comment.
Here are the patch links:
IXGBE vPMD:
http://patches.dpdk.org/patch/57649/
http://patches.dpdk.org/patch/57650/
I40e vPMD:
http://patches.dpdk.org/cover/57651/
http://patches.dpdk.org/patch/57652/
> >
> > Best Regards,
> > Gavin
> >
> > > -----Original Message-----
> > > From: Zhang, Qi Z <qi.z.zhang@intel.com>
> > > Sent: Thursday, July 25, 2019 8:11 PM
> > > To: Gavin Hu (Arm Technology China) <Gavin.Hu@arm.com>; Richardson,
> > > Bruce <bruce.richardson@intel.com>; Ananyev, Konstantin
> > > <konstantin.ananyev@intel.com>
> > > Cc: users@dpdk.org; Honnappa Nagarahalli
> > > <Honnappa.Nagarahalli@arm.com>; Phil Yang (Arm Technology China)
> > > <Phil.Yang@arm.com>
> > > Subject: RE: i40e vPMD fix out of order Rx read issue
> > >
> > > Hi Gavin:
> > >
> > > in vPMD, we read 4 or 8 packets as batch, we count DD bits for packet
> > > received, but not check the if they are continues or not, we assume it
> > > should always be 1000, 1100, 1110, 1111 ....(take batch size is 4 as
> > > example) while the out of order read instruction generated by compiler
> > > will cause driver to get un-continues DD bits, like 1011, the
> > > descriptor on the hole actually is invalid since when it is read ,
> > > descriptor is not write back yet, but we still process this as 1110, it cause
> an
> > corrupted mbuf returned.
> > >
> > > hope this is helpful.
> > >
> > > Regards
> > > Qi
> > >
> > > > -----Original Message-----
> > > > From: Gavin Hu (Arm Technology China) [mailto:Gavin.Hu@arm.com]
> > > > Sent: Thursday, July 25, 2019 5:57 PM
> > > > To: Zhang, Qi Z <qi.z.zhang@intel.com>; Richardson, Bruce
> > > > <bruce.richardson@intel.com>; Ananyev, Konstantin
> > > > <konstantin.ananyev@intel.com>
> > > > Cc: users@dpdk.org; Honnappa Nagarahalli
> > > > <Honnappa.Nagarahalli@arm.com>; Phil Yang (Arm Technology China)
> > > > <Phil.Yang@arm.com>
> > > > Subject: i40e vPMD fix out of order Rx read issue
> > > >
> > > > Hi Qi,
> > > >
> > > > I am working on optimizing the i40e vPMD on aarch64 and I see this
> > > > patch relevant.
> > > > Could you illuminate what issue this patch was fixing?
> > > > I understand the PMD works at the driver layer, for delivery of L2
> packets.
> > > > It does not own the responsibility to keep order(the responsibility
> > > > lies with
> > > the
> > > > protocol stack, like TCP)?
> > > >
> > > > http://patches.dpdk.org/patch/16665/
> > > >
> > > > Best regards,
> > > > Gavin
> > > >
> > > > IMPORTANT NOTICE: The contents of this email and any attachments are
> > > > confidential and may also be privileged. If you are not the intended
> > > > recipient, please notify the sender immediately and do not disclose
> > > > the contents to any other person, use it for any purpose, or store
> > > > or copy the information in any medium. Thank you.
> > IMPORTANT NOTICE: The contents of this email and any attachments are
> > confidential and may also be privileged. If you are not the intended
> recipient,
> > please notify the sender immediately and do not disclose the contents to
> any
> > other person, use it for any purpose, or store or copy the information in any
> > medium. Thank you.
IMPORTANT NOTICE: The contents of this email and any attachments are confidential and may also be privileged. If you are not the intended recipient, please notify the sender immediately and do not disclose the contents to any other person, use it for any purpose, or store or copy the information in any medium. Thank you.

  reply	other threads:[~2019-08-14  3:19 UTC|newest]

Thread overview: 11+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2019-07-25  9:56 Gavin Hu (Arm Technology China)
2019-07-25 12:11 ` Zhang, Qi Z
2019-07-26 10:00   ` Gavin Hu (Arm Technology China)
2019-07-27  1:32     ` Zhang, Qi Z
2019-08-14  3:19       ` Gavin Hu (Arm Technology China) [this message]
2019-08-14  3:38         ` Zhang, Qi Z
2019-08-16  2:50           ` Ye Xiaolong
2019-08-16  3:44             ` Zhang, Qi Z
2019-08-16  6:24               ` Gavin Hu (Arm Technology China)
2019-08-26  6:11             ` Honnappa Nagarahalli
2019-08-26 15:55               ` Honnappa Nagarahalli

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=VI1PR08MB53762725D991500FD0414CDC8FAD0@VI1PR08MB5376.eurprd08.prod.outlook.com \
    --to=gavin.hu@arm.com \
    --cc=Honnappa.Nagarahalli@arm.com \
    --cc=Phil.Yang@arm.com \
    --cc=Ruifeng.Wang@arm.com \
    --cc=bruce.richardson@intel.com \
    --cc=konstantin.ananyev@intel.com \
    --cc=qi.z.zhang@intel.com \
    --cc=users@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).