-
Notifications
You must be signed in to change notification settings - Fork 7
Commit
This commit does not belong to any branch on this repository, and may belong to a fork outside of the repository.
HVUI/Vulkan does not work on A14 yet. Reverted to GLES. Signed-off-by: Roman Stratiienko <[email protected]>
- Loading branch information
Showing
19 changed files
with
1,137 additions
and
115 deletions.
There are no files selected for viewing
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
29 changes: 0 additions & 29 deletions
29
patches-aosp/build/soong/0001-GLODROID-Don-t-use-clang-tidy.patch
This file was deleted.
Oops, something went wrong.
39 changes: 39 additions & 0 deletions
39
patches-aosp/frameworks/base/0001-HWUI-Fix-deadlock-in-graphics-pipeline.patch
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -0,0 +1,39 @@ | ||
From 2be0dae3f8fb3c912edb49dba2f2cd434b8d9401 Mon Sep 17 00:00:00 2001 | ||
From: Roman Stratiienko <[email protected]> | ||
Date: Thu, 19 Oct 2023 03:37:01 +0300 | ||
Subject: [PATCH] HWUI: Fix deadlock in graphics pipeline | ||
|
||
Mutex lock() causes deadlock when the queue is empty. | ||
Regression is caused by b013c8dee3b3e897b3557206f2d6c75b8c9f2f84 | ||
For some unknown reason, it is reproduced only on GloDroid :) | ||
So probably some other part of GD is to blame. | ||
Or, it is just a coincidence. | ||
|
||
TODO: Do one more investigation attempt. | ||
|
||
Change-Id: Ic981947babe62b83032e9ba1630cc0458c228f44 | ||
Signed-off-by: Roman Stratiienko <[email protected]> | ||
--- | ||
libs/hwui/renderthread/CanvasContext.cpp | 6 +++++- | ||
1 file changed, 5 insertions(+), 1 deletion(-) | ||
|
||
diff --git a/libs/hwui/renderthread/CanvasContext.cpp b/libs/hwui/renderthread/CanvasContext.cpp | ||
index 16b35ffcabac..a9dde31e83e7 100644 | ||
--- a/libs/hwui/renderthread/CanvasContext.cpp | ||
+++ b/libs/hwui/renderthread/CanvasContext.cpp | ||
@@ -586,7 +586,11 @@ void CanvasContext::draw(bool solelyTextureViewUpdates) { | ||
// not visible to IRenderPipeline much less FrameInfoVisualizer. And since this is | ||
// the thread we're primarily concerned about being responsive, this being too broad | ||
// shouldn't pose a performance issue. | ||
- std::scoped_lock lock(mFrameMetricsReporterMutex); | ||
+ | ||
+// Causes deadlock when the queue is empty. Regression is caused by b013c8dee3b3e897b3557206f2d6c75b8c9f2f84 | ||
+// For some unknown reason, it is reproduced only on GloDroid :) So probably some other part of GD is to blame. Or, it is just a coincidence. | ||
+// std::scoped_lock lock(mFrameMetricsReporterMutex); | ||
+ | ||
drawResult = mRenderPipeline->draw(frame, windowDirty, dirty, mLightGeometry, | ||
&mLayerUpdateQueue, mContentDrawBounds, mOpaque, | ||
mLightInfo, mRenderNodes, &(profiler()), mBufferParams); | ||
-- | ||
2.39.2 | ||
|
82 changes: 0 additions & 82 deletions
82
patches-aosp/frameworks/base/0001-Revert-Fix-crash-from-asynchronous-GPU-metrics.patch
This file was deleted.
Oops, something went wrong.
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
26 changes: 26 additions & 0 deletions
26
...sp/glodroid/configuration/0008-base-Remove-deprecated-BOARD_BUILD_SYSTEM_ROOT_IMAGE.patch
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -0,0 +1,26 @@ | ||
From 08bf277bb68ef649949a83eb908a35666dc7fccf Mon Sep 17 00:00:00 2001 | ||
From: Roman Stratiienko <[email protected]> | ||
Date: Fri, 6 Oct 2023 22:20:52 +0300 | ||
Subject: [PATCH 08/12] base: Remove deprecated BOARD_BUILD_SYSTEM_ROOT_IMAGE | ||
definition | ||
|
||
Signed-off-by: Roman Stratiienko <[email protected]> | ||
--- | ||
common/base/board.mk | 1 - | ||
1 file changed, 1 deletion(-) | ||
|
||
diff --git a/common/base/board.mk b/common/base/board.mk | ||
index 82c4292..73158c5 100644 | ||
--- a/common/base/board.mk | ||
+++ b/common/base/board.mk | ||
@@ -97,7 +97,6 @@ TARGET_COPY_OUT_VENDOR_DLKM := vendor_dlkm | ||
|
||
# Root image | ||
TARGET_COPY_OUT_ROOT := root | ||
-BOARD_BUILD_SYSTEM_ROOT_IMAGE := false | ||
|
||
BOARD_EXT4_SHARE_DUP_BLOCKS := true | ||
|
||
-- | ||
2.39.2 | ||
|
Oops, something went wrong.