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

2020-04-09 Thread Phil Yang
> -Original Message- > From: Carrillo, Erik G > Sent: Friday, April 10, 2020 3:29 AM > To: Honnappa Nagarahalli ; Phil Yang > ; rsanf...@akamai.com; dev@dpdk.org > Cc: david.march...@redhat.com; Burakov, Anatoly > ; tho...@monjalon.net; jer...@marvell.com; > hemant.agra...@nxp.com; Gavin H

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

2020-04-09 Thread Carrillo, Erik G
> -Original Message- > From: Honnappa Nagarahalli > Sent: Wednesday, April 8, 2020 4:56 PM > To: Carrillo, Erik G ; Phil Yang > ; rsanf...@akamai.com; dev@dpdk.org > Cc: david.march...@redhat.com; Burakov, Anatoly > ; tho...@monjalon.net; jer...@marvell.com; > hemant.agra...@nxp.com; Gavin

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

2020-04-08 Thread Honnappa Nagarahalli
> > > > > > Subject: [PATCH 2/2] lib/timer: relax barrier for status update > > > > > > > > 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. > > > > > > > >

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

2020-04-08 Thread Carrillo, Erik G
> -Original Message- > From: Honnappa Nagarahalli > Sent: Wednesday, April 8, 2020 4:16 PM > To: Carrillo, Erik G ; Phil Yang > ; rsanf...@akamai.com; dev@dpdk.org > Cc: david.march...@redhat.com; Burakov, Anatoly > ; tho...@monjalon.net; jer...@marvell.com; > hemant.agra...@nxp.com; Gavin

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

2020-04-08 Thread Honnappa Nagarahalli
> > Subject: [PATCH 2/2] lib/timer: relax barrier for status update > > > > 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 volatil

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

2020-04-08 Thread Carrillo, Erik G
> -Original Message- > From: Phil Yang > Sent: Monday, February 24, 2020 12:42 AM > To: rsanf...@akamai.com; Carrillo, Erik G ; > dev@dpdk.org > Cc: david.march...@redhat.com; Burakov, Anatoly > ; tho...@monjalon.net; jer...@marvell.com; > hemant.agra...@nxp.com; honnappa.nagaraha...@arm.c

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

2020-04-08 Thread Phil Yang
> -Original Message- > From: Phil Yang > Sent: Monday, February 24, 2020 2:42 PM > To: rsanf...@akamai.com; erik.g.carri...@intel.com; dev@dpdk.org > Cc: david.march...@redhat.com; anatoly.bura...@intel.com; > tho...@monjalon.net; jer...@marvell.com; hemant.agra...@nxp.com; > Honnappa Naga

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

2020-02-23 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