event/octeontx2: remove WFE from dual-slot dequeue
authorPavan Nikhilesh <pbhagavatula@marvell.com>
Fri, 14 Feb 2020 06:45:24 +0000 (12:15 +0530)
committerJerin Jacob <jerinj@marvell.com>
Sat, 4 Apr 2020 11:56:52 +0000 (13:56 +0200)
Each workslot is always bound to a specific lcore there is no multi-core
contention to cause cache trashing as a result it is safe to remove the
WFE. Also, in dual workslot dequeue work will mostlikely be available on
the pair workslot making WFE impractical.

Signed-off-by: Pavan Nikhilesh <pbhagavatula@marvell.com>
Reviewed-by: Gavin Hu <gavin.hu@arm.com>
drivers/event/octeontx2/otx2_worker_dual.h

index 5134e3d..c88420e 100644 (file)
@@ -29,11 +29,7 @@ otx2_ssogws_dual_get_work(struct otx2_ssogws_state *ws,
                rte_prefetch_non_temporal(lookup_mem);
 #ifdef RTE_ARCH_ARM64
        asm volatile(
-                       "        ldr %[tag], [%[tag_loc]]    \n"
-                       "        ldr %[wqp], [%[wqp_loc]]    \n"
-                       "        tbz %[tag], 63, done%=      \n"
-                       "        sevl                        \n"
-                       "rty%=:  wfe                         \n"
+                       "rty%=:                              \n"
                        "        ldr %[tag], [%[tag_loc]]    \n"
                        "        ldr %[wqp], [%[wqp_loc]]    \n"
                        "        tbnz %[tag], 63, rty%=      \n"