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

Dev/5.10/fix streamoff 1 #4599

Merged
merged 2 commits into from
Sep 23, 2021

Conversation

jc-kynesim
Copy link
Contributor

Fixes a CMA leak, also worked around by reordering streamoffs in FFmpeg

Remove unused ctx state tracking variable and associated defines.
Their presence implies they might be used, but they aren't.

Signed-off-by: John Cox <[email protected]>
Before uniniting the decode context sync with the IRQ queues to ensure
that decode no longer has any buffers in use.  This fixes a problem that
manifested as ffmpeg leaking CMA buffers when it did a stream off on
OUTPUT before CAPTURE, though in reality it was probably much more
dangerous than that.

Signed-off-by: John Cox <[email protected]>
@pelwell
Copy link
Contributor

pelwell commented Sep 22, 2021

Looks ok to me. Can someone kick the tyres?

@popcornmix
Copy link
Collaborator

popcornmix commented Sep 23, 2021

Tested with kodi. Was able to play/stop hevc a number of times without issue.

@pelwell pelwell merged commit 6ab9539 into raspberrypi:rpi-5.10.y Sep 23, 2021
popcornmix added a commit to raspberrypi/firmware that referenced this pull request Sep 23, 2021
See: raspberrypi/linux#4593

kernel: rpivid: Ensure IRQs have completed before uniniting context
See: raspberrypi/linux#4599

kernel: net: phy: lan87xx: Allow more time for link detect
See: raspberrypi/linux#4598
popcornmix added a commit to Hexxeh/rpi-firmware that referenced this pull request Sep 23, 2021
See: raspberrypi/linux#4593

kernel: rpivid: Ensure IRQs have completed before uniniting context
See: raspberrypi/linux#4599

kernel: net: phy: lan87xx: Allow more time for link detect
See: raspberrypi/linux#4598
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants