drm_hwcomposer: Make flattening thread-based instead of vsync-based
Using vsync means consume some CPU time every frame for
IRQ -> Kernel_Thread -> UserSpace_listener thread transitions.
Framework tries to reduce vsync usage to the minimum, by using timelines
instead.
New flattening controller thread wakes-up only once per second.
This commit also removes flattening info from dumpsys. Practice shows
that it is almost useless. Instead debugging can be done using ALOGV
dumps.
Signed-off-by: Roman Stratiienko <r.stratiienko@gmail.com>
diff --git a/hwc2_device/HwcDisplay.h b/hwc2_device/HwcDisplay.h
index 653aa4a..bf95c3e 100644
--- a/hwc2_device/HwcDisplay.h
+++ b/hwc2_device/HwcDisplay.h
@@ -23,6 +23,7 @@
#include <sstream>
#include "HwcDisplayConfigs.h"
+#include "compositor/FlatteningController.h"
#include "compositor/LayerData.h"
#include "drm/DrmAtomicStateManager.h"
#include "drm/ResourceManager.h"
@@ -165,10 +166,6 @@
return total_stats_;
}
- /* returns true if composition should be sent to client */
- bool ProcessClientFlatteningState(bool skip);
- void ProcessFlatenningVsyncInternal();
-
/* Headless mode required to keep SurfaceFlinger alive when all display are
* disconnected, Without headless mode Android will continuously crash.
* Only single internal (primary) display is required to be in HEADLESS mode
@@ -181,17 +178,11 @@
void Deinit();
+ auto GetFlatCon() {
+ return flatcon_;
+ }
+
private:
- enum ClientFlattenningState : int32_t {
- Disabled = -3,
- NotRequired = -2,
- Flattened = -1,
- ClientRefreshRequested = 0,
- VsyncCountdownMax = 60, /* 1 sec @ 60FPS */
- };
-
- std::atomic_int flattenning_state_{ClientFlattenningState::NotRequired};
-
HwcDisplayConfigs configs_;
DrmHwcTwo *const hwc2_;
@@ -205,10 +196,10 @@
DrmDisplayPipeline *pipeline_{};
std::unique_ptr<Backend> backend_;
+ std::shared_ptr<FlatteningController> flatcon_;
std::shared_ptr<VSyncWorker> vsync_worker_;
bool vsync_event_en_{};
- bool vsync_flattening_en_{};
bool vsync_tracking_en_{};
int64_t last_vsync_ts_{};