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

Fix playback 4k and 1080p video stuck issue #211

Closed
wants to merge 1 commit into from

Conversation

lsun30
Copy link
Contributor

@lsun30 lsun30 commented Dec 27, 2024

Issue is when playing 4k or 1080p video with gallery, video get stuck on android 15.

Cause is when c2 service change from hidl to aidl, block pool used in framework will change from buffer queue to IGBA, when allocating surface, surface metadata of igbp_id and igbp_slot will not be updated because they are buffer queue related concept. While in mediasdk_c2, it was checking igbp_id and igbp_slot during InitDecoder(), and caused IO pattern changed from video memory to system memory.

Solution is removing checking of igbp_id and igbp_slot for now. This way will cause some vts test fail, which will need be fixed.

Tracked-On: OAM-128397
Tracked-On: OAM-128324
Tracked-On: OAM-128315

Issue is when playing 4k or 1080p video with gallery, video get
stuck on android 15.

Cause is when c2 service change from hidl to aidl, block pool
used in framework will change from buffer queue to IGBA, when
allocating surface, surface metadata of igbp_id and igbp_slot
will not be updated because they are buffer queue related concept.
While in mediasdk_c2, it was checking igbp_id and igbp_slot during
InitDecoder(), and caused IO pattern changed from video memory to
system memory.

Solution is removing checking of igbp_id and igbp_slot for now.
This way will cause some vts test fail, which will need be fixed.

Tracked-On: OAM-128397
Tracked-On: OAM-128324
Tracked-On: OAM-128315
Signed-off-by: Lina Sun <[email protected]>
@lsun30
Copy link
Contributor Author

lsun30 commented Dec 30, 2024

moved to #206

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Engineering Build Not Started Engineering Build Not Started Pending Developer Approval Pending Developer Approval Pending PR Review Pending PR Review Valid commit message
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants