Re: [dpdk-dev] [PATCH v3 2/3] ring: synchronize the load and store of the tail

2018-09-29 Thread Jerin Jacob
-Original Message- > Date: Mon, 17 Sep 2018 16:17:23 +0800 > From: Gavin Hu > To: dev@dpdk.org > CC: gavin...@arm.com, honnappa.nagaraha...@arm.com, steve.cap...@arm.com, > ola.liljed...@arm.com, jerin.ja...@caviumnetworks.com, n...@arm.com, > sta...@dpdk.org > Subject: [PATCH v3 2/3] ri

Re: [dpdk-dev] [PATCH v3 2/3] ring: synchronize the load and store of the tail

2018-09-27 Thread Justin He
Hi Gavin > -Original Message- > From: Gavin Hu (Arm Technology China) > Sent: 2018年9月26日 17:30 > To: Gavin Hu (Arm Technology China) ; dev@dpdk.org > Cc: Honnappa Nagarahalli ; Steve Capper > ; Ola Liljedahl ; > jerin.ja...@caviumnetworks.com; nd ; sta...@dpdk.org; Justin > He > Subject:

Re: [dpdk-dev] [PATCH v3 2/3] ring: synchronize the load and store of the tail

2018-09-26 Thread Gavin Hu (Arm Technology China)
+Justin He for review. > -Original Message- > From: Gavin Hu > Sent: Monday, September 17, 2018 4:17 PM > To: dev@dpdk.org > Cc: Gavin Hu (Arm Technology China) ; Honnappa > Nagarahalli ; Steve Capper > ; Ola Liljedahl ; > jerin.ja...@caviumnetworks.com; nd ; sta...@dpdk.org > Subject: [P

[dpdk-dev] [PATCH v3 2/3] ring: synchronize the load and store of the tail

2018-09-17 Thread Gavin Hu
Synchronize the load-acquire of the tail and the store-release within update_tail, the store release ensures all the ring operations, enqueue or dequeue, are seen by the observers on the other side as soon as they see the updated tail. The load-acquire is needed here as the data dependency is not a