mirror of
https://github.com/Evolution-X/hardware_interfaces
synced 2026-02-01 11:36:00 +00:00
Revert "Add a DisplayCapability for multi-threaded present"
This reverts commit b3e336edd1.
Bug: 274954820
Test: build
Change-Id: I6265f8de3db31f07506906cee82a91fe3baac0bc
This commit is contained in:
@@ -42,5 +42,4 @@ enum DisplayCapability {
|
||||
AUTO_LOW_LATENCY_MODE = 5,
|
||||
SUSPEND = 6,
|
||||
DISPLAY_IDLE_TIMER = 7,
|
||||
MULTI_THREADED_PRESENT = 8,
|
||||
}
|
||||
|
||||
@@ -80,20 +80,4 @@ enum DisplayCapability {
|
||||
* IComposerCallback.onVsyncIdle.
|
||||
*/
|
||||
DISPLAY_IDLE_TIMER = 7,
|
||||
/**
|
||||
* Indicates that both the composer HAL implementation and the given display
|
||||
* support calling executeCommands concurrently from separate threads.
|
||||
* executeCommands for a particular display will never run concurrently to
|
||||
* any other executeCommands for the same display. In addition, the
|
||||
* CommandResultPayload must only reference displays included in the
|
||||
* DisplayCommands passed to executeCommands. Displays referenced from
|
||||
* separate threads must have minimal interference with one another. If a
|
||||
* HWC-managed display has this capability, SurfaceFlinger can run
|
||||
* executeCommands for this display concurrently with other displays with the
|
||||
* same capability.
|
||||
* @see IComposerClient.executeCommands
|
||||
* @see DisplayCommand.presentDisplay
|
||||
* @see DisplayCommand.validateDisplay
|
||||
*/
|
||||
MULTI_THREADED_PRESENT = 8,
|
||||
}
|
||||
|
||||
@@ -2494,20 +2494,6 @@ TEST_P(GraphicsComposerAidlTest, GetDisplayConfigNoRepetitions) {
|
||||
}
|
||||
}
|
||||
|
||||
TEST_P(GraphicsComposerAidlCommandTest, MultiThreadedPresent) {
|
||||
std::vector<VtsDisplay*> displays;
|
||||
for (auto& display : mDisplays) {
|
||||
if (hasDisplayCapability(display.getDisplayId(),
|
||||
DisplayCapability::MULTI_THREADED_PRESENT)) {
|
||||
displays.push_back(&display);
|
||||
}
|
||||
}
|
||||
if (displays.size() <= 1u) {
|
||||
return;
|
||||
}
|
||||
// TODO(b/251842321): Try to present on multiple threads.
|
||||
}
|
||||
|
||||
/**
|
||||
* Test Capability::SKIP_VALIDATE
|
||||
*
|
||||
@@ -2583,4 +2569,4 @@ int main(int argc, char** argv) {
|
||||
}
|
||||
|
||||
return RUN_ALL_TESTS();
|
||||
}
|
||||
}
|
||||
|
||||
Reference in New Issue
Block a user