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

Commit 53b2a9f8 authored by Tim Murray's avatar Tim Murray Committed by android-build-merger
Browse files

Disable SCHED_FIFO when turning off the display. am: f9d4e44f 

am: d406094f

Change-Id: Ia5020b7e8c2e09524e8801068c2863af933c7714
parents 34dd5530 d406094f

services/surfaceflinger/SurfaceFlinger.cpp

+13 −0
Original line number Diff line number Diff line
@@ -2626,6 +2626,7 @@ void SurfaceFlinger::setPowerModeInternal(const sp<DisplayDevice>& hw, 
    }



    if (currentMode == HWC_POWER_MODE_OFF) {

        // Turn on the display

        getHwComposer().setPowerMode(type, mode);

        if (type == DisplayDevice::DISPLAY_PRIMARY) {

            // FIXME: eventthread only knows about the main display right now
@@ -2636,7 +2637,19 @@ void SurfaceFlinger::setPowerModeInternal(const sp<DisplayDevice>& hw, 
        mVisibleRegionsDirty = true;

        mHasPoweredOff = true;

        repaintEverything();



        struct sched_param param = {0};

        param.sched_priority = 1;

        if (sched_setscheduler(0, SCHED_FIFO, &param) != 0) {

            ALOGW("Couldn't set SCHED_FIFO on display on");

        }

    } else if (mode == HWC_POWER_MODE_OFF) {

        // Turn off the display

        struct sched_param param = {0};

        if (sched_setscheduler(0, SCHED_OTHER, &param) != 0) {

            ALOGW("Couldn't set SCHED_OTHER on display off");

        }



        if (type == DisplayDevice::DISPLAY_PRIMARY) {

            disableHardwareVsync(true); // also cancels any in-progress resync

services/surfaceflinger/SurfaceFlinger_hwc1.cpp

+13 −0
Original line number Diff line number Diff line
@@ -2530,6 +2530,7 @@ void SurfaceFlinger::setPowerModeInternal(const sp<DisplayDevice>& hw, 
    }



    if (currentMode == HWC_POWER_MODE_OFF) {

        // Turn on the display

        getHwComposer().setPowerMode(type, mode);

        if (type == DisplayDevice::DISPLAY_PRIMARY) {

            // FIXME: eventthread only knows about the main display right now
@@ -2540,7 +2541,19 @@ void SurfaceFlinger::setPowerModeInternal(const sp<DisplayDevice>& hw, 
        mVisibleRegionsDirty = true;

        mHasPoweredOff = true;

        repaintEverything();



        struct sched_param param = {0};

        param.sched_priority = 1;

        if (sched_setscheduler(0, SCHED_FIFO, &param) != 0) {

            ALOGW("Couldn't set SCHED_FIFO on display on");

        }

    } else if (mode == HWC_POWER_MODE_OFF) {

        // Turn off the display

        struct sched_param param = {0};

        if (sched_setscheduler(0, SCHED_OTHER, &param) != 0) {

            ALOGW("Couldn't set SCHED_OTHER on display off");

        }



        if (type == DisplayDevice::DISPLAY_PRIMARY) {

            disableHardwareVsync(true); // also cancels any in-progress resync