Donate to e Foundation | Murena handsets with /e/OS | Own a part of Murena! Learn more

Commit 4d7094dc authored by Tom Cherry's avatar Tom Cherry Committed by android-build-merger
Browse files

Merge "Check for spurious wake ups" am: 3a3cd0a2 am: c2dcebe6 

am: 1344c034

Change-Id: I8ffc749ab6fbd825ce238fff4ba1d12609585c04
parents df2870df 1344c034

libs/hwui/renderthread/RenderProxy.cpp

+3 −1
Original line number Diff line number Diff line
@@ -703,7 +703,9 @@ void* RenderProxy::postAndWait(MethodInvokeRenderTask* task) { 
    SignalingRenderTask syncTask(task, &mSyncMutex, &mSyncCondition);

    AutoMutex _lock(mSyncMutex);

    mRenderThread.queue(&syncTask);

    while (!syncTask.hasRun()) {

        mSyncCondition.wait(mSyncMutex);

    }

    return retval;

}

libs/hwui/renderthread/RenderTask.cpp

+1 −0
Original line number Diff line number Diff line
@@ -26,6 +26,7 @@ namespace renderthread { 
void SignalingRenderTask::run() {

    mTask->run();

    mLock->lock();

    mHasRun = true;

    mSignal->signal();

    mLock->unlock();

}

libs/hwui/renderthread/RenderTask.h

+3 −1
Original line number Diff line number Diff line
@@ -60,13 +60,15 @@ class SignalingRenderTask : public RenderTask { 
public:

    // Takes ownership of task, caller owns lock and signal

    SignalingRenderTask(RenderTask* task, Mutex* lock, Condition* signal)

            : mTask(task), mLock(lock), mSignal(signal) {}

            : mTask(task), mLock(lock), mSignal(signal), mHasRun(false) {}

    virtual void run() override;

    bool hasRun() const { return mHasRun; }



private:

    RenderTask* mTask;

    Mutex* mLock;

    Condition* mSignal;

    bool mHasRun;

};



typedef void* (*RunnableMethod)(void* data);

libs/hwui/renderthread/RenderThread.cpp

+3 −1
Original line number Diff line number Diff line
@@ -374,8 +374,10 @@ void RenderThread::queueAndWait(RenderTask* task) { 


    AutoMutex _lock(mutex);

    queue(&syncTask);

    while (!syncTask.hasRun()) {

        condition.wait(mutex);

    }

}



void RenderThread::queueAtFront(RenderTask* task) {

    AutoMutex _lock(mLock);