Re: [FFmpeg-devel] [PATCH v3 1/2] dxva: wait until D3D11 buffer copies are done before submitting them

2020-08-11 Thread Steve Lhomme
On 2020-08-11 8:58, Soft Works wrote: -Original Message- From: ffmpeg-devel On Behalf Of Steve Lhomme Sent: Tuesday, August 11, 2020 6:35 AM To: ffmpeg-devel@ffmpeg.org Subject: Re: [FFmpeg-devel] [PATCH v3 1/2] dxva: wait until D3D11 buffer copies are done before submitting them [.

[FFmpeg-devel] [PATCH 2/2] tools/target_dec_fuzzer: Adjust threshold for AGM

2020-08-11 Thread Michael Niedermayer
Fixes: Timeout (142sec -> 2sec) Fixes: 24426/clusterfuzz-testcase-minimized-ffmpeg_AV_CODEC_ID_AGM_fuzzer-5639724379930624 Found-by: continuous fuzzing process https://github.com/google/oss-fuzz/tree/master/projects/ffmpeg Signed-off-by: Michael Niedermayer --- tools/target_dec_fuzzer.c | 1 +

[FFmpeg-devel] [PATCH 1/2] avcodec/agm: Mark fill_pixels() as av_always_inline

2020-08-11 Thread Michael Niedermayer
Speedup from 275sec to 142sec Testcase: 24426/clusterfuzz-testcase-minimized-ffmpeg_AV_CODEC_ID_AGM_fuzzer-5639724379930624 Signed-off-by: Michael Niedermayer --- libavcodec/agm.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/libavcodec/agm.c b/libavcodec/agm.c index bc9df

Re: [FFmpeg-devel] [PATCH 1/2] avcodec/agm: Mark fill_pixels() as av_always_inline

2020-08-11 Thread Paul B Mahol
LGTM On 8/11/20, Michael Niedermayer wrote: > Speedup from 275sec to 142sec > Testcase: > 24426/clusterfuzz-testcase-minimized-ffmpeg_AV_CODEC_ID_AGM_fuzzer-5639724379930624 > > Signed-off-by: Michael Niedermayer > --- > libavcodec/agm.c | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) >

Re: [FFmpeg-devel] [FFmpeg-cvslog] avfilter/vf_xfade: check that fps between inputs are valid

2020-08-11 Thread Nicolas George
Paul B Mahol (12020-08-11): > ffmpeg | branch: master | Paul B Mahol | Tue Aug 11 > 10:55:16 2020 +0200| [4dbb75c437117285cfa195c437a8a1ee898e9293] | committer: > Paul B Mahol > > avfilter/vf_xfade: check that fps between inputs are valid > > > http://git.videolan.org/gitweb.cgi/ffmpeg.git/?a=

Re: [FFmpeg-devel] [FFmpeg-cvslog] avfilter/vf_xfade: check that fps between inputs are valid

2020-08-11 Thread Paul B Mahol
On 8/11/20, Nicolas George wrote: > Paul B Mahol (12020-08-11): >> ffmpeg | branch: master | Paul B Mahol | Tue Aug 11 >> 10:55:16 2020 +0200| [4dbb75c437117285cfa195c437a8a1ee898e9293] | >> committer: Paul B Mahol >> >> avfilter/vf_xfade: check that fps between inputs are valid >> >> > http://gi

Re: [FFmpeg-devel] [PATCH] avcodec/bsf: Avoid allocation for AVBSFInternal

2020-08-11 Thread Nicolas George
Andreas Rheinhardt (12020-08-11): > Imagine the context to only contain elements that require a alignment of > 4 and the internal structure to require an alignment of eight. Then it > is perfectly possible for &ctx[1] to only have an alignment of four and > not of eight as internal requires it. Wi

Re: [FFmpeg-devel] [FFmpeg-cvslog] avfilter/vf_xfade: check that fps between inputs are valid

