Re: [FFmpeg-devel] [PATCH 3/4] fftools/ffmpeg: support variable dimension encode

2019-07-29 Thread Fu, Linjie
> -Original Message- > From: ffmpeg-devel [mailto:ffmpeg-devel-boun...@ffmpeg.org] On Behalf > Of James Almer > Sent: Tuesday, July 30, 2019 00:44 > To: ffmpeg-devel@ffmpeg.org > Subject: Re: [FFmpeg-devel] [PATCH 3/4] fftools/ffmpeg: support variable > dimension encode > > On 7/29/2019

[FFmpeg-devel] [PATCH, v2 3/4] fftools/ffmpeg: support variable dimension encode

2019-07-29 Thread Linjie Fu
Flush encoders when dimension change happens. Signed-off-by: Linjie Fu --- fftools/ffmpeg.c | 14 ++ 1 file changed, 14 insertions(+) diff --git a/fftools/ffmpeg.c b/fftools/ffmpeg.c index 5d52430..cb3adb2 100644 --- a/fftools/ffmpeg.c +++ b/fftools/ffmpeg.c @@ -130,6 +130,7 @@

[FFmpeg-devel] [PATCH, v2 2/4] avc/avcodec: add AV_CODEC_CAP_VARIABLE_DIMENSIONS flag

2019-07-29 Thread Linjie Fu
Add AV_CODEC_CAP_VARIABLE_DIMENSIONS flag to indicate whether encoder supports variable dimension encoding. Signed-off-by: Linjie Fu --- [v2]: update API changes. doc/APIchanges | 3 +++ fftools/cmdutils.c | 2 ++ libavcodec/avcodec.h | 5 + libavcodec/rawenc.c | 1 +

Re: [FFmpeg-devel] avcodec: add a WavPack DSD decoder

2019-07-29 Thread David Bryant
On 7/28/19 10:27 AM, Paul B Mahol wrote: > On Sun, Jul 28, 2019 at 6:02 AM David Bryant wrote: > >> On 7/24/19 12:26 AM, Paul B Mahol wrote: >>> On 7/23/19, David Bryant wrote: On 7/23/19 12:47 AM, Paul B Mahol wrote: > On 7/23/19, David Bryant wrote: >> On 7/21/19 11:23 PM, Paul B

Re: [FFmpeg-devel] [PATCH] Extract QP from h264 encoded videos

2019-07-29 Thread James Almer
On 7/29/2019 11:19 PM, Juan De León wrote: > On Mon, Jul 29, 2019 at 12:48 PM Mark Thompson wrote: > >> This doesn't belong in the commit message. >> >> What does belong here would be some commentary on why you want this >> feature. >> > Here is the, somewhat outdated, design document, this

[FFmpeg-devel] [PATCH] Extract QP from h264 encoded videos

2019-07-29 Thread Juan De León
I tried to fix all you suggested, please have a look and let me know what you think. design doc: https://docs.google.com/document/d/1WClt3EqhjwdGXhEw386O0wfn3IBQ1Ib-_5emVM1gbnA/edit?usp=sharing Signed-off-by: Juan De León --- libavutil/Makefile | 2 + libavutil/frame.h

Re: [FFmpeg-devel] [PATCH] Extract QP from h264 encoded videos

2019-07-29 Thread Juan De León
On Mon, Jul 29, 2019 at 12:48 PM Mark Thompson wrote: > This doesn't belong in the commit message. > > What does belong here would be some commentary on why you want this > feature. > Here is the, somewhat outdated, design document, this should explain it.

Re: [FFmpeg-devel] [PATCH V2] fate: add a case for ticket #3229

2019-07-29 Thread Li, Zhong
> From: ffmpeg-devel [mailto:ffmpeg-devel-boun...@ffmpeg.org] On Behalf > Of Michael Niedermayer > Sent: Tuesday, July 30, 2019 4:28 AM > To: FFmpeg development discussions and patches > > Subject: Re: [FFmpeg-devel] [PATCH V2] fate: add a case for ticket #3229 > > On Mon, Jul 29, 2019 at

[FFmpeg-devel] [PATCH 2/2] convert_from_tensorflow.py: support conv2d with dilation

