mbox

[v5,0/2] ring library with c11 memory model bug fix and optimization

Message ID 1541157688-40012-1-git-send-email-gavin.hu@arm.com (mailing list archive)
Headers

Message

Gavin Hu Nov. 2, 2018, 11:21 a.m. UTC
  v4->v5
1) Indicate the improvement by the change in the release note.

v3->v4
1) Move the variable declarations to the beginning of the block.

v2->v3
1) Reword the change and relocate it in the release note.

v1->v2
1) Add the changes to the 18.11 release note.

V1:
Updated the ring library with C11 memory model including the following changes
1) Synchronize the load and store of the tail to ensure the enqueue/dequeue
   operations are really completed before seen by the observers on the other
   sides.
2) Move the atomic load of head above the loop for the first iteration,it is
   not unnecessary and degrade performance for the other iteration as the head
   was loaded in the failure case of CAS.

Gavin Hu (2):
  ring: synchronize the load and store of the tail
  ring: move the atomic load of head above the loop

 doc/guides/rel_notes/release_18_11.rst |  7 +++++++
 lib/librte_ring/rte_ring_c11_mem.h     | 24 ++++++++++++++++++------
 2 files changed, 25 insertions(+), 6 deletions(-)