Re: [FFmpeg-devel] [Patch] Download dash content with byte range info

2017-11-14 Thread Steven Liu
2017-11-15 10:26 GMT+08:00 Colin NG : > Please ignore the previous "patch" email. > > > > > ___ > ffmpeg-devel mailing list > ffmpeg-devel@ffmpeg.org > http://ffmpeg.org/mailman/listinfo/ffmpeg-devel > diff --git

Re: [FFmpeg-devel] libavcodec/als: remove check for predictor order of a block

2017-11-14 Thread Umair Khan
Hi, On Wed, Nov 15, 2017 at 4:20 AM, Carl Eugen Hoyos wrote: > 2017-11-14 23:48 GMT+01:00 Carl Eugen Hoyos : >> 2017-11-13 21:07 GMT+01:00 Thilo Borgmann : >>> Am 13.11.17 um 21:06 schrieb Thilo Borgmann: Am 13.11.17 um 20:02

[FFmpeg-devel] [PATCH 08/11] avformat/replaygain: use FFABS to instead of abs

2017-11-14 Thread Steven Liu
Signed-off-by: Steven Liu --- libavformat/replaygain.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/libavformat/replaygain.c b/libavformat/replaygain.c index 707d3cd4f1..0d1fb7d11d 100644 --- a/libavformat/replaygain.c +++ b/libavformat/replaygain.c

[FFmpeg-devel] [PATCH 10/11] avformat/utils: use FFABS to instead of fabs

2017-11-14 Thread Steven Liu
Signed-off-by: Steven Liu --- libavformat/utils.c | 8 1 file changed, 4 insertions(+), 4 deletions(-) diff --git a/libavformat/utils.c b/libavformat/utils.c index 87887063be..c2c9afbbfd 100644 --- a/libavformat/utils.c +++ b/libavformat/utils.c @@ -3359,7 +3359,7

[FFmpeg-devel] [PATCH 07/11] avformat/mxf: use FFABS to instead of abs

2017-11-14 Thread Steven Liu
Signed-off-by: Steven Liu --- libavformat/mxf.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/libavformat/mxf.c b/libavformat/mxf.c index bfc3218b81..2e23b5d739 100644 --- a/libavformat/mxf.c +++ b/libavformat/mxf.c @@ -155,7 +155,7 @@ const

[FFmpeg-devel] [PATCH 09/11] avformat/swfenc: use FFABS to instead of abs

2017-11-14 Thread Steven Liu
Signed-off-by: Steven Liu --- libavformat/swfenc.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/libavformat/swfenc.c b/libavformat/swfenc.c index cada45ef9a..f53db0fb2b 100644 --- a/libavformat/swfenc.c +++ b/libavformat/swfenc.c @@ -69,7 +69,7 @@

[FFmpeg-devel] [PATCH 11/11] avformat/wavdecuse FFABS to instead of fabs

2017-11-14 Thread Steven Liu
Signed-off-by: Steven Liu --- libavformat/wavdec.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/libavformat/wavdec.c b/libavformat/wavdec.c index b016185a1b..9f3e01e5b8 100644 --- a/libavformat/wavdec.c +++ b/libavformat/wavdec.c @@ -531,7 +531,7 @@

[FFmpeg-devel] [PATCH 06/11] avformat/mux: use FFABS to instead of fabs

2017-11-14 Thread Steven Liu
Signed-off-by: Steven Liu --- libavformat/mux.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/libavformat/mux.c b/libavformat/mux.c index b1244c67f3..ec18c6d7b0 100644 --- a/libavformat/mux.c +++ b/libavformat/mux.c @@ -316,7 +316,7 @@

[FFmpeg-devel] [PATCH 05/11] avformat/mp3dec: use FFABS to instead of abs

2017-11-14 Thread Steven Liu
Signed-off-by: Steven Liu --- libavformat/mp3dec.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/libavformat/mp3dec.c b/libavformat/mp3dec.c index a5c4f2ea12..a76fe32e59 100644 --- a/libavformat/mp3dec.c +++ b/libavformat/mp3dec.c @@ -508,9 +508,9

[FFmpeg-devel] [PATCH 03/11] avformat/fitsdec: use FFABS to instead of abs

2017-11-14 Thread Steven Liu
Signed-off-by: Steven Liu --- libavformat/fitsdec.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/libavformat/fitsdec.c b/libavformat/fitsdec.c index 4b288b3903..b190118345 100644 --- a/libavformat/fitsdec.c +++ b/libavformat/fitsdec.c @@ -124,7 +124,7

[FFmpeg-devel] [PATCH 04/11] avformat/mov: use FFABS to instead of fabs

2017-11-14 Thread Steven Liu
Signed-off-by: Steven Liu --- libavformat/mov.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/libavformat/mov.c b/libavformat/mov.c index fd170baa57..e1a3e21e8d 100644 --- a/libavformat/mov.c +++ b/libavformat/mov.c @@ -4332,7 +4332,7 @@ static int

[FFmpeg-devel] [PATCH 01/11] avformat/hlsenc: use FFABS to instead of abs

2017-11-14 Thread Steven Liu
Signed-off-by: Steven Liu --- libavformat/hlsenc.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/libavformat/hlsenc.c b/libavformat/hlsenc.c index e0cef8b879..5af907b6f0 100644 --- a/libavformat/hlsenc.c +++ b/libavformat/hlsenc.c @@ -1113,7 +1113,7 @@

[FFmpeg-devel] [PATCH 02/11] avformat/dump: use FFABS to instead of llabs

2017-11-14 Thread Steven Liu
Signed-off-by: Steven Liu --- libavformat/dump.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/libavformat/dump.c b/libavformat/dump.c index 77043e3fdb..d414a6548f 100644 --- a/libavformat/dump.c +++ b/libavformat/dump.c @@ -586,8 +586,8 @@ void

Re: [FFmpeg-devel] [PATCH v3 2/2] avformat/{http, tls}: enable tcp_nodelay

