Hi Stephen, There is no api changes, but this is a significant change as ring is fundamental and widely used, it decreases latency by 25% in our tests, it may do even better for cases with more contending producers/consumers or deeper depth of rings.
Best regards Gavin Best regards Gavin ________________________________ 收件人: Stephen Hemminger <step...@networkplumber.org> 发送时间: 星期五, 十一月 2, 2018 1:26 上午 收件人: Gavin Hu (Arm Technology China) 抄送: dev@dpdk.org; tho...@monjalon.net; olivier.m...@6wind.com; chao...@linux.vnet.ibm.com; bruce.richard...@intel.com; konstantin.anan...@intel.com; jerin.ja...@caviumnetworks.com; Honnappa Nagarahalli; sta...@dpdk.org 主题: Re: [PATCH v4 2/2] ring: move the atomic load of head above the loop On Thu, 1 Nov 2018 17:53:51 +0800 Gavin Hu <gavin...@arm.com> wrote: > +* **Updated the ring library with C11 memory model.** > + > + Updated the ring library with C11 memory model including the following > changes: > + > + * Synchronize the load and store of the tail > + * Move the atomic load of head above the loop > + Does this really need to be in the release notes? Is it a user visible change or just an internal/optimization and fix. IMPORTANT NOTICE: The contents of this email and any attachments are confidential and may also be privileged. If you are not the intended recipient, please notify the sender immediately and do not disclose the contents to any other person, use it for any purpose, or store or copy the information in any medium. Thank you.
