> -----Original Message----- > From: Phil Yang <[email protected]> > Sent: Monday, February 24, 2020 2:42 PM > To: [email protected]; [email protected]; [email protected] > Cc: [email protected]; [email protected]; > [email protected]; [email protected]; [email protected]; > Honnappa Nagarahalli <[email protected]>; Gavin Hu > <[email protected]>; Phil Yang <[email protected]>; nd <[email protected]> > 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 volatile operation with c11 atomic operations > and one-way barrier to save the performance penalty. According to the > timer_perf_autotest benchmarking results, this patch can uplift 10%~16% > timer appending performance, 3%~20% timer resetting performance and 45% > timer callbacks scheduling performance on aarch64 and no loss in > performance for x86. > > Suggested-by: Honnappa Nagarahalli <[email protected]> > Signed-off-by: Phil Yang <[email protected]> > Reviewed-by: Gavin Hu <[email protected]> > --- > lib/librte_timer/rte_timer.c | 90 +++++++++++++++++++++++++++++++----
Ping. Thanks, Phil