2017-11-14 Thread Aman Gupta
On Tue, Nov 14, 2017 at 9:28 PM, Jeyapal, Karthick wrote: > > >On 11/15/17, 10:32 AM, "Aman Gupta" wrote: > > > >From: Aman Gupta > > > >Signed-off-by: Aman Gupta > >--- > > libavformat/http.c | 1 + > > libavformat/tls.c | 1

Re: [FFmpeg-devel] [PATCH v3 2/2] avformat/{http, tls}: enable tcp_nodelay

2017-11-14 Thread Jeyapal, Karthick
>On 11/15/17, 10:32 AM, "Aman Gupta" wrote: > >From: Aman Gupta > >Signed-off-by: Aman Gupta >--- > libavformat/http.c | 1 + > libavformat/tls.c | 1 + > 2 files changed, 2 insertions(+) Again, do want to hardcode tcp_nodelay at http and tls

[FFmpeg-devel] [PATCH 1/2] lavu/qsv: fix the warning of unused variable

2017-11-14 Thread Zhong Li
variable 's' is declared but not used. Signed-off-by: Zhong Li --- libavutil/hwcontext_qsv.c | 1 - 1 file changed, 1 deletion(-) diff --git a/libavutil/hwcontext_qsv.c b/libavutil/hwcontext_qsv.c index 8e43420..9b6040b 100644 --- a/libavutil/hwcontext_qsv.c +++

[FFmpeg-devel] [PATCH 2/2] lavf/qsv_overlay: fix compiling warning

2017-11-14 Thread Zhong Li
fix the compiling warning of "ignoring return value" Signed-off-by: Zhong Li --- libavfilter/vf_overlay_qsv.c | 7 +-- 1 file changed, 5 insertions(+), 2 deletions(-) diff --git a/libavfilter/vf_overlay_qsv.c b/libavfilter/vf_overlay_qsv.c index 1f50d4b..001a1c3 100644

[FFmpeg-devel] [PATCH v3 1/2] avformat/tcp: add option to enable TCP_NODELAY

2017-11-14 Thread Aman Gupta
From: Aman Gupta This can reduce latency and increase throughput, particularly on high latency networks. Signed-off-by: Aman Gupta Reviewed-by: Jeyapal, Karthick --- doc/protocols.texi| 3 +++ libavformat/network.h | 1 +

[FFmpeg-devel] [PATCH v3 2/2] avformat/{http, tls}: enable tcp_nodelay

2017-11-14 Thread Aman Gupta
From: Aman Gupta Signed-off-by: Aman Gupta --- libavformat/http.c | 1 + libavformat/tls.c | 1 + 2 files changed, 2 insertions(+) diff --git a/libavformat/http.c b/libavformat/http.c index 056d5f6583..f0a80b7add 100644 --- a/libavformat/http.c +++

Re: [FFmpeg-devel] [PATCH 2/2] avcodec/mpeg12dec: avoid adding PANSCAN side data unless present

2017-11-14 Thread Aman Gupta
On Tue, Nov 14, 2017 at 4:01 PM Michael Niedermayer wrote: > On Tue, Nov 14, 2017 at 10:04:02AM -0800, Aman Gupta wrote: > > From: Aman Gupta > > > > --- > > libavcodec/mpeg12dec.c | 20 ++-- > > 1 file changed, 14 insertions(+), 6

Re: [FFmpeg-devel] [PATCH] avformat/hlsenc: creation of hls variant streams with master playlist in a single hlsenc instance

2017-11-14 Thread 刘歧
> 在 2017年11月15日,07:14,Carl Eugen Hoyos 写道: > > 2017-11-14 11:25 GMT+01:00 刘歧 : > >> make fate failed: >> >> ../configure --enable-gpl --enable-memory-poisoning --enable-avresample >> --cc='ccache arm-linux-gnueabi-gcc' --target-exec='qemu-arm -L >>

Re: [FFmpeg-devel] [PATCH] avcodec: deprecate getters and setters for AVCodecContext and AVCodec fields

2017-11-14 Thread James Almer
On 11/13/2017 12:16 AM, James Almer wrote: > The fields can be accessed directly, so these are not needed anymore. > > Signed-off-by: James Almer > --- > libavcodec/avcodec.h | 20 > libavcodec/utils.c | 2 ++ > libavcodec/version.h | 3 +++ > 3 files

Re: [FFmpeg-devel] [PATCH v2 1/2] avformat/tcp: enable TCP_NODELAY by default

2017-11-14 Thread Jeyapal, Karthick
>On 11/14/17, 11:11 PM, "Aman Gupta" ffm...@tmm1.net> wrote: > >From: Aman Gupta > >This can reduce latency and increase throughput, particularly on high >latency networks. > >Signed-off-by: Aman Gupta >Reviewed-by: Jeyapal, Karthick

Re: [FFmpeg-devel] [PATCH 2/2] avcodec: Implement vc1 nvdec hwaccel

2017-11-14 Thread Philip Langdale
On Tue, 14 Nov 2017 21:14:58 +0100 Timo Rothenpieler wrote: > looks straight forward enough to me, struct naming matches up as well. > > add a Changelog and bump lavc version, and it's good to go. > Thanks. Pushed with Changelog and version bump. --phil

Re: [FFmpeg-devel] [PATCH 2/2] avcodec: Implement vc1 nvdec hwaccel

2017-11-14 Thread Philip Langdale
On Tue, 14 Nov 2017 21:07:45 +0100 Moritz Barsnick wrote: > On Tue, Nov 14, 2017 at 07:24:14 -0800, Philip Langdale wrote: > > This hwaccel is interesting because it also works for wmv3/9 > > content, which is not supported by the nvidia parser used by > > cuviddec. > >

[FFmpeg-devel] [PATCH] avcodec/mlpdsp: Fix undefined shift ff_mlp_pack_output()

