diff options
| author | 2020-03-04 17:32:44 -0800 | |
|---|---|---|
| committer | 2020-03-09 21:10:33 +0000 | |
| commit | cf1868c814eb466c06dcd572c787db59cf0e3729 (patch) | |
| tree | 5698800bb1f2ae9f140ab1adb1c1e6304d7fcb7b /vulkan/libvulkan/api.cpp | |
| parent | 14f49cd4d3566751258de3b050c2aea181fea9fc (diff) | |
SF: Remove a redundant write
The Layer compositionState structure was only intended to be updated
during the call from the LayerFE::prepareCompositionState() call, which
in turn calls preparePerFrameCompositionState().
Both BufferQueueLayer and BufferStateLayer were updating the buffer
entry in the composition state structure in updateActiveBuffer(), which
was redundant with an update of the buffer entry also being made there.
This *may* also have resulted in the StrongPointer code reporting a
write race, resulting in a reported bug.
Test: go/wm-smoke
Test: atest CtsColorModeTestCases
Test: atest CtsDisplayTestCases
Test: atest CtsGraphicsTestCases
Test: atest CtsUiRenderingTestCases
Test: atest CtsViewTestCases
Test: atest android.media.cts.EncodeVirtualDisplayWithCompositionTest
Bug: 149663608
Change-Id: I5b2fc552a92746adfe3693059dab315a573a4860
Diffstat (limited to 'vulkan/libvulkan/api.cpp')
0 files changed, 0 insertions, 0 deletions