2020-08-11 Thread Nicolas George
Paul B Mahol (12020-08-11): > You clearly do not know anything about xfade filter. I know that what you just pushed is a break in the API of libavfilter, I have explained the reason in my last mail. This is all I will say about it unless you prove willing to cooperate towards making the code bett

Re: [FFmpeg-devel] [FFmpeg-cvslog] avfilter/vf_xfade: check that fps between inputs are valid

2020-08-11 Thread Paul B Mahol
On 8/11/20, Nicolas George wrote: > Paul B Mahol (12020-08-11): >> You clearly do not know anything about xfade filter. > > I know that what you just pushed is a break in the API of libavfilter, I > have explained the reason in my last mail. Other filters check frame rate already. Cross fading c

Re: [FFmpeg-devel] [PATCH] avfilter: add radial and circular blur video filter

2020-08-11 Thread Paul B Mahol
On 7/29/20, Paul B Mahol wrote: > On 7/28/20, Nicolas George wrote: >> Paul B Mahol (12020-07-28): >>> >> Please add an example that produces high quality output. >>> >> Iam asking so the filter can be tested in relation to the concerns >>> >> that >>> >> have been raised in previous reviews >> >

Re: [FFmpeg-devel] [PATCH 2/7] avfilter: add ff_inlink_peek_samples and ff_inlink_skip samples

2020-08-11 Thread Paul B Mahol
Ping! On 7/21/20, Paul B Mahol wrote: > On 7/20/20, Michael Niedermayer wrote: >> On Thu, Jun 11, 2020 at 08:31:57PM +0200, Nicolas George wrote: >>> Paul B Mahol (12020-06-11): >>> > Signed-off-by: Paul B Mahol >>> > --- >>> > libavfilter/avfilter.c | 61 >>> >

Re: [FFmpeg-devel] [PATCH] avformat/ty: Remove write-only array and variable

2020-08-11 Thread Paul B Mahol
Make sure this does not break existing files, On 8/11/20, Andreas Rheinhardt wrote: > Up until now, the TiVo demuxer parse an array of SEQ entries, yet it has > never ever made any use of them. In fact, parse_master, the function > parsing said table, only influenced the outside world in three wa

Re: [FFmpeg-devel] [FFmpeg-cvslog] avfilter/vf_xfade: check that fps between inputs are valid

2020-08-11 Thread Nicolas George
Paul B Mahol (12020-08-11): > Other filters check frame rate already. If they do, then it is a bug too. > Cross fading can not work reliably if fps is variable or do not match > between inputs. You need to justify that statement. Because if they do not match, then users will just insert the fps

Re: [FFmpeg-devel] [FFmpeg-cvslog] avfilter/vf_xfade: check that fps between inputs are valid

2020-08-11 Thread Paul B Mahol
On 8/11/20, Nicolas George wrote: > Paul B Mahol (12020-08-11): >> Other filters check frame rate already. > > If they do, then it is a bug too. You are only one that now know what is bug and what is not bug and ignore others decisions? > >> Cross fading can not work reliably if fps is variable

Re: [FFmpeg-devel] [PATCH 4/6] avfilter/formats: Leave lists' ownership unchanged upon merge failure

2020-08-11 Thread Nicolas George
Andreas Rheinhardt (12020-08-08): > ff_merge_formats(), ff_merge_samplerates() and ff_merge_channel_layouts() > share common semantics: If merging succeeds, a non-NULL pointer is > returned and both input lists (of type AVFilterFormats resp. > AVFilterChannelLayouts) are to be treated as if they ha

Re: [FFmpeg-devel] [PATCH] avcodec/options: Add options for inputing HDR10-related metadata

2020-08-11 Thread Paul B Mahol
I think this should use some sort of side data instead or adding new fields to AVCodecContext. On 8/11/20, Kenny McClive wrote: > Previously, the only way to input the master display and content light > metadata required for HDR10 was through x265-params. Obviously, that only > worked with x265.

