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

Commit 02eeda14 authored by John Reck's avatar John Reck Committed by Zhao Wei Liew
Browse files

Avoid starving RT anims 

Test: Manual, usleep(16000) in DrawFrameTask and tap
on recents

Merged-In: I88bb30a2503bc908ec45650c7d36b6fb3cc750d0

Change-Id: I88bb30a2503bc908ec45650c7d36b6fb3cc750d0
parent 603931d0

libs/hwui/renderthread/RenderThread.cpp

+13 −4
Original line number Original line Diff line number Diff line
@@ -20,6 +20,7 @@ 
#include "CanvasContext.h"

#include "CanvasContext.h"

#include "EglManager.h"

#include "EglManager.h"

#include "RenderProxy.h"

#include "RenderProxy.h"

#include "utils/FatVector.h"





#include <gui/DisplayEventReceiver.h>

#include <gui/DisplayEventReceiver.h>

#include <gui/ISurfaceComposer.h>

#include <gui/ISurfaceComposer.h>
@@ -282,11 +283,19 @@ bool RenderThread::threadLoop() { 
                "RenderThread Looper POLL_ERROR!");

                "RenderThread Looper POLL_ERROR!");





        nsecs_t nextWakeup;

        nsecs_t nextWakeup;

        // Process our queue, if we have anything

        {

            FatVector<RenderTask*, 10> workQueue;

            // Process our queue, if we have anything. By first acquiring

            // all the pending events then processing them we avoid vsync

            // starvation if more tasks are queued while we are processing tasks.

            while (RenderTask* task = nextTask(&nextWakeup)) {

            while (RenderTask* task = nextTask(&nextWakeup)) {

                workQueue.push_back(task);

            }

            for (auto task : workQueue) {

                task->run();

                task->run();

                // task may have deleted itself, do not reference it again

                // task may have deleted itself, do not reference it again

            }

            }

        }

        if (nextWakeup == LLONG_MAX) {

        if (nextWakeup == LLONG_MAX) {

            timeoutMillis = -1;

            timeoutMillis = -1;

        } else {

        } else {