[FFmpeg-devel] [PATCH] lavfi: add opencl tonemap filter.

[Song, Ruiling]

> -----Original Message-----
> From: ffmpeg-devel [mailto:ffmpeg-devel-bounces at ffmpeg.org] On Behalf Of
> Mark Thompson
> Sent: Tuesday, May 22, 2018 8:41 PM
> To: ffmpeg-devel at ffmpeg.org
> Subject: Re: [FFmpeg-devel] [PATCH] lavfi: add opencl tonemap filter.
> 
> On 22/05/18 09:48, Song, Ruiling wrote:
> >> -----Original Message-----
> >> From: ffmpeg-devel [mailto:ffmpeg-devel-bounces at ffmpeg.org] On Behalf
> Of
> >> Mark Thompson
> >> Sent: Tuesday, May 22, 2018 8:19 AM
> >> To: ffmpeg-devel at ffmpeg.org
> >> Subject: Re: [FFmpeg-devel] [PATCH] lavfi: add opencl tonemap filter.
> >>
> >> On 21/05/18 07:50, Ruiling Song wrote:
> >>> This filter does HDR(HDR10/HLG) to SDR conversion with tone-mapping.
> >>>
> >>> An example command to use this filter with vaapi codecs:
> >>> FFMPEG -init_hw_device vaapi=va:/dev/dri/renderD128 -init_hw_device \
> >>> opencl=ocl at va -hwaccel vaapi -hwaccel_device va -
> hwaccel_output_format \
> >>> vaapi -i INPUT -filter_hw_device ocl -filter_complex \
> >>> '[0:v]hwmap,tonemap_opencl=t=bt2020:tonemap=linear:format=p010[x1];
> \
> >>> [x1]hwmap=derive_device=vaapi:reverse=1' -c:v hevc_vaapi -profile 2
> >> OUTPUT
> >>>
> >>> Signed-off-by: Ruiling Song <ruiling.song at intel.com>
> >>> ---

> >>> +
> >>> +    err = av_frame_copy_props(output, input);
> >>> +    if (err < 0)
> >>> +        goto fail;
> 
> av_frame_copy_props() copies the side-data which will include the
> mastering/light-level information, but that's no longer valid after tonemapping?
I think so, but I am not sure how to update this information correctly.
Using result peak and result color-space primaries to replace original metadata? Sounds ok?

Ruiling