[hw_composer]: Batch framebuffer releases into one
All checks were successful
eden-license / license-header (pull_request) Successful in 16s
All checks were successful
eden-license / license-header (pull_request) Successful in 16s
This commit is contained in:
parent
b5873e50dc
commit
0a83c34c52
1 changed files with 14 additions and 1 deletions
|
@ -53,7 +53,7 @@ u32 HardwareComposer::ComposeLocked(f32* out_speed_scale, Display& display,
|
||||||
// Set default speed limit to 100%.
|
// Set default speed limit to 100%.
|
||||||
*out_speed_scale = 1.0f;
|
*out_speed_scale = 1.0f;
|
||||||
|
|
||||||
// If no layers are available, skip the logic
|
// If no layers are available, skip the logic.
|
||||||
bool any_visible = false;
|
bool any_visible = false;
|
||||||
for (auto& layer : display.stack.layers) {
|
for (auto& layer : display.stack.layers) {
|
||||||
if (layer->visible) {
|
if (layer->visible) {
|
||||||
|
@ -132,6 +132,19 @@ u32 HardwareComposer::ComposeLocked(f32* out_speed_scale, Display& display,
|
||||||
nvdisp.Composite(composition_stack);
|
nvdisp.Composite(composition_stack);
|
||||||
}
|
}
|
||||||
|
|
||||||
|
// Batch framebuffer releases, instead of one-into-one.
|
||||||
|
std::vector<std::pair<Layer*, Framebuffer*>> to_release;
|
||||||
|
for (auto& [layer_id, framebuffer] : m_framebuffers) {
|
||||||
|
if (framebuffer.release_frame_number > m_frame_number || !framebuffer.is_acquired)
|
||||||
|
continue;
|
||||||
|
if (auto layer = display.stack.FindLayer(layer_id); layer)
|
||||||
|
to_release.emplace_back(layer.get(), &framebuffer);
|
||||||
|
}
|
||||||
|
for (auto& [layer, framebuffer] : to_release) {
|
||||||
|
layer->buffer_item_consumer->ReleaseBuffer(framebuffer->item, android::Fence::NoFence());
|
||||||
|
framebuffer->is_acquired = false;
|
||||||
|
}
|
||||||
|
|
||||||
// Advance by at least one frame.
|
// Advance by at least one frame.
|
||||||
const u32 frame_advance = swap_interval.value_or(1);
|
const u32 frame_advance = swap_interval.value_or(1);
|
||||||
m_frame_number += frame_advance;
|
m_frame_number += frame_advance;
|
||||||
|
|
Loading…
Add table
Add a link
Reference in a new issue