commit 5c2a8265c6821263e018c59b36fd8d4772546bb2
author android-build-prod (mdb) <android-build-team-robot@google.com> Mon May 07 18:07:21 2018 +0000
committer Gerrit Code Review <noreply-gerritcodereview@google.com> Mon May 07 18:07:21 2018 +0000
tree 94be2a8df9952e1894621b9f7ac75b5a15be1979
parent 515324a7a455c15dcad7902d456e9c32bc19b13b
parent 521d15c537b410c62cbe0e65c37c6a902f056a81

Merge changes from topic "lazy-dumpstate"

* changes:
  Make dumpstate HAL lazy.
  dumpstate: add interface into rc file

dumpstate/1.0/default/DumpstateDevice.cpp

diff --git a/dumpstate/1.0/default/DumpstateDevice.cpp b/dumpstate/1.0/default/DumpstateDevice.cpp
index 818a531..88623af 100644
--- a/dumpstate/1.0/default/DumpstateDevice.cpp
+++ b/dumpstate/1.0/default/DumpstateDevice.cpp
@@ -18,6 +18,7 @@
 
 #include "DumpstateDevice.h"
 
+#include <hidl/HidlBinderSupport.h>
 #include <log/log.h>
 
 #include "DumpstateUtil.h"
@@ -37,6 +38,11 @@
     // this interface - since HIDL_FETCH_IDumpstateDevice() is not defined, this function will never
     // be called by dumpstate.
 
+    // Exit when dump is completed since this is a lazy HAL.
+    addPostCommandTask([]() {
+        exit(0);
+    });
+
     if (handle == nullptr || handle->numFds < 1) {
         ALOGE("no FDs\n");
         return Void();

dumpstate/1.0/default/android.hardware.dumpstate@1.0-service.rc

diff --git a/dumpstate/1.0/default/android.hardware.dumpstate@1.0-service.rc b/dumpstate/1.0/default/android.hardware.dumpstate@1.0-service.rc
index 0f27248..dfbfb33 100644
--- a/dumpstate/1.0/default/android.hardware.dumpstate@1.0-service.rc
+++ b/dumpstate/1.0/default/android.hardware.dumpstate@1.0-service.rc
@@ -2,3 +2,6 @@
     class hal
     user system
     group system
+    interface android.hardware.dumpstate@1.0::IDumpstateDevice default
+    oneshot
+    disabled