Re: [FFmpeg-devel] [PATCH v3 1/3] lavc/decode: Warp get_hw_config function

2022-09-07 Thread Xiang, Haihao
On Wed, 2022-08-31 at 01:20 +, Wang, Fei W wrote: > On Tue, 2022-08-23 at 16:19 +0800, Fei Wang wrote: > > From: Linjie Fu > > > > Wrap the procedure of getting the hardware config from a pixel format > > into a function. > > > > Signed-off-by: Linjie Fu > > Signed-off-by: Fei Wang > > ---

Re: [FFmpeg-devel] [PATCH 0/5] Provide optimized neon implementation

2022-09-07 Thread Martin Storsjö
On Tue, 6 Sep 2022, Hubert Mazur wrote: Provide optimized implementations for me_cmp functions. This set of patches fixes all issues addressed in previous review. Major changes: - Remove redundant loads since the data can be reused. - Improve style. - Fix issues with unrecognized symbols. Than

Re: [FFmpeg-devel] [PATCH 2/5] lavc/aarch64: Add neon implementation of vsse16

2022-09-07 Thread Martin Storsjö
On Tue, 6 Sep 2022, Hubert Mazur wrote: Provide optimized implementation of vsse16 for arm64. Performance comparison tests are shown below. - vsse_0_c: 254.4 - vsse_0_neon: 64.7 Benchmarks and tests are run with checkasm tool on AWS Graviton 3. Signed-off-by: Hubert Mazur --- libavcodec/aarc

Re: [FFmpeg-devel] [PATCH 5/5] lavc/aarch64: Provide neon implementation of nsse16

2022-09-07 Thread Martin Storsjö
On Tue, 6 Sep 2022, Hubert Mazur wrote: Add vectorized implementation of nsse16 function. Performance comparison tests are shown below. - nsse_0_c: 707.0 - nsse_0_neon: 120.0 Benchmarks and tests run with checkasm tool on AWS Graviton 3. Signed-off-by: Hubert Mazur --- libavcodec/aarch64/me_

Re: [FFmpeg-devel] [PATCH] avocdec/flac_parser: another fix

2022-09-07 Thread Mattias Wadman
On Mon, Sep 5, 2022 at 8:16 PM Paul B Mahol wrote: > Patch attached. > Thanks and can confirm that the patch produces the same samples as the flac reference decoder for the original file in https://trac.ffmpeg.org/ticket/9621 that I could not share. But I'm not sure I follow how the patch works

Re: [FFmpeg-devel] [PATCH] avocdec/flac_parser: another fix

2022-09-07 Thread Paul B Mahol
On Wed, Sep 7, 2022 at 12:31 PM Mattias Wadman wrote: > On Mon, Sep 5, 2022 at 8:16 PM Paul B Mahol wrote: > > > Patch attached. > > > > Thanks and can confirm that the patch produces the same samples as the flac > reference decoder for the original file in > https://trac.ffmpeg.org/ticket/9621

Re: [FFmpeg-devel] [PATCH v3] libavfilter/x86/vf_convolution: add sobel filter optimization and unit test with intel AVX512 VNNI

2022-09-07 Thread Wang, Bin
-Original Message- From: ffmpeg-devel On Behalf Of Henrik Gramner Sent: Tuesday, September 6, 2022 9:02 PM To: FFmpeg development discussions and patches Subject: Re: [FFmpeg-devel] [PATCH v3] libavfilter/x86/vf_convolution: add sobel filter optimization and unit test with intel AVX512 V

Re: [FFmpeg-devel] [PATCH] avformat/matroska: Support HDR10+ metadata in Matroska.

2022-09-07 Thread Derek Buitenhuis
On 9/6/2022 10:47 PM, Mohammad Izadi wrote: > +if (side_data && side_data_size > 0) > + > ff_write_dynamic_hdr10_plus_to_full_itu_t_t35((AVDynamicHDRPlus*)side_data, > &hdr10_plus_itu_t_t35, &hdr10_plus_itu_t_t35_size); You can't use ff_-prefixed functions across library boundaries.

Re: [FFmpeg-devel] [PATCH] lavu: header and documentation for AVWriter

2022-09-07 Thread Nicolas George
Anton Khirnov (12022-09-02): > As I already said to you in private, I do not think the motivation and > use cases for this have been sufficiently established. > > You claim this will bring massive advantages all over the place. You > should support these claims with some actual patches that demons

