eventdev: remove redundant reset on timer cancel
authorPhil Yang <phil.yang@arm.com>
Tue, 7 Jul 2020 15:54:52 +0000 (23:54 +0800)
committerThomas Monjalon <thomas@monjalon.net>
Wed, 8 Jul 2020 16:16:41 +0000 (18:16 +0200)
There is no thread will access these impl_opaque data after timer
canceled. When new timer armed, it got refilled. So the cleanup
process is unnecessary.

Cc: stable@dpdk.org
Signed-off-by: Phil Yang <phil.yang@arm.com>
Reviewed-by: Dharmik Thakkar <dharmik.thakkar@arm.com>
lib/librte_eventdev/rte_event_timer_adapter.c

index 4ed3013..aa01b4d 100644 (file)
@@ -1167,8 +1167,6 @@ swtim_cancel_burst(const struct rte_event_timer_adapter *adapter,
                rte_mempool_put(sw->tim_pool, (void **)timp);
 
                evtims[i]->state = RTE_EVENT_TIMER_CANCELED;
-               evtims[i]->impl_opaque[0] = 0;
-               evtims[i]->impl_opaque[1] = 0;
 
                rte_smp_wmb();
        }