Re: [FFmpeg-devel] [PATCH]lavc/hevcdec: Silence warnings when decoding DolbyVision

2017-10-22 Thread James Almer
On 10/21/2017 9:31 PM, Carl Eugen Hoyos wrote: > Hi! > > Attached patch silences the many warnings shown when decoding streams > with DolbyVision content. > > Please comment, Carl Eugen > > > 0001-lavc-hevcdec-Silence-warnings-when-decoding-DolbyVis.patch > > > From

Re: [FFmpeg-devel] [PATCH] avutil/crc: always use precalculated CRC tables for known polynomials

2017-10-22 Thread Michael Niedermayer
On Sun, Oct 22, 2017 at 10:03:55AM -0300, James Almer wrote: > This prevents data races in av_crc_get_table() the modules which use a specific crc can init it during their initialization. the initialization should be thread safe as it never writes a different value in the same spot This would

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

2017-10-22 Thread Michael Niedermayer
On Sat, Oct 21, 2017 at 08:13:15AM +, Dixit, Vishwanath wrote: > Hi, [...] > @@ -1324,17 +1345,183 @@ static const char * > get_default_pattern_localtime_fmt(AVFormatContext *s) > return (HAVE_LIBC_MSVCRT || !strftime(b, sizeof(b), "%s", p) || > !strcmp(b, "%s")) ? "-%Y%m%d%H%M%S.ts" :

Re: [FFmpeg-devel] [RFC] v4l2_m2m: Fix races around freeing data on close

2017-10-22 Thread Mark Thompson
On 19/10/17 18:55, Jorge Ramirez wrote: > On 10/19/2017 11:49 AM, Mark Thompson wrote: >> On 19/10/17 08:28, Jorge Ramirez wrote: >>> On 10/19/2017 02:10 AM, Mark Thompson wrote: Refcount all of the context information. --- As discussed in the other thread, something like this.  We

[FFmpeg-devel] [PATCH] avcodec/tak: remove GetBitContext usage from avpriv_tak_parse_streaminfo()

2017-10-22 Thread James Almer
This prevents potential ABI issues with GetBitContext. Signed-off-by: James Almer --- libavcodec/tak.c | 17 +++-- libavcodec/tak.h | 8 ++-- libavformat/takdec.c | 7 +-- 3 files changed, 26 insertions(+), 6 deletions(-) diff --git

Re: [FFmpeg-devel] [PATCH] avutil/crc: always use precalculated CRC tables for known polynomials

2017-10-22 Thread Carl Eugen Hoyos
2017-10-22 15:03 GMT+02:00 James Almer : > This prevents data races in av_crc_get_table() Doesn't this mix two changes - moving the tables into a dedicated file and making them static - that are not necessarily related? Carl Eugen

Re: [FFmpeg-devel] [PATCH]lavc/bitstream_filter: Make a cast explicit

2017-10-22 Thread Carl Eugen Hoyos
2017-10-22 23:22 GMT+02:00 Michael Niedermayer : > On Wed, Mar 01, 2017 at 11:36:58PM +0100, Carl Eugen Hoyos wrote: >> Hi! >> >> Attached patch silences one of three warnings when compiling >> bitstream_filter.o, I suspect this cast is necessary. >> >> Please comment, Carl

Re: [FFmpeg-devel] [PATCH]lavc/avcodec: Constify the return value of av_bitstream_filter_next().

2017-10-22 Thread Carl Eugen Hoyos
2017-10-21 21:17 GMT+02:00 Carl Eugen Hoyos : > Hi! > > Attached patch fixes a warning when compiling with sufficiently new gcc. Working patch attached. Carl Eugen From 19077d0125e1e1eca81dc0311440c1a7da16e080 Mon Sep 17 00:00:00 2001 From: Carl Eugen Hoyos

Re: [FFmpeg-devel] [PATCH]lavc/bitstream_filter: Make a cast explicit