[FFmpeg-devel] [avcodec/amfenc: 10 bit support v2 1/3] amf: Update the min version to 1.4.23.0 for AMF SDK.

2022-09-07 Thread OvchinnikovDmitrii
--- configure | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/configure b/configure index 9e51abd0d3..a644e17c95 100755 --- a/configure +++ b/configure @@ -7038,7 +7038,7 @@ fi enabled amf && check_cpp_condition amf "AMF/core/Version.h" \ -"(AMF_VERSION_MAJOR <<

[FFmpeg-devel] [avcodec/amfenc: 10 bit support v2 2/3] avcodec/amfenc: Fixes the color information in the output.

2022-09-07 Thread OvchinnikovDmitrii
From: Michael Fabian 'Xaymar' Dirks added 10 bit support for amf hevc. before: command - ffmpeg.exe -hide_banner -y -hwaccel d3d11va -hwaccel_output_format d3d11 -i test_10bit_file.mkv -an -c:v h264_amf res.dx11_hw_h264.mkv output - Format of input frames context (p010le) is not supported by

[FFmpeg-devel] [avcodec/amfenc: 10 bit support v2 3/3] avcodec/amfenc: HDR metadata.

2022-09-07 Thread OvchinnikovDmitrii
From: nyanmisaka --- libavcodec/amfenc.c | 83 + 1 file changed, 83 insertions(+) diff --git a/libavcodec/amfenc.c b/libavcodec/amfenc.c index efff9b5ae8..346f86bf00 100644 --- a/libavcodec/amfenc.c +++ b/libavcodec/amfenc.c @@ -36,6 +36,57 @@ #inclu

Re: [FFmpeg-devel] [PATCH] avformat/mxfdec: do not log warning of multiple ANC packets if count is 0

2022-09-07 Thread Gavin Smith
On 06/08/2022 21:44, Tomas Härdin wrote: fre 2022-08-05 klockan 00:03 +0100 skrev Gavin Smith: Some NLVEs may insert a KLV packet for EIA-608 data even though the number of encapsulated ANC packets is zero. ---  libavformat/mxfdec.c | 2 +-  1 file changed, 1 insertion(+), 1 deletion(-) Looks

[FFmpeg-devel] [PATCH 1/2] tests/mxf: Fix test requirements

2022-09-07 Thread Andreas Rheinhardt
Signed-off-by: Andreas Rheinhardt --- tests/fate/mxf.mak | 38 ++ 1 file changed, 14 insertions(+), 24 deletions(-) diff --git a/tests/fate/mxf.mak b/tests/fate/mxf.mak index 3ab936b5de..3f13170eb9 100644 --- a/tests/fate/mxf.mak +++ b/tests/fate/mxf.mak @@ -1

[FFmpeg-devel] [PATCH 2/2] fate/mxf: Add ProRes remux test

2022-09-07 Thread Andreas Rheinhardt
Also covers writing mastering display metadata. Signed-off-by: Andreas Rheinhardt --- This is an updated version of https://ffmpeg.org/pipermail/ffmpeg-devel/2021-November/287641.html tests/fate/mxf.mak | 4 ++ tests/ref/fate/mxf-remux-applehdr10 | 67

[FFmpeg-devel] [PATCH 1/2] avcodec/ralf: Move frame allocation after error checks

2022-09-07 Thread Andreas Rheinhardt
Signed-off-by: Andreas Rheinhardt --- libavcodec/ralf.c | 11 +-- 1 file changed, 5 insertions(+), 6 deletions(-) diff --git a/libavcodec/ralf.c b/libavcodec/ralf.c index 20c5f060d0..8f8e49f535 100644 --- a/libavcodec/ralf.c +++ b/libavcodec/ralf.c @@ -451,12 +451,6 @@ static int decode_

[FFmpeg-devel] [PATCH 2/2] avcodec/ralf: Move variable from context to stack

2022-09-07 Thread Andreas Rheinhardt
Signed-off-by: Andreas Rheinhardt --- libavcodec/ralf.c | 17 - 1 file changed, 8 insertions(+), 9 deletions(-) diff --git a/libavcodec/ralf.c b/libavcodec/ralf.c index 8f8e49f535..591598d8fc 100644 --- a/libavcodec/ralf.c +++ b/libavcodec/ralf.c @@ -63,7 +63,6 @@ typedef struct

Re: [FFmpeg-devel] [PATCH v2 2/4] ffmpeg: Add display_matrix option

