[FFmpeg-devel] [PATCH] fftools/qsv: add extra_hw_frames support

2018-07-25 Thread Zhong Li
Currently extra_hw_frames can't be applied to qsv since it doesn't call function avcodec_get_hw_frames_parameters(). Give an option to fix ticket #7261 though it is not a perfect soultion (allocate the minimum pool size internally and automatically). Signed-off-by: Zhong Li --- fftools/ffmpeg_q

[FFmpeg-devel] [PATCH] lavfi: add lumakey_opencl filter

2018-07-25 Thread Danil Iashchenko
Add lumakey_opencl filter. Behaves like existing lumakey filter. --- configure | 1 + libavfilter/Makefile| 2 + libavfilter/allfilters.c| 1 + libavfilter/opencl/lumakey.cl | 43 +++ libavfilter/opencl_source.h | 1 + libavfilter/vf_lu

Re: [FFmpeg-devel] [PATCH 0/6] x86 SIMD for dirac 10-bit wavelet transforms

2018-07-25 Thread James Darnley
On 2018-07-19 17:23, Rostislav Pehlivanov wrote: > Could you provide standard overall transform results using START/STOP_TIMER > rather than overall decoding speed? Ask and ye shall receive. > haar horizontal compose > sse2: 3.67x faster (45248±108.1 vs. 12328±21.1 decicycles) compared with

Re: [FFmpeg-devel] [PATCH 1/3] avfilter: Add support for colour range as a link parameter

2018-07-25 Thread Kieran O Leary
Hi, On Thu, Feb 22, 2018 at 4:15 PM, Philip Langdale wrote: > -BEGIN PGP SIGNED MESSAGE- > Hash: SHA1 > > On Thu, 22 Feb 2018 12:39:16 +0100 > Nicolas George wrote: > >> Philip Langdale (2018-02-21): >> > Negotiation is part of Paul's larger changeset, and will be a useful >> > feature.

Re: [FFmpeg-devel] [PATCH 1/3] avfilter: Add support for colour range as a link parameter

2018-07-25 Thread Paul B Mahol
On 7/25/18, Kieran O Leary wrote: > Hi, > > On Thu, Feb 22, 2018 at 4:15 PM, Philip Langdale wrote: >> -BEGIN PGP SIGNED MESSAGE- >> Hash: SHA1 >> >> On Thu, 22 Feb 2018 12:39:16 +0100 >> Nicolas George wrote: >> >>> Philip Langdale (2018-02-21): >>> > Negotiation is part of Paul's large

Re: [FFmpeg-devel] [PATCH] lavfi: add lumakey_opencl filter

2018-07-25 Thread James Almer
On 7/25/2018 9:13 AM, Danil Iashchenko wrote: > Add lumakey_opencl filter. Behaves like existing lumakey filter. Isn't it possible to keep each of these new OpenCL filters as an optional codepath within the C version, using an AVOption like "opencl" or "hwaccel" to toggle one or another? Or maybe

[FFmpeg-devel] About the maintainer of mips

2018-07-25 Thread yinshiyou-hf
I hered from the previous mantainer for mips that he was no longer part of mips company,and as a result, my patch was still pending review. Will ffmpeg community asign new mantainer for mips? If so, I hope I can be one of the candidates. I have uploaded some loongson optimization patchs for libvp

Re: [FFmpeg-devel] [PATCH 3/5] Renamed reinterlace to tinterlace

2018-07-25 Thread Vasile Toncu
What would be the next steps? On Tue, Jul 24, 2018 at 12:17 PM, Vasile Toncu wrote: > Fixed tabs. > Thank you for the feedback. > > On Fri, Jul 20, 2018 at 7:27 PM, Paul B Mahol wrote: > >> On 7/20/18, Vasile Toncu wrote: >> > Hi, >> > >> > Did you have the time to take a look at patches 2 and

Re: [FFmpeg-devel] [PATCH 3/5] Renamed reinterlace to tinterlace

2018-07-25 Thread Paul B Mahol
On 7/25/18, Vasile Toncu wrote: > What would be the next steps? > Ping patch every second. ___ ffmpeg-devel mailing list ffmpeg-devel@ffmpeg.org http://ffmpeg.org/mailman/listinfo/ffmpeg-devel

Re: [FFmpeg-devel] [PATCH 3/5] Renamed reinterlace to tinterlace

2018-07-25 Thread Thomas Mundt
2018-07-25 17:11 GMT+02:00 Vasile Toncu : > What would be the next steps? > > On Tue, Jul 24, 2018 at 12:17 PM, Vasile Toncu > wrote: > > > Fixed tabs. > > Thank you for the feedback. > > > > On Fri, Jul 20, 2018 at 7:27 PM, Paul B Mahol wrote: > > > >> On 7/20/18, Vasile Toncu wrote: > >> > Hi

