summaryrefslogtreecommitdiff
path: root/libs/gui/ConsumerBase.cpp
diff options
context:
space:
mode:
author Yiwei Zhang <zzyiwei@google.com> 2019-04-16 11:03:01 -0700
committer Yiwei Zhang <zzyiwei@google.com> 2019-04-16 11:20:44 -0700
commitcb7dd008877ec881dc25d7c941bda802a0b467ff (patch)
tree4e07a58f6a85400851211a4750fc2940fe7b6929 /libs/gui/ConsumerBase.cpp
parentc4f335d6f4b00bac3fde8c439bcc363dff0a77ac (diff)
TimeStats: fix a racing case
setPostTime is plumbed without BufferQueue lock, and it's called at the end of queueBuffer. If the addAndGetFrameTimestamps is delayed by queueBuffer back pressure (this could happen on special SF/BQ configs), setAcquireFence will be called with a out-of-bound index internally. So this change will tweak the order of the back Pressure logic to make the post time correct when queueBuffer back pressure happens as well as guarding the TimeStats more strictly. Bug: 130515827 Test: all SurfaceFlinger tests Change-Id: I2eace6d8693cc647284de0f33e7b58a6bf79eaaf
Diffstat (limited to 'libs/gui/ConsumerBase.cpp')
0 files changed, 0 insertions, 0 deletions