Re: [FFmpeg-devel] [PATCH v3 1/2] dxva: wait until D3D11 buffer copies are done before submitting them

2020-08-11 Thread Soft Works
> -Original Message- > From: ffmpeg-devel On Behalf Of > Steve Lhomme > Sent: Tuesday, August 11, 2020 10:06 AM > To: ffmpeg-devel@ffmpeg.org > Subject: Re: [FFmpeg-devel] [PATCH v3 1/2] dxva: wait until D3D11 buffer > copies are done before submitting them > > On 2020-08-11 8:58, Soft

Re: [FFmpeg-devel] [PATCH] dnn: add backend options when load the model

2020-08-11 Thread Guo, Yejun
> -Original Message- > From: ffmpeg-devel On Behalf Of > Guo, Yejun > Sent: Friday, August 7, 2020 3:00 PM > To: ffmpeg-devel@ffmpeg.org > Subject: [FFmpeg-devel] [PATCH] dnn: add backend options when load the > model > > different backend might need different options for a better perfo

Re: [FFmpeg-devel] [PATCH v3 1/2] dxva: wait until D3D11 buffer copies are done before submitting them

2020-08-11 Thread Steve Lhomme
Sorry if you seem to know all the answers already, but I don't and so I have to investigate. Last year, I had literally worked this down to death. I followed every slightest hint from countless searches, read through hundreds of discussions, driven because I was unwilling to believe that up-/dow

Re: [FFmpeg-devel] [PATCH v3 1/2] dxva: wait until D3D11 buffer copies are done before submitting them

2020-08-11 Thread Soft Works
> -Original Message- > From: ffmpeg-devel On Behalf Of > Steve Lhomme > Sent: Tuesday, August 11, 2020 12:44 PM > To: ffmpeg-devel@ffmpeg.org > Subject: Re: [FFmpeg-devel] [PATCH v3 1/2] dxva: wait until D3D11 buffer > copies are done before submitting them > Even though the discussion

Re: [FFmpeg-devel] [PATCH] avcodec/bsf: Avoid allocation for AVBSFInternal

2020-08-11 Thread zhilizhao
> On Aug 11, 2020, at 5:48 PM, Nicolas George wrote: > > Andreas Rheinhardt (12020-08-11): >> Imagine the context to only contain elements that require a alignment of >> 4 and the internal structure to require an alignment of eight. Then it >> is perfectly possible for &ctx[1] to only have an a

Re: [FFmpeg-devel] [PATCH v3 1/2] dxva: wait until D3D11 buffer copies are done before submitting them

2020-08-11 Thread Steve Lhomme
On 2020-08-11 13:08, Soft Works wrote: -Original Message- From: ffmpeg-devel On Behalf Of Steve Lhomme Sent: Tuesday, August 11, 2020 12:44 PM To: ffmpeg-devel@ffmpeg.org Subject: Re: [FFmpeg-devel] [PATCH v3 1/2] dxva: wait until D3D11 buffer copies are done before submitting them

Re: [FFmpeg-devel] [PATCH 2/2] tools/target_dec_fuzzer: Adjust threshold for AGM

2020-08-11 Thread Michael Niedermayer
On Tue, Aug 11, 2020 at 10:32:15AM +0200, Michael Niedermayer wrote: > Fixes: Timeout (142sec -> 2sec) > Fixes: > 24426/clusterfuzz-testcase-minimized-ffmpeg_AV_CODEC_ID_AGM_fuzzer-5639724379930624 > > Found-by: continuous fuzzing process > https://github.com/google/oss-fuzz/tree/master/projects

Re: [FFmpeg-devel] [PATCH 1/2] avcodec/agm: Mark fill_pixels() as av_always_inline

2020-08-11 Thread Michael Niedermayer
On Tue, Aug 11, 2020 at 10:45:46AM +0200, Paul B Mahol wrote: > LGTM will apply thx [...] -- Michael GnuPG fingerprint: 9FF2128B147EF6730BADF133611EC787040B0FAB The real ebay dictionary, page 2 "100% positive feedback" - "All either got their money back or didnt complain" "Best seller ever

