Re: [FFmpeg-devel] [PATCH v4] avcodec: libdav1d AV1 decoder wrapper.

2018-11-05 Thread James Almer
On 10/29/2018 7:16 PM, James Almer wrote: > From: "Ronald S. Bultje" > > Originally written by Ronald S. Bultje, with fixes, optimizations and > improvements by James Almer. > > Signed-off-by: James Almer > --- > - s.n_frame_threads is now mapped to avctx->thread_count. > - colorspace,

[FFmpeg-devel] [PATCH] libvpxenc: extend auto-alt-ref range

2018-11-05 Thread James Zern
vp9 now supports [0, 6] Signed-off-by: James Zern --- doc/encoders.texi | 1 + libavcodec/libvpxenc.c | 6 -- 2 files changed, 5 insertions(+), 2 deletions(-) diff --git a/doc/encoders.texi b/doc/encoders.texi index 899faac49b..3c6f5cd70b 100644 --- a/doc/encoders.texi +++

Re: [FFmpeg-devel] [PATCH] libvpxenc: extend auto-alt-ref range

2018-11-05 Thread James Zern
On Mon, Nov 5, 2018 at 5:02 PM James Zern wrote: > > vp9 now supports [0, 6] > > Signed-off-by: James Zern > --- > doc/encoders.texi | 1 + > libavcodec/libvpxenc.c | 6 -- > 2 files changed, 5 insertions(+), 2 deletions(-) > This version moves the option from COMMON_OPTIONS for

Re: [FFmpeg-devel] [PATCH] libvpxenc: extend auto-alt-ref range

2018-11-05 Thread James Zern
On Mon, Nov 5, 2018 at 4:45 PM James Zern wrote: > > On Sat, Nov 3, 2018 at 2:31 PM James Almer wrote: > > > > On 11/3/2018 6:04 PM, James Zern wrote: > > > On Sat, Nov 3, 2018 at 2:02 PM James Zern wrote: > > >> [...] > > >> #define COMMON_OPTIONS \ > > >> { "auto-alt-ref","Enable

[FFmpeg-devel] [PATCH] avcodec/mips: [loongson] refine optimization in h264_chroma.

2018-11-05 Thread Shiyou Yin
Remove invalid operation in the case x and y all equal 0, this refine made about 2% speedup for H264 decode on loongson platform. --- libavcodec/mips/h264chroma_mmi.c | 970 +-- 1 file changed, 426 insertions(+), 544 deletions(-) diff --git

Re: [FFmpeg-devel] [PATCH 2/7] lavc/vaapi_encode_h264: Enable macro block level bit rate control.

2018-11-05 Thread myp...@gmail.com
On Tue, Nov 6, 2018 at 3:40 AM Mark Thompson wrote: > > On 03/11/18 02:52, Jun Zhao wrote: > > Enables macro block level bit rate control, that generally improves > > subjective visual quality. It may have a negative impact on > > performance and objective visual quality metrics. Default is off >

Re: [FFmpeg-devel] [PATCH] libvpxenc: extend auto-alt-ref range

2018-11-05 Thread James Zern
On Sat, Nov 3, 2018 at 2:31 PM James Almer wrote: > > On 11/3/2018 6:04 PM, James Zern wrote: > > On Sat, Nov 3, 2018 at 2:02 PM James Zern wrote: > >> [...] > >> #define COMMON_OPTIONS \ > >> { "auto-alt-ref","Enable use of alternate reference " \ > >> -

Re: [FFmpeg-devel] [PATCH 0/7] Enhancement VA-API encoder bitrate control

2018-11-05 Thread myp...@gmail.com
On Tue, Nov 6, 2018 at 2:49 AM Mark Thompson wrote: > > On 03/11/18 02:52, Jun Zhao wrote: > > V1: > > - add codec-specific bit rate control callback. > > I don't understand why you want this to be codec-specific at all? No option > is always supported in any particular case, and both of these

Re: [FFmpeg-devel] FFmpeg 4.1

2018-11-05 Thread Michael Niedermayer
On Thu, Nov 01, 2018 at 08:25:34PM +0100, Michael Niedermayer wrote: > On Fri, Sep 14, 2018 at 03:22:56AM +0200, Michael Niedermayer wrote: > > Hi all > > > > its almost 5 months since 4.0 so its time to make a new release. > > > > Are there any suggestions for a name ? > > If not ill pick

