commit af50624dfb592ad6c9f571e2da4cf91e45217b04
author Robert Carr <racarr@google.com> Wed Jul 28 14:30:18 2021 -0700
committer Robert Carr <racarr@google.com> Wed Jul 28 14:30:18 2021 -0700
tree 5a3c79e885229d3af60d1e532e6d228fd47a84c7
parent 6a409e17600f320134dea722247c15770030d7b2

Don't call invalidateHwcGeometry when handling transactions

This causes composition engine to trigger a full update when
preparing composition state (StateSubset::GeometryAndContent)
vs StateSubset::Content. In single buffer update scenarios
this costs ~14% of whole process CPU instructions or ~20% of main
thread CPU instructions.

Bug: 186200583
Test: Existing tests pass
Change-Id: I8e3be0e4d828823f665fb05ea275401a178eb861

services/surfaceflinger/SurfaceFlinger.cpp

diff --git a/services/surfaceflinger/SurfaceFlinger.cpp b/services/surfaceflinger/SurfaceFlinger.cpp
index 7270016..6dc2bbc 100644
--- a/services/surfaceflinger/SurfaceFlinger.cpp
+++ b/services/surfaceflinger/SurfaceFlinger.cpp
@@ -2481,7 +2481,6 @@
     handleTransactionLocked(transactionFlags);
 
     mDebugInTransaction = 0;
-    invalidateHwcGeometry();
     // here the transaction has been committed
 }