2019-07-29 Thread Guo, Yejun
conv2d with dilation > 1 generates tens of nodes in graph, it is not easy to parse each node one by one, so we do special tricks to parse the conv2d layer. Signed-off-by: Guo, Yejun --- tools/python/convert_from_tensorflow.py | 80 - 1 file changed, 59

[FFmpeg-devel] [PATCH 1/2] convert_from_tensorflow.py: add option to dump graph for visualization in tensorboard

2019-07-29 Thread Guo, Yejun
Signed-off-by: Guo, Yejun --- tools/python/convert.py | 6 +- tools/python/convert_from_tensorflow.py | 13 +++-- 2 files changed, 12 insertions(+), 7 deletions(-) diff --git a/tools/python/convert.py b/tools/python/convert.py index 662b429..64cf76b 100644 ---

[FFmpeg-devel] [PATCH] avcodec/scpr: Use av_memcpy_backptr() in type 17 and 33

2019-07-29 Thread Michael Niedermayer
This makes the changed code-path faster. Change not tested except with the fuzzer testcase as I found no other testcase. Improves: Timeout (136sec -> 74sec) Improves: 16040/clusterfuzz-testcase-minimized-ffmpeg_AV_CODEC_ID_SCPR_fuzzer-5705876062601216 Found-by: continuous fuzzing process

Re: [FFmpeg-devel] [PATCH 5/7] cbs_mpeg2: Fix parsing the last unit

2019-07-29 Thread Mark Thompson
On 29/07/2019 20:56, Andreas Rheinhardt wrote: > There is one way to find out if avpriv_find_start_code has found a start > code or not: One has to check whether the state variable contains a > start code, i.e. whether the three most serious bytes are 0x00 00 01. "most significant bytes" >

Re: [FFmpeg-devel] [PATCH] avcodec/rl2: set dimensions

2019-07-29 Thread Jean-Baptiste Kempf
Yo, On Tue, Jul 23, 2019, at 04:42, Lynne wrote: > Jul 23, 2019, 12:00 AM by mich...@niedermayer.cc: > >> Moreover, if neither the codec, nor container can change the resolution, > >> then how does that make anything different? > > > >> Please, stop it with those patches. It takes energy and

Re: [FFmpeg-devel] [PATCH V2] fate: add a case for ticket #3229

2019-07-29 Thread Michael Niedermayer
On Mon, Jul 29, 2019 at 06:20:39PM +0800, Zhong Li wrote: > Signed-off-by: Zhong Li > --- > https://patchwork.ffmpeg.org/patch/13725/ introduces a regression but not > found by fate, so add it. > Test clip produced by: > ffmpeg -i tickets/3229/bad.avi -vframes 6 -c:v copy >

[FFmpeg-devel] [PATCH 4/7] cbs_mpeg2: Rearrange start code search

2019-07-29 Thread Andreas Rheinhardt
1. Currently, cbs_mpeg2_split_fragment uses essentially three variables to hold the start code values found by avpriv_find_start_code. By rearranging the code, one of them can be omitted. 2. The return value of avpriv_find_start_code points to the byte after the byte containing the start code

[FFmpeg-devel] [PATCH 5/7] cbs_mpeg2: Fix parsing the last unit

2019-07-29 Thread Andreas Rheinhardt
There is one way to find out if avpriv_find_start_code has found a start code or not: One has to check whether the state variable contains a start code, i.e. whether the three most serious bytes are 0x00 00 01. Checking for whether the return value is the end of the designated buffer is not

[FFmpeg-devel] [PATCH 2/7] cbs_h264: Improve adding SEI messages

2019-07-29 Thread Andreas Rheinhardt
Up until now, if an SEI messages was to be added to a fragment, it was tried to add said SEI message to the first SEI NAL unit of the fragment and if this SEI NAL unit already contained H264_NAL_SEI SEI messages (an arbitrary limit imposed by cbs_h264), adding failed; if there was no SEI NAL unit,

[FFmpeg-devel] [PATCH 3/7] cbs_mpeg2: Decompose Sequence End

2019-07-29 Thread Andreas Rheinhardt
Sequence End units (or actually, sequence_end_codes) have up until now not been decomposed; in fact due to a bug in cbs_mpeg2_split_fragment they have mostly been treated as part of the preceding unit. So implement decomposing them as preparation for fixing said bug. Signed-off-by: Andreas

