[PATCH v4 5/5] eal: avoid deadlock in async IPC alarm callback

Burakov, Anatoly anatoly.burakov at intel.com
Tue Jun 9 10:04:22 CEST 2026


On 6/5/2026 4:29 PM, Anatoly Burakov wrote:
> async_reply_handle_thread_unsafe() can run while holding
> pending_requests.lock and currently calls rte_eal_alarm_cancel().
> 
> rte_eal_alarm_cancel() may spin-wait for an executing callback, which can
> deadlock if that callback is blocked on the same lock.
> 
> Remove callback-side alarm cancellation. It is safe to do so, because any
> callback triggered without a pending request becomes a noop.
> 
> Fixes: daf9bfca717e ("ipc: remove thread for async requests")
> Cc: stable at dpdk.org
> 
> Signed-off-by: Anatoly Burakov <anatoly.burakov at intel.com>
> ---

Okay, the AI review seems to keep flagging issues that are technically 
true in the patches, but are intentional and do get better once the 
complete patchset is applied.

Looks like I need to merge some of the patches or rethink the order in 
which the fixes are applied to avoid these issues.

-- 
Thanks,
Anatoly


More information about the dev mailing list