shinrich opened a new pull request #7667: URL: https://github.com/apache/trafficserver/pull/7667
We've been running with this code change for a couple months. @SolidWallOfCode suggested the PendingAction class to normalize how we store and clear the pending_action member of the HttpSM. For a long time we have had background level of crashes mostly from HostDB events that pop up long after the HttpSM has been deleted (and likely reallocated). The application of this class in our ATS9 has eliminated crashes like the following. ``` [ 00 ] libc-2.17.so raise ( raise.c:55 ) [ 01 ] libc-2.17.so abort ( abort.c:90 ) [ 02 ] libtscore.so.9.0.1 ink_abort ( ink_error.cc:99 ) [ 03 ] libtscore.so.9.0.1 _ink_assert ( ink_assert.cc:37 ) [ 04 ] traffic_server HostDBContinuation::probeEvent(int, Event*) ( HostDB.cc:1548 ) [ 05 ] traffic_server EThread::process_event(Event*, int) ( I_Continuation.h:167 ) [ 06 ] traffic_server EThread::execute_regular() ( UnixEThread.cc:241 ) [ 07 ] traffic_server execute ( UnixEThread.cc:332 ) [ 08 ] traffic_server EThread::execute() ( UnixEThread.cc:310 ) [ 09 ] traffic_server spawn_thread_internal ( Thread.cc:92 ) [ 10 ] libpthread-2.17.so start_thread ( pthread_create.c:307 ) ``` -- 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]