[FFmpeg-devel] [PATCH] libavformat/mov: Speed up fragmented mp4 parsing (Take 2)

2018-11-05 Thread Aaron Boushley
Sorry for the second thread, first one was corrupted with a newline because I attempted an inline patch. Hopefully the mime-type will be set correctly on this one. --- When parsing a fragmented MP4 with the use_mfra_for option set so that the mfra box is parsed we currently continue to parse

Re: [FFmpeg-devel] [PATCH] libavformat/mov: Speed up fragmented mp4 parsing

2018-11-05 Thread Aaron Boushley
On Mon, Nov 5, 2018 at 3:25 PM Aaron Boushley wrote: > > On Mon, Nov 5, 2018 at 3:13 PM Carl Eugen Hoyos wrote: > > > > 2018-11-06 0:03 GMT+01:00, Aaron Boushley : > > > On Mon, Nov 5, 2018 at 2:21 PM Michael Niedermayer > > > wrote: > > > > >> > diff --git a/libavformat/mov.c

Re: [FFmpeg-devel] [PATCH] libavformat/mov: Speed up fragmented mp4 parsing

2018-11-05 Thread Aaron Boushley
On Mon, Nov 5, 2018 at 3:13 PM Carl Eugen Hoyos wrote: > > 2018-11-06 0:03 GMT+01:00, Aaron Boushley : > > On Mon, Nov 5, 2018 at 2:21 PM Michael Niedermayer > > wrote: > > >> > diff --git a/libavformat/mov.c b/libavformat/mov.c > >> > index ec57a05803..1def594626 100644 > >> > ---

Re: [FFmpeg-devel] [PATCH v2 2/2] cbs_av1: Support redundant frame headers

2018-11-05 Thread Mark Thompson
On 05/11/18 18:51, James Almer wrote: > On 11/5/2018 3:33 PM, Mark Thompson wrote: >> On 05/11/18 18:16, James Almer wrote: >>> On 11/5/2018 12:25 PM, Mark Thompson wrote: --- On 05/11/18 00:55, James Almer wrote: > On 11/4/2018 9:10 PM, Mark Thompson wrote: >> ... >> +

Re: [FFmpeg-devel] [PATCH] libavformat/mov: Speed up fragmented mp4 parsing

2018-11-05 Thread Carl Eugen Hoyos
2018-11-06 0:03 GMT+01:00, Aaron Boushley : > On Mon, Nov 5, 2018 at 2:21 PM Michael Niedermayer > wrote: >> > diff --git a/libavformat/mov.c b/libavformat/mov.c >> > index ec57a05803..1def594626 100644 >> > --- a/libavformat/mov.c >> > +++ b/libavformat/mov.c >> > @@ -7344,6 +7344,8 @@ static

Re: [FFmpeg-devel] [PATCH] libavformat/mov: Speed up fragmented mp4 parsing

2018-11-05 Thread Aaron Boushley
On Mon, Nov 5, 2018 at 2:21 PM Michael Niedermayer wrote: > > On Mon, Nov 05, 2018 at 11:17:25AM -0800, Aaron Boushley wrote: > > When parsing a fragmented MP4 with the use_mfra_for option set so that > > the mfra box is parsed we currently continue to parse all the top level > > boxes in the

Re: [FFmpeg-devel] [PATCH] libavformat/mov: Speed up fragmented mp4 parsing

2018-11-05 Thread Michael Niedermayer
On Mon, Nov 05, 2018 at 11:17:25AM -0800, Aaron Boushley wrote: > When parsing a fragmented MP4 with the use_mfra_for option set so that > the mfra box is parsed we currently continue to parse all the top level > boxes in the mp4. We also avoid using the index in mov_seek_fragment > unless the

[FFmpeg-devel] [PATCH] avcodec/qpeg: Optimize full width runs in qpeg_decode_intra()

2018-11-05 Thread Michael Niedermayer
This improves the speed of decoding large patches of constant color Fixes: Timeout Fixes: 10967/clusterfuzz-testcase-minimized-ffmpeg_AV_CODEC_ID_QPEG_fuzzer-5630803793936384 Found-by: continuous fuzzing process https://github.com/google/oss-fuzz/tree/master/projects/ffmpeg Signed-off-by:

Re: [FFmpeg-devel] [PATCH 3/4] doc/filters: add tonemap_opencl document.

2018-11-05 Thread Lou Logan
On Mon, Oct 29, 2018, at 9:31 PM, Gyan wrote: > > LGTM. Pushed. ___ ffmpeg-devel mailing list ffmpeg-devel@ffmpeg.org http://ffmpeg.org/mailman/listinfo/ffmpeg-devel

Re: [FFmpeg-devel] [PATCH 1/4] doc/filters: add document for opencl filters

2018-11-05 Thread Lou Logan
On Sat, Nov 3, 2018, at 8:31 AM, Lou Logan wrote: > On Thu, Nov 1, 2018, at 9:13 PM, Gyan wrote: > > Sure, but I may not get to it until Monday. Pushed. ___ ffmpeg-devel mailing list ffmpeg-devel@ffmpeg.org

Re: [FFmpeg-devel] [PATCH v2 1/2] cbs_av1: Fix header writing when already aligned

2018-11-05 Thread James Almer
On 11/5/2018 12:25 PM, Mark Thompson wrote: > --- > libavcodec/cbs_av1.c | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > > diff --git a/libavcodec/cbs_av1.c b/libavcodec/cbs_av1.c > index 9bac9dde09..1c49d90f51 100644 > --- a/libavcodec/cbs_av1.c > +++ b/libavcodec/cbs_av1.c > @@

Re: [FFmpeg-devel] [PATCH 2/7] lavc/vaapi_encode_h264: Enable macro block level bit rate control.

2018-11-05 Thread Mark Thompson
On 03/11/18 02:52, Jun Zhao wrote: > Enables macro block level bit rate control, that generally improves > subjective visual quality. It may have a negative impact on > performance and objective visual quality metrics. Default is off > and can't compatible with Constant QP. > > Signed-off-by: Jun

[FFmpeg-devel] [PATCH] libavformat/mov: Speed up fragmented mp4 parsing

2018-11-05 Thread Aaron Boushley
When parsing a fragmented MP4 with the use_mfra_for option set so that the mfra box is parsed we currently continue to parse all the top level boxes in the mp4. We also avoid using the index in mov_seek_fragment unless the fragment index is marked as complete, which the mfra parsing code never

Re: [FFmpeg-devel] [PATCH v2 2/2] cbs_av1: Support redundant frame headers

