DPDK patches and discussions
 help / color / mirror / Atom feed
* [dpdk-dev] [PATCH v2 0/3] net: fix out of order Rx read issue
@ 2016-10-17 18:29 Qi Zhang
  2016-10-17 18:29 ` [dpdk-dev] [PATCH v2 1/3] net/i40e: " Qi Zhang
                   ` (3 more replies)
  0 siblings, 4 replies; 6+ messages in thread
From: Qi Zhang @ 2016-10-17 18:29 UTC (permalink / raw)
  To: jingjing.wu, helin.zhang; +Cc: dev, Qi Zhang

In vPMD, when load Rx desc with _mm_loadu_si128, 
volatile point will be cast into non-volatile point.
So GCC is allowed to reorder the load instructions, 
while Rx read's correctness is reply on these load 
instructions to follow a backward sequence strictly, 
so we add compile barrier to prevent compiler reorder.
We already met this issue on i40e with GCC6 and we 
fixed this on ixgbe and fm10k also.

v2: 
- fix check-git-log.sh warning.
- add more detail commit message.

Qi Zhang (3):
  net/i40e: fix out of order Rx read issue
  net/ixgbe: fix out of order Rx read issue
  net/fm10k: fix out of ofder Rx read issue

 drivers/net/fm10k/fm10k_rxtx_vec.c     | 3 +++
 drivers/net/i40e/i40e_rxtx_vec.c       | 3 +++
 drivers/net/ixgbe/ixgbe_rxtx_vec_sse.c | 3 +++
 3 files changed, 9 insertions(+)

-- 
2.7.4

^ permalink raw reply	[flat|nested] 6+ messages in thread

end of thread, other threads:[~2016-10-19  9:52 UTC | newest]

Thread overview: 6+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2016-10-17 18:29 [dpdk-dev] [PATCH v2 0/3] net: fix out of order Rx read issue Qi Zhang
2016-10-17 18:29 ` [dpdk-dev] [PATCH v2 1/3] net/i40e: " Qi Zhang
2016-10-17 18:29 ` [dpdk-dev] [PATCH v2 2/3] net/ixgbe: " Qi Zhang
2016-10-17 18:29 ` [dpdk-dev] [PATCH v2 3/3] net/fm10k: fix out of ofder " Qi Zhang
2016-10-18 11:33 ` [dpdk-dev] [PATCH v2 0/3] net: fix out of order " Ananyev, Konstantin
2016-10-19  9:52   ` Bruce Richardson

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).