diff options
| author | Tom Cherry <tomcherry@google.com> | 2020-07-31 15:21:54 -0700 |
|---|---|---|
| committer | Tom Cherry <tomcherry@google.com> | 2020-07-31 15:53:18 -0700 |
| commit | 2b3ebaf5c03acaffa1607d6385a2508f4bdd320e (patch) | |
| tree | d16b25a5d554f13b6e9ecc929ea746ded7fac285 /trusty/coverage/coverage.cpp | |
| parent | fcaed0effab54fa6face42a1b55c26760df0a074 (diff) | |
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
Diffstat (limited to 'trusty/coverage/coverage.cpp')
0 files changed, 0 insertions, 0 deletions