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

Commit 48ebb5bf authored by TreeHugger Robot's avatar TreeHugger Robot Committed by Automerger Merge Worker
Browse files

Merge "SF: Guard Power HAL access with lock" into rvc-dev am: c8f2d971 

Original change: https://googleplex-android-review.googlesource.com/c/platform/frameworks/native/+/11921176

Change-Id: Ia042d7b3c98f0b5f9e815f4cd7580b2652751329
parents 46423f50 c8f2d971

services/surfaceflinger/DisplayHardware/PowerAdvisor.cpp

+2 −0
Original line number Diff line number Diff line
@@ -85,6 +85,7 @@ void PowerAdvisor::setExpensiveRenderingExpected(DisplayId displayId, bool expec 


    const bool expectsExpensiveRendering = !mExpensiveDisplays.empty();

    if (mNotifiedExpensiveRendering != expectsExpensiveRendering) {

        std::lock_guard lock(mPowerHalMutex);

        HalWrapper* const halWrapper = getPowerHal();

        if (halWrapper == nullptr) {

            return;
@@ -108,6 +109,7 @@ void PowerAdvisor::notifyDisplayUpdateImminent() { 
    }



    if (mSendUpdateImminent.load()) {

        std::lock_guard lock(mPowerHalMutex);

        HalWrapper* const halWrapper = getPowerHal();

        if (halWrapper == nullptr) {

            return;

services/surfaceflinger/DisplayHardware/PowerAdvisor.h

+5 −2
Original line number Diff line number Diff line
@@ -19,6 +19,8 @@ 
#include <atomic>

#include <unordered_set>



#include <utils/Mutex.h>



#include "../Scheduler/OneShotTimer.h"

#include "DisplayIdentification.h"
@@ -56,10 +58,11 @@ public: 
    void notifyDisplayUpdateImminent() override;



private:

    HalWrapper* getPowerHal();

    HalWrapper* getPowerHal() REQUIRES(mPowerHalMutex);

    bool mReconnectPowerHal GUARDED_BY(mPowerHalMutex) = false;

    std::mutex mPowerHalMutex;



    std::atomic_bool mBootFinished = false;

    bool mReconnectPowerHal = false;



    std::unordered_set<DisplayId> mExpensiveDisplays;

    bool mNotifiedExpensiveRendering = false;