Re: [FFmpeg-devel] [PATCH] fate: add fate-sub-dvb test

2020-08-11 Thread Clément Bœsch
On Tue, Aug 11, 2020 at 12:06:40AM +0200, Andreas Rheinhardt wrote: > Clément Bœsch: > > On Mon, Aug 10, 2020 at 08:36:13PM +0200, Andreas Rheinhardt wrote: > >> Clément Bœsch: > >>> Source: Videolan sample database (samples/sub/dvbsub) > >>> --- > >>> Can someone upload > >>> https://streams.vide

Re: [FFmpeg-devel] [PATCH 4/6] avfilter/formats: Leave lists' ownership unchanged upon merge failure

2020-08-11 Thread Andreas Rheinhardt
Nicolas George: > Andreas Rheinhardt (12020-08-08): >> ff_merge_formats(), ff_merge_samplerates() and ff_merge_channel_layouts() >> share common semantics: If merging succeeds, a non-NULL pointer is >> returned and both input lists (of type AVFilterFormats resp. >> AVFilterChannelLayouts) are to be

Re: [FFmpeg-devel] [PATCH] avformat/ty: Remove write-only array and variable

2020-08-11 Thread Andreas Rheinhardt
Paul B Mahol: > Make sure this does not break existing files, > I tested with all the samples in https://samples.ffmpeg.org/TiVo/; of course, there was no change in output at all. So I'll apply this. - Andreas > On 8/11/20, Andreas Rheinhardt wrote: >> Up until now, the TiVo demuxer parse an a

Re: [FFmpeg-devel] [PATCH] avcodec/options: Add options for inputing HDR10-related metadata

2020-08-11 Thread lance . lmwang
On Mon, Aug 10, 2020 at 07:54:28PM -0600, Kenny McClive wrote: > Previously, the only way to input the master display and content light > metadata required for HDR10 was through x265-params. Obviously, that only > worked with x265. If you wanted to use a different encoder like nvenc, you > were

Re: [FFmpeg-devel] [PATCH] avformat/mxfenc: Write color metadata to MXF

2020-08-11 Thread Tomas Härdin
tor 2020-08-06 klockan 15:27 +0100 skrev Harry Mallon: > I attach a patch to apply the colour metadata that was read in my > previous commit during mxf encoding. ffmpeg previously wrote the > transfer characteristic only, and not for all supported transfer > curves. Now it will do transfer characte

Re: [FFmpeg-devel] [PATCH] avcodec/bsf: Avoid allocation for AVBSFInternal

2020-08-11 Thread James Almer
On 8/11/2020 6:48 AM, Nicolas George wrote: > Andreas Rheinhardt (12020-08-11): >> Imagine the context to only contain elements that require a alignment of >> 4 and the internal structure to require an alignment of eight. Then it >> is perfectly possible for &ctx[1] to only have an alignment of fou

Re: [FFmpeg-devel] [PATCH v2] doc/developer: origin of tables should be documented.

2020-08-11 Thread Tomas Härdin
lör 2020-08-08 klockan 19:57 +0200 skrev Nicolas George: > diff --git a/doc/developer.texi b/doc/developer.texi > index b33cab0fc7..c3103f31dc 100644 > --- a/doc/developer.texi > +++ b/doc/developer.texi > @@ -216,6 +216,14 @@ please use av_log() instead. > @item > Casts should be used only when

Re: [FFmpeg-devel] [PATCH] avcodec/options: Add options for inputing HDR10-related metadata

2020-08-11 Thread Lynne
Aug 11, 2020, 03:47 by kenny.mccl...@me.com: > Previously, the only way to input the master display and content light > metadata required for HDR10 was through x265-params. Obviously, that only > worked with x265. If you wanted to use a different encoder like nvenc, you > were out of luck. The

