[FFmpeg-devel] [PATCH] avcodec/mips: Improve hevc uni weighted hv mc msa functions

2017-10-31 Thread kaustubh.raste
From: Kaustubh Raste Use immediate unsigned saturation for clip to max saving one vector register. Signed-off-by: Kaustubh Raste --- libavcodec/mips/hevc_macros_msa.h |9 + libavcodec/mips/hevc_mc_uniw_msa.c | 1598

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

2017-10-31 Thread Mironov, Mikhail
> -Original Message- > From: ffmpeg-devel [mailto:ffmpeg-devel-boun...@ffmpeg.org] On Behalf > Of Marton Balint > Sent: October 30, 2017 9:26 PM > To: FFmpeg development discussions and patches de...@ffmpeg.org> > Subject: Re: [FFmpeg-devel] Added HW H.264 and HEVC encoding for AMD > GPUs

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

2017-10-31 Thread Mironov, Mikhail
> >>> +AMF_ASSIGN_PROPERTY_BOOL(res, ctx->encoder, > >> AMF_VIDEO_ENCODER_HEVC_DE_BLOCKING_FILTER_DISABLE, > >> deblocking_filter); > >> > >> What about SAO? > > > > SAO ??? > > You're looking at AV_CODEC_FLAG_LOOP_FILTER to disable this, so you > might want to consider both loop filters in

Re: [FFmpeg-devel] On in-tree external headers

2017-10-31 Thread Mark Thompson
On 31/10/17 16:34, Timo Rothenpieler wrote: > Removing the nvenc/cuvid headers from the tree would imply the following > procedure for anyone wanting to build ffmpeg with nvenc/cuvid: > > Register with nvidia to get access to their Video SDK Downloads. > Extract the headers from their massive

Re: [FFmpeg-devel] [PATCH] avcodec/mips: Improve hevc uni vt and hv mc msa functions