2017-11-14 Thread Michael Niedermayer
Fixes: runtime error: left shift of negative value -7862264 Fixes: 4074/clusterfuzz-testcase-minimized-4516104123711488 Found-by: continuous fuzzing process https://github.com/google/oss-fuzz/tree/master/projects/ffmpeg Signed-off-by: Michael Niedermayer ---

[FFmpeg-devel] [Patch] Fix for ticket 6658 (Dash demuxer segfault)

2017-11-14 Thread Colin NG
- Add a function to handle the base URL Processing described in section 5.6.5 of IEC_23009-1. - Fix for downloading dash content with byte range info diff --git a/libavformat/dashdec.c b/libavformat/dashdec.c index 0e3afd2..0be8a49 100644 --- a/libavformat/dashdec.c +++

Re: [FFmpeg-devel] [PATCH] lavc/libvpxenc: add tune-content option

2017-11-14 Thread James Zern
On Mon, Nov 13, 2017 at 6:05 PM, James Zern wrote: > On Mon, Nov 13, 2017 at 4:43 PM, James Almer wrote: >> On 11/13/2017 9:01 PM, James Zern wrote: >>> Signed-off-by: James Zern >>> --- >>> doc/encoders.texi | 2 ++ >>>

[FFmpeg-devel] [PATCH] avfoundation, unsupported framerate/activeVideoMinFrameDuration

2017-11-14 Thread Roman Puttkammer
Hi, Attached patch fixes an issue with avfoundation; the code continues to loop through the list of supported formats/framerates even after finding the chosen one, ending up with a bad format/rate combination. (Code breaks out of one loop only instead of two.) rgds, Roman - diff --git

[FFmpeg-devel] [Patch] Download dash content with byte range info

2017-11-14 Thread Colin NG
Please ignore the previous "patch" email. diff --git a/libavformat/dashdec.c b/libavformat/dashdec.c index 0e3afd2..68196e9 100644 --- a/libavformat/dashdec.c +++ b/libavformat/dashdec.c @@ -522,6 +522,22 @@ static enum AVMediaType get_content_type(xmlNodePtr node) return type; }

[FFmpeg-devel] [Patch] Download dash content with byte range info

2017-11-14 Thread Colin NG
diff --git a/doc/examples/Makefile b/doc/examples/Makefile index 58afd71..bff4af6 100644 --- a/doc/examples/Makefile +++ b/doc/examples/Makefile @@ -1,3 +1,16 @@ +# use pkg-config for getting CFLAGS and LDLIBS +FFMPEG_LIBS=libavdevice\ +libavformat

Re: [FFmpeg-devel] [PATCH 3/3] avcodec/dvbsubdec: Avoid re-computing clut

2017-11-14 Thread Michael Niedermayer
On Sat, Nov 04, 2017 at 01:19:21AM +0100, Michael Niedermayer wrote: > Fixes: Timeout > Fixes: 3218/clusterfuzz-testcase-5390672154591232 > > Found-by: continuous fuzzing process > https://github.com/google/oss-fuzz/tree/master/projects/ffmpeg > Signed-off-by: Michael Niedermayer

Re: [FFmpeg-devel] [PATCH 1/3] avcodec/h264dec: Fix potential array overread

2017-11-14 Thread Michael Niedermayer
On Sun, Oct 22, 2017 at 01:41:56AM +0200, Michael Niedermayer wrote: > add padding before scantable arrays > > See: 522d850e68ec4b77d3477b3c8f55b1ba00a9d69a > > Signed-off-by: Michael Niedermayer > --- > libavcodec/h264dec.h | 1 + > 1 file changed, 1 insertion(+) will

Re: [FFmpeg-devel] [PATCH v3 1/2] avcodec/mpeg12dec: parse A53 caption data embedded in SCTE-20 user data

2017-11-14 Thread Michael Niedermayer
On Tue, Nov 14, 2017 at 11:18:57AM -0800, Aman Gupta wrote: > From: Aman Gupta > > Signed-off-by: Aman Gupta > --- > libavcodec/mpeg12dec.c | 39 +++ > tests/fate/subtitles.mak | 3 +++ > tests/ref/fate/sub-cc-scte20

Re: [FFmpeg-devel] FATE on multiple platforms

2017-11-14 Thread Ronald S. Bultje
Hi, On Tue, Nov 14, 2017 at 7:56 PM, Jim DeLaHunt wrote: > On 2017-11-14 14:20, Carl Eugen Hoyos wrote: > >> 2017-11-14 18:28 GMT+01:00 Dixit, Vishwanath: >> >> One of my patches is in the review process. I am fairly new to >>> the patch submission

Re: [FFmpeg-devel] [PATCH] vf_zscale: Add more supported input properties

2017-11-14 Thread Carl Eugen Hoyos
2017-11-15 1:56 GMT+01:00 Vittorio Giovara : > @@ -2111,6 +2124,12 @@ static int mxf_parse_structural_metadata(MXFContext > *mxf) > st->codecpar->codec_id = container_ul->id; > st->codecpar->width = descriptor->width; >

Re: [FFmpeg-devel] [PATCH] vf_zscale: Add more supported input properties

2017-11-14 Thread Derek Buitenhuis
On 11/15/2017 12:56 AM, Vittorio Giovara wrote: > --- > libavfilter/vf_zscale.c | 26 -- > libavformat/mxfdec.c| 20 > 2 files changed, 44 insertions(+), 2 deletions(-) Accidental inclusion of MXF code? Also, Does the minimum zimg version need to

Re: [FFmpeg-devel] FATE on multiple platforms

2017-11-14 Thread Jim DeLaHunt
On 2017-11-14 14:20, Carl Eugen Hoyos wrote: 2017-11-14 18:28 GMT+01:00 Dixit, Vishwanath: One of my patches is in the review process. I am fairly new to the patch submission process and have a query regarding FATE testing on multiple platforms. The patch includes a new FATE

[FFmpeg-devel] [PATCH] vf_zscale: Add more supported input properties