[FFmpeg-devel] [PATCH 6/7] cbs_mpeg2: Remove zero byte stuffing

2019-07-29 Thread Andreas Rheinhardt
Remove superfluous trailing zeros from slices. Because MPEG-2 slices can end with zero bits a safe number of trailing zero bits is always kept. More explicitly, 6 + max{f_code[i][1] - 1, i = 0,1, f_code[i][1] != 0xf} is an upper bound for the number of possible trailing zeros that are part of the

[FFmpeg-devel] [PATCH 7/7] cbs_mpeg2: Drop fragments containing zero-sized units

2019-07-29 Thread Andreas Rheinhardt
They are invalid and can be easily detected, so discard them. Signed-off-by: Andreas Rheinhardt --- libavcodec/cbs_mpeg2.c | 6 ++ 1 file changed, 6 insertions(+) diff --git a/libavcodec/cbs_mpeg2.c b/libavcodec/cbs_mpeg2.c index 559793dc98..eb0e2c7ba9 100644 --- a/libavcodec/cbs_mpeg2.c

[FFmpeg-devel] [PATCH 1/7] cbs: Don't set AVBuffer's opaque

2019-07-29 Thread Andreas Rheinhardt
cbs is currently inconsistent regarding the opaque field that can be used as a special argument to av_buffer_create in order to be used during freeing the buffer: ff_cbs_alloc_unit_content and all the free functions used name this parameter as if it should contain a pointer to the unit whose

Re: [FFmpeg-devel] [PATCH] Extract QP from h264 encoded videos

2019-07-29 Thread Mark Thompson
On 29/07/2019 19:09, Juan De León wrote: > Changes to libavcodec, hope this addresses all your comments. > Cheers. This doesn't belong in the commit message. What does belong here would be some commentary on why you want this feature. > Signed-off-by: Juan De León > --- > libavutil/Makefile

Re: [FFmpeg-devel] [PATCH 2/4] avc/avcodec: add AV_CODEC_CAP_VARIABLE_DIMENSIONS flag

2019-07-29 Thread Michael Niedermayer
On Mon, Jul 29, 2019 at 10:43:52PM +0800, Linjie Fu wrote: > Add AV_CODEC_CAP_VARIABLE_DIMENSIONS flag to indicate whether encoder > supports variable dimension encoding. > > Signed-off-by: Linjie Fu > --- > fftools/cmdutils.c | 2 ++ > libavcodec/avcodec.h | 5 + > libavcodec/rawenc.c |

Re: [FFmpeg-devel] [PATCH] Extract QP from h264 encoded videos

2019-07-29 Thread Juan De León
Thank you for the feedback Andrey, I will fix it ASAP. On Mon, Jul 29, 2019 at 12:11 PM Andreas Håkon wrote: > Interesting patch. But I have a question: > Could you implement the same thing when using the hardware decoders? I believe this might be in the scope of my project. I'm not too

Re: [FFmpeg-devel] libavcodec: add timer bitstream filter

2019-07-29 Thread Paul B Mahol
Documentation for bsf does document single option, thus is incomplete. On Mon, Jul 29, 2019 at 9:14 PM Andreas Håkon wrote: > Ping > > ‐‐‐ Original Message ‐‐‐ > On Wednesday, 24 de July de 2019 9:16, Andreas Håkon < > andreas.ha...@protonmail.com> wrote: > > > Hi, > > > > ‐‐‐

Re: [FFmpeg-devel] libavcodec: add timer bitstream filter

2019-07-29 Thread Andreas Håkon
Ping ‐‐‐ Original Message ‐‐‐ On Wednesday, 24 de July de 2019 9:16, Andreas Håkon wrote: > Hi, > > ‐‐‐ Original Message ‐‐‐ > On Friday, 28 de June de 2019 10:44, Andreas Håkon > wrote: > >> Hi, >> This is a refined version of the initial version for the new “timer” BSF. >>

Re: [FFmpeg-devel] [PATCH] Extract QP from h264 encoded videos

2019-07-29 Thread Andreas Håkon
Hi Juan, ‐‐‐ Original Message ‐‐‐ On Monday, 29 de July de 2019 20:12, Juan De León wrote: > Here is the second patch, I sent the first one twice accidentaly. > First patch is libavutil, this patch is libavcodec. > Interesting patch. But I have a question: Could you implement the

