vmamidi commented on a change in pull request #7386:
URL: https://github.com/apache/trafficserver/pull/7386#discussion_r544412076



##########
File path: iocore/net/UnixNetVConnection.cc
##########
@@ -91,7 +91,7 @@ read_signal_and_update(int event, UnixNetVConnection *vc)
     case VC_EVENT_ACTIVE_TIMEOUT:
     case VC_EVENT_INACTIVITY_TIMEOUT:
       Debug("inactivity_cop", "event %d: null read.vio cont, closing vc %p", 
event, vc);
-      Warning("read: Closing orphaned vc %p", vc);
+      Warning("read: Closing orphaned vc vc=%p cont=%p event=%d", vc, 
vc->read.vio.cont, event);

Review comment:
       This is the normal scenario as the state machine may decide to go away 
for several reasons, but iocore might still keep the VC for clearing the 
events. Please let me know if I am missing something.




----------------------------------------------------------------
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]


Reply via email to