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

Commit 0ad5feff authored by TreeHugger Robot's avatar TreeHugger Robot Committed by Android (Google) Code Review
Browse files

Merge "surfaceflinger: make EventThread FIFO" into oc-dev

parents 7fccc207 22752851

services/surfaceflinger/SurfaceFlinger.cpp

+4 −1
Original line number Diff line number Diff line
@@ -548,12 +548,15 @@ void SurfaceFlinger::init() { 
        mSFEventThread = new EventThread(sfVsyncSrc, *this, true);

        mEventQueue.setEventThread(mSFEventThread);



        // set SFEventThread to SCHED_FIFO to minimize jitter

        // set EventThread and SFEventThread to SCHED_FIFO to minimize jitter

        struct sched_param param = {0};

        param.sched_priority = 2;

        if (sched_setscheduler(mSFEventThread->getTid(), SCHED_FIFO, &param) != 0) {

            ALOGE("Couldn't set SCHED_FIFO for SFEventThread");

        }

        if (sched_setscheduler(mEventThread->getTid(), SCHED_FIFO, &param) != 0) {

            ALOGE("Couldn't set SCHED_FIFO for EventThread");

        }



        // Get a RenderEngine for the given display / config (can't fail)

        mRenderEngine = RenderEngine::create(mEGLDisplay,

services/surfaceflinger/SurfaceFlinger_hwc1.cpp

+4 −2
Original line number Diff line number Diff line
@@ -520,13 +520,15 @@ void SurfaceFlinger::init() { 
    mSFEventThread = new EventThread(sfVsyncSrc, *this, true);

    mEventQueue.setEventThread(mSFEventThread);



    // set SFEventThread to SCHED_FIFO to minimize jitter

    // set EventThread and SFEventThread to SCHED_FIFO to minimize jitter

    struct sched_param param = {0};

    param.sched_priority = 2;

    if (sched_setscheduler(mSFEventThread->getTid(), SCHED_FIFO, &param) != 0) {

        ALOGE("Couldn't set SCHED_FIFO for SFEventThread");

    }



    if (sched_setscheduler(mEventThread->getTid(), SCHED_FIFO, &param) != 0) {

        ALOGE("Couldn't set SCHED_FIFO for EventThread");

    }



    // Initialize the H/W composer object.  There may or may not be an

    // actual hardware composer underneath.