commit dc8dd1071e51a5d915c24af25a1ee780e23b5881
author Satyam Kumar Singh <quic_satysing@quicinc.com> Wed Aug 03 18:22:23 2022 +0530
committer Gerrit - the friendly Code Review server <code-review@localhost> Mon Aug 07 04:01:46 2023 -0700
tree 1d651ba1aa5af65aea035ccb0722f0f748c5255e
parent a1dbeb8d51d61d050694ac92f0a2b99af0ac919b

sdm: handle concurrency between PostCommit and SetPanelBrightness

Change-Id: I14cce9cf69ece6200b9c806847a45fbdbd49c0ad

sdm/libs/core/display_builtin.cpp

diff --git a/sdm/libs/core/display_builtin.cpp b/sdm/libs/core/display_builtin.cpp
index 4cace41..1a853be 100644
--- a/sdm/libs/core/display_builtin.cpp
+++ b/sdm/libs/core/display_builtin.cpp
@@ -949,11 +949,14 @@
 
 DisplayError DisplayBuiltIn::PostCommit(HWLayersInfo *hw_layers_info) {
   DisplayBase::PostCommit(hw_layers_info);
-
-  if (pending_brightness_) {
-    Fence::Wait(retire_fence_);
-    SetPanelBrightness(cached_brightness_);
-    pending_brightness_ = false;
+  // Mutex scope
+  {
+    lock_guard<recursive_mutex> obj(brightness_lock_);
+    if (pending_brightness_) {
+      Fence::Wait(retire_fence_);
+      SetPanelBrightness(cached_brightness_);
+      pending_brightness_ = false;
+    }
   }
 
   if (commit_event_enabled_) {