tree d16b25a5d554f13b6e9ecc929ea746ded7fac285
parent fcaed0effab54fa6face42a1b55c26760df0a074
author Tom Cherry <tomcherry@google.com> 1596234114 -0700
committer Tom Cherry <tomcherry@google.com> 1596235998 -0700

SocketClient: don't ignore SIGPIPE

1) All current users are better off ignoring SIGPIPE at the beginning
   of their process instead of ignoring it just for SocketClient
2) This isn't thread safe if users did want it, since sigaction()
   ignores SIGPIPE for the entire process
3) This costs 5-10% of logd CPU time when logcat is reading logs

Also clean up the error handling in SocketClient::sendDataLockedv().

Test: kill logcat and see that logd doesn't crash
Test: run simpleperf and see that no cycles are going to sigaction
Change-Id: I6532c8a0d71338e534411707b9a9bd785145c730