Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Memory locality #30

Open
aboba opened this issue May 3, 2021 · 1 comment
Open

Memory locality #30

aboba opened this issue May 3, 2021 · 1 comment
Labels
question Further information is requested

Comments

@aboba
Copy link
Contributor

aboba commented May 3, 2021

@padenot is proposing a change to the WebCodecs API to improve memory locality. I would like to understand if this has any implications for Mediacapture-transform.

Currently, VideoFrame buffers can be released by calling close(), which will typically happen after they have been used as input (e.g. in WebCodecs encoder). Once released, VideoFrame buffers can presumably be reused, at the implementation's discretion. It therefore isn't clear to me whether there a way for mediacapture-transform to make use of the additional WebCodecs arguments that are being proposed. For example, would a BYOB reader be helpful?

@aboba aboba added the question Further information is requested label May 3, 2021
@chcunningham
Copy link

Copying a bit of my comment from the WC bug:

Re: capture, this too uses pooling internally. See here for an overview of the IPC API that dishes out buffers and to observers (render processes) which then release them at some later point.

To the core question raised, I'm not sure yet. Waiting to see some rough psuedo code.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
question Further information is requested
Projects
None yet
Development

No branches or pull requests

2 participants