app/eventdev: remove unnecessary barriers in perf test
authorFeifei Wang <feifei.wang2@arm.com>
Thu, 14 Jan 2021 07:08:27 +0000 (15:08 +0800)
committerJerin Jacob <jerinj@marvell.com>
Tue, 26 Jan 2021 13:52:56 +0000 (14:52 +0100)
commit9e9cf349fa031c5f3756d7f4fcee1b59e6fa1454
treea070715211ba19a9910be83d4a56b111c1b6d9b5
parentc7c033d173f2d4f14834c0fb942b84725ac940e8
app/eventdev: remove unnecessary barriers in perf test

For "processed_pkts" and "total_latency" functions, no operations should
keep the order that being executed before loading
"worker[i].processed_pkts". Thus rmb is unnecessary before loading.

For "perf_launch_lcores" function, wmb after that the main lcore
updates the variable "t->done", which represents the end of the test
signal, is unnecessary. Because after the main lcore updates this
siginal variable, it will jump out of the launch function loop, and wait
other lcores stop or return error in the main function(evt_main.c).
During this time, there is no important storing operation and thus no
need for wmb.

Signed-off-by: Feifei Wang <feifei.wang2@arm.com>
Reviewed-by: Ruifeng Wang <ruifeng.wang@arm.com>
Acked-by: Pavan Nikhilesh <pbhagavatula@marvell.com>
app/test-eventdev/test_perf_common.c