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

Pr vaapi 444/444_10 #31

Closed
wants to merge 2 commits into from
Closed

Conversation

fulinjie
Copy link
Collaborator

YUV444 blocked by:
intel/media-driver#577

YUV444_10 blocked by same issue in GST-VAAPI:
intel/media-driver#431

Signed-off-by: Linjie Fu <linjie.fu@intel.com>
HEVC_REXT will be map to {VAProfileHEVCMain422_10, VAProfileHEVCMain444,
VAProfileHEVCMain444_10} in vaapi_profile_map[], since need to be distinguished
to select the exact va_profile.

Add va_profile -> AV_PIX_FMT map for FF_PROFILE_HEVC_REXT to match the
exact va_profile.

Signed-off-by: Linjie Fu <linjie.fu@intel.com>
@uartie
Copy link
Collaborator

uartie commented Apr 24, 2019

This seems to break ffmpeg-vaapi mpeg2 decode test cases

@fulinjie
Copy link
Collaborator Author

Will update the patch set according to the comments from community and some pix_fmt support for packed format.
By the way, I got the check result for qsv in intel/ci/ubuntu/bionic/pr/check.
where can I get the result for ffmpeg-vaapi?
Thanks in advance.

@uartie
Copy link
Collaborator

uartie commented Apr 24, 2019

The intel/ci/ubuntu/bionic/pr/check compiles and tests both -vaapi and -qsv, so I'm not sure what you mean.

@uartie
Copy link
Collaborator

uartie commented Apr 24, 2019

The intel/ci/ubuntu/bionic/pr/check compiles and tests both -vaapi and -qsv, so I'm not sure what you mean.

On the "Build Overview" tab/page, there should be test summaries (near the bottom) for all -vaapi and -qsv (on both iHD and i965).

@fulinjie
Copy link
Collaborator Author

The intel/ci/ubuntu/bionic/pr/check compiles and tests both -vaapi and -qsv, so I'm not sure what you mean.

On the "Build Overview" tab/page, there should be test summaries (near the bottom) for all -vaapi and -qsv (on both iHD and i965).

Thanks Artie, found the right entry for the vaapi result.
Used to check the results by test suit, it seems that we should choose the exact test report (vaapi/qsv on exact platform) first.

@fulinjie
Copy link
Collaborator Author

fulinjie commented May 5, 2019

New patch set provided:
#41
Closed.

@fulinjie fulinjie closed this May 5, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
2 participants