shinrich commented on a change in pull request #7337:
URL: https://github.com/apache/trafficserver/pull/7337#discussion_r527806302
##########
File path: iocore/net/UnixNetVConnection.cc
##########
@@ -686,6 +680,14 @@ UnixNetVConnection::do_io_close(int alerrno /* = -1 */)
this->lerrno = alerrno;
}
+ // Must mark for closed last in case this is a
+ // cross thread migration scenario.
+ if (alerrno == -1) {
Review comment:
I can try moving clearing the vio contintations to the free. But even
reducing the race window this tiny amount seems to have eliminated the crash
for us.
----------------------------------------------------------------
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]