2022-09-07 Thread Thilo Borgmann
Am 22.08.22 um 14:30 schrieb Nicolas George: Thilo Borgman (12022-08-20): My two cents about it that the a=b:c=d syntax from AVDict is at least known and used in filters already. The function style a(b,c,d) thing from SVG would be completely new. Instead of the AVDict overhead, it adds a simplis

[FFmpeg-devel] [PATCH] Bonk, Bonk

2022-09-07 Thread Paul B Mahol
Patches attached. Could decoder be made faster? From 65a3956af251d296d88181db26fdfc8510be7031 Mon Sep 17 00:00:00 2001 From: Paul B Mahol Date: Wed, 7 Sep 2022 14:01:42 +0200 Subject: [PATCH 2/2] avformat: add bonk demuxer Signed-off-by: Paul B Mahol --- libavformat/Makefile | 1 + libavf

[FFmpeg-devel] [PATCH 1/1] avcodec/jpeg2000: Add support for High-Throughput JPEG 2000 (HTJ2K) decoding.

2022-09-07 Thread caleb
This patch satisfies the GSoC 2022 project with the goal of adding a native htj2k decoder to ffmpeg. HTJ2K is specified in Rec. ITU-T 814 | ISO/IEC 15444-15 (https://www.itu.int/rec/T-REC-T.814-201906-I/en) --- libavcodec/j2kenc.c| 28 +- libavcodec/jpeg2000.h | 115 +++- libav

Re: [FFmpeg-devel] [avcodec/amfenc: 10 bit support v2 1/3] amf: Update the min version to 1.4.23.0 for AMF SDK.

2022-09-07 Thread Jean-Baptiste Kempf
Isn't that a bit too recent? Or does it not matter because it still supports old drivers? jb On Wed, 7 Sep 2022, at 15:41, OvchinnikovDmitrii wrote: > --- > configure | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > > diff --git a/configure b/configure > index 9e51abd0d3..a644e17c95 10

[FFmpeg-devel] [PATCH] format/imfdec: improve error handling when selecting tracks for playback

2022-09-07 Thread pal
From: Pierre-Anthony Lemieux Addresses coverity 1512414 --- libavformat/imfdec.c | 15 --- 1 file changed, 12 insertions(+), 3 deletions(-) diff --git a/libavformat/imfdec.c b/libavformat/imfdec.c index 5bbe7a53f8..58b1455c9f 100644 --- a/libavformat/imfdec.c +++ b/libavformat/imfd

Re: [FFmpeg-devel] [PATCH] lavu: header and documentation for AVWriter

2022-09-07 Thread Jean-Baptiste Kempf
On Wed, 7 Sep 2022, at 15:30, Nicolas George wrote: > With that correction stated, I have already answered you that your > inability to see the benefits is only proof that your work on FFmpeg is [...] > last time it happened you used the worst possible implementation and we Please stop attacking p

Re: [FFmpeg-devel] [PATCH] lavu: header and documentation for AVWriter

2022-09-07 Thread Nicolas George
Jean-Baptiste Kempf (12022-09-07): > Open source communities are based on consensus. You cannot just discard an > opinion > as less important than other just because YOU consider it's not important. > > Especially about someone who is in the top 5 contributor of the project, in > the > last few

Re: [FFmpeg-devel] [PATCH v3 2/3] lavc/decode: Add internal surface re-allocate method for hwaccel

2022-09-07 Thread Mark Thompson
On 23/08/2022 09:19, Fei Wang wrote: From: Linjie Fu Add HWACCEL_CAP_INTERNAL_ALLOC flag to indicate hwaccels are able to re-allocate surface internally through ff_decode_get_hw_frames_ctx. So that hwaccels don't need to reinitialize all hw related configs when decode resolution change, just ne

Re: [FFmpeg-devel] [PATCH v3 1/3] lavc/decode: Warp get_hw_config function

2022-09-07 Thread Mark Thompson
On 07/09/2022 09:47, Xiang, Haihao wrote: On Wed, 2022-08-31 at 01:20 +, Wang, Fei W wrote: On Tue, 2022-08-23 at 16:19 +0800, Fei Wang wrote: From: Linjie Fu Wrap the procedure of getting the hardware config from a pixel format into a function. Signed-off-by: Linjie Fu Signed-off-by: F

[FFmpeg-devel] [PATCH 1/2] tests/fate-run: Allow to set input options for encoding pass

2022-09-07 Thread Andreas Rheinhardt
This will be useful in the next commit. Signed-off-by: Andreas Rheinhardt --- tests/fate-run.sh | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) diff --git a/tests/fate-run.sh b/tests/fate-run.sh index 4008bcbc16..61cc59acc0 100755 --- a/tests/fate-run.sh +++ b/tests/fate-run.sh @@ -247,