2017-10-22 Thread Michael Niedermayer
On Wed, Mar 01, 2017 at 11:36:58PM +0100, Carl Eugen Hoyos wrote: > Hi! > > Attached patch silences one of three warnings when compiling > bitstream_filter.o, I suspect this cast is necessary. > > Please comment, Carl Eugen probably ok [...] -- Michael GnuPG fingerprint:

Re: [FFmpeg-devel] fate/hap : add test for HAPQA decoding

2017-10-22 Thread Michael Niedermayer
On Sat, Oct 21, 2017 at 07:37:31PM +0200, Martin Vignali wrote: > Hello, > > Sample can be found here > https://we.tl/1XuI6QJ7Ra uploaded thx [...] -- Michael GnuPG fingerprint: 9FF2128B147EF6730BADF133611EC787040B0FAB Breaking DRM is a little like attempting to break through a door

Re: [FFmpeg-devel] order T-shirts

2017-10-22 Thread Thilo Borgmann
Am 22.10.17 um 18:18 schrieb Ronald S. Bultje: > Hi, > > On Sun, Oct 22, 2017 at 7:23 AM, Thilo Borgmann > wrote: > >> Am 22.10.17 um 09:55 schrieb Liu Steven: >>> 在 2017年10月22日,上午2:42,Lou Logan 写道: I was thinking of black shirt, green

[FFmpeg-devel] libavcodec/bswapdsp : add AVX2 for bswap_buf

2017-10-22 Thread Martin Vignali
Hello, In attach patch in order to add an AVX2 version for bswap_buf (swap uint32 in a buffer) Checkasm result (Kaby Lake 10.12) bswap_buf_c: 122.8 bswap_buf_sse2: 67.8 bswap_buf_ssse3: 34.3 bswap_buf_avx2: 21.0 Pass checkasm and fate test for me Martin Jokyo Images

Re: [FFmpeg-devel] order T-shirts

2017-10-22 Thread Ronald S. Bultje
Hi, On Sun, Oct 22, 2017 at 7:23 AM, Thilo Borgmann wrote: > Am 22.10.17 um 09:55 schrieb Liu Steven: > > > >> 在 2017年10月22日,上午2:42,Lou Logan 写道: > >> > >> I was thinking of black shirt, green logo, white "FFmpeg" lettering. > > I like this thinking too

Re: [FFmpeg-devel] [PATCH] avcodec/version: Postpone FF_API_DEBUG_MV

2017-10-22 Thread Ronald S. Bultje
Hi, On Sun, Oct 22, 2017 at 9:43 AM, James Almer wrote: > On 10/22/2017 8:26 AM, Ronald S. Bultje wrote: > > Hi, > > > > On Sat, Oct 21, 2017 at 9:23 PM, Michael Niedermayer > >> wrote: > > > >> On Sat, Oct 21, 2017 at 08:52:26PM -0400, Ronald S.

[FFmpeg-devel] [PATCH 1/2] ffserver: Fix off by 1 error in path

2017-10-22 Thread Michael Niedermayer
Signed-off-by: Michael Niedermayer --- fftools/ffserver.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/fftools/ffserver.c b/fftools/ffserver.c index d4885dfa0e..51f31bc704 100644 --- a/fftools/ffserver.c +++ b/fftools/ffserver.c @@ -499,9 +499,9

[FFmpeg-devel] [PATCH 2/2] tests/ffserver.regression.ref: update checksums to what ffserver currently produces

2017-10-22 Thread Michael Niedermayer
Signed-off-by: Michael Niedermayer --- tests/ffserver.regression.ref | 10 +- 1 file changed, 5 insertions(+), 5 deletions(-) diff --git a/tests/ffserver.regression.ref b/tests/ffserver.regression.ref index 398c285156..fd8536c853 100644 ---

Re: [FFmpeg-devel] [PATCH] mov: fix decode of fragments that overlap in time

