[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