Re: [FFmpeg-devel] [FFmpeg-cvslog] avfilter/vf_xfade: check that fps between inputs are valid

2020-08-11 Thread Nicolas George
Paul B Mahol (12020-08-11): > You are only one that now know what is bug and what is not bug and > ignore others decisions? As I am the one who designed and wrote this part of the code, it is not that much of a stretch to assume that I am in the best position to say what is supposed to work and wh

Re: [FFmpeg-devel] [PATCH 2/5] avutil/mpegts_audio_desc_metadata: add helper function for AC3 descriptor 0x6a

2020-08-11 Thread lance . lmwang
On Thu, Aug 06, 2020 at 03:52:33PM +0100, Kieran Kunhya wrote: > On Thu, 6 Aug 2020 at 15:48, wrote: > > > On Thu, Aug 06, 2020 at 07:51:55AM +0100, Kieran Kunhya wrote: > > > > > > > > It's audio descriptor information although it's stored in PSI table, > > but > > > > it'll map to > > > > speci

Re: [FFmpeg-devel] [PATCH] avcodec/mpeg12enc: support mpeg2 encoder const level

2020-08-11 Thread lance . lmwang
On Sun, Aug 02, 2020 at 09:07:04PM +0800, lance.lmw...@gmail.com wrote: > From: Limin Wang > > Signed-off-by: Limin Wang > --- > doc/encoders.texi | 10 ++ > libavcodec/mpeg12enc.c | 6 ++ > 2 files changed, 16 insertions(+) > > diff --git a/doc/encoders.texi b/doc/encoders.t

Re: [FFmpeg-devel] [FFmpeg-cvslog] avfilter/vf_xfade: check that fps between inputs are valid

2020-08-11 Thread Paul B Mahol
On 8/11/20, Nicolas George wrote: > Paul B Mahol (12020-08-11): >> You are only one that now know what is bug and what is not bug and >> ignore others decisions? > > As I am the one who designed and wrote this part of the code, it is not > that much of a stretch to assume that I am in the best pos

Re: [FFmpeg-devel] [FFmpeg-cvslog] avfilter/vf_xfade: check that fps between inputs are valid

2020-08-11 Thread James Almer
On 8/11/2020 11:51 AM, Paul B Mahol wrote: > On 8/11/20, Nicolas George wrote: >> Paul B Mahol (12020-08-11): >>> You are only one that now know what is bug and what is not bug and >>> ignore others decisions? >> >> As I am the one who designed and wrote this part of the code, it is not >> that mu

Re: [FFmpeg-devel] [PATCH 2/5] avcodec/libsvtav1: remove unneeded svt_av1_enc_deinit_handle()

2020-08-11 Thread lance . lmwang
On Sat, Aug 01, 2020 at 10:09:20AM -0300, James Almer wrote: > On 8/1/2020 8:22 AM, lance.lmw...@gmail.com wrote: > > From: Limin Wang > > > > Signed-off-by: Limin Wang > > --- > > libavcodec/libsvtav1.c | 6 -- > > 1 file changed, 6 deletions(-) > > > > diff --git a/libavcodec/libsvtav1.c

Re: [FFmpeg-devel] [FFmpeg-cvslog] avfilter/vf_xfade: check that fps between inputs are valid

2020-08-11 Thread Paul B Mahol
On 8/11/20, James Almer wrote: > On 8/11/2020 11:51 AM, Paul B Mahol wrote: >> On 8/11/20, Nicolas George wrote: >>> Paul B Mahol (12020-08-11): You are only one that now know what is bug and what is not bug and ignore others decisions? >>> >>> As I am the one who designed and wrote thi

[FFmpeg-devel] [PATCH] avfilter/vf_w3fdif: 255 * 256 * 128 -> max