2017-10-31 Thread Michael Niedermayer
On Tue, Oct 31, 2017 at 07:00:08AM +, Manojkumar Bhosale wrote: > LGTM will apply thx [...] -- Michael GnuPG fingerprint: 9FF2128B147EF6730BADF133611EC787040B0FAB Awnsering whenever a program halts or runs forever is On a turing machine, in general impossible (turings halting

[FFmpeg-devel] [PATCH 1/2] configure: Rename wincrypt to advapi32 for consistency with existing uses of check_lib

2017-10-31 Thread Aaron Levinson
Signed-off-by: Aaron Levinson --- configure | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/configure b/configure index ea22d692b9..572299c9d3 100755 --- a/configure +++ b/configure @@ -3327,7 +3327,7 @@ avformat_deps="avcodec avutil"

[FFmpeg-devel] [PATCH 2/2] configure: Fix backup libmfx MSVC build by adding advapi32 library dependency

2017-10-31 Thread Aaron Levinson
Signed-off-by: Aaron Levinson --- configure | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/configure b/configure index 572299c9d3..2ac6fed98a 100755 --- a/configure +++ b/configure @@ -5994,7 +5994,7 @@ enabled libkvazaar&&

Re: [FFmpeg-devel] [PATCH] avcodec/mips: Improve avc avg mc 22, 11, 31, 13 and 33 msa functions

2017-10-31 Thread Michael Niedermayer
On Mon, Oct 30, 2017 at 11:36:34AM +, Manojkumar Bhosale wrote: > LGTM applied [...] -- 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,

[FFmpeg-devel] [PATCH] avformat/mux: be less strict with bitstream filter failures

2017-10-31 Thread James Almer
This makes the autobsf feature behave the same as the manual bitstream filtering in ffmpeg.c Fixes ticket #6794 Signed-off-by: James Almer --- libavformat/mux.c | 4 +++- 1 file changed, 3 insertions(+), 1 deletion(-) diff --git a/libavformat/mux.c b/libavformat/mux.c index

[FFmpeg-devel] [PATCH] configure, fftools/Makefile: Copy .pdb files to bindir for MSVC builds for make install

2017-10-31 Thread Aaron Levinson
When ffmpeg is built using MSVC, symbols are stored separately from executables and libraries in .pdb files. However, unlike with gcc-based builds, when make install is invoked, symbols, in the form of .pdb files, which are important for debugging, are not copied to bindir. This change corrects

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

2017-10-31 Thread Marton Balint
On Tue, 31 Oct 2017, Mironov, Mikhail wrote: [...] I see some confusion. The user can call send_frame/receive_packet in any order, and you can implement send_frame and receive_packet any way you want, the only thing you have to guarantee is that you cannot return EAGAIN for both send_frame

Re: [FFmpeg-devel] [PATCH] avutil/crc: use ff_thread_once at av_crc_get_table

2017-10-31 Thread Muhammad Faiz
On Mon, Oct 30, 2017 at 8:07 PM, Michael Niedermayer wrote: > On Mon, Oct 30, 2017 at 02:14:35PM +0700, Muhammad Faiz wrote: >> On Tue, Oct 24, 2017 at 4:31 PM, Muhammad Faiz wrote: >> > Fix tsan warnings. >> > >> > Signed-off-by: Muhammad Faiz

Re: [FFmpeg-devel] [PATCH] avcodec/mips: Improve avc chroma avg hv mc msa functions

2017-10-31 Thread Michael Niedermayer
On Mon, Oct 30, 2017 at 11:36:41AM +, Manojkumar Bhosale wrote: > LGTM applied [...] -- Michael GnuPG fingerprint: 9FF2128B147EF6730BADF133611EC787040B0FAB If a bugfix only changes things apparently unrelated to the bug with no further explanation, that is a good sign that the bugfix

Re: [FFmpeg-devel] [RFC]lswr/rematrix: Support s32p

2017-10-31 Thread Michael Niedermayer
On Tue, Oct 31, 2017 at 12:40:24AM +0100, Carl Eugen Hoyos wrote: > 2017-10-30 9:47 GMT+01:00 Muhammad Faiz : > > On Sun, Oct 29, 2017 at 3:55 AM, Carl Eugen Hoyos > > wrote: > >> Hi! > >> > >> Attached patch fixes a random testcase for ticket #6785 here but

Re: [FFmpeg-devel] On in-tree external headers

2017-10-31 Thread Timo Rothenpieler
Removing the nvenc/cuvid headers from the tree would imply the following procedure for anyone wanting to build ffmpeg with nvenc/cuvid: Register with nvidia to get access to their Video SDK Downloads. Extract the headers from their massive SDK. Patch them so that ffmpeg can make use of them.

Re: [FFmpeg-devel] [PATCH]lavf/latmenc: Error out for invalid codecs

2017-10-31 Thread Michael Niedermayer
On Mon, Oct 30, 2017 at 11:51:30PM +0100, Carl Eugen Hoyos wrote: > Hi! > > Attached patch makes sure the loas muxer does not try to write > anything but aac and latm. > > Please comment, Carl Eugen > latmenc.c |4 > 1 file changed, 4 insertions(+) >

Re: [FFmpeg-devel] [PATCH v5 2/2 ] avformat/mpegts: opus demuxing for mapping family 255

2017-10-31 Thread Michael Niedermayer
On Sun, Oct 29, 2017 at 11:11:43PM +0100, pkv.stream wrote: > minor change, adding a test for number of channels for > mapping_family 1 when channel_config_code is > 0x82 > Passes fate + patcheck. > Regards > pkv > mpegts.c | 84 >

Re: [FFmpeg-devel] [PATCH] avcodec/mips: Improve hevc bi weighted copy, hz and vt mc msa functions

2017-10-31 Thread Michael Niedermayer
On Tue, Oct 31, 2017 at 06:59:57AM +, Manojkumar Bhosale wrote: > LGTM will apply thx [...] -- Michael GnuPG fingerprint: 9FF2128B147EF6730BADF133611EC787040B0FAB Avoid a single point of failure, be that a person or equipment. signature.asc Description: Digital signature

Re: [FFmpeg-devel] [PATCH] avcodec/mips: Improve hevc bi hz and hv mc msa functions

2017-10-31 Thread Michael Niedermayer
On Tue, Oct 31, 2017 at 07:00:03AM +, Manojkumar Bhosale wrote: > LGTM will apply [...] -- Michael GnuPG fingerprint: 9FF2128B147EF6730BADF133611EC787040B0FAB Breaking DRM is a little like attempting to break through a door even though the window is wide open and the only thing in the

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

2017-10-31 Thread Mironov, Mikhail
> -Original Message- > From: ffmpeg-devel [mailto:ffmpeg-devel-boun...@ffmpeg.org] On Behalf > Of Marton Balint > Sent: October 31, 2017 2:06 PM > To: FFmpeg development discussions and patches de...@ffmpeg.org> > Subject: Re: [FFmpeg-devel] Added HW H.264 and HEVC encoding for AMD > GPUs

Re: [FFmpeg-devel] [PATCH 1/2] lavc/vaapi_encode: correct the HRD buffer size calculate.

2017-10-31 Thread Jun Zhao
On 2017/10/31 18:19, Mark Thompson wrote: > On 31/10/17 02:37, Jun Zhao wrote: >> From 7eef9be1c8a92bf625d62a0f97f762f1342c6d78 Mon Sep 17 00:00:00 2001 >> From: Jun Zhao >> Date: Tue, 31 Oct 2017 10:13:42 +0800 >> Subject: [PATCH 1/2] lavc/vaapi_encode: correct the HRD

Re: [FFmpeg-devel] [PATCH 2/2] lavc/vaapi_encode_h264: correct bit_rate_scale setting.

2017-10-31 Thread Mark Thompson
On 31/10/17 15:18, Jun Zhao wrote: > On 2017/10/31 18:26, Mark Thompson wrote: >> On 31/10/17 02:37, Jun Zhao wrote: >>> From d1e105057e93e7c2788d6d684292db9008fbf3ac Mon Sep 17 00:00:00 2001 >>> From: Jun Zhao >>> Date: Tue, 31 Oct 2017 10:19:08 +0800 >>> Subject: [PATCH 2/2]

Re: [FFmpeg-devel] [PATCH] Fix missing used attribute for inline assembly variables

2017-10-31 Thread Thomas Köppe
+Teresa, who drafted the patch initially. On 31 October 2017 at 15:38, Michael Niedermayer wrote: > On Tue, Oct 31, 2017 at 12:16:18PM +, Thomas Köppe wrote: > > Variables used in inline assembly need to be marked with > attribute((used)). > > This should not be the

Re: [FFmpeg-devel] [PATCH 1/2] lavc/vaapi_encode: correct the HRD buffer size calculate.

2017-10-31 Thread Mark Thompson
On 31/10/17 16:00, Jun Zhao wrote: > On 2017/10/31 23:45, Mark Thompson wrote: >> On 31/10/17 15:11, Jun Zhao wrote:> On 2017/10/31 18:19, Mark Thompson wrote: On 31/10/17 02:37, Jun Zhao wrote: > From 7eef9be1c8a92bf625d62a0f97f762f1342c6d78 Mon Sep 17 00:00:00 2001 > From: Jun Zhao

Re: [FFmpeg-devel] [PATCH 2/2] lavc/vaapi_encode_h264: correct bit_rate_scale setting.

2017-10-31 Thread Jun Zhao
On 2017/10/31 18:26, Mark Thompson wrote: > On 31/10/17 02:37, Jun Zhao wrote: >> From d1e105057e93e7c2788d6d684292db9008fbf3ac Mon Sep 17 00:00:00 2001 >> From: Jun Zhao >> Date: Tue, 31 Oct 2017 10:19:08 +0800 >> Subject: [PATCH 2/2] lavc/vaapi_encode_h264: correct

Re: [FFmpeg-devel] [PATCH] Fix missing used attribute for inline assembly variables

2017-10-31 Thread Michael Niedermayer
On Tue, Oct 31, 2017 at 12:16:18PM +, Thomas Köppe wrote: > Variables used in inline assembly need to be marked with attribute((used)). This should not be the case. Variables referenced through in/out operands should not need that. Only ones accessed directly bypassing operands should need

Re: [FFmpeg-devel] [PATCH 1/2] lavc/vaapi_encode: correct the HRD buffer size calculate.

2017-10-31 Thread Mark Thompson
On 31/10/17 15:11, Jun Zhao wrote:> On 2017/10/31 18:19, Mark Thompson wrote: >> On 31/10/17 02:37, Jun Zhao wrote: >>> From 7eef9be1c8a92bf625d62a0f97f762f1342c6d78 Mon Sep 17 00:00:00 2001 >>> From: Jun Zhao >>> Date: Tue, 31 Oct 2017 10:13:42 +0800 >>> Subject: [PATCH 1/2]

[FFmpeg-devel] [PATCH] avformat/img2enc: add frame_pts option for make output filename

2017-10-31 Thread Steven Liu
when use frame_pts option, the output image name can be set with PTS of current frame. Signed-off-by: Steven Liu --- doc/muxers.texi | 9 + libavformat/img2enc.c | 7 +++ 2 files changed, 16 insertions(+) diff --git a/doc/muxers.texi b/doc/muxers.texi

Re: [FFmpeg-devel] [PATCH] Fix missing used attribute for inline assembly variables

2017-10-31 Thread Michael Niedermayer
On Tue, Oct 31, 2017 at 03:52:14PM +, Thomas Köppe wrote: > +Teresa, who drafted the patch initially. > > On 31 October 2017 at 15:38, Michael Niedermayer > wrote: > > > On Tue, Oct 31, 2017 at 12:16:18PM +, Thomas Köppe wrote: > > > Variables used in inline

Re: [FFmpeg-devel] [PATCH 1/2] lavc/vaapi_encode: correct the HRD buffer size calculate.

2017-10-31 Thread Jun Zhao
On 2017/10/31 23:45, Mark Thompson wrote: > On 31/10/17 15:11, Jun Zhao wrote:> On 2017/10/31 18:19, Mark Thompson wrote: >>> On 31/10/17 02:37, Jun Zhao wrote: From 7eef9be1c8a92bf625d62a0f97f762f1342c6d78 Mon Sep 17 00:00:00 2001 From: Jun Zhao Date: Tue, 31

Re: [FFmpeg-devel] [PATCH] avfilter/vf_tile: remove limit of max tile size

2017-10-31 Thread Nicolas George
Nack. Le septidi 7 brumaire, an CCXXVI, Paul B Mahol a écrit : > Signed-off-by: Paul B Mahol > --- > libavfilter/vf_tile.c | 12 +++- > 1 file changed, 3 insertions(+), 9 deletions(-) > > diff --git a/libavfilter/vf_tile.c b/libavfilter/vf_tile.c > index

Re: [FFmpeg-devel] [PATCH 1/2] configure: Rename wincrypt to advapi32 for consistency with existing uses of check_lib

2017-10-31 Thread Hendrik Leppkes
On Tue, Oct 31, 2017 at 6:50 PM, Aaron Levinson wrote: > Signed-off-by: Aaron Levinson > --- > configure | 4 ++-- > 1 file changed, 2 insertions(+), 2 deletions(-) > > diff --git a/configure b/configure > index ea22d692b9..572299c9d3 100755 >

[FFmpeg-devel] Support for h264/SVC over RTP and SVC base layer decoding in h264

2017-10-31 Thread Breeden, Joshua
Hello, First patch submission... hopefully I do all of this correctly.? Added basic support for H264 Annex G (Scalable Video Coding) to RTP module. Implement splitting of PACSI NAL units (type 30). Used existing H264 SPS decoder to parse Subset SPS NAL units (type 15) in h264 extradata. As a

[FFmpeg-devel] [PATCH] avfilter/vf_tile: remove limit of max tile size

2017-10-31 Thread Paul B Mahol
Signed-off-by: Paul B Mahol --- libavfilter/vf_tile.c | 17 + 1 file changed, 13 insertions(+), 4 deletions(-) diff --git a/libavfilter/vf_tile.c b/libavfilter/vf_tile.c index 87e0b940cf..8eb0dc2097 100644 --- a/libavfilter/vf_tile.c +++ b/libavfilter/vf_tile.c

Re: [FFmpeg-devel] [PATCH] Fix missing used attribute for inline assembly variables

2017-10-31 Thread Teresa Johnson
It's needed for the same reason the used attribute was already added to the "static const" variables. For those, when building with just -O2, they could be removed by optimization since they had local (file) scope, and we couldn't see the uses in the inline assembly (without the used attribute).

Re: [FFmpeg-devel] [PATCH] Fix quadratic memory use in ff_h2645_extract_rbsp() when multiple NALUs exist in packet.

2017-10-31 Thread Kieran Kunhya
> > This reduces memory consumption to linear from qudratic but i think > it still can be made to allocate very large amounts of memory. > That is with many small NAL units MAX_MBPAIR_SIZE would be allocated > for each.in worst case. > So this does fix the qudratic issue but not the OOM issue. >

Re: [FFmpeg-devel] [PATCH] avcodec/dca: return standard error codes in avpriv_dca_parse_core_frame_header()

2017-10-31 Thread Michael Niedermayer
On Tue, Oct 31, 2017 at 12:47:06AM -0300, James Almer wrote: > This prevents making the DCAParseError enum part of the ABI. > > Signed-off-by: James Almer > --- > libavcodec/dca.c | 11 --- > libavcodec/dca.h | 12 ++-- > 2 files changed, 18 insertions(+), 5

Re: [FFmpeg-devel] [PATCH 1/2] configure: Rename wincrypt to advapi32 for consistency with existing uses of check_lib

2017-10-31 Thread Aaron Levinson
On 10/31/2017 11:44 AM, Hendrik Leppkes wrote: On Tue, Oct 31, 2017 at 6:50 PM, Aaron Levinson wrote: Signed-off-by: Aaron Levinson --- configure | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/configure

[FFmpeg-devel] [PATCH] avformat/utils: Look at the first 3 frames if timestamps indicate frame reorder but decoder delay does not

2017-10-31 Thread Michael Niedermayer
Fixes: Ticket6487 Signed-off-by: Michael Niedermayer --- libavformat/avformat.h| 1 + libavformat/utils.c | 16 tests/ref/fate/mov-zombie | 4 ++-- 3 files changed, 15 insertions(+), 6 deletions(-) diff --git a/libavformat/avformat.h

[FFmpeg-devel] haunted by "undefined reference to `vaUnmapBuffer'"

2017-10-31 Thread Sampsa Riikonen
Dear List, I am compiling a stripped-down, static versions of the libraries, i.e. : libavcodec.a libavformat.a libavutil.a libswscale.a libavfilter.a libswresample.a and I'm "baking them in" into a custom shared-library. All external libraries are disabled, and configure script tells me

Re: [FFmpeg-devel] [PATCH] Fix quadratic memory use in ff_h2645_extract_rbsp() when multiple NALUs exist in packet.

2017-10-31 Thread Derek Buitenhuis
On 10/31/2017 2:25 AM, Michael Niedermayer wrote: > (though as said, this fix is not ideal or complete, I would very much > prefer if this would be fixed by using a single buffer or any other > solution that avoids the speedloss.) Using a single buffer would be marginally faster, but it does

Re: [FFmpeg-devel] [PATCH] Fix quadratic memory use in ff_h2645_extract_rbsp() when multiple NALUs exist in packet.

2017-10-31 Thread Niki Bowe
Thank you Kieran, Michael, Derek, and Carl. I ran some benchmarks and the patch I provided really does have a speed impact so I was considering alternative ways to achieve the same result. After a little thought last week I came to the same conclusion as Michael that a single rbsp_buffer per

[FFmpeg-devel] [PATCH] avcodec/dca: return standard error codes in avpriv_dca_parse_core_frame_header()

2017-10-31 Thread James Almer
This prevents making the DCAParseError enum part of the ABI. Signed-off-by: James Almer --- Now actually paying attention to -Wempty-body warnings. libavcodec/dca.c | 11 --- libavcodec/dca.h | 12 ++-- 2 files changed, 18 insertions(+), 5 deletions(-) diff

Re: [FFmpeg-devel] [PATCH] Fix quadratic memory use in ff_h2645_extract_rbsp() when multiple NALUs exist in packet.

2017-10-31 Thread Kieran Kunhya
> > Am I missing something here? > > P.S. I see Kieran mailed the same thing as I wrote this. > Further to Derek's excellent explanation, I think the best way is to go back to the old "in-place" NAL parsing code for H264. I will prepare a patch to do this. Kieran

Re: [FFmpeg-devel] [PATCH] Fix quadratic memory use in ff_h2645_extract_rbsp() when multiple NALUs exist in packet.

2017-10-31 Thread Kieran Kunhya
On Tue, 31 Oct 2017 at 22:36 Niki Bowe wrote: > I was going to prepare a patch, but it sounds like Kieran is going to. > Thank you Kieran. Let me know if there's anything you want me to do, or if > you would prefer me to do it for the experience. > > As for the

[FFmpeg-devel] [PATCH v6 2/2 ] avformat/mpegts: opus demuxing for mapping family 255

2017-10-31 Thread pkv.stream
Le 31/10/2017 à 6:28 PM, Michael Niedermayer a écrit : mpegts.c | 84 --- 1 file changed, 70 insertions(+), 14 deletions(-) 6648bab53735f1efaf0d6cd698dbc2398b483c3f 0002-libavf-mpegts-opus-demuxing-for-mapping-family-255.patch From

[FFmpeg-devel] [PATCH v6 1/2 ] avformat/mpegts: opus muxing for mapping family 255

2017-10-31 Thread pkv.stream
Le 29/10/2017 à 4:44 PM, Michael Niedermayer a écrit : On Sat, Oct 28, 2017 at 03:49:13AM +0200, pkv.stream wrote: mpegtsenc.c | 67 +--- 1 file changed, 60 insertions(+), 7 deletions(-) 1423a193788547952e3c4cdcb424b4724b0f1157

Re: [FFmpeg-devel] haunted by "undefined reference to `vaUnmapBuffer'"

2017-10-31 Thread Carl Eugen Hoyos
2017-10-31 20:38 GMT+01:00 Sampsa Riikonen : > I am compiling Please post on the user mailing list: http://ffmpeg.org/contact.html#MailingLists Carl Eugen ___ ffmpeg-devel mailing list ffmpeg-devel@ffmpeg.org

[FFmpeg-devel] [PATCH] lavf/mov.c: Refine edit list start seek, based on PTS computed from CTTS.

2017-10-31 Thread Sasi Inguva
Partially fixes t/6699. --- libavformat/mov.c | 125 +++-- tests/fate/mov.mak | 8 2 files changed, 90 insertions(+), 43 deletions(-) diff --git a/libavformat/mov.c b/libavformat/mov.c index 60f0228e2d..a295445651 100644 ---

Re: [FFmpeg-devel] [PATCH] lavf/mov.c: Refine edit list start seek, based on PTS computed from CTTS.

2017-10-31 Thread Sasi Inguva
Attaching the fate sample. On Tue, Oct 31, 2017 at 4:42 PM, Sasi Inguva wrote: > Partially fixes t/6699. > --- > libavformat/mov.c | 125 ++ > +-- > tests/fate/mov.mak | 8 > 2 files changed, 90 insertions(+), 43

Re: [FFmpeg-devel] On in-tree external headers

2017-10-31 Thread Carl Eugen Hoyos
2017-10-31 17:52 GMT+01:00 Mark Thompson : > On 31/10/17 16:34, Timo Rothenpieler wrote: >> We're not bundling entire 3rd party libs in-tree here, just >> API headers for system/driver APIs. > > What exactly would the policy be, then? > > "External headers for a system/driver API

Re: [FFmpeg-devel] [PATCH] avformat/movenc: let avpriv_ac3_parse_header() allocate the AC3HeaderInfo struct

2017-10-31 Thread Michael Niedermayer
On Tue, Oct 31, 2017 at 01:24:58AM -0300, James Almer wrote: > This removes sizeof(AC3HeaderInfo) from the ABI. > > Signed-off-by: James Almer > --- > libavformat/movenc.c | 72 > +++- > 1 file changed, 43 insertions(+), 29

[FFmpeg-devel] [PATCH 2/3] examples/filtering_video: suppress the build warning.

2017-10-31 Thread Jun Zhao
From e192b42ab0fdc4b8faad50ff7bb01c6028945f19 Mon Sep 17 00:00:00 2001 From: Jun Zhao Date: Tue, 31 Oct 2017 16:11:06 +0800 Subject: [PATCH 2/3] examples/filtering_video: suppress the build warning. MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8

[FFmpeg-devel] [PATCH 3/3] examples/filtering_audio: suppress the build warning.

2017-10-31 Thread Jun Zhao
From 6e8dbeb70f03312deee47a3cf0b5b47a7241ffb5 Mon Sep 17 00:00:00 2001 From: Jun Zhao Date: Tue, 31 Oct 2017 16:12:36 +0800 Subject: [PATCH 3/3] examples/filtering_audio: suppress the build warning. MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8

Re: [FFmpeg-devel] [PATCH 1/3] examples/transcoding: suppress build warning.

2017-10-31 Thread Steven Liu
2017-10-31 16:22 GMT+08:00 Jun Zhao : > > > ___ > ffmpeg-devel mailing list > ffmpeg-devel@ffmpeg.org > http://ffmpeg.org/mailman/listinfo/ffmpeg-devel > From 3ee6e7f01aa2c709afa7ad23f39af919c64f85d5 Mon Sep 17 00:00:00 2001 From:

Re: [FFmpeg-devel] [PATCH] lavfi/testsrc2: fix hang with very small sizes.

2017-10-31 Thread Nicolas George
Le decadi 10 brumaire, an CCXXVI, Martin Vignali a écrit : > lgtm Thanks, pushed. Regards, -- Nicolas George signature.asc Description: Digital signature ___ ffmpeg-devel mailing list ffmpeg-devel@ffmpeg.org

[FFmpeg-devel] [PATCH 1/3] examples/transcoding: suppress build warning.

2017-10-31 Thread Jun Zhao
From 3ee6e7f01aa2c709afa7ad23f39af919c64f85d5 Mon Sep 17 00:00:00 2001 From: Jun Zhao Date: Tue, 31 Oct 2017 16:09:45 +0800 Subject: [PATCH 1/3] examples/transcoding: suppress build warning. MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding:

Re: [FFmpeg-devel] [PATCH 3/3] examples/filtering_audio: suppress the build warning.

2017-10-31 Thread Steven Liu
2017-10-31 16:22 GMT+08:00 Jun Zhao : > > > ___ > ffmpeg-devel mailing list > ffmpeg-devel@ffmpeg.org > http://ffmpeg.org/mailman/listinfo/ffmpeg-devel > patchset LGTM ___ ffmpeg-devel

Re: [FFmpeg-devel] [PATCH] Fix quadratic memory use in ff_h2645_extract_rbsp() when multiple NALUs exist in packet.

2017-10-31 Thread Kieran Kunhya
On Tue, 31 Oct 2017 at 02:26 Michael Niedermayer wrote: > On Thu, Oct 19, 2017 at 11:46:47AM -0700, Nikolas Bowe wrote: > > Found via fuzzing. > > /tmp/poc is a 1 MB mpegts file generated via fuzzing, where 1 packet has > many NALUs > > Before this change: > > $

Re: [FFmpeg-devel] [PATCH] avcodec/mips: Improve hevc bi hz and hv mc msa functions

2017-10-31 Thread Manojkumar Bhosale
LGTM -Original Message- From: ffmpeg-devel [mailto:ffmpeg-devel-boun...@ffmpeg.org] On Behalf Of kaustubh.ra...@imgtec.com Sent: Friday, October 27, 2017 6:35 PM To: ffmpeg-devel@ffmpeg.org Cc: Kaustubh Raste Subject: [FFmpeg-devel] [PATCH] avcodec/mips: Improve hevc bi hz and hv mc msa

Re: [FFmpeg-devel] [PATCH] avcodec/mips: Improve hevc uni vt and hv mc msa functions

2017-10-31 Thread Manojkumar Bhosale
LGTM -Original Message- From: ffmpeg-devel [mailto:ffmpeg-devel-boun...@ffmpeg.org] On Behalf Of kaustubh.ra...@imgtec.com Sent: Friday, October 27, 2017 6:59 PM To: ffmpeg-devel@ffmpeg.org Cc: Kaustubh Raste Subject: [FFmpeg-devel] [PATCH] avcodec/mips: Improve hevc uni vt and hv mc msa

[FFmpeg-devel] [PATCH] avformat/img2enc: add frame_pts option for make output filename

2017-10-31 Thread Steven Liu
when use frame_pts option, the output image name can be set with PTS of current frame. Signed-off-by: Steven Liu --- doc/muxers.texi | 9 + libavformat/img2enc.c | 8 2 files changed, 17 insertions(+) diff --git a/doc/muxers.texi b/doc/muxers.texi

Re: [FFmpeg-devel] [PATCH 2/2] lavc/vaapi_encode_h264: correct bit_rate_scale setting.

2017-10-31 Thread Mark Thompson
On 31/10/17 02:37, Jun Zhao wrote: > From d1e105057e93e7c2788d6d684292db9008fbf3ac Mon Sep 17 00:00:00 2001 > From: Jun Zhao > Date: Tue, 31 Oct 2017 10:19:08 +0800 > Subject: [PATCH 2/2] lavc/vaapi_encode_h264: correct bit_rate_scale setting. > > As H264 Spec 2012 E.2.2,

Re: [FFmpeg-devel] [PATCH 1/2] lavc/vaapi_encode: correct the HRD buffer size calculate.

2017-10-31 Thread Mark Thompson
On 31/10/17 02:37, Jun Zhao wrote: > From 7eef9be1c8a92bf625d62a0f97f762f1342c6d78 Mon Sep 17 00:00:00 2001 > From: Jun Zhao > Date: Tue, 31 Oct 2017 10:13:42 +0800 > Subject: [PATCH 1/2] lavc/vaapi_encode: correct the HRD buffer size calculate. > > when rc_buffer_size didn't

[FFmpeg-devel] [PATCH] Fix missing used attribute for inline assembly variables

2017-10-31 Thread Thomas Köppe
Variables used in inline assembly need to be marked with attribute((used)). Static constants already were, via the define of DECLARE_ASM_CONST. But DECLARE_ALIGNED does not add this attribute, and some of the variables defined with it are const only used in inline assembly, and therefore appeared

Re: [FFmpeg-devel] On in-tree external headers

2017-10-31 Thread Carl Eugen Hoyos
2017-11-01 1:04 GMT+01:00 Mark Thompson : > This is why I'm concerned that we are facilitating anti-open > behaviour from Nvidia by making it easier to use the > closed software than more open alternatives. But you do agree that there is nothing "open" about the AMD driver in

Re: [FFmpeg-devel] haunted by "undefined reference to `vaUnmapBuffer'"

2017-10-31 Thread Moritz Barsnick
Hi Sampsa On Tue, Oct 31, 2017 at 21:38:42 +0200, Sampsa Riikonen wrote: > Dear List, > I am compiling a stripped-down, static versions of the libraries, i.e. : > > libavcodec.a libavformat.a libavutil.a libswscale.a libavfilter.a > libswresample.a > > and I'm "baking them in" into a custom

Re: [FFmpeg-devel] On in-tree external headers

2017-10-31 Thread Carl Eugen Hoyos
2017-11-01 1:23 GMT+01:00 Mark Thompson : > Or libx264, which also lacks headers inside ffmpeg. (We > could dynamically load libx264 using reverse-enginereed > non-GPL headers and then be able to enable it everywhere! But only with --enable-gpl, making the stunt less useful.

Re: [FFmpeg-devel] [PATCH] Fix missing used attribute for inline assembly variables

2017-10-31 Thread Michael Niedermayer
Hi On Tue, Oct 31, 2017 at 04:29:18PM +, Teresa Johnson wrote: > It's needed for the same reason the used attribute was already added to the > "static const" variables. For those, when building with just -O2, they > could be removed by optimization since they had local (file) scope, and we >

Re: [FFmpeg-devel] [PATCH] avformat/mux: be less strict with bitstream filter failures

2017-10-31 Thread Carl Eugen Hoyos
2017-10-31 19:00 GMT+01:00 James Almer : > This makes the autobsf feature behave the same as the manual > bitstream filtering in ffmpeg.c > > Fixes ticket #6794 > > Signed-off-by: James Almer > --- > libavformat/mux.c | 4 +++- > 1 file changed, 3

Re: [FFmpeg-devel] On in-tree external headers

2017-10-31 Thread Mark Thompson
On 31/10/17 23:52, Carl Eugen Hoyos wrote: > 2017-10-31 17:52 GMT+01:00 Mark Thompson : >> On 31/10/17 16:34, Timo Rothenpieler wrote: > >>> We're not bundling entire 3rd party libs in-tree here, just >>> API headers for system/driver APIs. >> >> What exactly would the policy be,

Re: [FFmpeg-devel] On in-tree external headers

2017-10-31 Thread Mark Thompson
On 01/11/17 00:29, Carl Eugen Hoyos wrote: > 2017-11-01 1:23 GMT+01:00 Mark Thompson : > >> Or libx264, which also lacks headers inside ffmpeg. (We >> could dynamically load libx264 using reverse-enginereed >> non-GPL headers and then be able to enable it everywhere! > > But

Re: [FFmpeg-devel] [PATCH] Fix quadratic memory use in ff_h2645_extract_rbsp() when multiple NALUs exist in packet.

2017-10-31 Thread Michael Niedermayer
On Tue, Oct 31, 2017 at 08:24:37PM +, Derek Buitenhuis wrote: > On 10/31/2017 2:25 AM, Michael Niedermayer wrote: > > (though as said, this fix is not ideal or complete, I would very much > > prefer if this would be fixed by using a single buffer or any other > > solution that avoids the

[FFmpeg-devel] [PATCH] lavf/mov.c: Refine edit list start seek, based on PTS computed from CTTS.

2017-10-31 Thread Sasi Inguva
Partially fixes t/6699. --- libavformat/mov.c| 125 +++ tests/fate/mov.mak | 8 +++ tests/ref/fate/mov-ibi-elst-starts-b | 33 + 3 files changed, 123 insertions(+), 43 deletions(-) create mode 100644

[FFmpeg-devel] (no subject)

2017-10-31 Thread Sasi Inguva
Added the fate ref file to the commit. Thx. ___ ffmpeg-devel mailing list ffmpeg-devel@ffmpeg.org http://ffmpeg.org/mailman/listinfo/ffmpeg-devel

Re: [FFmpeg-devel] [RFC]lswr/rematrix: Support s32p

2017-10-31 Thread Carl Eugen Hoyos
2017-10-31 17:28 GMT+01:00 Michael Niedermayer : >> rematrix.c | 21 +++-- >> 1 file changed, 15 insertions(+), 6 deletions(-) >> 9f50e61370cc72de000bb1174cbd8666388fdd9a 0001-lswr-Support-s32p.patch >> From a93b9309d74f5eadece371ee1e682d266af6cd83 Mon

Re: [FFmpeg-devel] On in-tree external headers

2017-10-31 Thread Mark Thompson
On 01/11/17 00:17, Mark Thompson wrote: > On 01/11/17 00:07, Carl Eugen Hoyos wrote: >> 2017-11-01 1:04 GMT+01:00 Mark Thompson : >>> This is why I'm concerned that we are facilitating anti-open >>> behaviour from Nvidia by making it easier to use the >>> closed software than more

Re: [FFmpeg-devel] [PATCH]lavf/latmenc: Error out for invalid codecs

2017-10-31 Thread Michael Niedermayer
On Wed, Nov 01, 2017 at 02:17:06AM +0100, Carl Eugen Hoyos wrote: > 2017-10-31 17:38 GMT+01:00 Michael Niedermayer : > > On Mon, Oct 30, 2017 at 11:51:30PM +0100, Carl Eugen Hoyos wrote: > >> Hi! > >> > >> Attached patch makes sure the loas muxer does not try to write > >>

Re: [FFmpeg-devel] Support for h264/SVC over RTP and SVC base layer decoding in h264

2017-10-31 Thread Carl Eugen Hoyos
2017-10-31 19:53 GMT+01:00 Breeden, Joshua : > Hello, > > First patch submission... hopefully I do all of this correctly.? Your patches contain a lot of trailing whitespace (afaict...) that cannot be committed to our repository, please remove it. To find it you can use the

Re: [FFmpeg-devel] [PATCH] lavf/vf_ocr: check ff_set_common_formats() return value

2017-10-31 Thread Steven Liu
2017-11-01 8:30 GMT+08:00 Moritz Barsnick : > Signed-off-by: Moritz Barsnick > --- > > The only remaining unchecked one I could find in lavf. Fixes a warning, > obviously. > > libavfilter/vf_ocr.c | 4 +--- > 1 file changed, 1 insertion(+), 3 deletions(-) > >

Re: [FFmpeg-devel] [PATCH] lavf/mov.c: Refine edit list start seek, based on PTS computed from CTTS.

2017-10-31 Thread Sasi Inguva
Thanks. Sent the corrected patch. On Tue, Oct 31, 2017 at 7:14 PM, Michael Niedermayer wrote: > On Tue, Oct 31, 2017 at 04:42:56PM -0700, Sasi Inguva wrote: > > Partially fixes t/6699. > > --- > > libavformat/mov.c | 125 ++ >

Re: [FFmpeg-devel] [PATCH]lavf/latmenc: Error out for invalid codecs

2017-10-31 Thread Carl Eugen Hoyos
2017-10-31 17:38 GMT+01:00 Michael Niedermayer : > On Mon, Oct 30, 2017 at 11:51:30PM +0100, Carl Eugen Hoyos wrote: >> Hi! >> >> Attached patch makes sure the loas muxer does not try to write >> anything but aac and latm. >> >> Please comment, Carl Eugen > >> latmenc.c |

Re: [FFmpeg-devel] On in-tree external headers

2017-10-31 Thread Carl Eugen Hoyos
2017-11-01 1:34 GMT+01:00 Mark Thompson : > On 01/11/17 00:29, Carl Eugen Hoyos wrote: >> 2017-11-01 1:23 GMT+01:00 Mark Thompson : >> >>> Or libx264, which also lacks headers inside ffmpeg. (We >>> could dynamically load libx264 using reverse-enginereed >>>

Re: [FFmpeg-devel] On in-tree external headers

2017-10-31 Thread Michael Niedermayer
On Wed, Nov 01, 2017 at 12:34:45AM +, Mark Thompson wrote: > On 01/11/17 00:29, Carl Eugen Hoyos wrote: > > 2017-11-01 1:23 GMT+01:00 Mark Thompson : > > > >> Or libx264, which also lacks headers inside ffmpeg. (We > >> could dynamically load libx264 using reverse-enginereed

Re: [FFmpeg-devel] [PATCH 3/3] examples/filtering_audio: suppress the build warning.

2017-10-31 Thread Michael Niedermayer
On Tue, Oct 31, 2017 at 04:29:02PM +0800, Steven Liu wrote: > 2017-10-31 16:22 GMT+08:00 Jun Zhao : > > > > > > ___ > > ffmpeg-devel mailing list > > ffmpeg-devel@ffmpeg.org > > http://ffmpeg.org/mailman/listinfo/ffmpeg-devel > > >

Re: [FFmpeg-devel] [PATCH] lavf/mov.c: Refine edit list start seek, based on PTS computed from CTTS.

2017-10-31 Thread Michael Niedermayer
On Tue, Oct 31, 2017 at 04:42:56PM -0700, Sasi Inguva wrote: > Partially fixes t/6699. > --- > libavformat/mov.c | 125 > +++-- > tests/fate/mov.mak | 8 > 2 files changed, 90 insertions(+), 43 deletions(-) Missing:

Re: [FFmpeg-devel] On in-tree external headers

2017-10-31 Thread Mark Thompson
On 01/11/17 00:07, Carl Eugen Hoyos wrote: > 2017-11-01 1:04 GMT+01:00 Mark Thompson : >> This is why I'm concerned that we are facilitating anti-open >> behaviour from Nvidia by making it easier to use the >> closed software than more open alternatives. > > But you do agree that

[FFmpeg-devel] [PATCH] lavf/vf_ocr: check ff_set_common_formats() return value

2017-10-31 Thread Moritz Barsnick
Signed-off-by: Moritz Barsnick --- The only remaining unchecked one I could find in lavf. Fixes a warning, obviously. libavfilter/vf_ocr.c | 4 +--- 1 file changed, 1 insertion(+), 3 deletions(-) diff --git a/libavfilter/vf_ocr.c b/libavfilter/vf_ocr.c index

Re: [FFmpeg-devel] [PATCH] avcodec/dca: return standard error codes in avpriv_dca_parse_core_frame_header()

2017-10-31 Thread Michael Niedermayer
On Tue, Oct 31, 2017 at 05:27:35PM -0300, James Almer wrote: > This prevents making the DCAParseError enum part of the ABI. > > Signed-off-by: James Almer > --- > Now actually paying attention to -Wempty-body warnings. > > libavcodec/dca.c | 11 --- >