saimayithri commented on PR #13362: URL: https://github.com/apache/trafficserver/pull/13362#issuecomment-4976090534
Thanks! I don't think I've completely proven that yet. From what I've traced so far, my change ensures the reenable event is scheduled onto the NetVConnection/NetHandler owner thread, but I haven't yet verified whether that also guarantees `nh->mutex` is already held (or otherwise guarantees the `MUTEX_TRY_LOCK` succeeds) by the time execution reaches `add_to_active_queue()`. Given @wuxcer's latest report, that seems to be the remaining question. I'll trace that path next and verify whether scheduling onto the owner thread is sufficient to satisfy the `MUTEX_TRY_LOCK` invariant, or whether there's another execution path where that invariant can still be violated. -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: [email protected] For queries about this service, please contact Infrastructure at: [email protected]
