commit cbfebdbadb283ee13c555e7d79037fb5089caa4a
author Tom Cherry <tomcherry@google.com> Tue Apr 24 14:13:30 2018 -0700
committer Tom Cherry <tomcherry@google.com> Tue Apr 24 14:19:32 2018 -0700
tree 397edaeb3e30bf1d99e096254b410dbb43a26136
parent 9aca01907d6f72bc8c481293a90bf6fe9395a2c7

logd: notify kernel log readers of new messages

LogBuffer::log() returns either a negative number on error or a
positive number indicating the length of the message written.
Therefore, the check to notify kernel log readers of a new message
should be that this function's return value is > 0.

Bug: 78209416
Test: `adb logcat -b kernel` updates when new log messages are present
Change-Id: Icc18c0c22e62340994e5c26aedb72282d61c1541

logd/LogKlog.cpp

diff --git a/logd/LogKlog.cpp b/logd/LogKlog.cpp
index 7a7ac7d..ab980ac 100755
--- a/logd/LogKlog.cpp
+++ b/logd/LogKlog.cpp
@@ -825,7 +825,7 @@
                          (unsigned short)n);
 
     // notify readers
-    if (!rc) {
+    if (rc > 0) {
         reader->notifyNewLog(static_cast<log_mask_t>(1 << LOG_ID_KERNEL));
     }