tree 9996e79f9463ddeeebded1ca9b1fa1149821bc9f
parent 94c1d740cfe001e5faf62d3c447c4e1e9f4aa325
author Yiwei Zhang <zzyiwei@google.com> 1555437781 -0700
committer Yiwei Zhang <zzyiwei@google.com> 1555439276 +0000

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
Merged-In: I2eace6d8693cc647284de0f33e7b58a6bf79eaaf