[FFmpeg-devel] [PATCH 2/2] fate/matroska: Add test for updating AV1 extradata

2022-09-07 Thread Andreas Rheinhardt
Signed-off-by: Andreas Rheinhardt --- tests/fate/matroska.mak | 6 + tests/ref/fate/webm-av1-extradata-update | 32 2 files changed, 38 insertions(+) create mode 100644 tests/ref/fate/webm-av1-extradata-update diff --git a/tests/fate/matroska.mak b

Re: [FFmpeg-devel] [PATCH v2] avcodec/mips: Fix MMI macro replaces in HEVC Decoder

2022-09-07 Thread Michael Niedermayer
On Tue, Sep 06, 2022 at 07:34:14PM +0800, Shiyou Yin wrote: > > > > 2022年8月18日 20:29,Shiyou Yin 写道: > > > > > > > >> 2022年8月18日 19:44,戚铁铮 mailto:qitiezh...@360.cn>> 写道: > >> > >> > >> At 2022/8/18 PM 7:01, "Qi Tiezheng" >> > wrote: > >> > >>> The latest commit of

Re: [FFmpeg-devel] [avcodec/amfenc: 10 bit support v2 1/3] amf: Update the min version to 1.4.23.0 for AMF SDK.

2022-09-07 Thread Dmitrii Ovchinnikov
>>Isn't that a bit too recent? Or does it not matter because it still supports old drivers? Old drivers are fully supported. ___ ffmpeg-devel mailing list ffmpeg-devel@ffmpeg.org https://ffmpeg.org/mailman/listinfo/ffmpeg-devel To unsubscribe, visit li

Re: [FFmpeg-devel] [PATCH] avformat/matroska: Support HDR10+ metadata in Matroska.

2022-09-07 Thread Mohammad Izadi
Thank you Derek for your comment ! It’s fixed. On Wed, Sep 7, 2022 at 6:13 AM Derek Buitenhuis wrote: > On 9/6/2022 10:47 PM, Mohammad Izadi wrote: > > +if (side_data && side_data_size > 0) > > + > ff_write_dynamic_hdr10_plus_to_full_itu_t_t35((AVDynamicHDRPlus*)side_data, > &hdr10_plus_itu

[FFmpeg-devel] [PATCH] avformat/matroska: Support HDR10+ metadata in Matroska.

2022-09-07 Thread Mohammad Izadi
The fate test file can be found here: https://drive.google.com/file/d/1jGW3f94rglLfr5WGmMQe3SEnp1YkbMRy/view?usp=drivesdk The video file needs to be copied to fate-suite/mkv/ --- libavcodec/dynamic_hdr10_plus.c | 269 +--- libavcodec/dynamic_hdr10_plus.h |

[FFmpeg-devel] [PATCH 1/2] swscale/input: Remove spec-incompliant '; '

2022-09-07 Thread Andreas Rheinhardt
These macros are definitions, not only declarations and therefore should not contain a semicolon. Such a semicolon is actually spec-incompliant, but compilers happen to accept them. Signed-off-by: Andreas Rheinhardt --- libswscale/input.c | 10 +- 1 file changed, 5 insertions(+), 5 delet

[FFmpeg-devel] [PATCH 2/2] swscale/input: Avoid calls to av_pix_fmt_desc_get()

2022-09-07 Thread Andreas Rheinhardt
Up until now, libswscale/input.c used a macro to read an input pixel which involved a call to av_pix_fmt_desc_get() to find out whether the input pixel format is BE or LE despite this being known at compile-time (there are templates per pixfmt). Even worse, these calls are made in a loop, so that e

[FFmpeg-devel] [PATCH 3/3] swscale/output: Don't call av_pix_fmt_desc_get() in a loop

2022-09-07 Thread Andreas Rheinhardt
Up until now, libswscale/output.c used a macro to write an output pixel which involved a call to av_pix_fmt_desc_get() to find out whether the input pixel format is BE or LE despite this being known at compile-time (there are templates per pixfmt). Even worse, these calls are made in a loop, so tha