[FFmpeg-devel] [PATCH 07/29] lavc: add a decoder option for configuring side data preference

[James Almer]

On 3/5/2024 11:29 AM, Anton Khirnov wrote:
> Quoting James Almer (2024-03-05 13:30:58)
>>> +    {"dynamic_hdr10_plus",          .default_val.i64 = AV_PKT_DATA_DYNAMIC_HDR10_PLUS,          .type = AV_OPT_TYPE_CONST, .flags = A|D, .unit = "side_data_pkt" },
>>
>> This one packet/frame level only, not global.
> 
> It is in sd_global_map

Then that's a mistake, and I'm probably he culprit. HDR10+ is per frame 
metadata. Static HDR metadata are mastering_display and ccl.

> 
>> Is this option meant to also choose which one of those to use?
> 
> ???

You can have packet side data at the container level that corresponds to 
the same thinga per frame side data at the bitstream level does. In 
HDR10+ case, Matroska may have it in block additional, and then afaik it 
could be present in the hevc bitstream.
One of them should have priority, or the user could be given the choice.