[FFmpeg-devel] [PATCH] libavformat/rtspdec.c:flush pes buffer while rtsp seek

2022-11-13 Thread tanwei (D)
Fixes ticket #9949. Signed-off-by: tanwei123 --- libavformat/mpegts.c| 20 libavformat/mpegts.h| 1 + libavformat/rtpdec.c| 7 +++ libavformat/rtpdec.h| 2 ++ libavformat/rtpdec_mpegts.c | 11 +++ libavformat/rtspdec.c |

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

2022-11-13 Thread Wang, Bin
-Original Message- From: ffmpeg-devel On Behalf Of James Almer Sent: Monday, November 14, 2022 10:43 AM To: ffmpeg-devel@ffmpeg.org Subject: Re: [FFmpeg-devel] [PATCH v7] libavfilter/x86/vf_convolution: add sobel filter optimization and unit test with intel AVX512 VNNI On 11/4/2022 5:29

[FFmpeg-devel] [PATCH] Changelog: Add missing lines for QSV

2022-11-13 Thread Xiang, Haihao
From: Haihao Xiang Signed-off-by: Haihao Xiang --- Changelog | 5 + 1 file changed, 5 insertions(+) diff --git a/Changelog b/Changelog index 166aa8b5e7..f0b3d870d8 100644 --- a/Changelog +++ b/Changelog @@ -20,6 +20,11 @@ version : - ViewQuest VQC decoder - backgroundkey filter - nvenc

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

2022-11-13 Thread James Almer
On 11/4/2022 5:29 AM, bin.wang-at-intel@ffmpeg.org wrote: +%macro FILTER_SOBEL 0 +%if UNIX64 +cglobal filter_sobel, 4, 15, 7, dst, width, matrix, ptr, c0, c1, c2, c3, c4, c5, c6, c7, c8, r, x +%else +cglobal filter_sobel, 4, 15, 7, dst, width, rdiv, bias, matrix, ptr, c0, c1, c2, c3, c4,

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

2022-11-13 Thread Xiang, Haihao
On Fri, 2022-11-11 at 03:00 +, Xiang, Haihao wrote: > On Mon, 2022-11-07 at 05:24 +, Xiang, Haihao wrote: > > On Fri, 2022-11-04 at 16:29 +0800, bin.wang-at-intel@ffmpeg.org wrote: > > > From: bwang30 > > > > > > This commit enabled assembly code with intel AVX512 VNNI and added unit

Re: [FFmpeg-devel] [PATCH v3 1/4] swscale/input: add rgbaf32 input support

2022-11-13 Thread Mark Reid
On Sun, Nov 13, 2022 at 1:25 PM Michael Niedermayer wrote: > On Wed, Nov 02, 2022 at 09:00:07PM -0700, mindm...@gmail.com wrote: > > From: Mark Reid > > > > --- > > libswscale/input.c | 172 + > > libswscale/utils.c | 4 ++ > > 2 files changed, 176

Re: [FFmpeg-devel] [PATCH v4 2/2] lavc/vaapi_decode: add support for HWACCEL_CAP_RESET_WITHOUT_UNINIT

2022-11-13 Thread Wang, Fei W
On Fri, 2022-11-11 at 03:45 +, Xiang, Haihao wrote: > On Tue, 2022-11-08 at 19:45 +0800, Fei Wang wrote: > > This can fix vp9 decode image corruption when the frame size is > > change, > > but the pervious frames still be referenced. > > > > Surfaces don't need to be bound to vaContext only

[FFmpeg-devel] [PATCH v5 2/2] lavc/vaapi_decode: add support for HWACCEL_CAP_RESET_WITHOUT_UNINIT

2022-11-13 Thread Fei Wang
This can fix vp9 decode image corruption when the frame size is change, but the pervious frames still be referenced. Surfaces don't need to be bound to vaContext only after VAAPI 1.0.0: https://github.com/intel/libva/commit/492b692005ccd0d8da190209d5b3ae7b7825f4b8 Signed-off-by: Fei Wang ---

[FFmpeg-devel] [PATCH v5 1/2] lavc: add HWACCEL_CAP_RESET_WITHOUT_UNINIT capacity for hwaccel

2022-11-13 Thread Fei Wang
The capacity can avoid hwaccel being uninited when do the reset. It provides the method for hwaccel if it still want to use the previous initialized configuration after reset. And the configuration can be updated in AVHWAccel.init() if needed. For example, when use vaapi vp9 decode dynamic