2017-11-14 Thread Vittorio Giovara
--- libavfilter/vf_zscale.c | 26 -- libavformat/mxfdec.c| 20 2 files changed, 44 insertions(+), 2 deletions(-) diff --git a/libavfilter/vf_zscale.c b/libavfilter/vf_zscale.c index 09fd842fe5..7d048da1ef 100644 --- a/libavfilter/vf_zscale.c +++

Re: [FFmpeg-devel] [PATCH 1/2] avcodec: Don't assume separate u and v planes in ff_alloc_picture

2017-11-14 Thread Michael Niedermayer
On Tue, Nov 14, 2017 at 07:24:13AM -0800, Philip Langdale wrote: > I'm confused at how none of the previous hwaccels hit this, but > alloc_frame_buffer in ff_alloc_picture asserts that the linesize > of planes 1 and 2 are the same. If the pixfmt has a single uv > plane, like NV12, this won't be

Re: [FFmpeg-devel] [PATCH] lavf/mov.c: Guess video codec delay based on PTS while parsing MOV header.

2017-11-14 Thread Michael Niedermayer
On Tue, Nov 14, 2017 at 03:36:49PM -0800, Thierry Foucu wrote: > On Tue, Nov 14, 2017 at 3:23 PM, Derek Buitenhuis < > derek.buitenh...@gmail.com> wrote: > > > On 11/14/2017 10:11 PM, Sasi Inguva wrote: > > > I don't know if the patch can be made more generic to work for all > > > demuxers,

Re: [FFmpeg-devel] [PATCH] lavf/mov.c: Guess video codec delay based on PTS while parsing MOV header.

2017-11-14 Thread Carl Eugen Hoyos
2017-11-15 0:36 GMT+01:00 Thierry Foucu : > On Tue, Nov 14, 2017 at 3:23 PM, Derek Buitenhuis < > derek.buitenh...@gmail.com> wrote: > >> On 11/14/2017 10:11 PM, Sasi Inguva wrote: >> > I don't know if the patch can be made more generic to work for all >> > demuxers, because this

Re: [FFmpeg-devel] [PATCH 2/2] avcodec/mpeg12dec: avoid adding PANSCAN side data unless present

2017-11-14 Thread Michael Niedermayer
On Tue, Nov 14, 2017 at 10:04:02AM -0800, Aman Gupta wrote: > From: Aman Gupta > > --- > libavcodec/mpeg12dec.c | 20 ++-- > 1 file changed, 14 insertions(+), 6 deletions(-) > > diff --git a/libavcodec/mpeg12dec.c b/libavcodec/mpeg12dec.c > index

Re: [FFmpeg-devel] [PATCH] lavf/mov.c: Guess video codec delay based on PTS while parsing MOV header.

2017-11-14 Thread Derek Buitenhuis
On 11/14/2017 11:36 PM, Thierry Foucu wrote: > One option i asked on IRC was to use the spec for max DPB when the bitstream > restriction flag was not set. > But people were worry about low latency usage. > Normally, if the bitstream restriction flag is not set, the DPB should be > set based on

Re: [FFmpeg-devel] Added HW H.264 and HEVC encoding for AMD GPUs based on AMF SDK

2017-11-14 Thread Mironov, Mikhail
> -Original Message- > From: ffmpeg-devel [mailto:ffmpeg-devel-boun...@ffmpeg.org] On Behalf > Of Mark Thompson > Sent: November 14, 2017 6:11 PM > To: ffmpeg-devel@ffmpeg.org > Subject: Re: [FFmpeg-devel] Added HW H.264 and HEVC encoding for AMD > GPUs based on AMF SDK > > On 14/11/17

Re: [FFmpeg-devel] [PATCH] lavf/mov.c: Guess video codec delay based on PTS while parsing MOV header.

2017-11-14 Thread Thierry Foucu
On Tue, Nov 14, 2017 at 3:23 PM, Derek Buitenhuis < derek.buitenh...@gmail.com> wrote: > On 11/14/2017 10:11 PM, Sasi Inguva wrote: > > I don't know if the patch can be made more generic to work for all > > demuxers, because this patch requires that PTS of all packets be > available > > in the

Re: [FFmpeg-devel] [PATCH] lavf/mov.c: Guess video codec delay based on PTS while parsing MOV header.

2017-11-14 Thread Derek Buitenhuis
On 11/14/2017 10:11 PM, Sasi Inguva wrote: > I don't know if the patch can be made more generic to work for all > demuxers, because this patch requires that PTS of all packets be available > in the header. The other route is to make it very specific to codecs with > B-frames. All PTS may not be

Re: [FFmpeg-devel] [PATCH] avformat/hlsenc: creation of hls variant streams with master playlist in a single hlsenc instance

2017-11-14 Thread Carl Eugen Hoyos
2017-11-14 11:25 GMT+01:00 刘歧 : > make fate failed: > > ../configure --enable-gpl --enable-memory-poisoning --enable-avresample > --cc='ccache arm-linux-gnueabi-gcc' --target-exec='qemu-arm -L > /usr/arm-linux-gnueabi/' --arch=armv5te --cpu=armv5te --enable-cross-compile

Re: [FFmpeg-devel] [PATCH] avformat/hlsenc: creation of hls variant streams with master playlist in a single hlsenc instance

2017-11-14 Thread Carl Eugen Hoyos
2017-11-14 7:42 GMT+01:00 Dixit, Vishwanath : > > >>On 11/13/17, 3:28 PM, "刘歧" wrote: >>root@localhost:~/ffmpeg# patch -p1 < >>~/0001-avformat-hlsenc-creation-of-hls-variant-streams-in-a.patch >>patching file doc/muxers.texi >>patching file

Re: [FFmpeg-devel] Added HW H.264 and HEVC encoding for AMD GPUs based on AMF SDK