2017-10-22 Thread John Stebbins
On 10/12/2017 10:59 AM, John Stebbins wrote: > When keyframe intervals of dash segments are not perfectly aligned, > fragments in the stream can overlap in time. The previous sorting by > timestamp causes packets to be read out of decode order and results > in decode errors. > > Insert new "trun"

Re: [FFmpeg-devel] [PATCH] avcodec/version: Postpone FF_API_DEBUG_MV

2017-10-22 Thread Clément Bœsch
On Sun, Oct 22, 2017 at 10:43:35AM -0300, James Almer wrote: [...] > Personally I'm with Clement in this. The deprecation was poorly handled, > and the feature currently has no replacement. Michael uses it for > debugging, so removing it does not seem productive. Not just Michael, it's generally

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

2017-10-22 Thread Paul B Mahol
On 10/22/17, Michael Niedermayer wrote: > On Sun, Oct 22, 2017 at 07:28:31AM -0400, Ronald S. Bultje wrote: >> Hi, >> >> On Sat, Oct 21, 2017 at 7:41 PM, Michael Niedermayer >> > > wrote: >> >> > add padding before scantable arrays >> > >> > See:

Re: [FFmpeg-devel] [PATCH] avcodec/version: Postpone FF_API_DEBUG_MV

2017-10-22 Thread James Almer
On 10/22/2017 8:26 AM, Ronald S. Bultje wrote: > Hi, > > On Sat, Oct 21, 2017 at 9:23 PM, Michael Niedermayer > wrote: > >> On Sat, Oct 21, 2017 at 08:52:26PM -0400, Ronald S. Bultje wrote: >>> Hi, >>> >>> On Sat, Oct 21, 2017 at 8:37 PM, Michael Niedermayer >>

Re: [FFmpeg-devel] [PATCH] avutil/crc: always use precalculated CRC tables for known polynomials