Re: [FFmpeg-devel] [PATCH] mpegts: identify and demux DSMCC-B/MHEG streams

2022-11-13 Thread Scott Theisen
Ping for review. On 9/17/22 13:08, Scott Theisen wrote: These changes are from MythTV. --- The `AV_CODEC_ID`s are probably in the wrong place since these are data codecs, but that is where they are In MythTV. There was also a related change to libavformat/demux.c's avformat_find_stream_info()

Re: [FFmpeg-devel] [PATCH] x86/intreadwrite: use intrinsics instead of inline asm for AV_ZERO128

2022-11-13 Thread James Almer
On 10/24/2022 5:03 PM, James Almer wrote: When called inside a loop, the inline asm version results in one pxor unnecessarely emitted per iteration, as the contents of the __asm__() block are opaque to the compiler's instruction scheduler. This is not the case with intrinsics, where pxor will be

Re: [FFmpeg-devel] [PATCH v2 3/3] avcodec/ass: specify a permissive encoding

2022-11-13 Thread Oneric
On Sun, Nov 13, 2022 at 21:15:04 +0100, Oneric wrote: > To apply the third patch you may use v1 or the attached patch file. The list also stripped the CRLF line endings from the (quoted-printable encoded) attachments :( Also no patchwork download has unmangled line endings anymore. All v1

Re: [FFmpeg-devel] [PATCH v2 0/3] Some small ASS conversion fixes

2022-11-13 Thread Oneric
On Sun, Nov 13, 2022 at 21:01:34 +, Soft Works wrote: > I am unable to apply any of those patches. Neither v1 nor v2 neither > when downloading the series mbox or [2/3] from patchwork nor when > saving [2/3] from the e-mail. Re patchwork, yes I rechecked and it appears all downloads are now

Re: [FFmpeg-devel] [PATCH v2 0/3] Some small ASS conversion fixes

2022-11-13 Thread Soft Works
> -Original Message- > From: ffmpeg-devel On Behalf Of > Oneric > Sent: Sunday, November 13, 2022 10:00 PM > To: FFmpeg development discussions and patches de...@ffmpeg.org> > Subject: Re: [FFmpeg-devel] [PATCH v2 0/3] Some small ASS conversion > fixes > > On Sun, Nov 13, 2022 at

Re: [FFmpeg-devel] [PATCH v2 0/3] Some small ASS conversion fixes

2022-11-13 Thread Soft Works
> -Original Message- > From: ffmpeg-devel On Behalf Of > Oneric > Sent: Sunday, November 13, 2022 9:27 PM > To: FFmpeg development discussions and patches de...@ffmpeg.org> > Subject: Re: [FFmpeg-devel] [PATCH v2 0/3] Some small ASS conversion > fixes > > On Sun, Nov 13, 2022 at

Re: [FFmpeg-devel] [PATCH v2 0/3] Some small ASS conversion fixes

2022-11-13 Thread Oneric
On Sun, Nov 13, 2022 at 21:27:11 +0100, Oneric wrote: > [...] You can easily verify emails are capable of it by > applying patch 2 yourself or using patchwork’s mbox file. A correction: it seems since last time we discussed this, some patchwork update has expanded the line-mangling to also affect

Re: [FFmpeg-devel] [PATCH v2 0/3] Some small ASS conversion fixes

2022-11-13 Thread Oneric
On Sun, Nov 13, 2022 at 20:15:23 +, Soft Works wrote: > > [...] it's not a Patchwork bug and it cannot be "fixed" > at all in any other way than using binary diffs. The problem lies in > the fact that it's being sent by e-mail where the different line > endings get lost (as the spec mandates

Re: [FFmpeg-devel] [PATCH v2 0/3] Some small ASS conversion fixes

2022-11-13 Thread Soft Works
> -Original Message- > From: ffmpeg-devel On Behalf Of > Oneric > Sent: Sunday, November 13, 2022 8:57 PM > To: ffmpeg-devel@ffmpeg.org > Subject: [FFmpeg-devel] [PATCH v2 0/3] Some small ASS conversion > fixes > > This fixes colours and font selection for files converted to ASS > >

Re: [FFmpeg-devel] [PATCH v2 3/3] avcodec/ass: specify a permissive encoding

2022-11-13 Thread Oneric
For some reason this message for the third patch was sent out as plain 7-bit us-ASCII, stripping the CRLF line endings. Using the very same git send-email command line except for an added --to= for sending it directly to myself, yields the expected "printed-quotable" encoded message preserving the

[FFmpeg-devel] [PATCH v2 3/3] avcodec/ass: specify a permissive encoding

2022-11-13 Thread Oneric
The Encoding field (and the \fe tag) allows to limit font selection to only those fonts declaring support for the specified codepage in their OS/2's table "Code Page Character Range" field. Particularly, Encoding=0 means only font's declaring support for "ANSI", or rather "Latin (Western

[FFmpeg-devel] [PATCH v2 2/3] avcodec/ass: accurately preserve colours

2022-11-13 Thread Oneric
Colour values used in ASS files without a "YCbCr Matrix" header set to "None" are subject to colour mangling, due to how ASS was historically conceived. A more in-depth description can be found in the documetation inside libass' public ass_types.h header. The important part is, if this header is

[FFmpeg-devel] [PATCH v2 1/3] avcodec/ass: fix comment

2022-11-13 Thread Oneric
There is no v4 ASS. There are versions 1 to 4 of SSA (with only v4 being supported by softsub renderers), ASS which declares itself as v4+ or v4.00+, and the rarely used v4++/v4.00++, which isn't fully supported by libass. As reflected by the [V4+ Styles] section header, FFmpeg uses ASS, not SSA

[FFmpeg-devel] [PATCH v2 0/3] Some small ASS conversion fixes

2022-11-13 Thread Oneric
This fixes colours and font selection for files converted to ASS v2 == On Nicolas George’s request resent with plain text diff for the updated refernce files. There is no other change and applying the diff of v1 yields equal results to v2. Those files do intentionally contain CRLF line endings,

Re: [FFmpeg-devel] [PATCH 2/3] avcodec/ass: accurately preserve colours

2022-11-13 Thread Soft Works
> -Original Message- > From: ffmpeg-devel On Behalf Of > Oneric > Sent: Sunday, November 13, 2022 8:22 PM > To: FFmpeg development discussions and patches de...@ffmpeg.org> > Subject: Re: [FFmpeg-devel] [PATCH 2/3] avcodec/ass: accurately > preserve colours > > Thanks for taking a

Re: [FFmpeg-devel] [PATCH 2/3] avcodec/ass: accurately preserve colours

2022-11-13 Thread Oneric
Thanks for taking a look! On Sun, Nov 13, 2022 at 20:12:46 +0100, Nicolas George wrote: > Oneric (12022-11-13): > > [...] > > tests/ref/fate/sub-webvtt2 | Bin 1648 -> 1668 bytes > > These are text files. Please fix this so that we can review the patch. As explained in the cover letter,

Re: [FFmpeg-devel] [PATCH 2/3] avcodec/ass: accurately preserve colours

2022-11-13 Thread Nicolas George
Oneric (12022-11-13): > tests/ref/fate/sub-aqtitle | Bin 3213 -> 3233 bytes > tests/ref/fate/sub-cc| Bin 839 -> 859 bytes > tests/ref/fate/sub-cc-realtime | Bin 1524 -> 1544 bytes > tests/ref/fate/sub-cc-scte20 | Bin 945 -> 965 bytes > tests/ref/fate/sub-charenc

[FFmpeg-devel] [PATCH 2/3] avcodec/ass: accurately preserve colours

2022-11-13 Thread Oneric
Colour values used in ASS files without a "YCbCr Matrix" header set to "None" are subject to colour mangling, due to how ASS was historically conceived. A more in-depth description can be found in the documetation inside libass' public ass_types.h header. The important part is, if this header is

[FFmpeg-devel] [PATCH 3/3] avcodec/ass: specify a permissive encoding

2022-11-13 Thread Oneric
The Encoding field (and the \fe tag) allows to limit font selection to only those fonts declaring support for the specified codepage in their OS/2's table "Code Page Character Range" field. Particularly, Encoding=0 means only font's declaring support for "ANSI", or rather "Latin (Western

[FFmpeg-devel] [PATCH 0/3] Some small ASS conversion fixes

2022-11-13 Thread Oneric
This fixes colours and font selection for files converted to ASS. To avoid a bug with patchwork’s diff download stripping CRLFs, I formatted the updates of reference files as binary. They intentionally contain CRLF characters and without them do not pass FATE. tests/ref/fate/sub-scc keeps a text

[FFmpeg-devel] [PATCH 1/3] avcodec/ass: fix comment

2022-11-13 Thread Oneric
There is no v4 ASS. There are versions 1 to 4 of SSA (with only v4 being supported by softsub renderers), ASS which declares itself as v4+ or v4.00+, and the rarely used v4++/v4.00++, which isn't fully supported by libass. As reflected by the [V4+ Styles] section header, FFmpeg uses ASS, not SSA

[FFmpeg-devel] [PATCH 3/3] avformat/electronicarts: add option to return alpha channel in the main video stream in VP6A codec

2022-11-13 Thread Marton Balint
Signed-off-by: Marton Balint --- doc/demuxers.texi| 18 libavformat/electronicarts.c | 42 +++- libavformat/version.h| 2 +- 3 files changed, 56 insertions(+), 6 deletions(-) diff --git a/doc/demuxers.texi b/doc/demuxers.texi

[FFmpeg-devel] [PATCH 2/3] avformat/electronicarts: set packet_read in one place

2022-11-13 Thread Marton Balint
Signed-off-by: Marton Balint --- libavformat/electronicarts.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/libavformat/electronicarts.c b/libavformat/electronicarts.c index 0a0d6249e8..0532264f38 100644 --- a/libavformat/electronicarts.c +++

[FFmpeg-devel] [PATCH 1/3] avformat/electronicarts: fix EOF check

2022-11-13 Thread Marton Balint
Similar to feof(), avio_feof() is only true after an actual overread. Signed-off-by: Marton Balint --- libavformat/electronicarts.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/libavformat/electronicarts.c b/libavformat/electronicarts.c index e7f0cb9b9e..0a0d6249e8 100644

Re: [FFmpeg-devel] [Question] Technical reason why FFmpeg sets a mstack-alignment value on compile time?

2022-11-13 Thread Ronald S. Bultje
Hi, On Sun, Nov 13, 2022 at 8:47 AM Nomis101 wrote: > Am 12.11.22 um 23:09 schrieb Ronald S. Bultje: > > Hi, > > > > On Sat, Nov 12, 2022 at 1:57 PM Nomis101 wrote: > > > >> Hi.If building FFmpeg, configure does set a value for mstack-alignment, > >> for example on macOS > >>

Re: [FFmpeg-devel] [Question] Technical reason why FFmpeg sets a mstack-alignment value on compile time?

2022-11-13 Thread Nomis101
Am 12.11.22 um 23:09 schrieb Ronald S. Bultje: Hi, On Sat, Nov 12, 2022 at 1:57 PM Nomis101 wrote: Hi.If building FFmpeg, configure does set a value for mstack-alignment, for example on macOS -mstack-alignment=16. On HandBrake we found there is an issue with current Clang in Xcode about

[FFmpeg-devel] [PATCH] avfilter/af_firequalizer: switch to TX from lavu

2022-11-13 Thread Paul B Mahol
Patch attached. From bf0c69dc29fa4a41de7e47308f6907adf1215df3 Mon Sep 17 00:00:00 2001 From: Paul B Mahol Date: Sat, 12 Nov 2022 12:02:08 +0100 Subject: [PATCH] avfilter/af_firequalizer: switch to TX from lavu Signed-off-by: Paul B Mahol --- configure | 3 -

[FFmpeg-devel] [PATCH] fftools/ffprobe: support 2D arrays in print_list_fmt()

2022-11-13 Thread James Almer
Should fix undefined behavior. Signed-off-by: James Almer --- fftools/ffprobe.c | 24 +--- 1 file changed, 13 insertions(+), 11 deletions(-) diff --git a/fftools/ffprobe.c b/fftools/ffprobe.c index 99adf615ae..f46925618c 100644 --- a/fftools/ffprobe.c +++

Re: [FFmpeg-devel] [Internet][PATCH 00/12] Add MediaCodec encoder and NDK MediaCodec support

2022-11-13 Thread Zhao Zhili
> From: ffmpeg-devel On Behalf Of Olivier > Ayache > Sent: 2022年11月12日 18:00 > To: FFmpeg development discussions and patches > Cc: matthieu.bou...@stupeflix.com; a...@tmm1.net > Subject: Re: [FFmpeg-devel] [Internet][PATCH 00/12] Add MediaCodec encoder > and NDK MediaCodec support > > Hello