2017-11-14 Thread Mark Thompson
On 14/11/17 22:10, Mironov, Mikhail wrote: >> On 14/11/17 17:14, Mironov, Mikhail wrote: > +res = ctx->factory->pVtbl->CreateContext(ctx->factory, > + >>> context); > +AMF_RETURN_IF_FALSE(ctx, res == AMF_OK, AVERROR_UNKNOWN, "CreateContext()

Re: [FFmpeg-devel] avcodec/qsvenc Question for Intel QSV low latency

2017-11-14 Thread Carl Eugen Hoyos
2017-11-14 17:02 GMT+01:00 Natsuki Kai : > Hi Moritz, > > Thank you for explaining in details. It really helped me. > It seems gmail or other mail tool shouldn't be used, but only "git > send-email" is accepted. Just send your patch - made with git format-patch - as

Re: [FFmpeg-devel] Added HW H.264 and HEVC encoding for AMD GPUs based on AMF SDK

2017-11-14 Thread Mironov, Mikhail
> -Original Message- > From: ffmpeg-devel [mailto:ffmpeg-devel-boun...@ffmpeg.org] On Behalf > Of Hendrik Leppkes > Sent: November 14, 2017 5:32 PM > To: FFmpeg development discussions and patches de...@ffmpeg.org> > Subject: Re: [FFmpeg-devel] Added HW H.264 and HEVC encoding for AMD >

Re: [FFmpeg-devel] libavcodec/als: remove check for predictor order of a block

2017-11-14 Thread Carl Eugen Hoyos
2017-11-14 23:48 GMT+01:00 Carl Eugen Hoyos : > 2017-11-13 21:07 GMT+01:00 Thilo Borgmann : >> Am 13.11.17 um 21:06 schrieb Thilo Borgmann: >>> Am 13.11.17 um 20:02 schrieb Umair Khan: Hi, On Mon, Nov 13, 2017 at 11:06 PM, Thilo Borgmann

Re: [FFmpeg-devel] [DEVEL][PATCH v3] ffmpeg: fix channel_layout bug on non-default layout

2017-11-14 Thread pkv.stream
Le 14/11/2017 à 1:13 PM, Michael Niedermayer a écrit : On Sun, Nov 12, 2017 at 06:26:18PM +0100, pkv.stream wrote: Le 12/11/2017 à 5:38 PM, Michael Niedermayer a écrit : On Sun, Nov 12, 2017 at 05:07:04PM +0100, Kv Pham wrote: Le 12 nov. 2017 5:01 PM, "Michael

Re: [FFmpeg-devel] libavcodec/als: remove check for predictor order of a block

2017-11-14 Thread Carl Eugen Hoyos
2017-11-13 21:07 GMT+01:00 Thilo Borgmann : > Am 13.11.17 um 21:06 schrieb Thilo Borgmann: >> Am 13.11.17 um 20:02 schrieb Umair Khan: >>> Hi, >>> >>> On Mon, Nov 13, 2017 at 11:06 PM, Thilo Borgmann >>> wrote: Hi, > On Mon, Nov 13,

Re: [FFmpeg-devel] [PATCH] aacenc: WIP support for PCEs

2017-11-14 Thread pkv.stream
Le 14/11/2017 à 11:14 PM, Carl Eugen Hoyos a écrit : 2017-11-14 21:01 GMT+01:00 pkv.stream : Le 14/11/2017 à 1:20 AM, Carl Eugen Hoyos a écrit : 2017-11-09 9:01 GMT+01:00 pkv.stream : 1) there are changes to make to the list of channel layouts not

Re: [FFmpeg-devel] [PATCH] Add FAQs about running in background

2017-11-14 Thread Carl Eugen Hoyos
2017-11-14 12:38 GMT+01:00 Jim DeLaHunt : > Those are good comments. Thank you. I can see how to improve the wording. > I could add them to my current branch, and submit a new patch. Yes, this is what is expected. Please do not top-post here, Carl Eugen

Re: [FFmpeg-devel] Added HW H.264 and HEVC encoding for AMD GPUs based on AMF SDK

2017-11-14 Thread Hendrik Leppkes
On Tue, Nov 14, 2017 at 6:45 PM, Mark Thompson wrote: > > Decode via D3D11 does work for me on Windows 7 with both AMD and Intel; I > don't know anything about how, though. (I don't really care about Windows 7 > - this was just a set of parts mashed together into a working

Re: [FFmpeg-devel] [PATCH 00/15] OpenCL infrastructure, filters

2017-11-14 Thread Carl Eugen Hoyos
2017-11-14 20:47 GMT+01:00 Mark Thompson : > Silly example using everything (for i965 VAAPI + Beignet): > > ./ffmpeg_g -y -init_hw_device vaapi=va:/dev/dri/renderD128 > -init_hw_device opencl=ocl@va -hwaccel vaapi > -hwaccel_device va -hwaccel_output_format vaapi -i in.mp4 > -f

Re: [FFmpeg-devel] avcodec/hapqa_extract_bsf : add bsf filter for haqqa (to hapq or hapalpha only) conversion (WIP)

2017-11-14 Thread Carl Eugen Hoyos
2017-11-14 10:55 GMT+01:00 Martin Vignali : > 2017-11-14 1:26 GMT+01:00 Carl Eugen Hoyos : > >> 2017-11-13 22:43 GMT+01:00 Martin Vignali : >> >> > In attach patch to add a new bitstream filter >> > >> > The goal is to convert

Re: [FFmpeg-devel] FATE on multiple platforms

2017-11-14 Thread Carl Eugen Hoyos
2017-11-14 18:28 GMT+01:00 Dixit, Vishwanath : > One of my patches is in the review process. I am fairly new to > the patch submission process and have a query regarding > FATE testing on multiple platforms. > The patch includes a new FATE test and I am expected to get > it

Re: [FFmpeg-devel] [PATCH]lavf/matroskaenc: Do not write 0 duration for subtitles

2017-11-14 Thread Carl Eugen Hoyos
2017-11-14 19:02 GMT+01:00 Jerome Martinez : > On 12/11/2017 03:12, Carl Eugen Hoyos wrote: >> >> -put_ebml_uint(pb, MATROSKA_ID_BLOCKDURATION, duration); >> +if (duration > 0) >> +put_ebml_uint(pb, MATROSKA_ID_BLOCKDURATION, duration);

