bdgranger commented on issue #7581: URL: https://github.com/apache/trafficserver/issues/7581#issuecomment-800532421
@masaori335 @shinrich we are still seeing crashes near the tip of the 8.1.x branch. They are not all related to Http2, but they do all go through either add_to_keep_alive_queue or manage_keep_alive_queue and in every case it appears that the m_ptr of the mutex is null. @shinrich we do have the code from #4936 that was to address #4504. However, we do not have the assert code from #4379, which is also not in the 8.1.x branch. It seems like it would be good to pick that up. Over the past week or so in this production environment, we have seen more than a dozen crashes spread across multiple servers, although not all the servers in the deployment. It appears to happen only under production load. The crashes fall fairly evenly across 4 call stacks, all of which eventually end up trying to add to or manage the keep_alive queue: * Http2Stream::destroy => Http2ConnectionState::release_stream => add_to_keep_alive_queue * InactivityCop::check_inactivity => manage_keep_alive_queue * Http1ClientSession::start => add_to_keep_alive_queue * Http1ClientSession::release => add_to_keep_alive_queue Will add some example stack traces and one history dump below. ---------------------------------------------------------------- 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. For queries about this service, please contact Infrastructure at: [email protected]
