Skip to content

android: limit frame timestamps to SurfaceFlinger - #10832

Open
z3moon wants to merge 1 commit into
mainfrom
zw/unconditional-frame-timestamps-i10698
Open

z3moon wants to merge 1 commit into
mainfrom
zw/unconditional-frame-timestamps-i10698

Conversation

@z3moon

@z3moon z3moon commented Nov 12, 2026

Copy link
Copy Markdown
Contributor

Filament enabled frame timestamps on every native window swap chain. On some devices this crashes the vendor codec process behind a MediaCodec encoder input surface, and the app aborts with it.

Enable them only when the window queues to SurfaceFlinger, on both the GL and Vulkan backends.

Also fix NativeWindow::QUEUES_TO_WINDOW_COMPOSER, which was 7 (NATIVE_WINDOW_DEFAULT_HEIGHT) instead of 4.

Fixes #10698

Filament enabled frame timestamps on every native window swap chain.
On some devices this crashes the vendor codec process behind a
MediaCodec encoder input surface, and the app aborts with it.

Enable them only when the window queues to SurfaceFlinger, on both the
GL and Vulkan backends.

Also fix NativeWindow::QUEUES_TO_WINDOW_COMPOSER, which was 7
(NATIVE_WINDOW_DEFAULT_HEIGHT) instead of 4.

Fixes #10698

This branch has not been deployed

No deployments
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

internal Issue/PR does not affect clients

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Android Crash: SwapChain on a MediaCodec encoder input surface crashes media.codec and the app (EGL_TIMESTAMPS_ANDROID / enableFrameTimestamps)

1 participant