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

2018-11-01 Thread Gavin Hu (Arm Technology China)
tin.anan...@intel.com; jerin.ja...@caviumnetworks.com; > Honnappa Nagarahalli ; sta...@dpdk.org > Subject: Re: [dpdk-dev] [PATCH v3 1/2] ring: synchronize the load and store > of the tail > > On Wed, 31 Oct 2018 18:26:26 +0800 > Gavin Hu wrote: > > > -/* > > - * The s

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

2018-10-31 Thread Stephen Hemminger
On Wed, 31 Oct 2018 18:26:26 +0800 Gavin Hu wrote: > - /* > - * The subtraction is done between two unsigned 32bits value > + /* load-acquire synchronize with store-release of ht->tail > + * in update_tail. > + */ > + con

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

2018-10-31 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