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

Support secure decoder #27

Open
wants to merge 11 commits into
base: main
Choose a base branch
from

Conversation

zhangyichix
Copy link
Contributor

No description provided.

zhangyichix and others added 10 commits April 25, 2024 10:44
 * Add the return of videoSignal value in av1

Tracked-On: OAM-112687
Signed-off-by: zhangyichix <[email protected]>
case:android.mediav2.cts.AdaptivePlaybackTest#testAdaptivePlayback

Some av1 videos may contain reserved units. We should ignore it instead of returning an error.

Reference source:
https://aomediacodec.github.io/av1-spec/av1-spec.pdf
Section 6.2.6: Reserved units are for future use and shall be ignored by AV1 decoder.

Tracked-On: OAM-102526
Signed-off-by: zhangyichix <[email protected]>
Tracked-On:OAM-117146
Signed-off-by: zhepeng.xu <[email protected]>
Signed-off-by: zhangyichix <[email protected]>
Some CTS tests of android.videocodec.cts.VideoEncoderMaxBFrameTest#
testMaxBFrameSupport failed with "Number of BFrames in a SubGOP
exceeds maximum number of BFrames configured".

Cause is for HEVC encode, CO3->GPB is set to "on" by default, which
makes GPB frames are used instead of P frames. In CTS test, GPB
frames are considered B frames.

Solution is for HEVC encode, set CO3->GPB to "off" in mediasdk_c2
to use P frames not GPB frames. But in onevpl-intel-gpu, HEVC encode
caps did not enable "GPB off" before, change HEVC encode caps to
enable it.

Tracked-On: OAM-118626
Signed-off-by: Lina Sun <[email protected]>
Value: MFX_LEVEL_AVC_1:10 MFX_LEVEL_AVC_1b:9
The functions `GetLevelLimitByDpbSize()`, `GetLevelLimitByFrameSize()`, `GetLevelLimitByMbps()`,
and `GetLevelLimitByMaxBitrate()` return the minimum level according to the frame size, FPS and profile.
Since the capability of level MFX_LEVEL_AVC_1b is greater than level MFX_LEVEL_AVC_1, the CodecLevel
should not be updated in this case.

Cases:
1.android.media.recorder.cts.MediaRecorderTest#testProfileAvcBaselineLevel1
2.android.mediav2.cts.EncoderProfileLevelTest#testValidateProfileLevel
[8_c2.intel.avc.encoder_video/avc_128kbps_176x144_15fps_yuv420flexible_2_0-bframes]
3.android.mediav2.cts.EncoderProfileLevelTest#testValidateProfileLevel
[10_c2.intel.avc.encoder_video/avc_128kbps_176x144_15fps_yuv420flexible_2_2-bframes]

Tracked-On: OAM-119037
Signed-off-by: Nana Zhang <[email protected]>
Temporarily change to only open renderD128.

Tracked-On: OAM-121112
Signed-off-by: Zhang, YichiX <[email protected]>
…509)"

This reverts commit c40736a.

Tracked-On: OAM-118913
Signed-off-by: Shaofeng Tang <[email protected]>
In order to pass Cts#testDefaultOutputColorFormat,
we need to determine the output bit format by mfxparam
instead of bitstream.

Tracked-On: OAM-119832
Signed-off-by: Zhang, YichiX <[email protected]>
@sysopenci
Copy link

Improper Commit Message
Tracked on not found in commit message,
make sure Tracked-On: Jira-ticket is present.

@sysopenci
Copy link

Improper Commit Message
Tracked on not found in commit message,
make sure Tracked-On: Jira-ticket is present.

@sysopenci
Copy link

Improper Commit Message
Tracked on not found in commit message,
make sure Tracked-On: Jira-ticket is present.

@sysopenci
Copy link

Improper Commit Message
Tracked on not found in commit message,
make sure Tracked-On: Jira-ticket is present.

@sysopenci
Copy link

Improper Commit Message
Tracked on not found in commit message,
make sure Tracked-On: Jira-ticket is present.

Signed-off-by: Zhang, YichiX <[email protected]>
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 Invalid commit message Pending Developer Approval Pending Developer Approval Pending PR Review Pending PR Review
Projects
None yet
Development

Successfully merging this pull request may close these issues.

6 participants