JosiahWI opened a new pull request, #11623: URL: https://github.com/apache/trafficserver/pull/11623
I would like to extract a new class to handle the responsibility of document evacuation, but that is hard because the evacuation logic uses the thread holding the `StripeSM` mutex to do the `EvacuationBlock` allocation. I think this is a hacky way to get the current thread, so I want to switch the calls to `this_ethread()` which doesn't depend on the mutex's internal data and describes precisely what it does. I'm PRing this change all by itself because from the perspective of the `StripeSM` API it is a behavior change. I don't think there can be too many debug asserts for this change so I've added them to every call into the evacuation logic, although if any of these asserts were violated we'd probably already have seen horrendous damage to the unprotected doubly-linked `evacuate` list and more. -- 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]
