Re: [FFmpeg-devel] [PATCH]lavc/x264: Improve level setting

Le nonidi 29 nivôse, an CCXXIV, Carl Eugen Hoyos a écrit : > The original patch (that does not care about tail > since it can't be reached anyway) uses atoi(). > Is that not ok? atoi() has undefined behaviours in case of error, and has a trivial replacement that works best: it should not be used

[FFmpeg-devel] [PATCH v4] lavc/rawdec: Use AV_PIX_FMT_PAL8 for 1-bit raw QuickTime video

Eliminated some calculations inside loops. Mats >From e836bfee99b6de39acfe023c3a50b4e5b53f82e1 Mon Sep 17 00:00:00 2001 From: Mats Peterson Date: Mon, 18 Jan 2016 08:40:57 +0100 Subject: [PATCH v4] lavc/rawdec: Use AV_PIX_FMT_PAL8 for 1-bit raw QuickTime video Match the use

[FFmpeg-devel] [PATCH v2 2/5] ffmpeg: hwaccel helper for VAAPI decode

From 845be894e8e0e2a966b6e51229cad1aadca36704 Mon Sep 17 00:00:00 2001 From: Mark Thompson Date: Sun, 17 Jan 2016 22:13:45 + Subject: [PATCH 2/5] ffmpeg: hwaccel helper for VAAPI decode --- Makefile | 1 + ffmpeg.h | 2 + ffmpeg_opt.c | 3 +

[FFmpeg-devel] [PATCH v2 3/5] libavcodec: add VAAPI H.264 encoder

From 108355504c2eaa11101e1599ef7e01f18a2a8ac5 Mon Sep 17 00:00:00 2001 From: Mark Thompson Date: Sun, 17 Jan 2016 22:15:06 + Subject: [PATCH 3/5] libavcodec: add VAAPI H.264 encoder --- configure | 1 + libavcodec/Makefile | 1 +

[FFmpeg-devel] [PATCH v2 1/5] libavutil: some VAAPI infrastructure

From 45a803b627d0180c1aac928756924bd39ddf529d Mon Sep 17 00:00:00 2001 From: Mark Thompson Date: Sun, 17 Jan 2016 22:13:20 + Subject: [PATCH 1/5] libavutil: some VAAPI infrastructure --- configure | 4 + libavutil/Makefile | 1 + libavutil/vaapi.c | 782

Re: [FFmpeg-devel] [PATCH] lavf/img2dec: change the default pattern to none

Marton Balint passwd.hu> writes: > The current default pattern type, glob_sequence is > deprecated for almost 3.5 years, Sorry if I misunderstand: When was it deprecated? > before removing it, we need a new, sensible pattern > default. I suggest "none" which turns off pattern > matching,

[FFmpeg-devel] [PATCH 7/7] x86inc: Add debug symbols indicating sizes of compiled functions

From: Geza Lore Some debuggers/profilers use this metadata to determine which function a given instruction is in; without it they get can confused by local labels (if you haven't stripped those). On the other hand, some tools are still confused even with this metadata. e.g.

[FFmpeg-devel] [PATCH 6/7] x86inc: Avoid creating unnecessary local labels

The REP_RET workaround is only needed on old AMD cpus, and the labels clutter up the symbol table and confuse debugging/profiling tools, so use EQU to create SHN_ABS symbols instead of creating local labels. Furthermore, skip the workaround completely in functions that definitely won't run on such

[FFmpeg-devel] [PATCH 2/7] x86inc: Improve FMA instruction handling

* Correctly handle FMA instructions with memory operands. * Print a warning if FMA instructions are used without the correct cpuflag. * Simplify the instantiation code. * Clarify documentation. Only the last operand in FMA3 instructions can be a memory operand. When converting FMA4

[FFmpeg-devel] [PATCH 4/7] x86inc: Use more consistent indentation

--- libavutil/x86/x86inc.asm | 134 +++ 1 file changed, 67 insertions(+), 67 deletions(-) diff --git a/libavutil/x86/x86inc.asm b/libavutil/x86/x86inc.asm index a8c316c..f2635ea 100644 --- a/libavutil/x86/x86inc.asm +++ b/libavutil/x86/x86inc.asm @@

[FFmpeg-devel] [PATCH 0/7] x86inc: Sync changes from x264

The following patches were recently pushed to x264. Geza Lore (1): x86inc: Add debug symbols indicating sizes of compiled functions Henrik Gramner (6): x86inc: Be more verbose in assertion failures x86inc: Improve FMA instruction handling x86inc: Preserve arguments when allocating stack

[FFmpeg-devel] [PATCH 1/7] x86inc: Be more verbose in assertion failures

--- libavutil/x86/x86inc.asm | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/libavutil/x86/x86inc.asm b/libavutil/x86/x86inc.asm index 69ec57d..5bc5c46 100644 --- a/libavutil/x86/x86inc.asm +++ b/libavutil/x86/x86inc.asm @@ -295,7 +295,7 @@ DECLARE_REG_TMP_SIZE

[FFmpeg-devel] [PATCH 5/7] x86inc: Simplify AUTO_REP_RET

cpuflags is never undefined any more, it's set to 0 instead. Also fix an incorrect comment. --- libavutil/x86/x86inc.asm | 6 ++ 1 file changed, 2 insertions(+), 4 deletions(-) diff --git a/libavutil/x86/x86inc.asm b/libavutil/x86/x86inc.asm index f2635ea..3ad149c 100644 ---

[FFmpeg-devel] [PATCH 3/7] x86inc: Preserve arguments when allocating stack space

When allocating stack space with a larger alignment than the known stack alignment a temporary register is used for storing the stack pointer. Ensure that this isn't one of the registers used for passing arguments. --- libavutil/x86/x86inc.asm | 6 -- 1 file changed, 4 insertions(+), 2

Re: [FFmpeg-devel] [PATCH 1/5] libavutil: Some VAAPI infrastructure

On 17/01/16 17:53, wm4 wrote: On Sun, 17 Jan 2016 17:34:55 + Mark Thompson wrote: From 2442c1aca8778167c2e60a34d03ed452737f1366 Mon Sep 17 00:00:00 2001 From: Mark Thompson Date: Sun, 17 Jan 2016 15:48:54 + Subject: [PATCH 1/5] libavutil: Some VAAPI

Re: [FFmpeg-devel] [PATCH] avfilter/vf_crop: make it possible to use frame metadata when cropping

On Sun, Jan 17, 2016 at 20:15:45 +0100, Paul B Mahol wrote: Nice feature. > +@item metadata > +If set to 1 it will use frame metadata instead to obtain all parameters. > +To make use of this feature you need to start @ref{cropdetect} filter before > +this filter. I would probably put it in

[FFmpeg-devel] [PATCH v2 0/5] VAAPI support infrastructure, encoders

Hi, Here is a new version of this patchset. See previous email for the full summary. This fixes the main thread-safety complaint, that libva initialisation and operations weren't mutually excluded. It guards the

Re: [FFmpeg-devel] [PATCH v2 1/2] ffplay: Seek only when pressing the right mouse button on the video window.

Vittorio Gambaletta (VittGam vittgam.net> writes: > This commit changes that behaviour to seek only when the right > mouse button is used to click and drag on the window. Please add a line to the Changelog. Carl Eugen ___ ffmpeg-devel mailing list

Re: [FFmpeg-devel] [PATCH] lavf/img2dec: change the default pattern to none

On Sun, Jan 17, 2016 at 08:28:32PM +0100, Marton Balint wrote: > > On Sun, 17 Jan 2016, Carl Eugen Hoyos wrote: > >Marton Balint passwd.hu> writes: > > > >>The current default pattern type, glob_sequence is deprecated > >>for almost 3.5 years, > > > >Sorry if I misunderstand: When was it

[FFmpeg-devel] [PATCH v2 4/5] libavcodec: add VAAPI H.265 encoder

From 7f4e62e0682786681fef1c6f52429fea6443a843 Mon Sep 17 00:00:00 2001 From: Mark Thompson Date: Sun, 17 Jan 2016 22:15:49 + Subject: [PATCH 4/5] libavcodec: add VAAPI H.265 encoder --- configure |1 + libavcodec/Makefile |1 +

[FFmpeg-devel] [PATCH v2 5/5] libavfilter: add VAAPI surface converter

From d1ddb63818c6ee04c7a25c5223fda9c50e19f4f4 Mon Sep 17 00:00:00 2001 From: Mark Thompson Date: Sun, 17 Jan 2016 22:16:13 + Subject: [PATCH 5/5] libavfilter: add VAAPI surface converter --- configure | 1 + libavfilter/Makefile| 1 +

Re: [FFmpeg-devel] [PATCH 1/5] libavutil: Some VAAPI infrastructure

On Sun, Jan 17, 2016 at 7:13 PM, Mark Thompson wrote: > On 17/01/16 17:53, wm4 wrote: >> >> On Sun, 17 Jan 2016 17:34:55 + >> Mark Thompson wrote: >> >>> From 2442c1aca8778167c2e60a34d03ed452737f1366 Mon Sep 17 00:00:00 2001 >>> From: Mark Thompson

Re: [FFmpeg-devel] [PATCH 1/5] libavutil: Some VAAPI infrastructure

On Sun, 17 Jan 2016 18:13:50 + Mark Thompson wrote: > On 17/01/16 17:53, wm4 wrote: > > On Sun, 17 Jan 2016 17:34:55 + > > Mark Thompson wrote: > > > >> From 2442c1aca8778167c2e60a34d03ed452737f1366 Mon Sep 17 00:00:00 2001 > >> From: Mark Thompson

Re: [FFmpeg-devel] [PATCH] lavf/img2dec: change the default pattern to none

On Sun, 17 Jan 2016, Carl Eugen Hoyos wrote: Marton Balint passwd.hu> writes: The current default pattern type, glob_sequence is deprecated for almost 3.5 years, Sorry if I misunderstand: When was it deprecated? In commit 3a06ea84 at 2012-08-06 by Stefano. before removing it, we need

[FFmpeg-devel] [PATCH] avfilter/vf_crop: make it possible to use frame metadata when cropping

Signed-off-by: Paul B Mahol --- doc/filters.texi | 6 ++ libavfilter/vf_crop.c | 54 +++ 2 files changed, 48 insertions(+), 12 deletions(-) diff --git a/doc/filters.texi b/doc/filters.texi index f4bda6a..aee4e66 100644

Re: [FFmpeg-devel] [PATCH 1/5] libavutil: Some VAAPI infrastructure

On 17/01/16 19:46, Mark Thompson wrote: On 17/01/16 18:46, wm4 wrote: There are two issues: 1. global state in libav* which is not synchronized 2. thread-safety within 1. is is completely unacceptable, because it can trigger undefined behavior if there is more than 1 libav* user in the same

[FFmpeg-devel] [PATCH v3] lavc/rawdec: Use AV_PIX_FMT_PAL8 for 1-bit raw QuickTime video

Alright Michael, I'm not entirely sure of what I'm doing regarding the FFALIGN(avctx->width, 32), but at least the two odd-width (113 and 129 pixels) 1 bpp files that failed to render correctly with the last patch now seem to be OK. Please give it a try. Original description follows: Match

[FFmpeg-devel] [PATCH] doc/filters: add yet another fftfilt example

Hi, patch attached. From 60bf479bb8c502163084cc9ed545f98b23ec3d9f Mon Sep 17 00:00:00 2001 From: Paul B Mahol Date: Sun, 17 Jan 2016 11:13:33 +0100 Subject: [PATCH] doc/filters: add yet another fftfilt example Signed-off-by: Paul B Mahol ---

[FFmpeg-devel] [PATCH] checkasm: add videodsp emulated_edge_mc test.

--- tests/checkasm/Makefile | 1 + tests/checkasm/checkasm.c | 3 ++ tests/checkasm/checkasm.h | 1 + tests/checkasm/videodsp.c | 88 +++ 4 files changed, 93 insertions(+) create mode 100644 tests/checkasm/videodsp.c diff --git

Re: [FFmpeg-devel] [PATCH 2/5] ffmpeg: hwaccel helper for VAAPI decode

On Sun, Jan 17, 2016 at 05:35:36PM +, Mark Thompson wrote: [...] > +static int vaapi_retrieve_data(AVCodecContext *avctx, AVFrame *input_frame) > +{ > +InputStream *ist = avctx->opaque; > +VAAPIDecoderContext *ctx = ist->hwaccel_ctx; > +AVVAAPISurfaceConfig *output = >output; > +

Re: [FFmpeg-devel] [PATCH] avfilter/vf_crop: make it possible to use frame metadata when cropping

On 1/17/16, Moritz Barsnick wrote: > On Sun, Jan 17, 2016 at 20:15:45 +0100, Paul B Mahol wrote: > > Nice feature. > >> +@item metadata >> +If set to 1 it will use frame metadata instead to obtain all parameters. >> +To make use of this feature you need to start @ref{cropdetect}

Re: [FFmpeg-devel] [PATCH]lavc/x264: Improve level setting

Michael Niedermayer niedermayer.cc> writes: > > > > > this should check that teres no "tail" after the integer > > > > > > > > For which command line will this make a difference? > > > > I didn't find one... > indeed, i guess x264 checks this internally before > our code gets a chance to fail

Re: [FFmpeg-devel] [PATCH v2 1/5] libavutil: some VAAPI infrastructure

On Sun, Jan 17, 2016 at 10:45:37PM +, Mark Thompson wrote: > From 45a803b627d0180c1aac928756924bd39ddf529d Mon Sep 17 00:00:00 2001 > From: Mark Thompson > Date: Sun, 17 Jan 2016 22:13:20 + > Subject: [PATCH 1/5] libavutil: some VAAPI infrastructure > > --- > configure

Re: [FFmpeg-devel] Allow MPEGTS and KLV metadata remuxing

On Sun, Jan 17, 2016 at 01:24:29PM +0100, Stefano Sabatini wrote: > Hi, > > the attached patches allow to remux MPEGTS files containing KLV > metadata. Since there are two kinds of metadata (synchronous and > asynchronous), the first patch creates side data which stores the > stream ID used to

Re: [FFmpeg-devel] [PATCH] lavf/img2dec: change the default pattern to none

Marton Balint passwd.hu> writes: > >> The current default pattern type, glob_sequence is > >> deprecated for almost 3.5 years, > > > > Sorry if I misunderstand: When was it deprecated? > > In commit 3a06ea84 at 2012-08-06 by Stefano. The way I understand the commit it was only planned to

Re: [FFmpeg-devel] [PATCH 0/7] x86inc: Sync changes from x264

On Mon, Jan 18, 2016 at 01:33:44AM +0100, Michael Niedermayer wrote: > On Sun, Jan 17, 2016 at 11:59:07PM +0100, Henrik Gramner wrote: > > The following patches were recently pushed to x264. > > > > Geza Lore (1): > > x86inc: Add debug symbols indicating sizes of compiled functions > > > >

Re: [FFmpeg-devel] [PATCH]lavc/x264: Improve level setting

On Sun, Jan 17, 2016 at 11:17:36PM +, Carl Eugen Hoyos wrote: > Michael Niedermayer niedermayer.cc> writes: > > > > > > > this should check that teres no "tail" after the integer > > > > > > > > > > For which command line will this make a difference? > > > > > I didn't find one... > > >

Re: [FFmpeg-devel] [PATCH v2 1/5] libavutil: some VAAPI infrastructure

On Sun, Jan 17, 2016 at 11:45 PM, Mark Thompson wrote: > From 45a803b627d0180c1aac928756924bd39ddf529d Mon Sep 17 00:00:00 2001 > From: Mark Thompson > Date: Sun, 17 Jan 2016 22:13:20 + > Subject: [PATCH 1/5] libavutil: some VAAPI infrastructure > > --- >

Re: [FFmpeg-devel] [PATCH]lavc/x264: Improve level setting

Michael Niedermayer niedermayer.cc> writes: > > So is the patch ok? > > id use strtol instead of strtod to parse a value that > is expeted to be in integer, or is it not expected to > be an integer ? The original patch (that does not care about tail since it can't be reached anyway) uses

Re: [FFmpeg-devel] [PATCH] doc/filters: add yet another fftfilt example

On Sun, Jan 17, 2016 at 06:28:13PM +, Paul B Mahol wrote: > Hi, > > patch attached. > filters.texi |6 ++ > 1 file changed, 6 insertions(+) > 319d3bfd40717f1381a4f57b95bdd65e4a17377e > 0001-doc-filters-add-yet-another-fftfilt-example.patch > From

Re: [FFmpeg-devel] [PATCH v3] lavc/rawdec: Use AV_PIX_FMT_PAL8 for 1-bit raw QuickTime video

On 01/17/2016 10:50 PM, Mats Peterson wrote: Alright Michael, I'm not entirely sure of what I'm doing regarding the FFALIGN(avctx->width, 32), but at least the two odd-width (113 and 129 In fact, I *am* sure of what I'm doing, since the original rows are obviously 4-byte aligned, and for 1

Re: [FFmpeg-devel] [PATCH v2 1/5] libavutil: some VAAPI infrastructure

On 17/01/16 23:55, Michael Niedermayer wrote: > On Sun, Jan 17, 2016 at 10:45:37PM +, Mark Thompson wrote: >> From 45a803b627d0180c1aac928756924bd39ddf529d Mon Sep 17 00:00:00 2001 >> From: Mark Thompson >> Date: Sun, 17 Jan 2016 22:13:20 + >> Subject: [PATCH 1/5]

Re: [FFmpeg-devel] [PATCH]lavf/icodec: Improve probe function

Michael Bradshaw gmail.com> writes: > > +return AVPROBE_SCORE_MAX / 4 + 1; > > A score of 26 seems low to me, but maybe that's just me. Increased the score and pushed. Thank you, Carl Eugen ___ ffmpeg-devel mailing list ffmpeg-devel@ffmpeg.org

Re: [FFmpeg-devel] [PATCH 0/7] x86inc: Sync changes from x264

On Sun, Jan 17, 2016 at 11:59:07PM +0100, Henrik Gramner wrote: > The following patches were recently pushed to x264. > > Geza Lore (1): > x86inc: Add debug symbols indicating sizes of compiled functions > > Henrik Gramner (6): > x86inc: Be more verbose in assertion failures > x86inc:

Re: [FFmpeg-devel] [PATCH 0/7] x86inc: Sync changes from x264

On 2016-01-17 23:59, Henrik Gramner wrote: > The following patches were recently pushed to x264. > > Geza Lore (1): > x86inc: Add debug symbols indicating sizes of compiled functions > > Henrik Gramner (6): > x86inc: Be more verbose in assertion failures > x86inc: Improve FMA instruction

[FFmpeg-devel] Question about 2 bpp Macintosh palette

Where does the seemingly weird 2 bpp palette ff_qt_default_palette_4[] in qtpalette.h come from? The file below will look mighty strange when using this one (play it with FFplay, or possibly VLC, since MPlayer still thinks 'raw' QuickTime video is 24-bit RGB).

[FFmpeg-devel] Allow MPEGTS and KLV metadata remuxing

Hi, the attached patches allow to remux MPEGTS files containing KLV metadata. Since there are two kinds of metadata (synchronous and asynchronous), the first patch creates side data which stores the stream ID used to distinguish the two kind of packets, which is otherwise lost. At the moment,

Re: [FFmpeg-devel] [PATCH] remove the deprecated avpicture_get_size() function

On Sat, Jan 16, 2016 at 09:41:54PM -0800, Eddie Hao wrote: > --- > libavcodec/libutvideodec.cpp | 3 ++- > libavcodec/libutvideoenc.cpp | 3 ++- > libavdevice/decklink_dec.cpp | 2 +- > libavdevice/lavfi.c | 3 ++- > libavformat/frmdec.c | 3 ++- > 5 files changed, 9

[FFmpeg-devel] [PATCH] avcodec/ass_split: Fix null pointer dereference in ff_ass_style_get()

From: Michael Niedermayer Fixes: 55d71971da50365d542ed14b65565fe1/signal_sigsegv_4765a4_8499_f146af090a94f591d6254515c7700ef5.mkv Found-by: Mateusz "j00ru" Jurczyk and Gynvael Coldwind Signed-off-by: Michael Niedermayer ---

Re: [FFmpeg-devel] [PATCH] avcodec/diracdec: Fix qfactor/offset tables

On Wed, Jan 13, 2016 at 11:51:47PM +0100, Michael Niedermayer wrote: > From: Michael Niedermayer > > It seems the previous tables where calculated with 32bit integers ignoring > overflows. > Also check for the max qindex, the value is choosen so that the qfactor/offset >

Re: [FFmpeg-devel] [PATCH 1/4] fate: add 10-bit v210 encoder tests

On 2016-01-17 03:11, James Darnley wrote: > On 2016-01-15 20:07, James Darnley wrote: >> ... > > If nobody has further comments about the patches I will probably push > these after I wake up. > A little later than planned but now pushed. signature.asc Description: OpenPGP digital signature

[FFmpeg-devel] [PATCH 0/5] VAAPI support infrastructure, encoders

Hi all, This patch series contains the following: * Some VAAPI infrastructure in libavutil. Initialisation code, along with routines to handle mapping and copying to/from VA surfaces. Doesn't really fit as a public API, but has to go there because it's used by the parts in both libavfilter

[FFmpeg-devel] [PATCH 1/5] libavutil: Some VAAPI infrastructure

From 2442c1aca8778167c2e60a34d03ed452737f1366 Mon Sep 17 00:00:00 2001 From: Mark Thompson Date: Sun, 17 Jan 2016 15:48:54 + Subject: [PATCH 1/5] libavutil: Some VAAPI infrastructure --- configure | 4 + libavutil/Makefile | 1 + libavutil/vaapi.c | 732

[FFmpeg-devel] [PATCH 2/5] ffmpeg: hwaccel helper for VAAPI decode

From 390d4fdacbc2954489f6baa44190e7bf2f2621cc Mon Sep 17 00:00:00 2001 From: Mark Thompson Date: Sun, 17 Jan 2016 15:55:32 + Subject: [PATCH 2/5] ffmpeg: hwaccel helper for VAAPI decode --- Makefile | 1 + ffmpeg.h | 2 + ffmpeg_opt.c | 3 +

[FFmpeg-devel] [PATCH 4/5] libavcodec: add VAAPI H.265 encoder

From 5ea0c6f4e9a0086a1972bb142e9bd9b6b779a6c1 Mon Sep 17 00:00:00 2001 From: Mark Thompson Date: Sun, 17 Jan 2016 15:59:01 + Subject: [PATCH 4/5] libavcodec: add VAAPI H.265 encoder --- configure |1 + libavcodec/Makefile |1 +

Re: [FFmpeg-devel] [PATCH 1/5] libavutil: Some VAAPI infrastructure

On Sun, 17 Jan 2016 17:34:55 + Mark Thompson wrote: > From 2442c1aca8778167c2e60a34d03ed452737f1366 Mon Sep 17 00:00:00 2001 > From: Mark Thompson > Date: Sun, 17 Jan 2016 15:48:54 + > Subject: [PATCH 1/5] libavutil: Some VAAPI infrastructure > > + >