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.

Sign up for GitHub

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

Jump to bottom

Safari から送信された H.264 のデータを VPL デコーダで受信すると遅延が起きたりセグフォすることがあるのを修正 #92

Merged
merged 10 commits into from
Apr 15, 2024
Merged

Safari から送信された H.264 のデータを VPL デコーダで受信すると遅延が起きたりセグフォすることがあるのを修正 #92

merged 10 commits into from
Apr 15, 2024

Conversation

melpon
Copy link
Contributor

@melpon melpon commented Apr 7, 2024 •
edited
Loading

#91 に余計な実験的なコミットを入れてしまったので作り直しです。

  • 遅延は、Decode 関数の中で１回しかデコードしないため、MFX_ERR_MORE_DATA やその他復帰可能なエラーが起きた時に、デコードしないといけないデータがずっと溜まっていってしまうのが原因
    • →Decode 関数の中で MFX_ERR_MORE_DATA が出るまでひたすらデコードするようにしました
  • セグフォは、DecodeFrameAsync を呼んだ後に mfxBitstream のバッファを動かしたり削除していたのが原因っぽい
    • →mfxBitstream のデータをキューにして、DecodeFrameAsync 以降は一切触らないようにすることで直しました
    • →ちゃんと mfxBitstream が消費される（＝MORE_DATA が返却される）までループを回せばキューにする必要すら無さそうだったので、キューを削除しました。

@melpon melpon mentioned this pull request Apr 7, 2024
Closed
@enm10k
Copy link
Contributor

enm10k commented Apr 15, 2024

動作確認の結果、事象が改善していることを確認したためマージします
ありがとうございました

@enm10k enm10k merged commit edee486 into develop Apr 15, 2024
8 checks passed
@enm10k enm10k deleted the feature/fix-vpl-decoder2 branch April 15, 2024 08:29
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@melpon @enm10k