diff options
| author | 2020-02-27 12:06:10 -0800 | |
|---|---|---|
| committer | 2020-05-12 19:05:37 +0000 | |
| commit | 6193c3b2e787cd19cbd14b72f5bbff1a4dd302f0 (patch) | |
| tree | c91887c9fb0de471989cc713dddb9970a26e3d4c /opengl/libagl/TextureObjectManager.cpp | |
| parent | fa5cda8260547494301c2dffa9256a7837d28327 (diff) | |
[RenderEngine] Reorder when ImageManager thread starts
Previously the background thread for managing EglImages was intialized
as a member variable before the constructor completed running. In very
rare circumstances this can potentially stop the running thread if the
thread ran before mRunning was initialized. To be less error-prone,
require that ImageManger::initThread be explicitly called to guarantee
that class member variables are well-defined when the thread begins
executing.
In case the issue resurfaces after this change, added a debug log to add
context.
Bug: 146416748
Bug: 155852355
Test: builds. There is no reliable repro, so this fix is speculative
Test: systrace to verify thread is running and configured correctly.
Change-Id: I141c86240b90c7f87c22b3768c2e188293987b76
Merged-In: I141c86240b90c7f87c22b3768c2e188293987b76
Diffstat (limited to 'opengl/libagl/TextureObjectManager.cpp')
0 files changed, 0 insertions, 0 deletions