Cursor and overlay elements overwrite primary plane in `RenderFrameResult::blit_frame_result`

[Ottatop]

Cursor and overlay elements overwrite primary plane in `RenderFrameResult::blit_frame_result`

[cmeissl]

Thanks for testing, can you try if #1381 fixes the issue while keeping ColorTansformations active?

[Drakulix]

cc @cmeissl

[cmeissl]

Thanks for reporting this issue!

Disabling the overlay and cursor planes fixes the problem. Digging into the method, I think it's because elements on those planes are being RenderElement::drawn into the target buffer after blitting the primary plane in. Is that supposed to overwrite pixels already in the buffer even if what's getting drawn in has transparency?

Yes, RenderElement::draw is used to composite elements placed on drm planes into the bound buffer to align it with what drm would do during scan-out. For doing that more efficiently the blit_frame_result function uses glBlitFramebuffer for blitting the already composited. The blit_from/blit_to functions are optional and not part of the Frame Trait used for drawing. This is the reason why the function splits rendering into 3 stages, clear, blit and draw the overlay elements. Using the default capabilites for the GlesRenderer might enable color transformations which currently requires the use of a shadow buffer. My guess is that we fail to keep the content we already have drawn in this case.

blit_from does directly blit to the backing buffer and does not use the shadow buffer. But even if it did GlesRenderer::render would clear the shadow buffer anyway. Also GlesFrame::finish disables blending before blitting the shadow buffer. So it seems that it is currently not possible to render in multiple passes in case a shadow buffer is active.

@Ottatop can you try to verify this by disabling the Capability::ColorTransformations? This should be possible by using GlesRenderer::with_capabilites and in case you use the multi gpu backend by using a factory function.

[Ottatop]

Can confirm that disabling ColorTransformations works around that problem.

[Ottatop]

Yep, it does fix the issue!