2018-11-05 Thread James Almer
On 11/5/2018 3:33 PM, Mark Thompson wrote: > On 05/11/18 18:16, James Almer wrote: >> On 11/5/2018 12:25 PM, Mark Thompson wrote: >>> --- >>> On 05/11/18 00:55, James Almer wrote: On 11/4/2018 9:10 PM, Mark Thompson wrote: > ... > +xf(1, frame_header_copy[k], b, b, b,

Re: [FFmpeg-devel] [PATCH 0/7] Enhancement VA-API encoder bitrate control

2018-11-05 Thread Mark Thompson
On 03/11/18 02:52, Jun Zhao wrote: > V1: > - add codec-specific bit rate control callback. I don't understand why you want this to be codec-specific at all? No option is always supported in any particular case, and both of these options are at least queryable. > - enable macro block level bit

Re: [FFmpeg-devel] [PATCH v2 2/2] cbs_av1: Support redundant frame headers

2018-11-05 Thread Mark Thompson
On 05/11/18 18:16, James Almer wrote: > On 11/5/2018 12:25 PM, Mark Thompson wrote: >> --- >> On 05/11/18 00:55, James Almer wrote: >>> On 11/4/2018 9:10 PM, Mark Thompson wrote: ... +xf(1, frame_header_copy[k], b, b, b, 1, k); >>> This is making a lot of noise in the

Re: [FFmpeg-devel] [PATCH v2 7/7] lavc/qsvenc: add an option to set MB numbers of each slice

2018-11-05 Thread Mark Thompson
On 05/11/18 12:15, Zhong Li wrote: > This option specifies suggested macroblocks numbers in each slice. > MSDK may adjust it based on platform capability. > And slice_number will be ignored if this option set to a non-zero > value. > > Signed-off-by: Zhong Li > --- > libavcodec/qsvenc.c |

Re: [FFmpeg-devel] [PATCH v2 2/2] cbs_av1: Support redundant frame headers

2018-11-05 Thread James Almer
On 11/5/2018 12:25 PM, Mark Thompson wrote: > --- > On 05/11/18 00:55, James Almer wrote: >> On 11/4/2018 9:10 PM, Mark Thompson wrote: >>> ... >>> +xf(1, frame_header_copy[k], b, b, b, 1, k); >> This is making a lot of noise in the trace output for no real gain, >> since it prints

Re: [FFmpeg-devel] [PATCH v2 6/7] lavc/qsvenc: add an option to set h264 pps for every frame

2018-11-05 Thread Mark Thompson
On 05/11/18 12:15, Zhong Li wrote: > RepeatPPS is enabled by default in mfx. It is not necessary mostly and > wasting encoding bits. > Add an option to control it and disable it by default. > > Signed-off-by: Zhong Li > --- > libavcodec/qsvenc.c | 1 + > libavcodec/qsvenc.h | 2 ++ >

Re: [FFmpeg-devel] [PATCH v2 2/7] lavc/qsvenc: add forced_idr opiton

2018-11-05 Thread Mark Thompson
On 05/11/18 12:15, Zhong Li wrote: > This option can be used to repect original input I/IDR frame type. > > Signed-off-by: Zhong Li > --- > libavcodec/qsvenc.c | 7 +++ > libavcodec/qsvenc.h | 2 ++ > 2 files changed, 9 insertions(+) > > diff --git a/libavcodec/qsvenc.c

Re: [FFmpeg-devel] [PATCH 1/4] proresenc_nanatoliy: Rename a profile name with the correct one

2018-11-05 Thread Vittorio Giovara
On Sat, Nov 3, 2018 at 10:23 AM Paul B Mahol wrote: > On 11/2/18, Vittorio Giovara wrote: > > In all Apple documentation, this profile is called Prores . > > --- > > libavcodec/proresenc_anatoliy.c | 16 > > 1 file changed, 8 insertions(+), 8 deletions(-) > > > There is

Re: [FFmpeg-devel] [PATCH v2 1/7] lavc/qsvenc: enable ICQ and ICQ_LA on Linux

2018-11-05 Thread Mark Thompson
On 05/11/18 12:15, Zhong Li wrote: > ICQ/ICQ_LA are enabled with MSDK V1.28 > > One example of ICQ_LA: > ffmpeg -hwaccel qsv -c:v h264_qsv -i in.mp4 -c:v h264_qsv -global_quality 25 > -look_ahead 1 out.mp4 > > Signed-off-by: Zhong Li > --- > ICQ mode will fail with some cases: >

[FFmpeg-devel] [PATCH] vaapi_encode: Improve log message for unsupported profiles

2018-11-05 Thread Mark Thompson
--- For example: [h264_vaapi @ 0x563e95f73d00] Compatible profile VAProfileH264High (7) is not supported by driver. [h264_vaapi @ 0x563e95f73d00] Compatible profile VAProfileH264Main (6) is not supported by driver. [h264_vaapi @ 0x563e95f73d00] Using VAAPI profile

Re: [FFmpeg-devel] [PATCH] avcodec/vaapi_encode: modify the initialization postion of profile_string

2018-11-05 Thread Mark Thompson
On 05/11/18 09:39, Linjie Fu wrote: > Currently, profile_string was initialized in the "for" loop. If > it didn't enter this loop or accidently break, profile_string may be > uninitialized. > > Modify to initialize the profile_string after the loop to avoid > using the uninitialized value when

Re: [FFmpeg-devel] [PATCH] lavf/isom: add "dvhe" fourCC for HEVC

2018-11-05 Thread Jan Ekström
On Mon, Nov 5, 2018 at 6:34 PM Rodger Combs wrote: > > This refers to "Dolby Vision", which can have some additional extensions, > but (usually?) is also valid HEVC. This probably will have to be referenced against

Re: [FFmpeg-devel] [PATCH 4/4] mjpeg: Use profile names in the encoder and decoder

2018-11-05 Thread Vittorio Giovara
On Fri, Nov 2, 2018 at 5:29 PM Carl Eugen Hoyos wrote: > 2018-11-02 20:35 GMT+01:00, Vittorio Giovara : > > --- > > libavcodec/codec_desc.c | 1 + > > libavcodec/mjpegdec.c | 2 ++ > > libavcodec/profiles.c | 9 + > > libavcodec/profiles.h | 1 + > > libavcodec/version.h| 2 +-

[FFmpeg-devel] [PATCH] lavf/isom: add "dvhe" fourCC for HEVC

2018-11-05 Thread Rodger Combs
This refers to "Dolby Vision", which can have some additional extensions, but (usually?) is also valid HEVC. --- libavformat/isom.c | 1 + 1 file changed, 1 insertion(+) diff --git a/libavformat/isom.c b/libavformat/isom.c index ca9d22e4f7..fbfa0dc057 100644 --- a/libavformat/isom.c +++

Re: [FFmpeg-devel] [PATCH v2] avcodec/libopenh264enc.c: Handle sample_aspect_ratio in libopenh264 encoder

2018-11-05 Thread Mark Thompson
On 01/11/18 13:24, Valery Kot wrote: > On Thu, Nov 1, 2018 at 1:55 PM Valery Kot wrote: >> >>> I think this would look nicer (and generate less code) as a table + loop >>> rather than an if-ladder making each fraction structure inline? >>> >>> E.g. something like >>>

Re: [FFmpeg-devel] [PATCH 2/3] doc/encoders: Add libaom-av1

2018-11-05 Thread Gyan
On Mon, Nov 5, 2018 at 9:04 PM Mark Thompson wrote: > On 05/11/18 15:20, Gyan wrote: > > On Mon, Nov 5, 2018 at 8:23 PM Mark Thompson wrote: > > > >> --- > >> Updated to add some more explanation of the rate control mode selection. > >> > >> > >> doc/encoders.texi | 108

Re: [FFmpeg-devel] [PATCH 2/3] doc/encoders: Add libaom-av1

2018-11-05 Thread Mark Thompson
On 05/11/18 15:20, Gyan wrote: > On Mon, Nov 5, 2018 at 8:23 PM Mark Thompson wrote: > >> --- >> Updated to add some more explanation of the rate control mode selection. >> >> >> doc/encoders.texi | 108 ++ >> 1 file changed, 108 insertions(+) >> >>

Re: [FFmpeg-devel] [PATCH 2/3] doc/encoders: Add libaom-av1

2018-11-05 Thread Gyan
On Mon, Nov 5, 2018 at 8:23 PM Mark Thompson wrote: > --- > Updated to add some more explanation of the rate control mode selection. > > > doc/encoders.texi | 108 ++ > 1 file changed, 108 insertions(+) > > diff --git a/doc/encoders.texi

[FFmpeg-devel] [PATCH v2 2/2] cbs_av1: Support redundant frame headers

2018-11-05 Thread Mark Thompson
--- On 05/11/18 00:55, James Almer wrote: > On 11/4/2018 9:10 PM, Mark Thompson wrote: >> ... >> +xf(1, frame_header_copy[k], b, b, b, 1, k); > > This is making a lot of noise in the trace output for no real gain, > since it prints every single bit as its own line. Can you silence

[FFmpeg-devel] [PATCH v2 1/2] cbs_av1: Fix header writing when already aligned

2018-11-05 Thread Mark Thompson
--- libavcodec/cbs_av1.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/libavcodec/cbs_av1.c b/libavcodec/cbs_av1.c index 9bac9dde09..1c49d90f51 100644 --- a/libavcodec/cbs_av1.c +++ b/libavcodec/cbs_av1.c @@ -1179,7 +1179,7 @@ static int

Re: [FFmpeg-devel] [PATCH 1/3] doc/encoders: Move libxavs2 to correct place in alphabetical order

2018-11-05 Thread Gyan
On Mon, Nov 5, 2018 at 8:23 PM Mark Thompson wrote: > --- > doc/encoders.texi | 92 +++ > 1 file changed, 46 insertions(+), 46 deletions(-) > > diff --git a/doc/encoders.texi b/doc/encoders.texi > index 899faac49b..1ca0ef1543 100644 > ---

Re: [FFmpeg-devel] Reimbursement request

2018-11-05 Thread Michael Niedermayer
On Mon, Nov 05, 2018 at 12:06:46PM -0200, Pedro Arthur wrote: > Hi, > I'm requesting the reimbursement of travel expenses for the Google > Mentor Summit. > I mentored the Super Resolution project, more details can be found in [1]. > > > Flight (BRL) R$ 3271.03 > One night

[FFmpeg-devel] [PATCH 3/3] libaomenc: Drop unused noise-sensitivity option

2018-11-05 Thread Mark Thompson
--- On 28/10/18 21:02, Carl Eugen Hoyos wrote: > 2018-10-28 21:08 GMT+01:00, Mark Thompson : >> --- >> This was in the 4.0 release, unfortunately (and did nothing there as well). > > If it has never worked, it can be removed imo. Sure, here is a new version. Thanks, - Mark

[FFmpeg-devel] [PATCH 2/3] doc/encoders: Add libaom-av1

2018-11-05 Thread Mark Thompson
--- Updated to add some more explanation of the rate control mode selection. doc/encoders.texi | 108 ++ 1 file changed, 108 insertions(+) diff --git a/doc/encoders.texi b/doc/encoders.texi index 1ca0ef1543..f2e4199017 100644 --- a/doc/encoders.texi

[FFmpeg-devel] [PATCH 1/3] doc/encoders: Move libxavs2 to correct place in alphabetical order

2018-11-05 Thread Mark Thompson
--- doc/encoders.texi | 92 +++ 1 file changed, 46 insertions(+), 46 deletions(-) diff --git a/doc/encoders.texi b/doc/encoders.texi index 899faac49b..1ca0ef1543 100644 --- a/doc/encoders.texi +++ b/doc/encoders.texi @@ -2180,6 +2180,52 @@ ffmpeg -i

Re: [FFmpeg-devel] [PATCH 1/3] cbs_av1: Fix header writing when already aligned

2018-11-05 Thread James Almer
On 11/5/2018 11:12 AM, Mark Thompson wrote: > On 05/11/18 13:14, James Almer wrote: >> On 11/4/2018 9:10 PM, Mark Thompson wrote: >>> --- >>> Previously it lost the trailing bits byte but did include it in the OBU >>> size, completely breaking the stream. >>> >>> >>> libavcodec/cbs_av1.c | 2 +-

Re: [FFmpeg-devel] [PATCH 1/3] cbs_av1: Fix header writing when already aligned

2018-11-05 Thread Mark Thompson
On 05/11/18 13:14, James Almer wrote: > On 11/4/2018 9:10 PM, Mark Thompson wrote: >> --- >> Previously it lost the trailing bits byte but did include it in the OBU >> size, completely breaking the stream. >> >> >> libavcodec/cbs_av1.c | 2 +- >> 1 file changed, 1 insertion(+), 1 deletion(-) >>

[FFmpeg-devel] Reimbursement request

2018-11-05 Thread Pedro Arthur
Hi, I'm requesting the reimbursement of travel expenses for the Google Mentor Summit. I mentored the Super Resolution project, more details can be found in [1]. Flight (BRL) R$ 3271.03 One night Airbnb (BRL) R$ 235.70 Trasnportation (USD) $ 23.00 Total

Re: [FFmpeg-devel] [PATCH 2/2] libx264: Pass the reordered_opaque field through the encoder

2018-11-05 Thread Martin Storsjö
On Thu, 1 Nov 2018, Martin Storsjö wrote: On Thu, 1 Nov 2018, Derek Buitenhuis wrote: On 31/10/2018 21:41, Martin Storsjö wrote: Even though we do allow reconfiguration, it doesn't look like we support changing any parameters which would actually affect the delay, only RC method and targets

Re: [FFmpeg-devel] [PATCH 1/3] cbs_av1: Fix header writing when already aligned

2018-11-05 Thread James Almer
On 11/4/2018 9:10 PM, Mark Thompson wrote: > --- > Previously it lost the trailing bits byte but did include it in the OBU size, > completely breaking the stream. > > > libavcodec/cbs_av1.c | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > > diff --git a/libavcodec/cbs_av1.c

Re: [FFmpeg-devel] [PATCH] avcodec/vaapi_encode: modify the initialization postion of profile_string

2018-11-05 Thread Li, Zhong
> From: ffmpeg-devel [mailto:ffmpeg-devel-boun...@ffmpeg.org] On Behalf > Of Linjie Fu > Sent: Monday, November 5, 2018 5:39 PM > To: ffmpeg-devel@ffmpeg.org > Cc: Fu, Linjie > Subject: [FFmpeg-devel] [PATCH] avcodec/vaapi_encode: modify the > initialization postion of profile_string > >

Re: [FFmpeg-devel] [PATCH] avcodec/qsvenc: add default initiaize for pict_type

2018-11-05 Thread Li, Zhong
> From: ffmpeg-devel [mailto:ffmpeg-devel-boun...@ffmpeg.org] On Behalf > Of Linjie Fu > Sent: Monday, November 5, 2018 5:41 PM > To: ffmpeg-devel@ffmpeg.org > Cc: Fu, Linjie > Subject: [FFmpeg-devel] [PATCH] avcodec/qsvenc: add default initiaize for > pict_type > > add default initiaize for

Re: [FFmpeg-devel] [PATCH] avfilter/vf_overlay_qsv: initialize default pix_fmt

2018-11-05 Thread Li, Zhong
> From: ffmpeg-devel [mailto:ffmpeg-devel-boun...@ffmpeg.org] On Behalf > Of Linjie Fu > Sent: Monday, November 5, 2018 5:37 PM > To: ffmpeg-devel@ffmpeg.org > Cc: Fu, Linjie > Subject: [FFmpeg-devel] [PATCH] avfilter/vf_overlay_qsv: initialize default > pix_fmt > > add default initiaization for

[FFmpeg-devel] [PATCH v2 7/7] lavc/qsvenc: add an option to set MB numbers of each slice

2018-11-05 Thread Zhong Li
This option specifies suggested macroblocks numbers in each slice. MSDK may adjust it based on platform capability. And slice_number will be ignored if this option set to a non-zero value. Signed-off-by: Zhong Li --- libavcodec/qsvenc.c | 1 + libavcodec/qsvenc.h | 1 +

[FFmpeg-devel] [PATCH v2 4/7] lavc/qsvenc: make hevc alignment same as h264 for 1.19+ version

2018-11-05 Thread Zhong Li
It is to clean up to the code and To-Do list. Signed-off-by: Zhong Li --- libavcodec/qsvenc.c | 17 +++-- 1 file changed, 11 insertions(+), 6 deletions(-) diff --git a/libavcodec/qsvenc.c b/libavcodec/qsvenc.c index d160db3..3e7d623 100644 --- a/libavcodec/qsvenc.c +++

[FFmpeg-devel] [PATCH v2 6/7] lavc/qsvenc: add an option to set h264 pps for every frame

2018-11-05 Thread Zhong Li
RepeatPPS is enabled by default in mfx. It is not necessary mostly and wasting encoding bits. Add an option to control it and disable it by default. Signed-off-by: Zhong Li --- libavcodec/qsvenc.c | 1 + libavcodec/qsvenc.h | 2 ++ libavcodec/qsvenc_h264.c | 2 ++ libavcodec/version.h

[FFmpeg-devel] [PATCH v2 3/7] lavc/qsvenc: remove redundant code

2018-11-05 Thread Zhong Li
Signed-off-by: Zhong Li --- libavcodec/qsvenc.c | 11 --- 1 file changed, 4 insertions(+), 7 deletions(-) diff --git a/libavcodec/qsvenc.c b/libavcodec/qsvenc.c index 7098ec7..d160db3 100644 --- a/libavcodec/qsvenc.c +++ b/libavcodec/qsvenc.c @@ -444,6 +444,7 @@ static int

[FFmpeg-devel] [PATCH v2 2/7] lavc/qsvenc: add forced_idr opiton

2018-11-05 Thread Zhong Li
This option can be used to repect original input I/IDR frame type. Signed-off-by: Zhong Li --- libavcodec/qsvenc.c | 7 +++ libavcodec/qsvenc.h | 2 ++ 2 files changed, 9 insertions(+) diff --git a/libavcodec/qsvenc.c b/libavcodec/qsvenc.c index 948751d..7098ec7 100644 ---

[FFmpeg-devel] [PATCH v2 5/7] lavc/qsvenc: dump log for frame rate

2018-11-05 Thread Zhong Li
Signed-off-by: Zhong Li --- libavcodec/qsvenc.c | 4 1 file changed, 4 insertions(+) diff --git a/libavcodec/qsvenc.c b/libavcodec/qsvenc.c index 3e7d623..7745086 100644 --- a/libavcodec/qsvenc.c +++ b/libavcodec/qsvenc.c @@ -263,6 +263,10 @@ static void dump_video_param(AVCodecContext

[FFmpeg-devel] [PATCH v2 1/7] lavc/qsvenc: enable ICQ and ICQ_LA on Linux

2018-11-05 Thread Zhong Li
ICQ/ICQ_LA are enabled with MSDK V1.28 One example of ICQ_LA: ffmpeg -hwaccel qsv -c:v h264_qsv -i in.mp4 -c:v h264_qsv -global_quality 25 -look_ahead 1 out.mp4 Signed-off-by: Zhong Li --- ICQ mode will fail with some cases: https://github.com/Intel-Media-SDK/MediaSDK/issues/863. Don't merge

[FFmpeg-devel] [PATCH v2 0/7] qsv encoder enhancement and code refine

2018-11-05 Thread Zhong Li
Zhong Li (7): lavc/qsvenc: enable ICQ and ICQ_LA on Linux lavc/qsvenc: add forced_idr opiton lavc/qsvenc: remove redundant code lavc/qsvenc: make hevc alignment same as h264 for 1.19+ version lavc/qsvenc: dump log for frame rate lavc/qsvenc: add an option to set h264 pps for every

Re: [FFmpeg-devel] [PATCH] [V2] avcodec/qsvenc: add VDENC support for H264 and HEVC

2018-11-05 Thread Fu, Linjie
> -Original Message- > On Mon, Nov 05, 2018 at 09:50:13 +0800, Linjie Fu wrote: > > It's an experimental function(like lowpower in vaapi) with > > Should it be under "-strict experimental" then? > > > module parameter(i915.enable_guc=2 for linux kerner version >= 4.16) >

[FFmpeg-devel] [PATCH] avcodec/mips: [loongson] refine optimization in h264_chroma.

2018-11-05 Thread Shiyou Yin
Remove invalid operation in the case x and y all equal 0, this refine made about 2% speedup for H264 decode on loongson platform. --- libavcodec/mips/h264chroma_mmi.c | 970 +-- 1 file changed, 426 insertions(+), 544 deletions(-) diff --git

[FFmpeg-devel] [PATCH] avcodec/qsvenc: add default initiaize for pict_type

2018-11-05 Thread Linjie Fu
add default initiaize for pict_type in function ff_qsv_encode to avoid using uninitialized value: FF_DISABLE_DEPRECATION_WARNINGS avctx->coded_frame->pict_type = pict_type; FF_ENABLE_DEPRECATION_WARNINGS Signed-off-by: Linjie Fu --- libavcodec/qsvenc.c | 2 ++ 1 file changed, 2

[FFmpeg-devel] [PATCH] avcodec/vaapi_encode: modify the initialization postion of profile_string

2018-11-05 Thread Linjie Fu
Currently, profile_string was initialized in the "for" loop. If it didn't enter this loop or accidently break, profile_string may be uninitialized. Modify to initialize the profile_string after the loop to avoid using the uninitialized value when calling av_log. Signed-off-by: Linjie Fu ---

[FFmpeg-devel] [PATCH] avfilter/vf_overlay_qsv: initialize default pix_fmt

2018-11-05 Thread Linjie Fu
add default initiaization for pix_fmt in function have_alpha_panar to avoid using uninitialized value. Signed-off-by: Linjie Fu --- libavfilter/vf_overlay_qsv.c | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) diff --git a/libavfilter/vf_overlay_qsv.c b/libavfilter/vf_overlay_qsv.c

Re: [FFmpeg-devel] [PATCH] [V2] avcodec/qsvenc: add VDENC support for H264 and HEVC

2018-11-05 Thread Moritz Barsnick
On Mon, Nov 05, 2018 at 09:50:13 +0800, Linjie Fu wrote: > It's an experimental function(like lowpower in vaapi) with Should it be under "-strict experimental" then? > module parameter(i915.enable_guc=2 for linux kerner version >= 4.16) ^ kernel >