commit 9f57e3c9da8923649adff62e63c644537f565e90
author Mikhail Naganov <mnaganov@google.com> Mon Dec 05 12:54:36 2016 -0800
committer Mikhail Naganov <mnaganov@google.com> Fri Dec 09 16:02:36 2016 -0800
tree 4ecc43b149d5801f9b378c493c725a4c790bcf42
parent e158da59b3b525cf271968bec3967414b21159fa

Crash audioserver if EPIPE (DEAD_OBJECT) is received from the HAL

audioflinger doesn't have a proper way to re-establish all
the opened streams if HAL crashes. The only valid way of dealing
with them is to restart the audioserver.

Although broadcastradio and sountrigger have cleaner protocols
for dealing with HAL failures and can handle HAL crash and restart,
since the audio side will anyway get broken, it makes sense
to also kill the audioserver when they detect HAL crash.

Change-Id: I5f8e8c1d0981042cdd435a3ddbb6526e9f226e6b
Test: kill audio-hal process, watch dmesg and logcat

media/libaudiohal/EffectsFactoryHalHidl.cpp

diff --git a/media/libaudiohal/EffectsFactoryHalHidl.cpp b/media/libaudiohal/EffectsFactoryHalHidl.cpp
index 4f2eef0..bacbe4e 100644
--- a/media/libaudiohal/EffectsFactoryHalHidl.cpp
+++ b/media/libaudiohal/EffectsFactoryHalHidl.cpp
@@ -20,6 +20,7 @@
 #include <cutils/native_handle.h>
 #include <media/EffectsFactoryApi.h>
 
+#include "ConversionHelperHidl.h"
 #include "EffectHalHidl.h"
 #include "EffectsFactoryHalHidl.h"
 #include "HidlUtils.h"
@@ -63,6 +64,7 @@
         return retval == Result::OK ? OK : NO_INIT;
     }
     mLastDescriptors.resize(0);
+    ConversionHelperHidl::crashIfHalIsDead(ret.getStatus());
     return ret.getStatus().transactionError();
 }
 
@@ -107,6 +109,7 @@
         else if (retval == Result::INVALID_ARGUMENTS) return NAME_NOT_FOUND;
         else return NO_INIT;
     }
+    ConversionHelperHidl::crashIfHalIsDead(ret.getStatus());
     return ret.getStatus().transactionError();
 }
 
@@ -130,6 +133,7 @@
         else if (retval == Result::INVALID_ARGUMENTS) return NAME_NOT_FOUND;
         else return NO_INIT;
     }
+    ConversionHelperHidl::crashIfHalIsDead(ret.getStatus());
     return ret.getStatus().transactionError();
 }
 
@@ -139,6 +143,7 @@
     hidlHandle->data[0] = fd;
     Return<void> ret = mEffectsFactory->debugDump(hidlHandle);
     native_handle_delete(hidlHandle);
+    ConversionHelperHidl::crashIfHalIsDead(ret.getStatus());
     return ret.getStatus().transactionError();
 }