ywkaras commented on a change in pull request #7382:
URL: https://github.com/apache/trafficserver/pull/7382#discussion_r540485711



##########
File path: proxy/logging/LogObject.h
##########
@@ -283,7 +284,7 @@ class LogObject : public RefCountObj
   int m_min_rolled;            // minimum number of rolled logs to be kept, 0 
no limit
   bool m_reopen_after_rolling; // reopen log file after rolling (normally it 
is just renamed and closed)
 
-  head_p m_log_buffer; // current work buffer
+  ts::Atomic_versioned_ptr m_log_buffer; // current work buffer

Review comment:
       This is now less of a problem as I'm switching back to favoring using 
128-bit CAS with the version stored separately from the pointer.




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