From: Hiroshi Shimamoto <h-shimamoto@ct.jp.nec.com>
To: Thomas Monjalon <thomas.monjalon@6wind.com>,
"dev@dpdk.org" <dev@dpdk.org>
Cc: Hayato Momma <h-momma@ce.jp.nec.com>
Subject: Re: [dpdk-dev] [memnic PATCH 4/7] pmd: use compiler barrier
Date: Thu, 25 Sep 2014 00:35:51 +0000 [thread overview]
Message-ID: <7F861DC0615E0C47A872E6F3C5FCDDBD02AD9F40@BPXM14GP.gisp.nec.co.jp> (raw)
In-Reply-To: <1855491.X6obi8qxOV@xps13>
> Subject: Re: [dpdk-dev] [memnic PATCH 4/7] pmd: use compiler barrier
>
> 2014-09-11 07:48, Hiroshi Shimamoto:
> > x86 can keep store ordering with standard operations.
>
> Are we sure it's always the case (including old 32-bit CPU)?
> I would prefer to have a reference here. I know we already discussed
> this kind of things but having a reference in commit log could help
> for future discussions.
>
> > Using memory barrier is much expensive in main packet processing loop.
> > Removing this improves xmit/recv packet performance.
> >
> > We can see performance improvements with memnic-tester.
> > Using Xeon E5-2697 v2 @ 2.70GHz, 4 vCPU.
> > size | before | after
> > 64 | 4.18Mpps | 4.59Mpps
> > 128 | 3.85Mpps | 4.87Mpps
> > 256 | 4.01Mpps | 4.72Mpps
> > 512 | 3.52Mpps | 4.41Mpps
> > 1024 | 3.18Mpps | 3.64Mpps
> > 1280 | 2.86Mpps | 3.15Mpps
> > 1518 | 2.59Mpps | 2.87Mpps
> >
> > Note: we have to take care if we use temporal cache.
>
> Please, could you explain this last sentence?
Oops, I have mistaken the word, "temporal" should be "non-temporal".
By the way, there are some instructions which use non-temporal
cache liek MOVNTx series.
The store ordering of these instructions is not kept.
Ref. Intel Software Developer Manual
Vol.1 10.4.6.2 Caching of Temporal vs. Non-Temporal Data
Vol.3 8.2 Memory Ordering
thanks,
Hiroshi
>
> Thanks
> --
> Thomas
prev parent reply other threads:[~2014-09-25 0:29 UTC|newest]
Thread overview: 3+ messages / expand[flat|nested] mbox.gz Atom feed top
2014-09-11 7:48 Hiroshi Shimamoto
2014-09-24 15:18 ` Thomas Monjalon
2014-09-25 0:35 ` Hiroshi Shimamoto [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=7F861DC0615E0C47A872E6F3C5FCDDBD02AD9F40@BPXM14GP.gisp.nec.co.jp \
--to=h-shimamoto@ct.jp.nec.com \
--cc=dev@dpdk.org \
--cc=h-momma@ce.jp.nec.com \
--cc=thomas.monjalon@6wind.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).