summaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorAntoine Labour <piman@google.com>2014-07-25 18:14:42 -0700
committerAntoine Labour <piman@google.com>2014-07-25 18:47:25 -0700
commit11f14871db607718090ae6aa2e5dee3f490b8830 (patch)
tree4cdd0f7eaa5475c60caf0734be59822a502b8705
parentc268068c55afaaa441fda903b1b84a5b5c8a0a01 (diff)
downloadframeworks_native-11f14871db607718090ae6aa2e5dee3f490b8830.zip
frameworks_native-11f14871db607718090ae6aa2e5dee3f490b8830.tar.gz
frameworks_native-11f14871db607718090ae6aa2e5dee3f490b8830.tar.bz2
BufferQueueProducer: fix which slot gets pre-allocated
Commit 78014f32da6d0ebf52fb34ebb7663863000520a0 introduced a bug that made us pre-allocate buffers into the last available free slots instead of the first available ones. This in turn caused more re-allocations, and possibly triggered driver bugs. Change-Id: Ic4a70e676b4f2bbb054bc873be62ced26e3099a0
-rw-r--r--libs/gui/BufferQueueProducer.cpp2
1 files changed, 1 insertions, 1 deletions
diff --git a/libs/gui/BufferQueueProducer.cpp b/libs/gui/BufferQueueProducer.cpp
index cdc810d..a53775f 100644
--- a/libs/gui/BufferQueueProducer.cpp
+++ b/libs/gui/BufferQueueProducer.cpp
@@ -890,7 +890,7 @@ void BufferQueueProducer::allocateBuffers(bool async, uint32_t width,
continue;
}
- freeSlots.push_front(slot);
+ freeSlots.push_back(slot);
}
}