Re: [FFmpeg-devel] [PATCH] Extract QP from h264 encoded videos

2019-07-29 Thread Andrey Semashev
Just a few random comments. Disclaimer: I'm not a maintainer. On 7/29/19 9:09 PM, Juan De León wrote: Changes to libavcodec, hope this addresses all your comments. Cheers. Signed-off-by: Juan De León --- libavutil/Makefile | 2 + libavutil/frame.h | 6 ++

Re: [FFmpeg-devel] [PATCH 1/2] cbs_h264: Fix missing inferred colour description fields

2019-07-29 Thread Mark Thompson
On 29/07/2019 00:31, Andreas Rheinhardt wrote: > Mark Thompson: >> With video_signal_type_present_flag set but colour_description_present_flag >> unset the colour fields would not have had their correct values inferred. >> --- >> libavcodec/cbs_h264_syntax_template.c | 4 >> 1 file changed,

Re: [FFmpeg-devel] [PATCH] avcodec/arm/sbcenc: save callee preserved vfp registers

2019-07-29 Thread Reimar Döffinger
Seems sensible to me, though extra points if you or someone has numbers on performance impact. To know whether it would be worthwhile to check if it can be optimized... On 28.07.2019, at 23:46, James Cowgill wrote: > When compiling FFmpeg with GCC-9, some very random segfaults were > observed

Re: [FFmpeg-devel] [PATCH v3] avutil/mips: Avoid instruction exception caused by gssqc1/gslqc1.

2019-07-29 Thread Reimar Döffinger
On 29.07.2019, at 11:54, "Shiyou Yin" wrote: >> > DECLARE_ALIGNED is defined in ' libavutil/mem.h ' and related to compiler. No > matter mips or x86, > it's definition is ' #define DECLARE_ALIGNED(n,t,v) t __attribute__ > ((aligned (n))) v' when build > with gcc or clang (Specific

[FFmpeg-devel] [PATCH] Extract QP from h264 encoded videos

2019-07-29 Thread Juan De León
Signed-off-by: Juan De León --- libavutil/Makefile | 2 + libavutil/frame.h | 6 ++ libavutil/quantization_params.c | 41 libavutil/quantization_params.h | 106 4 files changed, 155 insertions(+) create mode 100644

[FFmpeg-devel] [PATCH] Extract QP from h264 encoded videos

2019-07-29 Thread Juan De León
Here is the second patch, I sent the first one twice accidentaly. First patch is libavutil, this patch is libavcodec. Signed-off-by: Juan De León --- libavcodec/avcodec.h | 1 + libavcodec/h264dec.c | 44 ++ libavcodec/options_table.h | 1 + 3

[FFmpeg-devel] [PATCH] Extract QP from h264 encoded videos

2019-07-29 Thread Juan De León
Changes to libavcodec, hope this addresses all your comments. Cheers. Signed-off-by: Juan De León --- libavutil/Makefile | 2 + libavutil/frame.h | 6 ++ libavutil/quantization_params.c | 41 libavutil/quantization_params.h | 106

Re: [FFmpeg-devel] [PATCH 2/3] avcodec/eatqi: Check for minimum frame size

2019-07-29 Thread Michael Niedermayer
On Mon, Jul 29, 2019 at 08:47:07AM +0200, Paul B Mahol wrote: > LGTM will apply thanks [...] -- Michael GnuPG fingerprint: 9FF2128B147EF6730BADF133611EC787040B0FAB Any man who breaks a law that conscience tells him is unjust and willingly accepts the penalty by staying in jail in order

Re: [FFmpeg-devel] [PATCH 1/3] avcodec/eatgv: Check remaining size after the keyframe header

2019-07-29 Thread Michael Niedermayer
On Mon, Jul 29, 2019 at 08:49:40AM +0200, Paul B Mahol wrote: > Actually remove log message completely. ok, will apply with that change > It is invalid because it is warning > while you return error immediately. Thats true for most of the error messages in the file. Thanks [...] -- Michael

Re: [FFmpeg-devel] [PATCH 3/4] fftools/ffmpeg: support variable dimension encode

