oneVPL-intel-gpu icon indicating copy to clipboard operation
oneVPL-intel-gpu copied to clipboard

[Feature]: support MFX_EXTBUFF_MASTERING_DISPLAY_COLOUR_VOLUME and MFX_EXTBUFF_CONTENT_LIGHT_LEVEL_INFO in AV1 decode and encode

Open xhaihao opened this issue 3 years ago • 5 comments

What Feature?

User may use MFX_EXTBUFF_MASTERING_DISPLAY_COLOUR_VOLUME and MFX_EXTBUFF_CONTENT_LIGHT_LEVEL_INFO to fetch / pass HDR parameters from / to oneVPL for hevc, we expect they can work for av1 too.

What's the usage scenario would be benifited?

Transcode for media delivery, Playback

What impacted?

No response

Do you want to contribute a patch to develop this feature?

None

xhaihao avatar Apr 26 '23 03:04 xhaihao

Auto Created VSMGWL-64006 for further analysis.

intel-mediadev avatar Apr 26 '23 03:04 intel-mediadev

https://github.com/oneapi-src/oneVPL-intel-gpu/blob/main/_studio/mfx_lib/encode_hw/av1/agnostic/base/av1ehw_base_hdr.cpp

https://github.com/oneapi-src/oneVPL-intel-gpu/blob/main/_studio/mfx_lib/encode_hw/av1/agnostic/base/av1ehw_base_hdr.h

VPL GPU runtime open source has already supported it.

FurongZhang avatar Aug 15 '23 01:08 FurongZhang

@FurongZhang The link provided by you is for encoding only, we can't fetch hdr metadata via MFX_EXTBUFF_MASTERING_DISPLAY_COLOUR_VOLUME and MFX_EXTBUFF_CONTENT_LIGHT_LEVEL_INFO for av1 decoding. This works for hevc decoding (see https://github.com/oneapi-src/oneVPL-intel-gpu/blob/main/_studio/mfx_lib/decode/h265/src/mfx_h265_dec_decode.cpp)

xhaihao avatar Aug 15 '23 04:08 xhaihao

@FurongZhang The link provided by you is for encoding only, we can't fetch hdr metadata via MFX_EXTBUFF_MASTERING_DISPLAY_COLOUR_VOLUME and MFX_EXTBUFF_CONTENT_LIGHT_LEVEL_INFO for av1 decoding. This works for hevc decoding (see https://github.com/oneapi-src/oneVPL-intel-gpu/blob/main/_studio/mfx_lib/decode/h265/src/mfx_h265_dec_decode.cpp)

The recent commit 0d0ba3c should have fix it. Then this PR for ffmpeg can continue.

nyanmisaka avatar Dec 08 '23 13:12 nyanmisaka

@nyanmisaka , thank you for the information. I knew that.

FurongZhang avatar Dec 11 '23 03:12 FurongZhang