[FFmpeg-devel] [PATCH 1/4] vf_tonemap: Update the default peak values

2018-07-25 Thread Vittorio Giovara
When there is no metadata attached to a frame, take into account both the PQ and HLG transfers, and change the HLG default value to 10: the value of 12 is the maximum range in scene referred light, but the reference OOTF maps this from 0 to 1000 cd/m² on the ideal HLG monitor. This matches what vf

[FFmpeg-devel] [PATCH 3/4] vf_tonemap_opencl: Move update_metadata() to a shared file

2018-07-25 Thread Vittorio Giovara
--- libavfilter/colorspace.c| 17 + libavfilter/colorspace.h| 1 + libavfilter/vf_tonemap_opencl.c | 19 +-- 3 files changed, 19 insertions(+), 18 deletions(-) diff --git a/libavfilter/colorspace.c b/libavfilter/colorspace.c index d6f6055401..c6682

[FFmpeg-devel] [PATCH 2/4] vf_tonemap[_opencl]: Move determine_signal_peak() to a shared file

2018-07-25 Thread Vittorio Giovara
The two functions are identical. Use the shared LumaCoeffients type too. --- libavfilter/Makefile| 2 +- libavfilter/colorspace.c| 29 + libavfilter/colorspace.h| 6 ++ libavfilter/vf_tonemap.c| 38 +++--

[FFmpeg-devel] [PATCH 4/4] vf_tonemap: Update hdr metadata with the new peak value

2018-07-25 Thread Vittorio Giovara
Less effective than the approach in vf_tonemap_opencl because there is no peak detection, but it's still a good idea to implement this. --- libavfilter/vf_tonemap.c | 2 ++ 1 file changed, 2 insertions(+) diff --git a/libavfilter/vf_tonemap.c b/libavfilter/vf_tonemap.c index b62532946b..98a2c4bd2

Re: [FFmpeg-devel] [PATCH]lavf/mov: Make auxiliary_offsets consistently size_t.

2018-07-25 Thread Michael Niedermayer
On Tue, Jul 24, 2018 at 10:36:06PM +0200, Carl Eugen Hoyos wrote: > 2018-07-21 22:43 GMT+02:00, Michael Niedermayer : > > On Sat, Jul 21, 2018 at 03:52:19PM +0200, Carl Eugen Hoyos wrote: > >> Hi! > >> > >> Attached patch fixes a warning on ppc32: > >> libavformat/mov.c: In function ‘mov_read_saio’

Re: [FFmpeg-devel] [PATCH]lavf/mov: Make auxiliary_offsets consistently size_t.

2018-07-25 Thread Carl Eugen Hoyos
2018-07-25 20:03 GMT+02:00, Michael Niedermayer : > On Tue, Jul 24, 2018 at 10:36:06PM +0200, Carl Eugen Hoyos wrote: >> 2018-07-21 22:43 GMT+02:00, Michael Niedermayer : >> > On Sat, Jul 21, 2018 at 03:52:19PM +0200, Carl Eugen Hoyos wrote: >> >> Hi! >> >> >> >> Attached patch fixes a warning on p

Re: [FFmpeg-devel] About the maintainer of mips

2018-07-25 Thread Carl Eugen Hoyos
2018-07-25 16:28 GMT+02:00, yinshiyou...@loongson.cn : > I hered from the previous mantainer for mips that he was no longer part of > mips company,and as a result, my patch was still pending review. > Will ffmpeg community asign new mantainer for mips? No, you have to send a patch that changes the

Re: [FFmpeg-devel] [PATCH 1/4] vf_tonemap: Update the default peak values

