ADHOC/replicant-frameworks_native
3
0
Fork 0
Code Issues 7 Pull Requests Releases Wiki Activity

am 86d1d747: Merge "Add lock before calling initEglImage"

Browse Source 
* commit '86d1d74762e65b6f64c2d4758aa5fd4af6275696':
  Add lock before calling initEglImage
This commit is contained in:
Conley Owens 2011-04-27 12:43:19 -07:00 committed by Android Git Automerger
commit a97b487178
1 changed files with 10 additions and 2 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

12
services/surfaceflinger/Layer.cpp
View File

@ -874,8 +874,16 @@ status_t Layer::BufferManager::initEglImage(EGLDisplay dpy,
ssize_t index = mActiveBuffer; ssize_t index = mActiveBuffer;
if (index >= 0) { if (index >= 0) {
if (!mFailover) { if (!mFailover) {
Image& texture(mBufferData[index].texture); {
err = mTextureManager.initEglImage(&texture, dpy, buffer); // Without that lock, there is a chance of race condition
// where while composing a specific index, requestBuf
// with the same index can be executed and touch the same data
// that is being used in initEglImage.
// (e.g. dirty flag in texture)
Mutex::Autolock _l(mLock);
Image& texture(mBufferData[index].texture);
err = mTextureManager.initEglImage(&texture, dpy, buffer);
}
// if EGLImage fails, we switch to regular texture mode, and we // if EGLImage fails, we switch to regular texture mode, and we
// free all resources associated with using EGLImages. // free all resources associated with using EGLImages.
if (err == NO_ERROR) { if (err == NO_ERROR) {