commit 5d1fbece58cfda5486d8f729b7b28ce553e62261
author Tom Cherry <tomcherry@google.com> Mon Sep 21 15:10:20 2020 -0700
committer Tom Cherry <tomcherry@google.com> Mon Sep 21 15:10:20 2020 -0700
tree a6b217b73a2ec52506bf88ab7cde99dec6597a84
parent b92ac53aeb27a954dbd0eed6d743739929c13db4

logd: additional logging if read_offset > buffer_it->write_offset()

I didn't expect this to ever be hit, but apparently it is, so add
additional logging to track it down.

Bug: 168870781
Test: build
Change-Id: Ia472bc7458a3727e4d407365672f7394426ae515

logd/SerializedFlushToState.cpp

diff --git a/logd/SerializedFlushToState.cpp b/logd/SerializedFlushToState.cpp
index 378cf20..fdf1dd3 100644
--- a/logd/SerializedFlushToState.cpp
+++ b/logd/SerializedFlushToState.cpp
@@ -91,7 +91,8 @@
         }
     } else {
         // read_offset > buffer_it->write_offset() should never happen.
-        CHECK(false);
+        LOG(FATAL) << "read_offset (" << read_offset << ") > buffer_it->write_offset() ("
+                   << buffer_it->write_offset() << ")";
     }
 }