diff options
| author | Mathias Agopian <mathias@google.com> | 2009-10-02 18:12:30 -0700 |
|---|---|---|
| committer | Mathias Agopian <mathias@google.com> | 2009-10-02 18:12:30 -0700 |
| commit | 0c4cec7e4df87181486d280c98fba9c0f4774c37 (patch) | |
| tree | 4d3aa4fe0101104a1816af9208d367d588bb4f4d /libs/surfaceflinger/BufferAllocator.cpp | |
| parent | 7b16834adc1003f492cd2be4b1bcc3fb73a78c23 (diff) | |
| download | frameworks_base-0c4cec7e4df87181486d280c98fba9c0f4774c37.zip frameworks_base-0c4cec7e4df87181486d280c98fba9c0f4774c37.tar.gz frameworks_base-0c4cec7e4df87181486d280c98fba9c0f4774c37.tar.bz2 | |
Attempt to fix [2152536] ANR in browser
The ANR is caused by SurfaceFlinger waiting for buffers of a removed surface to become availlable.
When it is removed from the current list, a Surface is marked as NO_INIT, which causes SF to return
immediately in the above case. For some reason, the surface here wasn't marked as NO_INIT.
This change makes the code more robust by always (irregadless or errors) setting the NO_INIT status
in all code paths where a surface is removed from the list.
Additionaly added more information in the logs, should this happen again.
Diffstat (limited to 'libs/surfaceflinger/BufferAllocator.cpp')
| -rw-r--r-- | libs/surfaceflinger/BufferAllocator.cpp | 2 |
1 files changed, 0 insertions, 2 deletions
diff --git a/libs/surfaceflinger/BufferAllocator.cpp b/libs/surfaceflinger/BufferAllocator.cpp index caf9bec..3e37bc3 100644 --- a/libs/surfaceflinger/BufferAllocator.cpp +++ b/libs/surfaceflinger/BufferAllocator.cpp @@ -15,8 +15,6 @@ ** limitations under the License. */ -#include <sys/mman.h> -#include <cutils/ashmem.h> #include <cutils/log.h> #include <utils/Singleton.h> |