[FFmpeg-devel] [PATCH v2 1/4] lavu/frame: Add Dolby Vision metadata side data type

[Marton Balint]

On Thu, 9 Dec 2021, Niklas Haas wrote:

> On Thu, 09 Dec 2021 17:21:58 +0000 Derek Buitenhuis <derek.buitenhuis at gmail.com> wrote:
>> On 12/8/2021 10:12 AM, Niklas Haas wrote:
>>> +/* based on guesswork, see mkvtoolnix and dovi_tool */
>>> +int av_dovi_profile(const AVDOVIRpuDataHeader *hdr)
>>> +{
>>
>> The correct way to find the profile is from the stream level
>> DOVI configuration record side data, if available.
>
> So, I changed the API to accept the DOVI configuration record, but
> actually getting that record into hevcdec isimpossible given the current
> design of FFmpeg where stream-level properties are not mode available to
> the decoder. Possibilities:
>
> 1. Add patch to propagate stream-level properties to
>   AVCodecContext.coded_side_data automatically
> 2. Add patch to automatically propagate stream-level properties to each
>   AVPacket somehow?

Don't we already have av_format_inject_global_side_data() for something 
like this? It is not enabled by default however, only for ffplay, but I am 
not sure why.

Regards,
Marton

> 3. Have the code read the profile from the AVPacket even though the side
>   data doesn't exist for them, and let this just be an open bug.
> 4. Something else?
>
> Thoughts?
> _______________________________________________
> ffmpeg-devel mailing list
> ffmpeg-devel at ffmpeg.org
> https://ffmpeg.org/mailman/listinfo/ffmpeg-devel
>
> To unsubscribe, visit link above, or email
> ffmpeg-devel-request at ffmpeg.org with subject "unsubscribe".
>