2019-07-29 Thread James Almer
On 7/29/2019 11:44 AM, Linjie Fu wrote: > Flush encoders when dimension change happens, reset draining to resume > encode. > > If encoder doesn't support variable dimension, stop encoding and > report errors. > > Signed-off-by: Linjie Fu > --- > fftools/ffmpeg.c| 13 + >

Re: [FFmpeg-devel] [FFmpeg-cvslog] avcodec/cfhd: add bayer support

2019-07-29 Thread Paul B Mahol
On Mon, Jul 29, 2019 at 6:22 PM Carl Eugen Hoyos wrote: > Am Mo., 29. Juli 2019 um 18:07 Uhr schrieb Paul B Mahol >: > > > > On Mon, Jul 29, 2019 at 6:05 PM Carl Eugen Hoyos > wrote: > > > > > Am Mo., 29. Juli 2019 um 16:35 Uhr schrieb Paul B Mahol < > g...@videolan.org > > > >: > > > > > > >

Re: [FFmpeg-devel] [FFmpeg-cvslog] avcodec/cfhd: add bayer support

2019-07-29 Thread Carl Eugen Hoyos
Am Mo., 29. Juli 2019 um 18:07 Uhr schrieb Paul B Mahol : > > On Mon, Jul 29, 2019 at 6:05 PM Carl Eugen Hoyos wrote: > > > Am Mo., 29. Juli 2019 um 16:35 Uhr schrieb Paul B Mahol > >: > > > > > +static inline void process_bayer(AVFrame *frame) > > > +{ > > > +const int linesize =

Re: [FFmpeg-devel] [FFmpeg-cvslog] avcodec/cfhd: add bayer support

2019-07-29 Thread Paul B Mahol
On Mon, Jul 29, 2019 at 6:05 PM Carl Eugen Hoyos wrote: > Am Mo., 29. Juli 2019 um 16:35 Uhr schrieb Paul B Mahol >: > > > +static inline void process_bayer(AVFrame *frame) > > +{ > > +const int linesize = frame->linesize[0]; > > +uint16_t *r = (uint16_t *)frame->data[0]; > > +

Re: [FFmpeg-devel] [FFmpeg-cvslog] avcodec/cfhd: add bayer support

2019-07-29 Thread Carl Eugen Hoyos
Am Mo., 29. Juli 2019 um 16:35 Uhr schrieb Paul B Mahol : > +static inline void process_bayer(AVFrame *frame) > +{ > +const int linesize = frame->linesize[0]; > +uint16_t *r = (uint16_t *)frame->data[0]; > +uint16_t *g1 = (uint16_t *)(frame->data[0] + 2); > +uint16_t *g2 =

Re: [FFmpeg-devel] [PATCH V2 1/3] dnn: add layer pad which is equivalent to tf.pad

2019-07-29 Thread Pedro Arthur
LGTM. Pushed, thanks! Em dom, 28 de jul de 2019 às 22:59, Guo, Yejun escreveu: > > the reason to add this layer first is that vf_sr uses it in its > tensorflow model, and the next plan is to update the python script > to convert tf.pad into native model. > > Signed-off-by: Guo, Yejun > --- >

Re: [FFmpeg-devel] [PATCH V2 3/3] dnn: convert tf.pad to native model in python script, and load/execute it in the c code.

2019-07-29 Thread Pedro Arthur
LGTM. Pushed, thanks! Em dom, 28 de jul de 2019 às 23:00, Guo, Yejun escreveu: > > since tf.pad is enabled, the conv2d(valid) changes back to its original > behavior. > > Signed-off-by: Guo, Yejun > --- > libavfilter/dnn/dnn_backend_native.c| 35 > + >

Re: [FFmpeg-devel] [PATCH V2 2/3] fate: add unit test for dnn-layer-pad

2019-07-29 Thread Pedro Arthur
LGTM. Pushed, thanks! Em dom, 28 de jul de 2019 às 22:59, Guo, Yejun escreveu: > > 'make fate-dnn-layer-pad' to run the test > > Signed-off-by: Guo, Yejun > --- > tests/Makefile | 5 +- > tests/dnn/Makefile | 11 +++ > tests/dnn/dnn-layer-pad-test.c | 203 >

Re: [FFmpeg-devel] [PATCH, v4 1/2] fftools/ffmpeg_filter: add -autoscale to disable/enable the default scale

2019-07-29 Thread Fu, Linjie
> -Original Message- > From: ffmpeg-devel [mailto:ffmpeg-devel-boun...@ffmpeg.org] On Behalf > Of Michael Niedermayer > Sent: Monday, July 29, 2019 00:48 > To: FFmpeg development discussions and patches de...@ffmpeg.org> > Subject: Re: [FFmpeg-devel] [PATCH, v4 1/2] fftools/ffmpeg_filter:

[FFmpeg-devel] [PATCH 4/4] lavc/libvpxenc: add dynamic resolution encode support for libvpx

2019-07-29 Thread Linjie Fu
According to spec, libvpx should support dynamic resolution changes. Add dynamic resolution encoding support in libvpx. Format change should also be supported, but I didn't test it so just leave it open. cmdline: ffmpeg -noautoscale -y -i ./reinit-large_420_8-to-small_420_8.h264 -pix_fmt

[FFmpeg-devel] [PATCH 3/4] fftools/ffmpeg: support variable dimension encode

2019-07-29 Thread Linjie Fu
Flush encoders when dimension change happens, reset draining to resume encode. If encoder doesn't support variable dimension, stop encoding and report errors. Signed-off-by: Linjie Fu --- fftools/ffmpeg.c| 13 + libavcodec/encode.c | 4 2 files changed, 17 insertions(+)

[FFmpeg-devel] [PATCH 2/4] avc/avcodec: add AV_CODEC_CAP_VARIABLE_DIMENSIONS flag

2019-07-29 Thread Linjie Fu
Add AV_CODEC_CAP_VARIABLE_DIMENSIONS flag to indicate whether encoder supports variable dimension encoding. Signed-off-by: Linjie Fu --- fftools/cmdutils.c | 2 ++ libavcodec/avcodec.h | 5 + libavcodec/rawenc.c | 1 + libavcodec/version.h | 2 +- 4 files changed, 9 insertions(+), 1

[FFmpeg-devel] [PATCH, v4 1/4] fftools/ffmpeg_filter: add -autoscale to disable/enable the default scale

2019-07-29 Thread Linjie Fu
Currently, ffmpeg inserts scale filter by default in the filter graph to force the whole decoded stream to scale into the same size with the first frame. It's not quite make sense in resolution changing cases if user wants the rawvideo without any scale. Using autoscale/noautoscale to indicate

Re: [FFmpeg-devel] [PATCH] avcodec/cfhd: add bayer support

2019-07-29 Thread Paul B Mahol
Hi, Fixed possible issue with invalid files. New patch attached. > 0001-avcodec-cfhd-add-bayer-support.patch Description: Binary data ___ ffmpeg-devel mailing list ffmpeg-devel@ffmpeg.org https://ffmpeg.org/mailman/listinfo/ffmpeg-devel To

Re: [FFmpeg-devel] [PATCH] fate: add a case for ticket #3229

2019-07-29 Thread Li, Zhong
> From: ffmpeg-devel [mailto:ffmpeg-devel-boun...@ffmpeg.org] On Behalf > Of Michael Niedermayer > Sent: Monday, July 15, 2019 2:56 AM > To: FFmpeg development discussions and patches > > Subject: Re: [FFmpeg-devel] [PATCH] fate: add a case for ticket #3229 > > On Sun, Jul 14, 2019 at 06:16:19PM

Re: [FFmpeg-devel] libavformat/mpegtsenc: fix incorrect PCR with multiple programs [v3]

2019-07-29 Thread Andreas Håkon
Hi Andriy, A new updated version here: https://patchwork.ffmpeg.org/patch/14121/ ‐‐‐ Original Message ‐‐‐ On Monday, 29 de July de 2019 8:46, Andriy Gelman wrote: > > The MPEG-TS muxer has a serious bug related to the PCR pid selection. > > This bug appears when more than one program

[FFmpeg-devel] libavformat/mpegtsenc: fix incorrect PCR with multiple programs [v4]

2019-07-29 Thread Andreas Håkon
Hi, An updated version that fixes all previous bugs: https://patchwork.ffmpeg.org/patch/14109/ https://patchwork.ffmpeg.org/patch/14099/ https://patchwork.ffmpeg.org/patch/14036/ It passes the tests pointed by Michael Niedermayer (Sample_cut.ts) and Andriy Gelman (day_flight.mpg). I hope this

[FFmpeg-devel] [PATCH V2] fate: add a case for ticket #3229

2019-07-29 Thread Zhong Li
Signed-off-by: Zhong Li --- https://patchwork.ffmpeg.org/patch/13725/ introduces a regression but not found by fate, so add it. Test clip produced by: ffmpeg -i tickets/3229/bad.avi -vframes 6 -c:v copy /fate-suite/mjpeg/mjpeg_field_order.avi tests/fate/video.mak| 3 +++

Re: [FFmpeg-devel] [PATCH] avcodec/dsddec: add slice threading support

2019-07-29 Thread Peter Ross
On Sun, Jul 28, 2019 at 11:35:45PM +0200, Paul B Mahol wrote: > Hi, > > patch attached. > From ddfb95d32213406bc68a2803c13c7cfb2d99ee6a Mon Sep 17 00:00:00 2001 > From: Paul B Mahol > Date: Sun, 28 Jul 2019 22:27:34 +0200 > Subject: [PATCH 2/2] avcodec/dsddec: add slice threading support > >

Re: [FFmpeg-devel] [PATCH] avcodec/dstdec: add slice threading support

2019-07-29 Thread Peter Ross
On Sun, Jul 28, 2019 at 11:34:51PM +0200, Paul B Mahol wrote: > Hi, > > patch attached. > From b0d0e0e025fc696270ef59a451d9ee93e2ba27b5 Mon Sep 17 00:00:00 2001 > From: Paul B Mahol > Date: Sun, 28 Jul 2019 22:27:10 +0200 > Subject: [PATCH 1/2] avcodec/dstdec: add slice threading support > >

Re: [FFmpeg-devel] [PATCH v3] avutil/mips: Avoid instruction exception caused by gssqc1/gslqc1.

2019-07-29 Thread Shiyou Yin
>-Original Message- >From: ffmpeg-devel-boun...@ffmpeg.org [mailto:ffmpeg-devel-boun...@ffmpeg.org] >On Behalf Of >Reimar D?ffinger >Sent: Sunday, July 28, 2019 6:37 AM >To: FFmpeg development discussions and patches >Subject: Re: [FFmpeg-devel] [PATCH v3] avutil/mips: Avoid instruction

[FFmpeg-devel] [PATCH v4] avutil/mips: Avoid instruction exception caused by gssqc1/gslqc1.

2019-07-29 Thread Shiyou Yin
Ensure the address accesed by gssqc1/gslqc1 are 16-byte aligned. --- libavcodec/mips/simple_idct_mmi.c | 2 +- libavutil/mips/mmiutils.h | 2 +- 2 files changed, 2 insertions(+), 2 deletions(-) diff --git a/libavcodec/mips/simple_idct_mmi.c b/libavcodec/mips/simple_idct_mmi.c index

Re: [FFmpeg-devel] [PATCH 2/2] h264_metadata: Support overscan_appropriate_flag

2019-07-29 Thread Andreas Rheinhardt
Mark Thompson: > On 29/07/2019 00:40, Andreas Rheinhardt wrote: >> Mark Thompson: >>> Fixes #8041. >>> --- >>> Requested in . >>> >>> >>> doc/bitstream_filters.texi | 4 >>> libavcodec/h264_metadata_bsf.c | 11 +++ >>> 2 files changed, 15

Re: [FFmpeg-devel] [PATCH 2/2] h264_metadata: Support overscan_appropriate_flag

2019-07-29 Thread Mark Thompson
On 29/07/2019 00:40, Andreas Rheinhardt wrote: > Mark Thompson: >> Fixes #8041. >> --- >> Requested in . >> >> >> doc/bitstream_filters.texi | 4 >> libavcodec/h264_metadata_bsf.c | 11 +++ >> 2 files changed, 15 insertions(+) >> >> diff

Re: [FFmpeg-devel] [PATCH 25/31] cbs: Add function to update video codec parameters

2019-07-29 Thread Mark Thompson
On 29/07/2019 02:33, Andreas Rheinhardt wrote: > Mark Thompson: >> On 20/06/2019 00:45, Andreas Rheinhardt wrote: >>> If any of the *_metadata filter based upon cbs currently updates a video >>> codec parameter like color information, the AVCodecParameters are not >>> updated accordingly, so that

Re: [FFmpeg-devel] [PATCH] avfilter/fade: don't allow nb_frames == 0

2019-07-29 Thread Gyan
On 29-07-2019 12:16 PM, Paul B Mahol wrote: LGTM Pushed as 43891ea8ab286a08e731ebac78123c4f1b1d35ca Thanks, Gyan ___ ffmpeg-devel mailing list ffmpeg-devel@ffmpeg.org https://ffmpeg.org/mailman/listinfo/ffmpeg-devel To unsubscribe, visit link

Re: [FFmpeg-devel] [PATCH 2/3] avcodec/eatqi: Check for minimum frame size

2019-07-29 Thread Paul B Mahol
LGTM On Mon, Jul 29, 2019 at 1:11 AM Michael Niedermayer wrote: > The minimum header is 8 bytes, the smallest bitstream that is passed to > the MB decode code is 4 bytes > > Fixes: Timeout (35sec -> 18sec) > Fixes: >

Re: [FFmpeg-devel] [PATCH] avfilter/fade: don't allow nb_frames == 0

2019-07-29 Thread Paul B Mahol
LGTM On Mon, Jul 29, 2019 at 7:30 AM Gyan wrote: > > Fixes crash when user inputs a sub-unit fractional number for nb_frames. > > Gyan > ___ > ffmpeg-devel mailing list > ffmpeg-devel@ffmpeg.org > https://ffmpeg.org/mailman/listinfo/ffmpeg-devel > >

Re: [FFmpeg-devel] libavformat/mpegtsenc: fix incorrect PCR with multiple programs [v3]

2019-07-29 Thread Andriy Gelman
Andreas, On Sun, 28. Jul 19:07, Andreas Håkon wrote: > Hi, > > This last version fixes the small bug discovered by Michael Niedermayer: > https://patchwork.ffmpeg.org/patch/14099/ > > This version is finally clean. > > Regards. > A.H. > > --- > From 08565b81aa2b6d75043e5e984da143480891c3b0

Re: [FFmpeg-devel] [PATCH 1/3] avcodec/eatgv: Check remaining size after the keyframe header

2019-07-29 Thread Paul B Mahol
Actually remove log message completely. It is invalid because it is warning while you return error immediately. On Mon, Jul 29, 2019 at 8:48 AM Paul B Mahol wrote: > LGTM > > On Mon, Jul 29, 2019 at 1:11 AM Michael Niedermayer > wrote: > >> The minimal size which unpack() will not fail on is 5

Re: [FFmpeg-devel] [PATCH v15 1/2] lavc/svt_hevc: add libsvt hevc encoder wrapper

2019-07-29 Thread Sun, Jing A
The just updated one fixed such typo: .pix_fmts = (const enum AVPixelFormat[]){ AV_PIX_FMT_YUV420P, AV_PIX_FMT_YUV420P10, AV_PIX_FMT_YUV422P, -

Re: [FFmpeg-devel] [PATCH 1/3] avcodec/eatgv: Check remaining size after the keyframe header

2019-07-29 Thread Paul B Mahol
LGTM On Mon, Jul 29, 2019 at 1:11 AM Michael Niedermayer wrote: > The minimal size which unpack() will not fail on is 5 bytes > Fixes: Timeout (14sec -> 77ms) (testcase 15508) > Fixes: > 15508/clusterfuzz-testcase-minimized-ffmpeg_AV_CODEC_ID_EATGV_fuzzer-5700053513011200 > Fixes: >

[FFmpeg-devel] [PATCH v15 1/2] lavc/svt_hevc: add libsvt hevc encoder wrapper

2019-07-29 Thread Jing Sun
Signed-off-by: Zhengxu Huang Signed-off-by: Hassene Tmar Signed-off-by: Jun Zhao Signed-off-by: Jing Sun --- configure| 4 + libavcodec/Makefile | 1 + libavcodec/allcodecs.c | 1 + libavcodec/libsvt_hevc.c | 501 +++