Re: [FFmpeg-devel] [PATCH] aacenc: WIP support for PCEs

2017-11-14 Thread Carl Eugen Hoyos
2017-11-14 21:01 GMT+01:00 pkv.stream : > Le 14/11/2017 à 1:20 AM, Carl Eugen Hoyos a écrit : >> >> 2017-11-09 9:01 GMT+01:00 pkv.stream : >> >>> 1) there are changes to make to the list of channel layouts not requiring >>> PCE >>> >>> ==>

Re: [FFmpeg-devel] [PATCH] lavf/mov.c: Guess video codec delay based on PTS while parsing MOV header.

2017-11-14 Thread Sasi Inguva
I don't know if the patch can be made more generic to work for all demuxers, because this patch requires that PTS of all packets be available in the header. The other route is to make it very specific to codecs with B-frames. On Tue, Nov 14, 2017 at 12:40 PM, Derek Buitenhuis <

Re: [FFmpeg-devel] Added HW H.264 and HEVC encoding for AMD GPUs based on AMF SDK

2017-11-14 Thread Mironov, Mikhail
> On 14/11/17 17:14, Mironov, Mikhail wrote: > >>> +res = ctx->factory->pVtbl->CreateContext(ctx->factory, > >>> + > > context); > >>> +AMF_RETURN_IF_FALSE(ctx, res == AMF_OK, > >> AVERROR_UNKNOWN, > >> "CreateContext() failed with error %d\n", res); > >>> +//

[FFmpeg-devel] [PATCH] Add DECLARE_ASM_ALIGNED macro for DJGPP architecture.

2017-11-14 Thread Thomas Köppe
The macro was added in 43171a2a738f5114768d34a7278e56e5fde714bc, but I forgot to add it to the DJGPP architecture in that change. --- libavutil/mem.h | 1 + 1 file changed, 1 insertion(+) diff --git a/libavutil/mem.h b/libavutil/mem.h index 9e344bd2c3..7e0b12a8a7 100644 --- a/libavutil/mem.h

Re: [FFmpeg-devel] [PATCH v2 1/2] avformat/tcp: enable TCP_NODELAY by default

2017-11-14 Thread Aman Gupta
On Tue, Nov 14, 2017 at 9:40 AM, Aman Gupta wrote: > From: Aman Gupta > > This can reduce latency and increase throughput, particularly on high > latency networks. > Oops, I forgot to remove "enabled by default" from the commit message subject. Fixed locally.

Re: [FFmpeg-devel] [PATCH v2 1/2] avcodec/videotoolboxenc: add hevc_videotoolbox encoder

2017-11-14 Thread Aman Gupta
On Sun, Nov 12, 2017 at 10:22 AM, Aman Gupta wrote: > From: Aman Gupta > > --- > configure| 2 + > libavcodec/allcodecs.c | 1 + > libavcodec/videotoolboxenc.c | 153 ++ > + > 3 files changed,

Re: [FFmpeg-devel] [PATCH] lavf/mov.c: Guess video codec delay based on PTS while parsing MOV header.

2017-11-14 Thread Derek Buitenhuis
On 11/14/2017 8:28 PM, Sasi Inguva wrote: > For H264 files with no bitstream restriction flag, we aren't able to guess > the delay correctly. Especially if it's MOV container, because for MOV > container we just probe the 1st frame and stop in avformat_find_streaminfo . You do not appear to be

Re: [FFmpeg-devel] [PATCH] lavf/mov.c: Guess video codec delay based on PTS while parsing MOV header.

2017-11-14 Thread Hendrik Leppkes
On Tue, Nov 14, 2017 at 9:28 PM, Sasi Inguva wrote: > For H264 files with no bitstream restriction flag, we aren't able to guess > the delay correctly. Especially if it's MOV container, because for MOV > container we just probe the 1st frame and stop in

Re: [FFmpeg-devel] [PATCH] lavf/mov.c: Guess video codec delay based on PTS while parsing MOV header.

2017-11-14 Thread Michael Niedermayer
On Tue, Nov 14, 2017 at 09:39:33AM -0800, Sasi Inguva wrote: > Sorry forgot to attach the fate samples. Pls find them attached. uploaded thx [...] -- Michael GnuPG fingerprint: 9FF2128B147EF6730BADF133611EC787040B0FAB Its not that you shouldnt use gotos but rather that you should write

Re: [FFmpeg-devel] [PATCH] lavf/mov.c: Guess video codec delay based on PTS while parsing MOV header.

2017-11-14 Thread Sasi Inguva
For H264 files with no bitstream restriction flag, we aren't able to guess the delay correctly. Especially if it's MOV container, because for MOV container we just probe the 1st frame and stop in avformat_find_streaminfo . When we are decoding , we increase the has_b_frames value from zero to 1

Re: [FFmpeg-devel] [PATCH 2/2] avcodec: Implement vc1 nvdec hwaccel

2017-11-14 Thread Timo Rothenpieler
looks straight forward enough to me, struct naming matches up as well. add a Changelog and bump lavc version, and it's good to go. smime.p7s Description: S/MIME Cryptographic Signature ___ ffmpeg-devel mailing list ffmpeg-devel@ffmpeg.org

Re: [FFmpeg-devel] [PATCH v2 1/2] avformat/tcp: enable TCP_NODELAY by default

2017-11-14 Thread Moritz Barsnick
On Tue, Nov 14, 2017 at 09:40:58 -0800, Aman Gupta wrote: > From: Aman Gupta > Subject: avformat/tcp: enable TCP_NODELAY by default The commit message is no longer correct -> avformat/tcp: allow setting TCP_NODELAY Moritz ___

Re: [FFmpeg-devel] [PATCH 2/2] avcodec: Implement vc1 nvdec hwaccel

