diff options
author | Christopher Ferris <cferris@google.com> | 2015-02-10 12:15:01 -0800 |
---|---|---|
committer | Christopher Ferris <cferris@google.com> | 2015-02-25 19:59:29 -0800 |
commit | 54bcc5ffd5a79f4f194089c58d3de571532bf39b (patch) | |
tree | 9b35e579cfc14be60b978ee1c6129333c71d83d1 /opengl/specs | |
parent | d80268ba19f9217c5459a2952580090ef84a7b78 (diff) | |
download | frameworks_native-54bcc5ffd5a79f4f194089c58d3de571532bf39b.zip frameworks_native-54bcc5ffd5a79f4f194089c58d3de571532bf39b.tar.gz frameworks_native-54bcc5ffd5a79f4f194089c58d3de571532bf39b.tar.bz2 |
Add timeout for dump_file.
It turns out dump_file is used on a number of /proc and system files.
In one case, the read of a file stalled and caused a bugreport to
hang forever. It's still possible if there is a kernel bug that this
could stall forever, but less likely.
Also, change the return type of nanotime to uint64_t.
Testing:
- Created a named fifo and verified that dump_file fails with a timeout.
- Created a large /data/anr/traces.txt to verify that large files still
dump properly and that the additional NONBLOCK parameter doesn't cause
a problem.
- Created a dummy /data/tombstones/tombstone_00 to verify that the
dump of these files still works.
- Compared a dump using the old dumpstate to the new dumpstate to verify
nothing obviously different.
Bug: 19117030
Change-Id: I0d3dd27583c853cdaccd2fd278748cb5f9ccd4fb
Diffstat (limited to 'opengl/specs')
0 files changed, 0 insertions, 0 deletions