2020-08-11 Thread lance . lmwang
From: Limin Wang max is initialized to ((1 << depth) - 1) * 256 * 128 before. Signed-off-by: Limin Wang --- libavfilter/vf_w3fdif.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/libavfilter/vf_w3fdif.c b/libavfilter/vf_w3fdif.c index 5d64dbd..0f9efbf 100644 --- a/libavfil

Re: [FFmpeg-devel] [PATCH] avfilter/vf_w3fdif: 255 * 256 * 128 -> max

2020-08-11 Thread Paul B Mahol
NAK On 8/11/20, lance.lmw...@gmail.com wrote: > From: Limin Wang > > max is initialized to ((1 << depth) - 1) * 256 * 128 before. > > Signed-off-by: Limin Wang > --- > libavfilter/vf_w3fdif.c | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > > diff --git a/libavfilter/vf_w3fdif.c b/lib

Re: [FFmpeg-devel] [PATCH] avfilter/vf_w3fdif: 255 * 256 * 128 -> max

2020-08-11 Thread lance . lmwang
On Tue, Aug 11, 2020 at 05:24:06PM +0200, Paul B Mahol wrote: > NAK > > On 8/11/20, lance.lmw...@gmail.com wrote: > > From: Limin Wang > > > > max is initialized to ((1 << depth) - 1) * 256 * 128 before. > > > > Signed-off-by: Limin Wang > > --- > > libavfilter/vf_w3fdif.c | 2 +- > > 1 file c

Re: [FFmpeg-devel] [PATCH] avfilter/vf_w3fdif: 255 * 256 * 128 -> max

2020-08-11 Thread Paul B Mahol
On 8/11/20, lance.lmw...@gmail.com wrote: > On Tue, Aug 11, 2020 at 05:24:06PM +0200, Paul B Mahol wrote: >> NAK >> >> On 8/11/20, lance.lmw...@gmail.com wrote: >> > From: Limin Wang >> > >> > max is initialized to ((1 << depth) - 1) * 256 * 128 before. >> > >> > Signed-off-by: Limin Wang >> >

Re: [FFmpeg-devel] [PATCH] avcodec/mpeg12enc: support mpeg2 encoder const level

2020-08-11 Thread Paul B Mahol
On 8/11/20, lance.lmw...@gmail.com wrote: > On Sun, Aug 02, 2020 at 09:07:04PM +0800, lance.lmw...@gmail.com wrote: >> From: Limin Wang >> >> Signed-off-by: Limin Wang >> --- >> doc/encoders.texi | 10 ++ >> libavcodec/mpeg12enc.c | 6 ++ >> 2 files changed, 16 insertions(+) >

Re: [FFmpeg-devel] [PATCH v6 01/22] cbs: Mention all codecs in unit type comment