2017-11-14 Thread Moritz Barsnick
On Tue, Nov 14, 2017 at 07:24:14 -0800, Philip Langdale wrote: > This hwaccel is interesting because it also works for wmv3/9 content, > which is not supported by the nvidia parser used by cuviddec. Nits: Changelog entry? Version bump? > +case AV_CODEC_ID_VC1: return cudaVideoCodec_VC1; >

Re: [FFmpeg-devel] [PATCH] lavf/mov.c: Don't correct edit list start to zero, when we can't find a frame before edit list start. After c2a8f0fcbe57ea9ccaa864130f078af10516c3c1 this can happen on norma

2017-11-14 Thread Michael Niedermayer
On Mon, Nov 13, 2017 at 06:09:28PM -0800, Sasi Inguva wrote: > Signed-off-by: Sasi Inguva > --- > libavformat/mov.c| 4 +- > tests/fate/mov.mak | 6 + > tests/ref/fate/mov-bbi-elst-starts-b | 391 > +++ >

Re: [FFmpeg-devel] [PATCH] aacenc: WIP support for PCEs

2017-11-14 Thread pkv.stream
Le 14/11/2017 à 1:20 AM, Carl Eugen Hoyos a écrit : 2017-11-09 9:01 GMT+01:00 pkv.stream : 1) there are changes to make to the list of channel layouts not requiring PCE ==> AV_CH_LAYOUT_5POINT0 to AV_CH_LAYOUT_5POINT0_BACK since the previous is 5.0(side) while the latter

Re: [FFmpeg-devel] Avoid corrupting diagnostic state with _Pragma changes.

2017-11-14 Thread Michael Niedermayer
On Fri, Nov 03, 2017 at 05:13:58PM -0700, Dale Curtis wrote: > The macros for ICC and MSVC correctly push and pop the diagnostic > state of the compiler when disabling deprecation warnings. The > ones for clang/gcc should do the same. Without this, if a blanket > deprecation warning is applied to

Re: [FFmpeg-devel] [PATCH] fate/mov: add test for overlapping fragments

2017-11-14 Thread Michael Niedermayer
On Mon, Nov 13, 2017 at 08:09:30AM -0800, John Stebbins wrote: > --- > tests/fate/mov.mak | 4 ++ > tests/ref/fate/mov-frag-overlap | 105 > > 2 files changed, 109 insertions(+) > create mode 100644 tests/ref/fate/mov-frag-overlap will

[FFmpeg-devel] [PATCH 09/15] lavfi: Add filter to run an arbitrary OpenCL program on frames

2017-11-14 Thread Mark Thompson
--- configure | 1 + libavfilter/Makefile| 1 + libavfilter/allfilters.c| 1 + libavfilter/vf_program_opencl.c | 254 4 files changed, 257 insertions(+) create mode 100644 libavfilter/vf_program_opencl.c

[FFmpeg-devel] [PATCH 14/15] hwcontext: Perform usual uninitialisation on derived frames contexts

2017-11-14 Thread Mark Thompson
--- libavutil/hwcontext.c | 17 +++-- 1 file changed, 7 insertions(+), 10 deletions(-) diff --git a/libavutil/hwcontext.c b/libavutil/hwcontext.c index b26e68fe77..f47158f811 100644 --- a/libavutil/hwcontext.c +++ b/libavutil/hwcontext.c @@ -217,19 +217,16 @@ static void

[FFmpeg-devel] [PATCH 13/15] vf_hwmap: Pass mapping mode when deriving frames context on an existing device

2017-11-14 Thread Mark Thompson
To match creation on a newly-derived device. (This was missed earlier because the mode is only used in some cases.) --- libavfilter/vf_hwmap.c | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) diff --git a/libavfilter/vf_hwmap.c b/libavfilter/vf_hwmap.c index 8277241dc4..d5fc3c46e6 100644

[FFmpeg-devel] [PATCH 08/15] lavfi: Add some common code for OpenCL filtering

2017-11-14 Thread Mark Thompson
--- libavfilter/opencl.c | 251 +++ libavfilter/opencl.h | 87 ++ 2 files changed, 338 insertions(+) create mode 100644 libavfilter/opencl.c create mode 100644 libavfilter/opencl.h diff --git a/libavfilter/opencl.c

[FFmpeg-devel] [PATCH 12/15] lavfi: Add OpenCL unsharp mask filter

2017-11-14 Thread Mark Thompson
Intended to replace existing opencl mode of the unsharp filter. Supports many more pixel formats and works without immediate upload and download of frame data. The options are compatible with the existing filter. --- The most obvious missing optimisation is to split the kernels into multiple

[FFmpeg-devel] [PATCH 11/15] lavfi: Add OpenCL overlay filter

2017-11-14 Thread Mark Thompson
Input and output formats must be the same, the overlay format must be the same as the input except possibly with an additional alpha component. --- configure | 1 + libavfilter/Makefile| 2 + libavfilter/allfilters.c| 1 +

[FFmpeg-devel] [PATCH 07/15] hwcontext_opencl: DRM to OpenCL mapping for ARM

2017-11-14 Thread Mark Thompson
Using cl_arm_import_memory. Unfortunately, despite this not being a standard extension, the function clImportMemoryARM() is not accessible via clGetExtensionFunctionAddressForPlatform(). This means that it has to be linked directly to the ARM OpenCL binary, so making a portable binary is not

[FFmpeg-devel] [PATCH 06/15] hwcontext_opencl: D3D11 to OpenCL mapping

2017-11-14 Thread Mark Thompson
Using cl_khr_d3d11_sharing and cl_intel_d3d11_nv12_media_sharing. --- configure| 6 + libavutil/hwcontext_opencl.c | 321 ++- 2 files changed, 325 insertions(+), 2 deletions(-) diff --git a/configure b/configure index

[FFmpeg-devel] [PATCH 10/15] lavfi: Add infrastructure for building OpenCL source into libavfilter

