commit 69aff753905f7415535bfabec0e02fd2663c733d
author Ryan Campbell <ryanjcampbell@google.com> Thu Jul 27 13:49:46 2017 -0700
committer Ryan Campbell <ryanjcampbell@google.com> Tue Aug 01 16:08:35 2017 +0000
tree 93c1d182261f41cd302211d0f74b91bb9469b4eb
parent 537bac1f59d3f6525876eaadd95f2b138a5fa3f3

Add support for coverage flushing.

Register a callback to allow for coverage flushing on the HAL server.
The callback is tied to the property "hal.coverage.enable" and should
execute whenever the property has value "true".

Test: vts_sancov_configure
Bug: 64019182
Change-Id: I08f19ce84a1307dfb321ce2d46cafcf095643c5c

base/HidlInternal.cpp

diff --git a/base/HidlInternal.cpp b/base/HidlInternal.cpp
index 3bb27f8..843fcad 100644
--- a/base/HidlInternal.cpp
+++ b/base/HidlInternal.cpp
@@ -25,13 +25,16 @@
 #include <dirent.h>
 #include <dlfcn.h>
 #include <regex>
+#include <utils/misc.h>
+
+extern "C" __attribute__((weak)) void __sanitizer_cov_dump();
 #endif
 
 namespace android {
 namespace hardware {
 namespace details {
 
-void logAlwaysFatal(const char *message) {
+void logAlwaysFatal(const char* message) {
     LOG(FATAL) << message;
 }
 
@@ -42,9 +45,21 @@
         const std::string &interface)
         : mInstrumentationLibPackage(package), mInterfaceName(interface) {
     configureInstrumentation(false);
+#ifdef LIBHIDL_TARGET_DEBUGGABLE
+    if (__sanitizer_cov_dump != nullptr) {
+        ::android::add_sysprop_change_callback(
+            []() {
+                bool enableCoverage = property_get_bool("hal.coverage.enable", false);
+                if (enableCoverage) {
+                    __sanitizer_cov_dump();
+                }
+            },
+            0);
+    }
+#endif
 }
 
-HidlInstrumentor:: ~HidlInstrumentor() {}
+HidlInstrumentor::~HidlInstrumentor() {}
 
 void HidlInstrumentor::configureInstrumentation(bool log) {
     bool enable_instrumentation = property_get_bool(