2018-07-25 Thread Carl Eugen Hoyos
2018-07-25 17:46 GMT+02:00, Vittorio Giovara : > When there is no metadata attached to a frame, take into account both > the PQ and HLG transfers, and change the HLG default value to 10 Sounds like two independent changes to me. (While splitting a function and adding it to a second filter can be

Re: [FFmpeg-devel] [PATCH]tools/qt-faststart: Allow free atoms after moov atom

2018-07-25 Thread Carl Eugen Hoyos
2018-07-19 23:18 GMT+02:00, Carl Eugen Hoyos : > 2018-07-12 1:28 GMT+02:00, Carl Eugen Hoyos : > >> Attached patch allows to fast-start mov files with free atoms after >> the moov atom. >> Tested with the sample from ticket #7277. > > Ping. > > As this is not trivial to test, I would prefer not to

Re: [FFmpeg-devel] [PATCH] avutil/hwcontext_d3d11va: fix type arguments passed to IDXGIAdapter2_GetDesc()

2018-07-25 Thread James Almer
On 7/24/2018 1:06 PM, Jean-Baptiste Kempf wrote: > LGTM. > > DXGI_ADAPTER_DESC2 should only be used for IDXGIAdapter2::GetDesc2() Pushed, thanks. ___ ffmpeg-devel mailing list ffmpeg-devel@ffmpeg.org http://ffmpeg.org/mailman/listinfo/ffmpeg-devel

Re: [FFmpeg-devel] [PATCH]configure: Force pie for Android

2018-07-25 Thread Carl Eugen Hoyos
2018-07-13 20:52 GMT+02:00, Carl Eugen Hoyos : > A user pointed out that Android requires pie since Lollipop. Patch applied. Carl Eugen ___ ffmpeg-devel mailing list ffmpeg-devel@ffmpeg.org http://ffmpeg.org/mailman/listinfo/ffmpeg-devel

Re: [FFmpeg-devel] [PATCH] swresample/swresample: Fix input channel count in resample_first computation

2018-07-25 Thread Michael Niedermayer
On Tue, Jul 24, 2018 at 07:32:52PM -0700, Marcin Gorzel wrote: > Looks good, thanks Michael! will apply thx [...] -- Michael GnuPG fingerprint: 9FF2128B147EF6730BADF133611EC787040B0FAB I have never wished to cater to the crowd; for what I know they do not approve, and what they approve I do

Re: [FFmpeg-devel] [PATCH] doc/formats: Add documentation for skip_estimate_duration_from_pts

2018-07-25 Thread Michael Niedermayer
On Wed, Jul 25, 2018 at 10:13:58AM +0530, Gyan Doshi wrote: > > > On 25-07-2018 04:19 AM, Michael Niedermayer wrote: > > >+ > >+@item skip_estimate_duration_from_pts @var{bool} (@emph{input}) > >+skip duration calculation from pts. > > The function this is in is only called for MPEG-PS or seeka

Re: [FFmpeg-devel] [PATCH] lavfi: add lumakey_opencl filter

2018-07-25 Thread Michael Niedermayer
On Wed, Jul 25, 2018 at 10:50:43AM -0300, James Almer wrote: > On 7/25/2018 9:13 AM, Danil Iashchenko wrote: > > Add lumakey_opencl filter. Behaves like existing lumakey filter. > > Isn't it possible to keep each of these new OpenCL filters as an > optional codepath within the C version, using an

Re: [FFmpeg-devel] [FFmpeg-user] fMP4 generation speed

2018-07-25 Thread Ronak Patel
Hi, We’d like to start working on this real soon and want to make sure we properly fix this. We understand this code is used by more than just fMP4 assets and even live streaming use cases. Can anyone please help shed some light on all use cases we should make sure are working after we make ou

[FFmpeg-devel] [PATCH] avformat/movenc: implicitly enable negative CTS offsets for ismv

2018-07-25 Thread Jan Ekström
ISMV lacks any sort of edit list support, as well as tfxd is effectively the PTS of the fragment for most intents and purposes. Thus, if b-frames are requested without negative CTS offsets you end up with N frames' worth of delay (tfxd PTS plus the CTS offset of the first sample). Negative CTS off

Re: [FFmpeg-devel] [PATCH] avformat/mov: Check default_encrypted_sample before use in mov_read_sample_encryption_info()

2018-07-25 Thread Michael Niedermayer
On Sat, Jul 21, 2018 at 06:14:38PM +0200, Michael Niedermayer wrote: > Fixes: 2018-0721-sample > Fixes: null pointer dereference > > Found-by: Nikita Knyzhov (knnik...@yandex.ru) > Signed-off-by: Michael Niedermayer > --- > libavformat/mov.c | 5 + > 1 file changed, 5 insertions(+) will app

[FFmpeg-devel] [PATCH 1/5] avcodec/dump_extradata_bsf: don't allocate a new AVPacket per filtered packet

2018-07-25 Thread James Almer
~4x speedup in dump_extradata() Signed-off-by: James Almer --- libavcodec/dump_extradata_bsf.c | 7 --- 1 file changed, 4 insertions(+), 3 deletions(-) diff --git a/libavcodec/dump_extradata_bsf.c b/libavcodec/dump_extradata_bsf.c index 98703749f7..188a1c619b 100644 --- a/libavcodec/dump_ex

[FFmpeg-devel] [PATCH 3/5] avcodec/av1_parse: check for OBU header overread

2018-07-25 Thread James Almer
Signed-off-by: James Almer --- libavcodec/av1_parse.h | 4 1 file changed, 4 insertions(+) diff --git a/libavcodec/av1_parse.h b/libavcodec/av1_parse.h index 84fc71c925..3a4151491a 100644 --- a/libavcodec/av1_parse.h +++ b/libavcodec/av1_parse.h @@ -118,6 +118,10 @@ static inline int parse_

[FFmpeg-devel] [PATCH 2/5] avcodec/libaomdec: auto insert dump_extra bitstream filter

2018-07-25 Thread James Almer
Some containers, like Matroska, may propagate key frames with no Sequence Header OBU since it's provided in extradata instead. With this change, the Sequence Header will be appended to the packet data before calling aom_codec_decode(). Signed-off-by: James Almer --- configure | 1 +

[FFmpeg-devel] [PATCH 4/5] avformat/av1: return an error when no data is provided to ff_isom_write_av1c()

2018-07-25 Thread James Almer
Signed-off-by: James Almer --- libavformat/av1.c | 3 +++ 1 file changed, 3 insertions(+) diff --git a/libavformat/av1.c b/libavformat/av1.c index 7db29c8d76..7c55a100bf 100644 --- a/libavformat/av1.c +++ b/libavformat/av1.c @@ -82,6 +82,9 @@ int ff_isom_write_av1c(AVIOContext *pb, const uint8_t

[FFmpeg-devel] [PATCH 5/5] avformat/matroskaenc: update AV1 support

2018-07-25 Thread James Almer
Make sure to not write forbidden OBUs to CodecPrivate, and do the same with unnecessary OBUs for packets. Signed-off-by: James Almer --- libavformat/matroskaenc.c | 6 ++ 1 file changed, 6 insertions(+) diff --git a/libavformat/matroskaenc.c b/libavformat/matroskaenc.c index b7ff1950d3..816

Re: [FFmpeg-devel] [FFmpeg-user] fMP4 generation speed

2018-07-25 Thread Liu Steven
> 在 2018年7月26日,上午7:04,Ronak Patel 写道: > > Hi, > > We’d like to start working on this real soon and want to make sure we > properly fix this. We understand this code is used by more than just fMP4 > assets and even live streaming use cases. > > Can anyone please help shed some light on all u

Re: [FFmpeg-devel] [FFmpeg-user] fMP4 generation speed

2018-07-25 Thread Ronak Patel
> On Jul 25, 2018, at 11:12 PM, Liu Steven wrote: > > > >> 在 2018年7月26日,上午7:04,Ronak Patel 写道: >> >> Hi, >> >> We’d like to start working on this real soon and want to make sure we >> properly fix this. We understand this code is used by more than just fMP4 >> assets and even live stream

[FFmpeg-devel] [PATCH] fate: remove unnecessary reference file for fate-encryption-info

2018-07-25 Thread James Almer
Signed-off-by: James Almer --- tests/fate/libavutil.mak | 1 + tests/ref/fate/encryption-info | 0 2 files changed, 1 insertion(+) delete mode 100644 tests/ref/fate/encryption-info diff --git a/tests/fate/libavutil.mak b/tests/fate/libavutil.mak index a1d9313c3d..b12d15727b 100644 --- a/t

Re: [FFmpeg-devel] [FFmpeg-user] fMP4 generation speed

2018-07-25 Thread Ronak Patel
> On Jul 25, 2018, at 11:25 PM, Ronak Patel > wrote: > > > >> On Jul 25, 2018, at 11:12 PM, Liu Steven wrote: >> >> >> >>> 在 2018年7月26日,上午7:04,Ronak Patel 写道: >>> >>> Hi, >>> >>> We’d like to start working on this real soon and want to make sure we >>> properly fix this. We understan

Re: [FFmpeg-devel] [FFmpeg-user] fMP4 generation speed

2018-07-25 Thread Liu Steven
> 在 2018年7月26日,上午11:30,Ronak Patel 写道: > > > >> On Jul 25, 2018, at 11:25 PM, Ronak Patel >> wrote: >> >> >> >>> On Jul 25, 2018, at 11:12 PM, Liu Steven wrote: >>> >>> >>> 在 2018年7月26日,上午7:04,Ronak Patel 写道: Hi, We’d like to start working on this real soo

[FFmpeg-devel] [PATCH] MAINTAINERS: add myself to MIPS section

2018-07-25 Thread Shiyou Yin
Signed-off-by: Shiyou Yin --- MAINTAINERS | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/MAINTAINERS b/MAINTAINERS index 234b655..3dd26e3 100644 --- a/MAINTAINERS +++ b/MAINTAINERS @@ -524,7 +524,7 @@ Operating systems / CPU architectures

Re: [FFmpeg-devel] [PATCH] doc/formats: Add documentation for skip_estimate_duration_from_pts

2018-07-25 Thread Gyan Doshi
On 26-07-2018 03:37 AM, Michael Niedermayer wrote: On Wed, Jul 25, 2018 at 10:13:58AM +0530, Gyan Doshi wrote: Wouldn't it be better to move this as a private option for those two demuxers? iam not sure, it could be used by others in the future too what do people prefer ? I have no real op