2017-11-14 Thread Mark Thompson
--- libavfilter/Makefile| 6 ++ libavfilter/opencl_source.h | 22 ++ tools/cl2c | 20 3 files changed, 48 insertions(+) create mode 100644 libavfilter/opencl_source.h create mode 100755 tools/cl2c diff --git

[FFmpeg-devel] [PATCH 05/15] hwcontext_opencl: DXVA2 to OpenCL mapping

2017-11-14 Thread Mark Thompson
Using cl_khr_dx9_media_sharing. --- configure| 6 + libavutil/hwcontext_opencl.c | 397 +++ 2 files changed, 403 insertions(+) diff --git a/configure b/configure index 7649fe8de1..dee44eff85 100755 --- a/configure +++ b/configure @@

[FFmpeg-devel] [PATCH 03/15] hwcontext_opencl: VAAPI to OpenCL mapping for Intel i965+beignet

2017-11-14 Thread Mark Thompson
Supports all surface formats in common between the two. --- configure| 6 + libavutil/hwcontext_opencl.c | 298 +++ 2 files changed, 304 insertions(+) diff --git a/configure b/configure index 167db274f1..dcdb5fee1f 100755 ---

[FFmpeg-devel] [PATCH 04/15] hwcontext_opencl: QSV to OpenCL mapping for Intel Media SDK

2017-11-14 Thread Mark Thompson
Uses the cl_intel_va_api_media_sharing extension, which supports only NV12 surfaces and only mapping from QSV to OpenCL. --- configure| 5 + libavutil/hwcontext_opencl.c | 365 +++ 2 files changed, 370 insertions(+) diff --git

[FFmpeg-devel] [PATCH 02/15] lavu: OpenCL hwcontext implementation

2017-11-14 Thread Mark Thompson
--- configure |4 +- doc/APIchanges |4 + libavutil/Makefile |2 + libavutil/hwcontext.c |4 + libavutil/hwcontext.h |1 + libavutil/hwcontext_internal.h |1 + libavutil/hwcontext_opencl.c | 1303

[FFmpeg-devel] [PATCH 00/15] OpenCL infrastructure, filters

2017-11-14 Thread Mark Thompson
Changes since the last time this was posted: * Add unsharp filter (to replace existing unsharp). * Remove old experimental API. * Miscellaneous fixes. Now also tested with AMD OpenCL on Windows (DXVA2 mapping works nicely, D3D11 does not because it wants the Intel extension for NV12 support).

[FFmpeg-devel] [PATCH 01/15] lavu: Add OpenCL hardware pixfmt

2017-11-14 Thread Mark Thompson
--- doc/APIchanges | 3 +++ libavutil/pixdesc.c | 4 libavutil/pixfmt.h | 7 +++ 3 files changed, 14 insertions(+) diff --git a/doc/APIchanges b/doc/APIchanges index d336f6ce22..3f68bcccd2 100644 --- a/doc/APIchanges +++ b/doc/APIchanges @@ -15,6 +15,9 @@ libavutil: 2017-10-21

[FFmpeg-devel] [PATCH v3 1/2] avcodec/mpeg12dec: parse A53 caption data embedded in SCTE-20 user data

2017-11-14 Thread Aman Gupta
From: Aman Gupta Signed-off-by: Aman Gupta --- libavcodec/mpeg12dec.c | 39 +++ tests/fate/subtitles.mak | 3 +++ tests/ref/fate/sub-cc-scte20 | 15 +++ 3 files changed, 57 insertions(+) create mode

[FFmpeg-devel] [PATCH v3 2/2] avcodec/mpeg12dec: ensure a53_caption_size is reset on malloc failures

2017-11-14 Thread Aman Gupta
From: Aman Gupta Signed-off-by: Aman Gupta Reviewed-by: Michael Niedermayer --- libavcodec/mpeg12dec.c | 13 ++--- 1 file changed, 10 insertions(+), 3 deletions(-) diff --git a/libavcodec/mpeg12dec.c b/libavcodec/mpeg12dec.c index

Re: [FFmpeg-devel] [PATCH] configure: call flatten_extralibs in a subshell

2017-11-14 Thread James Almer
On 11/14/2017 11:35 AM, Timo Rothenpieler wrote: > Am 14.11.2017 um 14:50 schrieb Timo Rothenpieler: >> By putting the call in a subshell, the problem of it spilling >> cleanup-decision from a previous library to other libraries is avoided. >> >> For example, it could have already cleaned up

Re: [FFmpeg-devel] [PATCH 1/2] avcodec: fix whitespace on AVPanScan struct definition

2017-11-14 Thread Aman Gupta
On Tue, Nov 14, 2017 at 10:23 AM, James Almer wrote: > On 11/14/2017 3:04 PM, Aman Gupta wrote: > > From: Aman Gupta > > > > --- > > libavcodec/avcodec.h | 4 ++-- > > 1 file changed, 2 insertions(+), 2 deletions(-) > > > > diff --git a/libavcodec/avcodec.h

Re: [FFmpeg-devel] [PATCH 1/2] avcodec: fix whitespace on AVPanScan struct definition

2017-11-14 Thread James Almer
On 11/14/2017 3:04 PM, Aman Gupta wrote: > From: Aman Gupta > > --- > libavcodec/avcodec.h | 4 ++-- > 1 file changed, 2 insertions(+), 2 deletions(-) > > diff --git a/libavcodec/avcodec.h b/libavcodec/avcodec.h > index 74b097cdd8..0875ae3ba0 100644 > --- a/libavcodec/avcodec.h

Re: [FFmpeg-devel] [PATCH] lavf/mov.c: Guess video codec delay based on PTS while parsing MOV header.

2017-11-14 Thread Derek Buitenhuis
On 11/14/2017 2:15 AM, Sasi Inguva wrote: > Signed-off-by: Sasi Inguva > --- > libavformat/mov.c| 48 > > tests/fate/mov.mak | 5 + > tests/ref/fate/mov-guess-delay-1 | 3 +++ >

  1   2   >