2017-10-22 Thread Derek Buitenhuis
On 10/22/2017 2:11 PM, James Almer wrote: > It was suggested, but nobody gave it a try (Or they did but found it > wasn't as simple as first thought?). [...] > Thread sanitizer complains about this in every other run, and the tables > are at most 1k each, so this is not a bad solution and can be

Re: [FFmpeg-devel] [PATCH] avutil/crc: always use precalculated CRC tables for known polynomials

2017-10-22 Thread James Almer
On 10/22/2017 10:08 AM, Derek Buitenhuis wrote: > On 10/22/2017 2:03 PM, James Almer wrote: >> This prevents data races in av_crc_get_table() >> >> Signed-off-by: James Almer >> --- >> libavutil/Makefile |1 + >> libavutil/crc.c| 295 +- >>

Re: [FFmpeg-devel] [PATCH] avutil/crc: always use precalculated CRC tables for known polynomials

2017-10-22 Thread Derek Buitenhuis
On 10/22/2017 2:03 PM, James Almer wrote: > This prevents data races in av_crc_get_table() > > Signed-off-by: James Almer > --- > libavutil/Makefile |1 + > libavutil/crc.c| 295 +- > libavutil/crc_tables.c | 1030 >

[FFmpeg-devel] [PATCH] avutil/crc: always use precalculated CRC tables for known polynomials

2017-10-22 Thread James Almer
This prevents data races in av_crc_get_table() Signed-off-by: James Almer --- libavutil/Makefile |1 + libavutil/crc.c| 295 +- libavutil/crc_tables.c | 1030 libavutil/crc_tables.h | 33 ++ 4

Re: [FFmpeg-devel] [PATCH 0/6] Patchset to remove ffserver

2017-10-22 Thread James Almer
On 10/22/2017 7:15 AM, Paul B Mahol wrote: > On 10/22/17, Michael Niedermayer wrote: >> On Sun, Oct 22, 2017 at 10:37:28AM +0200, Clement Boesch wrote: >>> On Sun, Oct 22, 2017 at 02:55:38AM +0200, Michael Niedermayer wrote: On Sat, Oct 21, 2017 at 04:15:37PM -0300,

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

2017-10-22 Thread Michael Niedermayer
On Sun, Oct 22, 2017 at 07:28:31AM -0400, Ronald S. Bultje wrote: > Hi, > > On Sat, Oct 21, 2017 at 7:41 PM, Michael Niedermayer > wrote: > > > add padding before scantable arrays > > > > See: 522d850e68ec4b77d3477b3c8f55b1ba00a9d69a > > > > Signed-off-by: Michael

[FFmpeg-devel] libavcodec/utvideodsp : add avx2 version

2017-10-22 Thread Martin Vignali
Hello, In attach patch to add AVX2 version for the utvideodsp Checkasm result (Kaby Lake, os 10.12) restore_rgb_planes_c: 8371.0 restore_rgb_planes_sse2: 6583.7 restore_rgb_planes_avx2: 3596.5 restore_rgb_planes10_c: 16735.7 restore_rgb_planes10_sse2: 11478.5 restore_rgb_planes10_avx2: 7193.7

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

2017-10-22 Thread Ronald S. Bultje
Hi, On Sat, Oct 21, 2017 at 7:41 PM, Michael Niedermayer wrote: > add padding before scantable arrays > > See: 522d850e68ec4b77d3477b3c8f55b1ba00a9d69a > > Signed-off-by: Michael Niedermayer > --- > libavcodec/h264dec.h | 1 + > 1 file changed,

Re: [FFmpeg-devel] [PATCH] avcodec/version: Postpone FF_API_DEBUG_MV

2017-10-22 Thread Ronald S. Bultje
Hi, On Sat, Oct 21, 2017 at 9:23 PM, Michael Niedermayer wrote: > On Sat, Oct 21, 2017 at 08:52:26PM -0400, Ronald S. Bultje wrote: > > Hi, > > > > On Sat, Oct 21, 2017 at 8:37 PM, Michael Niedermayer > > > wrote: > > > > > This is different

Re: [FFmpeg-devel] order T-shirts

2017-10-22 Thread Thilo Borgmann
Am 22.10.17 um 09:55 schrieb Liu Steven: > >> 在 2017年10月22日,上午2:42,Lou Logan 写道: >> >> I was thinking of black shirt, green logo, white "FFmpeg" lettering. > I like this thinking too :D I must say that the black/white style comes closer to what our website has color-wise... So

Re: [FFmpeg-devel] [PATCH 0/6] Patchset to remove ffserver

2017-10-22 Thread Michael Niedermayer
On Sun, Oct 22, 2017 at 12:15:25PM +0200, Paul B Mahol wrote: > On 10/22/17, Michael Niedermayer wrote: > > On Sun, Oct 22, 2017 at 10:37:28AM +0200, Clement Boesch wrote: > >> On Sun, Oct 22, 2017 at 02:55:38AM +0200, Michael Niedermayer wrote: > >> > On Sat, Oct 21, 2017

[FFmpeg-devel] GSoC 2017 mentor summit review

2017-10-22 Thread Thilo Borgmann
Hi, Carl Eugen and me went to the mentor summit this year. We've met a lot of known and some yet unknown users and had a lot of interesting talks with them, as well as a lot conversations with representatives of other projects about OSS development in general and GSoC matters in particular. We

Re: [FFmpeg-devel] [PATCH 0/6] Patchset to remove ffserver

2017-10-22 Thread Carl Eugen Hoyos
2017-10-22 12:40 GMT+02:00 Clément Bœsch : > Removal was announced, people were aware, we got discussions, > but no one cared enough to do something about it. Wasn't the (only) question what should be done? Carl Eugen ___ ffmpeg-devel

Re: [FFmpeg-devel] [PATCH 0/6] Patchset to remove ffserver

2017-10-22 Thread Clément Bœsch
On Sun, Oct 22, 2017 at 10:51:38AM +0200, Marton Balint wrote: > > On Sun, 22 Oct 2017, Clément Bœsch wrote: > > > On Sun, Oct 22, 2017 at 02:55:38AM +0200, Michael Niedermayer wrote: > > > On Sat, Oct 21, 2017 at 04:15:37PM -0300, James Almer wrote: > > > > On 10/21/2017 3:54 PM, Rostislav

Re: [FFmpeg-devel] [PATCH 0/6] Patchset to remove ffserver

2017-10-22 Thread Paul B Mahol
On 10/22/17, Michael Niedermayer wrote: > On Sun, Oct 22, 2017 at 10:37:28AM +0200, Clement Boesch wrote: >> On Sun, Oct 22, 2017 at 02:55:38AM +0200, Michael Niedermayer wrote: >> > On Sat, Oct 21, 2017 at 04:15:37PM -0300, James Almer wrote: >> > > On 10/21/2017 3:54 PM,

Re: [FFmpeg-devel] [PATCH 0/6] Patchset to remove ffserver

2017-10-22 Thread Michael Niedermayer
On Sun, Oct 22, 2017 at 10:37:28AM +0200, Clément Bœsch wrote: > On Sun, Oct 22, 2017 at 02:55:38AM +0200, Michael Niedermayer wrote: > > On Sat, Oct 21, 2017 at 04:15:37PM -0300, James Almer wrote: > > > On 10/21/2017 3:54 PM, Rostislav Pehlivanov wrote: > > > > This patchset removes the

[FFmpeg-devel] [PATCH] swscale: Reduce verbosity of misalignment reporting

2017-10-22 Thread Henrik Gramner
It's a bit overzealous to complain about misalignment with AV_LOG_WARNING, especially since memory bandwidth is much more likely to be the bottleneck compared to data alignment which the user may not even have control over. --- libswscale/swscale.c | 18 +++--- 1 file changed, 3

Re: [FFmpeg-devel] [PATCH 0/6] Patchset to remove ffserver

2017-10-22 Thread Marton Balint
On Sun, 22 Oct 2017, Clément Bœsch wrote: On Sun, Oct 22, 2017 at 02:55:38AM +0200, Michael Niedermayer wrote: On Sat, Oct 21, 2017 at 04:15:37PM -0300, James Almer wrote: On 10/21/2017 3:54 PM, Rostislav Pehlivanov wrote: This patchset removes the long-deprecated ffserver program and all

Re: [FFmpeg-devel] [PATCH 0/6] Patchset to remove ffserver

2017-10-22 Thread Clément Bœsch
On Sun, Oct 22, 2017 at 02:55:38AM +0200, Michael Niedermayer wrote: > On Sat, Oct 21, 2017 at 04:15:37PM -0300, James Almer wrote: > > On 10/21/2017 3:54 PM, Rostislav Pehlivanov wrote: > > > This patchset removes the long-deprecated ffserver program and all > > > its privately exposed things

Re: [FFmpeg-devel] [PATCH] avcodec/version: Postpone FF_API_DEBUG_MV

2017-10-22 Thread Clément Bœsch
On Sat, Oct 21, 2017 at 11:23:29PM -0300, James Almer wrote: [...] > >> The commit even states it should have removed "all traces of its use", > >> but it looks like with the years more and more code was added to it, > >> seeing the removal commit 8933ac2079644fb09916f1875c569103aefe84b1 in > >>

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

2017-10-22 Thread Liu Steven
> 在 2017年10月21日,下午4:13,Dixit, Vishwanath 写道: > > Hi, > > Please find the attached patches which add support to create multiple HLS > variant streams and master playlist. > > Key advantages: > > 1. A single HLS encoder instance can handle multiple variant streams now. >

Re: [FFmpeg-devel] order T-shirts

2017-10-22 Thread Liu Steven
> 在 2017年10月22日,上午2:42,Lou Logan 写道: > > I was thinking of black shirt, green logo, white "FFmpeg" lettering. I like this thinking too :D > ___ > ffmpeg-devel mailing list > ffmpeg-devel@ffmpeg.org >