Re: [dpdk-dev] [PATCH v3] lib/timer: relax barrier for status update

2020-04-26 Thread Carrillo, Erik G
...@redhat.com; > > Honnappa Nagarahalli ; Gavin Hu > > ; nd ; nd > > Subject: RE: [dpdk-dev] [PATCH v3] lib/timer: relax barrier for status > > update > > > > > > > > > -Original Message- > > > From: Phil Yang > > > Sent:

Re: [dpdk-dev] [PATCH v3] lib/timer: relax barrier for status update

2020-04-26 Thread Phil Yang
; david.march...@redhat.com; Honnappa Nagarahalli > > ; Gavin Hu ; nd > > ; nd > > Subject: RE: [dpdk-dev] [PATCH v3] lib/timer: relax barrier for status > update > > > > > -Original Message- > > > From: Thomas Monjalon > > > Sent:

Re: [dpdk-dev] [PATCH v3] lib/timer: relax barrier for status update

2020-04-26 Thread Carrillo, Erik G
avid.march...@redhat.com; Honnappa Nagarahalli > > ; Gavin Hu ; nd > > > > Subject: Re: [dpdk-dev] [PATCH v3] lib/timer: relax barrier for status > > update > > > > 24/04/2020 09:24, Phil Yang: > > > Volatile has no ordering semantics. The rte_timer stru

Re: [dpdk-dev] [PATCH v3] lib/timer: relax barrier for status update

2020-04-26 Thread Phil Yang
> -Original Message- > From: Thomas Monjalon > Sent: Sunday, April 26, 2020 1:18 AM > To: Phil Yang > Cc: erik.g.carri...@intel.com; rsanf...@akamai.com; dev@dpdk.org; > david.march...@redhat.com; Honnappa Nagarahalli > ; Gavin Hu ; nd > > Subject: Re: [dpd

Re: [dpdk-dev] [PATCH v3] lib/timer: relax barrier for status update

2020-04-25 Thread Thomas Monjalon
24/04/2020 09:24, Phil Yang: > Volatile has no ordering semantics. The rte_timer structure defines > timer status as a volatile variable and uses the rte_r/wmb barrier > to guarantee inter-thread visibility. > > This patch optimized the volatile operation with c11 atomic operations > and one-way b

[dpdk-dev] [PATCH v3] lib/timer: relax barrier for status update

2020-04-24 Thread Phil Yang
Volatile has no ordering semantics. The rte_timer structure defines timer status as a volatile variable and uses the rte_r/wmb barrier to guarantee inter-thread visibility. This patch optimized the volatile operation with c11 atomic operations and one-way barrier to save the performance penalty. A