2020-08-11 Thread Andreas Rheinhardt
Mark Thompson: > --- > libavcodec/cbs.h | 2 ++ > 1 file changed, 2 insertions(+) > > diff --git a/libavcodec/cbs.h b/libavcodec/cbs.h > index e897e348a2..07faf18c5b 100644 > --- a/libavcodec/cbs.h > +++ b/libavcodec/cbs.h > @@ -45,8 +45,10 @@ struct CodedBitstreamType; > /** > * The codec-spe

Re: [FFmpeg-devel] [PATCH] avcodec/mpeg12enc: support mpeg2 encoder const level

2020-08-11 Thread lance . lmwang
On Tue, Aug 11, 2020 at 05:43:24PM +0200, Paul B Mahol wrote: > On 8/11/20, lance.lmw...@gmail.com wrote: > > On Sun, Aug 02, 2020 at 09:07:04PM +0800, lance.lmw...@gmail.com wrote: > >> From: Limin Wang > >> > >> Signed-off-by: Limin Wang > >> --- > >> doc/encoders.texi | 10 ++ >

Re: [FFmpeg-devel] [PATCH] avfilter/vf_w3fdif: 255 * 256 * 128 -> max

2020-08-11 Thread lance . lmwang
On Tue, Aug 11, 2020 at 05:41:58PM +0200, Paul B Mahol wrote: > On 8/11/20, lance.lmw...@gmail.com wrote: > > On Tue, Aug 11, 2020 at 05:24:06PM +0200, Paul B Mahol wrote: > >> NAK > >> > >> On 8/11/20, lance.lmw...@gmail.com wrote: > >> > From: Limin Wang > >> > > >> > max is initialized to ((1

Re: [FFmpeg-devel] [PATCH] avfilter/vf_w3fdif: 255 * 256 * 128 -> max

2020-08-11 Thread Paul B Mahol
On 8/11/20, lance.lmw...@gmail.com wrote: > On Tue, Aug 11, 2020 at 05:41:58PM +0200, Paul B Mahol wrote: >> On 8/11/20, lance.lmw...@gmail.com wrote: >> > On Tue, Aug 11, 2020 at 05:24:06PM +0200, Paul B Mahol wrote: >> >> NAK >> >> >> >> On 8/11/20, lance.lmw...@gmail.com wrote: >> >> > From:

Re: [FFmpeg-devel] [PATCH v3] [RFC] lavc, lavfmt: add FLIF decoding support

2020-08-11 Thread Andreas Rheinhardt
Anamitra Ghorui: > This patch adds support for non animated, animated, non interlaced and > interlaced FLIF images. > Hopefully, all previously mentioned mistakes have been resolved. > However, there are a few things I want to ask: > > * The decoder can accept arbitary packet sizes. How do I make

Re: [FFmpeg-devel] [PATCH v6 15/22] cbs_h264: Add support for frame packing arrangement SEI messages

2020-08-11 Thread Andreas Rheinhardt
Mark Thompson: > --- > libavcodec/cbs_h264.h | 24 > libavcodec/cbs_h2645.c| 1 + > libavcodec/cbs_h264_syntax_template.c | 40 +++ > 3 files changed, 65 insertions(+) > > diff --git a/libavcodec/cbs_h264.h b/libavcodec/cbs

[FFmpeg-devel] Uploaded sample file to 'help'

2020-08-11 Thread Nathanael Newton
This is a video from a genie wireless security camera. They claimed it was encrypted, in fact this was a lie, it's just obfuscated by being renamed to .data Error while decoding stream #0:0: Invalid data found when processing input [h264 @ 028038d69d40] Invalid NAL unit 5, skipping. [h264 @ 00

Re: [FFmpeg-devel] [PATCH v3 2/4] lavc/avs3_paeser: add avs3 parser

2020-08-11 Thread Mark Thompson
Typo "paeser" in the title. On 05/08/2020 17:18, hwr...@126.com wrote: From: hwren Signed-off-by: hbj Signed-off-by: hwren --- libavcodec/Makefile | 1 + libavcodec/avs3_parser.c | 184 +++ libavcodec/parsers.c | 1 + 3 files changed, 18

Re: [FFmpeg-devel] [PATCH v3 4/4] lavc, doc: add libuavs3d video decoder wrapper

2020-08-11 Thread Mark Thompson
On 05/08/2020 17:18, hwr...@126.com wrote: From: hwren Signed-off-by: hbj Signed-off-by: hwren --- Changelog | 1 + configure | 4 + doc/decoders.texi | 21 +++ doc/general.texi | 8 ++ libavcodec/Makefile| 1 + libavcodec/allcodecs.c |

Re: [FFmpeg-devel] [PATCH v2] doc/developer: origin of tables should be documented.

2020-08-11 Thread Alexander Strasser
On 2020-08-09 13:13 +0200, Nicolas George wrote: > Alexander Strasser (12020-08-09): > > >> Andreas: [...] > > > At least the aspect Mark mentioned below. > > > > In general I think it could be worded a bit clearer and easier to grasp. > > Basically it should ask for a short comment in front of no

[FFmpeg-devel] Copying EIT (EPG) to output

2020-08-11 Thread Michal Rybarik
Hello, I am using ffmpeg to transcode video in live stream - both input and output are MPEG-TS multicasts (single program transport stream). Input MPEG-TS contains EIT (EPG) and I want to copy it to the output. Current ffmpeg treats EPG/EIT as private data stream. I have mapped it to be copied to

Re: [FFmpeg-devel] [PATCH v6 19/22] cbs_h265: Add functions to turn HDR metadata into SEI

2020-08-11 Thread Andreas Rheinhardt
Mark Thompson: > --- > libavcodec/Makefile | 2 +- > libavcodec/cbs_h265.c | 100 ++ > libavcodec/cbs_h265.h | 18 > 3 files changed, 119 insertions(+), 1 deletion(-) > create mode 100644 libavcodec/cbs_h265.c > > diff --git a/libavcodec/Make

Re: [FFmpeg-devel] [PATCH] Increasing the max length of control URI to 2048 characters, as urls of some RTSP streams do exceed this limit.

2020-08-11 Thread Yeet
Hi, the patch file I have sent via git-email does not seem to be listed in the patchwork list of patches. Can somebody comment on the file/mimetype, and if everything looks good? First time submitting a patch, please let me know if there is any other thing I need to do. *• **Yiğit (Yeet) Uyan* *

Re: [FFmpeg-devel] [PATCH v6 21/22] h264_metadata_bsf: Refactor the filter function into smaller parts

2020-08-11 Thread Andreas Rheinhardt
First of all: I only looked at the sei_user_data stuff yet. Mark Thompson: > --- > libavcodec/h264_metadata_bsf.c | 443 ++--- > 1 file changed, 242 insertions(+), 201 deletions(-) > > diff --git a/libavcodec/h264_metadata_bsf.c b/libavcodec/h264_metadata_bsf.c > inde

Re: [FFmpeg-devel] [PATCH] avcodec/options: Add options for inputing HDR10-related metadata

2020-08-11 Thread Kenny McClive
Lynne, Please see my responses below. I am not opposed to taking a different approach, but I could use some guidance on how to go about that. This is my first contribution to FFmpeg, and I struggled to get this done. > For HDR transcoding/tonemapping we rely purely on frame side data atm. How

[FFmpeg-devel] [PATCH] libavutil/hwcontext_d3d11va: missing unlock if d3d11va_create_staging_texture failed

2020-08-11 Thread xie.kunming
Signed-off-by: xie.kunming --- libavutil/hwcontext_d3d11va.c | 4 +++- 1 file changed, 3 insertions(+), 1 deletion(-) diff --git a/libavutil/hwcontext_d3d11va.c b/libavutil/hwcontext_d3d11va.c index c8ae58f908..4524b01c20 100644 --- a/libavutil/hwcontext_d3d11va.c +++ b/libavutil/hwcontext_d3d11

Re: [FFmpeg-devel] [PATCH V2 0/3] avformat/av1dec: add low overhead obu demux

2020-08-11 Thread Xu, Guangxin
Hi James, All issues addressed. Could you help review it again? Thanks for you great help on this. > -Original Message- > From: Xu, Guangxin > Sent: Monday, August 10, 2020 5:34 PM > To: ffmpeg-devel@ffmpeg.org; jamr...@gmail.com > Cc: Xu, Guangxin > Subject: [PATCH V2 0/3] avformat/av1d

Re: [FFmpeg-devel] [PATCH] dnn_backend_native_layer_mathunary: add round support

2020-08-11 Thread Guo, Yejun
> -Original Message- > From: ffmpeg-devel On Behalf Of Mingyu > Yin > Sent: 2020年8月10日 19:06 > To: ffmpeg-devel@ffmpeg.org > Subject: [FFmpeg-devel] [PATCH] dnn_backend_native_layer_mathunary: add > round support > > Signed-off-by: Mingyu Yin > --- > libavfilter/dnn/dnn_backend_native