> -----Original Message-----
> From: Thomas Monjalon <[email protected]>
> Sent: Monday, July 31, 2023 7:31 AM
> To: Wathsala Wathawana Vithanage <[email protected]>
> Cc: Honnappa Nagarahalli <[email protected]>;
> [email protected]; Ruifeng Wang <[email protected]>;
> [email protected]; nd <[email protected]>; [email protected];
> [email protected]; Morten Brørup <[email protected]>; Tyler
> Retzlaff <[email protected]>
> Subject: Re: [RFC] ring: further performance improvements with C11
>
> 15/06/2023 22:13, Wathsala Vithanage:
> > For improved performance over the current C11 based ring
> > implementation following changes were made.
> > (1) Replace tail store with RELEASE semantics in
> > __rte_ring_update_tail with a RELEASE fence. Replace load of the tail
> > with ACQUIRE semantics in __rte_ring_move_prod_head and
> > __rte_ring_move_cons_head with ACQUIRE fences.
> > (2) Remove ACQUIRE fences between load of the old_head and load of the
> > cons_tail in __rte_ring_move_prod_head and __rte_ring_move_cons_head.
> > These two fences are not required for the safety of the ring library.
> >
> > Signed-off-by: Wathsala Vithanage <[email protected]>
> > Reviewed-by: Honnappa Nagarahalli <[email protected]>
> > Reviewed-by: Ruifeng Wang <[email protected]>
>
> Are we waiting for more reviews?
We do not have a good solution. We should discuss this in the Techboard meeting.
>
