Re: [libav-devel] [PATCH] vp56: only compile MMX SIMD on x86-32.

2012-07-27 Thread Luca Barbato
On 07/27/2012 07:09 AM, Ronald S. Bultje wrote: From: Ronald S. Bultje rsbul...@gmail.com All x86-64 CPUs have SSE2, so the MMX version will never be used. This leads to smaller binaries. --- libavcodec/x86/vp56dsp.asm|2 ++ libavcodec/x86/vp56dsp_init.c |2 ++ 2 files

Re: [libav-devel] [PATCH] vp3: don't compile mmx IDCT functions on x86-64.

2012-07-27 Thread Luca Barbato
On 07/27/2012 07:16 AM, Ronald S. Bultje wrote: From: Ronald S. Bultje rsbul...@gmail.com 64-bit CPUs always have SSE2, and a SSE2 version exists, thus the MMX version will never be used. --- libavcodec/x86/vp3dsp.asm|3 +++ libavcodec/x86/vp3dsp_init.c |2 ++ 2 files

Re: [libav-devel] [PATCH 1/6] build: Only compile and run dct-test if AAN DCT tables are enabled

2012-07-27 Thread Diego Biurrun
On Thu, Jul 26, 2012 at 05:23:05PM -0700, Ronald S. Bultje wrote: On Thu, Jul 26, 2012 at 5:15 PM, Diego Biurrun di...@biurrun.de wrote: --- libavcodec/Makefile |2 +- tests/fate/dct.mak |2 +- 2 files changed, 2 insertions(+), 2 deletions(-) This test tests a lot more than

[libav-devel] [PATCH] x11grab: cosmetics: consistent naming for x11grab-related things

2012-07-27 Thread Diego Biurrun
--- Patch rebased on top of master and now also drops the silly _device part from the x11grab name. configure|2 +- libavdevice/Makefile |2 +- libavdevice/alldevices.c |2 +- libavdevice/avdevice.h |2 +- libavdevice/x11grab.c| 25

Re: [libav-devel] [PATCH] x11grab: cosmetics: consistent naming for x11grab-related things

2012-07-27 Thread Luca Barbato
On 07/27/2012 10:27 AM, Diego Biurrun wrote: --- Patch rebased on top of master and now also drops the silly _device part from the x11grab name. Ok -- Luca Barbato Gentoo/linux http://dev.gentoo.org/~lu_zero ___ libav-devel mailing list

Re: [libav-devel] [PATCH 2/2] G.723.1 demuxer and decoder

2012-07-27 Thread Mohamed Naufal
On 22 July 2012 11:33, Kostya Shishkov kostya.shish...@gmail.com wrote: On Wed, Jul 18, 2012 at 04:44:24PM -0400, Justin Ruggles wrote: On 07/18/2012 01:23 PM, Kostya Shishkov wrote: +memset(p-frame.data[0], 0, FRAME_LEN * 2); [...] +memcpy(p-frame.data[0], p-audio +

Re: [libav-devel] [PATCH 2/2] G.723.1 demuxer and decoder

2012-07-27 Thread Kostya Shishkov
On Fri, Jul 27, 2012 at 03:15:26PM +0530, Mohamed Naufal wrote: On 22 July 2012 11:33, Kostya Shishkov kostya.shish...@gmail.com wrote: On Wed, Jul 18, 2012 at 04:44:24PM -0400, Justin Ruggles wrote: On 07/18/2012 01:23 PM, Kostya Shishkov wrote: +memset(p-frame.data[0], 0,

Re: [libav-devel] [PATCH 1/6] build: Only compile and run dct-test if AAN DCT tables are enabled

2012-07-27 Thread Måns Rullgård
Diego Biurrun di...@biurrun.de writes: --- libavcodec/Makefile |2 +- tests/fate/dct.mak |2 +- 2 files changed, 2 insertions(+), 2 deletions(-) diff --git a/libavcodec/Makefile b/libavcodec/Makefile index 3a74338..7f1cadb 100644 --- a/libavcodec/Makefile +++

Re: [libav-devel] [PATCH 2/6] configure: Drop some special-casing for unsupported gcc 2.95

2012-07-27 Thread Måns Rullgård
Diego Biurrun di...@biurrun.de writes: --- configure |3 --- 1 files changed, 0 insertions(+), 3 deletions(-) diff --git a/configure b/configure index fd90369..4502fc0 100755 --- a/configure +++ b/configure @@ -2065,9 +2065,6 @@ probe_cc(){ gcc_pkg_ver=$(expr $gcc_version

Re: [libav-devel] [PATCH 4/6] build: Simplify CFLAGS handling in header compilation rule

2012-07-27 Thread Måns Rullgård
Diego Biurrun di...@biurrun.de writes: --- Makefile |2 +- 1 files changed, 1 insertions(+), 1 deletions(-) diff --git a/Makefile b/Makefile index 37faa76..93c6621 100644 --- a/Makefile +++ b/Makefile @@ -46,7 +46,7 @@ COMPILE_S = $(call COMPILE,AS) $(COMPILE_S) %.ho:

Re: [libav-devel] [PATCH 3/6] build: Rename YASMDEP variable to DEPYASM for consistency

2012-07-27 Thread Måns Rullgård
Diego Biurrun di...@biurrun.de writes: --- Makefile|2 +- configure |2 +- library.mak |2 +- 3 files changed, 3 insertions(+), 3 deletions(-) diff --git a/Makefile b/Makefile index 3716e9e..37faa76 100644 --- a/Makefile +++ b/Makefile @@ -11,7 +11,7 @@ ifndef V Q

Re: [libav-devel] [PATCH 5/6] configure: Place non-user-selectable components in CONFIG_EXTRA

2012-07-27 Thread Måns Rullgård
Diego Biurrun di...@biurrun.de writes: --- configure | 16 1 files changed, 8 insertions(+), 8 deletions(-) That's a weird description. Those options are currently user-selectable by virtue of being in CONFIG_LIST. diff --git a/configure b/configure index

[libav-devel] [PATCH] configure: Add missing descriptions to help output

2012-07-27 Thread Diego Biurrun
--- configure |5 - 1 files changed, 4 insertions(+), 1 deletions(-) diff --git a/configure b/configure index be666ab..fd04699 100755 --- a/configure +++ b/configure @@ -207,16 +207,18 @@ Advanced options (experts only): --ar=AR use archive tool AR [$ar_default]

[libav-devel] [PATCH] build: Use portable compiler flag constructs in header compilation rule

2012-07-27 Thread Diego Biurrun
--- Makefile |2 +- 1 files changed, 1 insertions(+), 1 deletions(-) diff --git a/Makefile b/Makefile index 37faa76..5a7d591 100644 --- a/Makefile +++ b/Makefile @@ -46,7 +46,7 @@ COMPILE_S = $(call COMPILE,AS) $(COMPILE_S) %.ho: %.h - $(CC) $(CPPFLAGS) $(CFLAGS) -c -o $@ -x

Re: [libav-devel] [PATCH] configure: Add missing descriptions to help output

2012-07-27 Thread Måns Rullgård
Diego Biurrun di...@biurrun.de writes: + --disable-thumb disable ARM Thumb instruction set This is still a bad description. Firstly, it is off by default. Secondly, it is not an optional extra that can be enabled or disabled. The _entire thing_ is compiled either as ARM or Thumb.

Re: [libav-devel] [PATCH] build: Use portable compiler flag constructs in header compilation rule

2012-07-27 Thread Måns Rullgård
Diego Biurrun di...@biurrun.de writes: --- Makefile |2 +- 1 files changed, 1 insertions(+), 1 deletions(-) diff --git a/Makefile b/Makefile index 37faa76..5a7d591 100644 --- a/Makefile +++ b/Makefile @@ -46,7 +46,7 @@ COMPILE_S = $(call COMPILE,AS) $(COMPILE_S) %.ho:

Re: [libav-devel] [PATCH] build: Use portable compiler flag constructs in header compilation rule

2012-07-27 Thread Diego Biurrun
On Fri, Jul 27, 2012 at 11:57:40AM +0100, Måns Rullgård wrote: Diego Biurrun di...@biurrun.de writes: --- a/Makefile +++ b/Makefile @@ -46,7 +46,7 @@ COMPILE_S = $(call COMPILE,AS) %.ho: %.h - $(CC) $(CPPFLAGS) $(CFLAGS) -c -o $@ -x c $ + $(CC) $(CCFLAGS) -c $(CC_O) -x c $

[libav-devel] [PATCH] rtmp: add ff_amf_read_* functions

2012-07-27 Thread Jordi Ortiz
--- libavformat/rtmppkt.c | 57 + libavformat/rtmppkt.h | 43 + 2 files changed, 100 insertions(+) diff --git a/libavformat/rtmppkt.c b/libavformat/rtmppkt.c index 4ce238d..c592f61 100644 ---

Re: [libav-devel] [PATCH 5/6] configure: Place non-user-selectable components in CONFIG_EXTRA

2012-07-27 Thread Diego Biurrun
On Fri, Jul 27, 2012 at 11:29:02AM +0100, Måns Rullgård wrote: Diego Biurrun di...@biurrun.de writes: --- configure | 16 1 files changed, 8 insertions(+), 8 deletions(-) That's a weird description. Those options are currently user-selectable by virtue of being in

Re: [libav-devel] [PATCH 5/6] configure: Place non-user-selectable components in CONFIG_EXTRA

2012-07-27 Thread Martin Storsjö
On Fri, 27 Jul 2012, Måns Rullgård wrote: Diego Biurrun di...@biurrun.de writes: pic rdft -rtpdec What does this option do? It's used as an intermediate component so that all demuxers that depend on it don't need to list all the 20ish object files that comprise the rtpdec

[libav-devel] VC-1 security fixes cherry-picked from FFmpeg git

2012-07-27 Thread Mashiat Sarker Shakkhar
These patches won't probably improve anything in terms of decoded picture. ___ libav-devel mailing list libav-devel@libav.org https://lists.libav.org/mailman/listinfo/libav-devel

[libav-devel] [PATCH 1/6] vc1: avoid reading beyond the last line in vc1_draw_sprites()

2012-07-27 Thread Mashiat Sarker Shakkhar
From: Michael Niedermayer michae...@gmx.at Fixes overread Found-by: Mateusz j00ru Jurczyk and Gynvael Coldwind Signed-off-by: Michael Niedermayer michae...@gmx.at --- libavcodec/vc1dec.c |4 ++-- 1 files changed, 2 insertions(+), 2 deletions(-) diff --git a/libavcodec/vc1dec.c

[libav-devel] [PATCH 2/6] vc1dec: dont ignore ff_vc1_parse_frame_header_advs return value

2012-07-27 Thread Mashiat Sarker Shakkhar
From: Michael Niedermayer michae...@gmx.at Found-by: Mateusz j00ru Jurczyk and Gynvael Coldwind Signed-off-by: Michael Niedermayer michae...@gmx.at --- libavcodec/vc1dec.c | 14 ++ 1 files changed, 10 insertions(+), 4 deletions(-) diff --git a/libavcodec/vc1dec.c

[libav-devel] [PATCH 3/6] vc1dec: Sanity-check macroblock quantizer

2012-07-27 Thread Mashiat Sarker Shakkhar
From: Michael Niedermayer michae...@gmx.at Found-by: Mateusz j00ru Jurczyk and Gynvael Coldwind Signed-off-by: Michael Niedermayer michae...@gmx.at --- libavcodec/vc1dec.c |3 +++ 1 files changed, 3 insertions(+), 0 deletions(-) diff --git a/libavcodec/vc1dec.c b/libavcodec/vc1dec.c index

[libav-devel] [PATCH 4/6] vc1dec: check end_mb_y / start_mb_y validity

2012-07-27 Thread Mashiat Sarker Shakkhar
From: Michael Niedermayer michae...@gmx.at Found-by: Mateusz j00ru Jurczyk and Gynvael Coldwind Signed-off-by: Michael Niedermayer michae...@gmx.at --- libavcodec/vc1dec.c |4 1 files changed, 4 insertions(+), 0 deletions(-) diff --git a/libavcodec/vc1dec.c b/libavcodec/vc1dec.c index

[libav-devel] [PATCH 5/6] vc1dec: check that coded slice positions and interlacing match.

2012-07-27 Thread Mashiat Sarker Shakkhar
From: Michael Niedermayer michae...@gmx.at This fixes out of array writes Found-by: Mateusz j00ru Jurczyk and Gynvael Coldwind Signed-off-by: Michael Niedermayer michae...@gmx.at --- libavcodec/vc1dec.c |4 1 files changed, 4 insertions(+), 0 deletions(-) diff --git

[libav-devel] [PATCH 6/6] vc1dec: dont apply the loop filter on fields

2012-07-27 Thread Mashiat Sarker Shakkhar
From: Michael Niedermayer michae...@gmx.at Fixes read of uninitialized memory Signed-off-by: Michael Niedermayer michae...@gmx.at --- libavcodec/vc1dec.c|2 +- tests/ref/fate/vc1_sa10143 | 58 ++-- 2 files changed, 30 insertions(+), 30

Re: [libav-devel] [PATCH 3/6] vc1dec: Sanity-check macroblock quantizer

2012-07-27 Thread Kostya Shishkov
On Fri, Jul 27, 2012 at 06:49:41PM +0600, Mashiat Sarker Shakkhar wrote: From: Michael Niedermayer michae...@gmx.at Found-by: Mateusz j00ru Jurczyk and Gynvael Coldwind Signed-off-by: Michael Niedermayer michae...@gmx.at --- libavcodec/vc1dec.c |3 +++ 1 files changed, 3

Re: [libav-devel] [PATCH 4/6] vc1dec: check end_mb_y / start_mb_y validity

2012-07-27 Thread Kostya Shishkov
On Fri, Jul 27, 2012 at 06:49:42PM +0600, Mashiat Sarker Shakkhar wrote: From: Michael Niedermayer michae...@gmx.at Found-by: Mateusz j00ru Jurczyk and Gynvael Coldwind Signed-off-by: Michael Niedermayer michae...@gmx.at --- libavcodec/vc1dec.c |4 1 files changed, 4

Re: [libav-devel] [PATCH 5/6] vc1dec: check that coded slice positions and interlacing match.

2012-07-27 Thread Kostya Shishkov
On Fri, Jul 27, 2012 at 06:49:43PM +0600, Mashiat Sarker Shakkhar wrote: From: Michael Niedermayer michae...@gmx.at This fixes out of array writes Found-by: Mateusz j00ru Jurczyk and Gynvael Coldwind Signed-off-by: Michael Niedermayer michae...@gmx.at --- libavcodec/vc1dec.c |4

Re: [libav-devel] [PATCH 6/6] vc1dec: dont apply the loop filter on fields

2012-07-27 Thread Kostya Shishkov
On Fri, Jul 27, 2012 at 06:49:44PM +0600, Mashiat Sarker Shakkhar wrote: From: Michael Niedermayer michae...@gmx.at Fixes read of uninitialized memory Signed-off-by: Michael Niedermayer michae...@gmx.at --- libavcodec/vc1dec.c|2 +- tests/ref/fate/vc1_sa10143 | 58

Re: [libav-devel] [PATCH 1/6] vc1: avoid reading beyond the last line in vc1_draw_sprites()

2012-07-27 Thread Kostya Shishkov
On Fri, Jul 27, 2012 at 06:49:39PM +0600, Mashiat Sarker Shakkhar wrote: From: Michael Niedermayer michae...@gmx.at Fixes overread Found-by: Mateusz j00ru Jurczyk and Gynvael Coldwind Signed-off-by: Michael Niedermayer michae...@gmx.at --- libavcodec/vc1dec.c |4 ++-- 1 files

Re: [libav-devel] [PATCH 2/6] vc1dec: dont ignore ff_vc1_parse_frame_header_advs return value

2012-07-27 Thread Kostya Shishkov
On Fri, Jul 27, 2012 at 06:49:40PM +0600, Mashiat Sarker Shakkhar wrote: From: Michael Niedermayer michae...@gmx.at Found-by: Mateusz j00ru Jurczyk and Gynvael Coldwind Signed-off-by: Michael Niedermayer michae...@gmx.at --- libavcodec/vc1dec.c | 14 ++ 1 files changed, 10

Re: [libav-devel] [PATCH 6/6] vc1dec: dont apply the loop filter on fields

2012-07-27 Thread Mashiat Sarker Shakkhar
On 7/27/2012 7:04 PM, Kostya Shishkov wrote: On Fri, Jul 27, 2012 at 06:49:44PM +0600, Mashiat Sarker Shakkhar wrote: From: Michael Niedermayer michae...@gmx.at Fixes read of uninitialized memory Signed-off-by: Michael Niedermayer michae...@gmx.at --- libavcodec/vc1dec.c|2 +-

Re: [libav-devel] [PATCH 6/6] vc1dec: dont apply the loop filter on fields

2012-07-27 Thread Kostya Shishkov
On Fri, Jul 27, 2012 at 07:19:49PM +0600, Mashiat Sarker Shakkhar wrote: On 7/27/2012 7:04 PM, Kostya Shishkov wrote: On Fri, Jul 27, 2012 at 06:49:44PM +0600, Mashiat Sarker Shakkhar wrote: From: Michael Niedermayer michae...@gmx.at Fixes read of uninitialized memory Signed-off-by:

Re: [libav-devel] [PATCH] vp3: don't compile mmx IDCT functions on x86-64.

2012-07-27 Thread Ronald S. Bultje
Hi, On Thu, Jul 26, 2012 at 11:40 PM, Luca Barbato lu_z...@gentoo.org wrote: On 07/27/2012 07:16 AM, Ronald S. Bultje wrote: From: Ronald S. Bultje rsbul...@gmail.com 64-bit CPUs always have SSE2, and a SSE2 version exists, thus the MMX version will never be used. ---

[libav-devel] [PATCH 1/4] lavf: remove commented out cruft in avformat_find_stream_info()

2012-07-27 Thread Anton Khirnov
--- libavformat/utils.c |8 +--- 1 file changed, 1 insertion(+), 7 deletions(-) diff --git a/libavformat/utils.c b/libavformat/utils.c index 156c527..8dbf1e5 100644 --- a/libavformat/utils.c +++ b/libavformat/utils.c @@ -2403,8 +2403,6 @@ int avformat_find_stream_info(AVFormatContext

[libav-devel] [PATCH 2/4] lavf: use dts difference instead of AVPacket.duration for framerate estimation

2012-07-27 Thread Anton Khirnov
AVPacket.duration is mostly made up and thus completely useless, this is especially true for video streams. --- libavformat/avformat.h |9 + libavformat/utils.c| 22 +++--- 2 files changed, 28 insertions(+), 3 deletions(-) diff --git a/libavformat/avformat.h

[libav-devel] [PATCH 3/4] lavf: round estimated average fps to a standard fps.

2012-07-27 Thread Anton Khirnov
--- libavformat/utils.c | 18 ++ 1 file changed, 18 insertions(+) diff --git a/libavformat/utils.c b/libavformat/utils.c index 2a43645..6c570a7 100644 --- a/libavformat/utils.c +++ b/libavformat/utils.c @@ -2469,10 +2469,28 @@ int avformat_find_stream_info(AVFormatContext *ic,

[libav-devel] [PATCH 4/4] lavf: deprecate r_frame_rate.

2012-07-27 Thread Anton Khirnov
According to its description, it is supposed to be the LCM of all the frame durations. The usability of such a thing is vanishingly small, especially since we cannot determine it with any amount of reliability. Therefore get rid of it after the next bump. Replace it with the average framerate

Re: [libav-devel] [PATCH 5/6] configure: Place non-user-selectable components in CONFIG_EXTRA

2012-07-27 Thread Måns Rullgård
Diego Biurrun di...@biurrun.de writes: On Fri, Jul 27, 2012 at 11:29:02AM +0100, Måns Rullgård wrote: Diego Biurrun di...@biurrun.de writes: --- configure | 16 1 files changed, 8 insertions(+), 8 deletions(-) That's a weird description. Those options are currently

Re: [libav-devel] [PATCH 4/6] vc1dec: check end_mb_y / start_mb_y validity

2012-07-27 Thread Mashiat Sarker Shakkhar
On 7/27/2012 7:01 PM, Kostya Shishkov wrote: On Fri, Jul 27, 2012 at 06:49:42PM +0600, Mashiat Sarker Shakkhar wrote: From: Michael Niedermayer michae...@gmx.at Found-by: Mateusz j00ru Jurczyk and Gynvael Coldwind Signed-off-by: Michael Niedermayer michae...@gmx.at --- libavcodec/vc1dec.c |

Re: [libav-devel] [PATCH 2/4] lavf: use dts difference instead of AVPacket.duration for framerate estimation

2012-07-27 Thread Justin Ruggles
On 07/27/2012 10:30 AM, Anton Khirnov wrote: AVPacket.duration is mostly made up and thus completely useless, this is especially true for video streams. --- libavformat/avformat.h |9 + libavformat/utils.c| 22 +++--- 2 files changed, 28 insertions(+), 3

Re: [libav-devel] [PATCH] h264_loopfilter: port x86 simd to cpuflags.

2012-07-27 Thread Loren Merritt
On Thu, 26 Jul 2012, Ronald S. Bultje wrote: From: Ronald S. Bultje rsbul...@gmail.com --- libavcodec/x86/h264_deblock.asm | 124 +++-- libavcodec/x86/h264_deblock_10bit.asm | 77 ++-- libavcodec/x86/h264dsp_mmx.c | 60

[libav-devel] [PATCH 1/4] vc1: avoid reading beyond the last line in vc1_draw_sprites()

2012-07-27 Thread Mashiat Sarker Shakkhar
Fixes overread Found-by: Mateusz j00ru Jurczyk and Gynvael Coldwind Signed-off-by: Michael Niedermayer michae...@gmx.at --- libavcodec/vc1dec.c |8 ++-- 1 files changed, 6 insertions(+), 2 deletions(-) diff --git a/libavcodec/vc1dec.c b/libavcodec/vc1dec.c index 51124cf..3feb312 100644

[libav-devel] [PATCH 2/4] vc1dec: Do not ignore ff_vc1_parse_frame_header_adv return value

2012-07-27 Thread Mashiat Sarker Shakkhar
Found-by: Mateusz j00ru Jurczyk and Gynvael Coldwind Signed-off-by: Michael Niedermayer michae...@gmx.at --- libavcodec/vc1dec.c | 14 ++ 1 files changed, 10 insertions(+), 4 deletions(-) diff --git a/libavcodec/vc1dec.c b/libavcodec/vc1dec.c index 3feb312..2ff1315 100644 ---

[libav-devel] [PATCH 3/4] vc1dec: Sanity-check macroblock quantizer

2012-07-27 Thread Mashiat Sarker Shakkhar
Found-by: Mateusz j00ru Jurczyk and Gynvael Coldwind Signed-off-by: Michael Niedermayer michae...@gmx.at --- libavcodec/vc1dec.c |4 1 files changed, 4 insertions(+), 0 deletions(-) diff --git a/libavcodec/vc1dec.c b/libavcodec/vc1dec.c index 2ff1315..5b0fe46 100644 ---

[libav-devel] [PATCH 4/4] vc1dec: check that coded slice positions and interlacing match.

2012-07-27 Thread Mashiat Sarker Shakkhar
This fixes out of array writes Found-by: Mateusz j00ru Jurczyk and Gynvael Coldwind Signed-off-by: Michael Niedermayer michae...@gmx.at --- libavcodec/vc1dec.c |4 1 files changed, 4 insertions(+), 0 deletions(-) diff --git a/libavcodec/vc1dec.c b/libavcodec/vc1dec.c index

[libav-devel] [PATCH] Add missing libavutil/time.h includes.

2012-07-27 Thread Anton Khirnov
--- libavdevice/fbdev.c |1 + libavdevice/jack_audio.c |1 + libavdevice/oss_audio.c |1 + libavdevice/x11grab.c|1 + libavformat/rtpdec.c |1 + libavformat/rtsp.c |1 + libavformat/rtspdec.c|1 + libavformat/rtspenc.c|1 +

Re: [libav-devel] [PATCH 1/4] vc1: avoid reading beyond the last line in vc1_draw_sprites()

2012-07-27 Thread Derek Buitenhuis
On 27/07/2012 11:10 AM, Mashiat Sarker Shakkhar wrote: Fixes overread Found-by: Mateusz j00ru Jurczyk and Gynvael Coldwind Signed-off-by: Michael Niedermayer michae...@gmx.at --- libavcodec/vc1dec.c |8 ++-- 1 files changed, 6 insertions(+), 2 deletions(-) Shouldn't some (or

Re: [libav-devel] [PATCH 1/4] vc1: avoid reading beyond the last line in vc1_draw_sprites()

2012-07-27 Thread Mashiat Sarker Shakkhar
On 7/27/2012 9:23 PM, Derek Buitenhuis wrote: On 27/07/2012 11:10 AM, Mashiat Sarker Shakkhar wrote: Fixes overread Found-by: Mateusz j00ru Jurczyk and Gynvael Coldwind Signed-off-by: Michael Niedermayer michae...@gmx.at --- libavcodec/vc1dec.c |8 ++-- 1 files changed, 6

Re: [libav-devel] [PATCH 1/3] dsputil: x86: convert PMINSD, PMAXSD, and CLIPD macros to use cpuflags

2012-07-27 Thread Justin Ruggles
On 07/21/2012 05:39 PM, Justin Ruggles wrote: --- Updated patch to allow float vs. dword min/max as a parameter to CLIPD instead of using 2 separate macros. libavcodec/x86/dsputil_mmx.c|6 ++-- libavcodec/x86/dsputil_yasm.asm | 66 +++

Re: [libav-devel] [PATCH 1/3] dsputil: x86: convert PMINSD, PMAXSD, and CLIPD macros to use cpuflags

2012-07-27 Thread Ronald S. Bultje
Hi, On Fri, Jul 27, 2012 at 8:54 AM, Justin Ruggles justin.rugg...@gmail.com wrote: On 07/21/2012 05:39 PM, Justin Ruggles wrote: --- Updated patch to allow float vs. dword min/max as a parameter to CLIPD instead of using 2 separate macros. libavcodec/x86/dsputil_mmx.c|6 ++--

Re: [libav-devel] [PATCH 2/3] dsputil: x86: convert some of the SPLATD macros to use cpuflags

2012-07-27 Thread Ronald S. Bultje
Hi, On Sat, Jul 21, 2012 at 2:39 PM, Justin Ruggles justin.rugg...@gmail.com wrote: %macro VECTOR_CLIP_INT32 2 cglobal vector_clip_int32, 5,5,11, dst, src, min, max, len +SPLATD_LOW m4, minm +SPLATD_LOW m5, maxm %if notcpuflag(sse4) cpuflag(sse2) notcpuflag(atom) -cvtsi2ss

Re: [libav-devel] [PATCH] x86/dsputilenc: bury inline asm under HAVE_INLINE_ASM.

2012-07-27 Thread Ronald S. Bultje
Hi, On Thu, Jul 26, 2012 at 6:42 AM, Måns Rullgård m...@mansr.com wrote: Ronald S. Bultje rsbul...@gmail.com writes: Hi, On Thu, Jul 26, 2012 at 2:23 AM, Måns Rullgård m...@mansr.com wrote: Ronald S. Bultje rsbul...@gmail.com writes: From: Ronald S. Bultje rsbul...@gmail.com ---

Re: [libav-devel] [PATCH] swscale: bury one more piece of inline asm under HAVE_INLINE_ASM.

2012-07-27 Thread Ronald S. Bultje
Hi, On Thu, Jul 26, 2012 at 9:46 AM, Ronald S. Bultje rsbul...@gmail.com wrote: On Thu, Jul 26, 2012 at 9:05 AM, Måns Rullgård m...@mansr.com wrote: Ronald S. Bultje rsbul...@gmail.com writes: On Thu, Jul 26, 2012 at 7:30 AM, Martin Storsjö mar...@martin.st wrote: On Thu, 26 Jul 2012, Ronald

[libav-devel] [PATCH] x86inc: clip num_args to 7 on x86-32.

2012-07-27 Thread Ronald S. Bultje
From: Loren Merritt lor...@u.washington.edu This allows us to unconditionally set the cglobal num_args parameter to a bigger value, thus making writing yasm code even easier than before. Signed-off-by: Ronald S. Bultje rsbul...@gmail.com --- libavutil/x86/x86inc.asm |3 +++ 1 file changed,

[libav-devel] [PATCH] h264: convert loop filter strength dsp function to yasm.

2012-07-27 Thread Ronald S. Bultje
From: Ronald S. Bultje rsbul...@gmail.com This completes the conversion of h264dsp to yasm; note that h264 also uses some dsputil functions, most notably qpel. Performance-wise, the yasm-version is ~10 cycles faster (182-172) on x86-64, and ~8 cycles faster (201-193) on x86-32. ---

[libav-devel] [PATCH 1/2] tls: Return AVERROR_EOF if the TLS_read/write functions return 0

2012-07-27 Thread Martin Storsjö
OpenSSL returns 0 when the peer has closed the connection. GnuTLS doesn't return that though, but returns GNUTLS_E_UNEXPECTED_PACKET_LENGTH if the connection simply is closed without a clean close notify packet. --- libavformat/tls.c |4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-)

[libav-devel] [PATCH 2/2] hls: Proceed to the next segment at any error code

2012-07-27 Thread Martin Storsjö
Previously, we returned any error code except AVERROR_EOF to the caller - only if AVERROR_EOF or 0 was returned, we proceeded to the next segment. With some setups of web servers, using Connection: close in https and GnuTLS, we don't get a clean error code at the end of segments. In those cases,

[libav-devel] [PATCH] af_amix: move per-input MixContext fields into a separate struct

2012-07-27 Thread Justin Ruggles
simplifies allocation --- libavfilter/af_amix.c | 76 - 1 files changed, 37 insertions(+), 39 deletions(-) diff --git a/libavfilter/af_amix.c b/libavfilter/af_amix.c index d79a626..ae16db0 100644 --- a/libavfilter/af_amix.c +++

Re: [libav-devel] [PATCH] proresdsp: port x86 assembly to cpuflags.

2012-07-27 Thread Diego Biurrun
On Thu, Jul 26, 2012 at 08:38:27PM -0700, Ronald S. Bultje wrote: --- a/libavcodec/x86/proresdsp.asm +++ b/libavcodec/x86/proresdsp.asm @@ -406,27 +405,25 @@ cglobal prores_idct_put_10_%1, 4, 4, %2 -INIT_XMM -%define SIGNEXTEND signextend_sse2 -idct_put_fn sse2, 16 -INIT_XMM -%define

Re: [libav-devel] [PATCH] h264: Split H.264 decoder off from H.264 common code.

2012-07-27 Thread Diego Elio Pettenò
Il 10/04/2012 16:00, Diego Biurrun ha scritto: So size increases for the default case. I would suspect that some tables are getting duplicated somewhere. I'll not go to the trouble of investigating issue now, so patch dropped. Please don't use `ls -l` alone to judge binary size, use tools

[libav-devel] MJPEG via RTP (RFC 2435) support

2012-07-27 Thread lswa156
Hi, I'm working with RTSP/RTP player for the MJPEG RTP stream from remote NVR. Sniffer show that incoming network flow is a valid RTP stream with RFC 2435 JPEG data. But have a problem: leon@leon-R519:~/Downloads/libraries/libav$ avplay -i rtsp://user:passw...@rtsp.server.com/some_video

Re: [libav-devel] [PATCH] proresdsp: port x86 assembly to cpuflags.

2012-07-27 Thread Ronald S. Bultje
Hi, On Fri, Jul 27, 2012 at 11:39 AM, Diego Biurrun di...@biurrun.de wrote: On Thu, Jul 26, 2012 at 08:38:27PM -0700, Ronald S. Bultje wrote: --- a/libavcodec/x86/proresdsp.asm +++ b/libavcodec/x86/proresdsp.asm @@ -406,27 +405,25 @@ cglobal prores_idct_put_10_%1, 4, 4, %2 -INIT_XMM

Re: [libav-devel] [PATCH] proresdsp: port x86 assembly to cpuflags.

2012-07-27 Thread Diego Biurrun
On Fri, Jul 27, 2012 at 11:42:04AM -0700, Ronald S. Bultje wrote: On Fri, Jul 27, 2012 at 11:39 AM, Diego Biurrun di...@biurrun.de wrote: On Thu, Jul 26, 2012 at 08:38:27PM -0700, Ronald S. Bultje wrote: --- a/libavcodec/x86/proresdsp.asm +++ b/libavcodec/x86/proresdsp.asm @@ -406,27

Re: [libav-devel] [PATCH] rtmp: add ff_amf_read_* functions

2012-07-27 Thread Martin Storsjö
On Fri, 27 Jul 2012, Jordi Ortiz wrote: --- libavformat/rtmppkt.c | 57 + libavformat/rtmppkt.h | 43 + 2 files changed, 100 insertions(+) diff --git a/libavformat/rtmppkt.c b/libavformat/rtmppkt.c index

Re: [libav-devel] [PATCH 1/4] lavf: remove commented out cruft in avformat_find_stream_info()

2012-07-27 Thread Luca Barbato
On 07/27/2012 04:30 PM, Anton Khirnov wrote: --- libavformat/utils.c |8 +--- 1 file changed, 1 insertion(+), 7 deletions(-) Ok. -- Luca Barbato Gentoo/linux http://dev.gentoo.org/~lu_zero ___ libav-devel mailing list

Re: [libav-devel] [PATCH 4/4] lavf: deprecate r_frame_rate.

2012-07-27 Thread Luca Barbato
On 07/27/2012 04:30 PM, Anton Khirnov wrote: According to its description, it is supposed to be the LCM of all the frame durations. The usability of such a thing is vanishingly small, especially since we cannot determine it with any amount of reliability. Therefore get rid of it after the next

Re: [libav-devel] [PATCH 3/4] lavf: round estimated average fps to a standard fps.

2012-07-27 Thread Luca Barbato
On 07/27/2012 04:30 PM, Anton Khirnov wrote: --- libavformat/utils.c | 18 ++ 1 file changed, 18 insertions(+) I'm scared about 30fps and 29.7fps mismatches. -- Luca Barbato Gentoo/linux http://dev.gentoo.org/~lu_zero ___

Re: [libav-devel] [PATCH 1/2] tls: Return AVERROR_EOF if the TLS_read/write functions return 0

2012-07-27 Thread Luca Barbato
On 07/27/2012 07:33 PM, Martin Storsjö wrote: OpenSSL returns 0 when the peer has closed the connection. GnuTLS doesn't return that though, but returns GNUTLS_E_UNEXPECTED_PACKET_LENGTH if the connection simply is closed without a clean close notify packet. --- libavformat/tls.c |4 ++--

Re: [libav-devel] [PATCH] rtmp: add ff_amf_read_* functions

2012-07-27 Thread Luca Barbato
On 07/27/2012 08:51 PM, Martin Storsjö wrote: +int ff_amf_read_bool(const uint8_t **src, int src_size, int *val) +{ +GetByteContext bsc; +bytestream2_init(bsc, *src, src_size); +if (bytestream2_get_byte(bsc) != AMF_DATA_TYPE_BOOL) +return AVERROR_INVALIDDATA; +*val =

Re: [libav-devel] [PATCH] Add missing libavutil/time.h includes.

2012-07-27 Thread Luca Barbato
On 07/27/2012 05:15 PM, Anton Khirnov wrote: --- libavdevice/fbdev.c |1 + libavdevice/jack_audio.c |1 + libavdevice/oss_audio.c |1 + libavdevice/x11grab.c|1 + libavformat/rtpdec.c |1 + libavformat/rtsp.c |1 + libavformat/rtspdec.c|

Re: [libav-devel] [PATCH] vp56: port x86 simd to cpuflags.

2012-07-27 Thread Diego Biurrun
On Thu, Jul 26, 2012 at 10:07:29PM -0700, Ronald S. Bultje wrote: From: Ronald S. Bultje rsbul...@gmail.com --- libavcodec/x86/vp56dsp.asm | 34 +++--- 1 file changed, 15 insertions(+), 19 deletions(-) LGTM Diego

Re: [libav-devel] [PATCH 1/6] vc1: avoid reading beyond the last line in vc1_draw_sprites()

2012-07-27 Thread Luca Barbato
On 07/27/2012 02:49 PM, Mashiat Sarker Shakkhar wrote: From: Michael Niedermayer michae...@gmx.at Fixes overread Found-by: Mateusz j00ru Jurczyk and Gynvael Coldwind Signed-off-by: Michael Niedermayer michae...@gmx.at --- libavcodec/vc1dec.c |4 ++-- 1 files changed, 2

Re: [libav-devel] [PATCH 3/6] vc1dec: Sanity-check macroblock quantizer

2012-07-27 Thread Luca Barbato
On 07/27/2012 02:49 PM, Mashiat Sarker Shakkhar wrote: From: Michael Niedermayer michae...@gmx.at Found-by: Mateusz j00ru Jurczyk and Gynvael Coldwind Signed-off-by: Michael Niedermayer michae...@gmx.at --- libavcodec/vc1dec.c |3 +++ 1 files changed, 3 insertions(+), 0 deletions(-)

Re: [libav-devel] [PATCH] vp3: don't compile mmx IDCT functions on x86-64.

2012-07-27 Thread Luca Barbato
On 07/27/2012 04:15 PM, Ronald S. Bultje wrote: Hi, On Thu, Jul 26, 2012 at 11:40 PM, Luca Barbato lu_z...@gentoo.org wrote: On 07/27/2012 07:16 AM, Ronald S. Bultje wrote: From: Ronald S. Bultje rsbul...@gmail.com 64-bit CPUs always have SSE2, and a SSE2 version exists, thus the MMX

Re: [libav-devel] [PATCH] h264: convert loop filter strength dsp function to yasm.

2012-07-27 Thread Loren Merritt
On Fri, 27 Jul 2012, Ronald S. Bultje wrote: From: Ronald S. Bultje rsbul...@gmail.com This completes the conversion of h264dsp to yasm; note that h264 also uses some dsputil functions, most notably qpel. Performance-wise, the yasm-version is ~10 cycles faster (182-172) on x86-64, and ~8

Re: [libav-devel] [PATCH 3/6] vc1dec: Sanity-check macroblock quantizer

2012-07-27 Thread Mashiat Sarker Shakkhar
On 7/28/2012 1:15 AM, Luca Barbato wrote: On 07/27/2012 02:49 PM, Mashiat Sarker Shakkhar wrote: From: Michael Niedermayer michae...@gmx.at Found-by: Mateusz j00ru Jurczyk and Gynvael Coldwind Signed-off-by: Michael Niedermayer michae...@gmx.at --- libavcodec/vc1dec.c |3 +++ 1 files

Re: [libav-devel] [PATCH 3/6] vc1dec: Sanity-check macroblock quantizer

2012-07-27 Thread Luca Barbato
On 07/27/2012 10:54 PM, Mashiat Sarker Shakkhar wrote: On 7/28/2012 1:15 AM, Luca Barbato wrote: On 07/27/2012 02:49 PM, Mashiat Sarker Shakkhar wrote: From: Michael Niedermayer michae...@gmx.at Found-by: Mateusz j00ru Jurczyk and Gynvael Coldwind Signed-off-by: Michael Niedermayer

Re: [libav-devel] [PATCH] swscale: bury one more piece of inline asm under HAVE_INLINE_ASM.

2012-07-27 Thread Luca Barbato
On 07/27/2012 06:30 PM, Ronald S. Bultje wrote: Hi, On Thu, Jul 26, 2012 at 9:46 AM, Ronald S. Bultje rsbul...@gmail.com wrote: On Thu, Jul 26, 2012 at 9:05 AM, Måns Rullgård m...@mansr.com wrote: Ronald S. Bultje rsbul...@gmail.com writes: On Thu, Jul 26, 2012 at 7:30 AM, Martin Storsjö

Re: [libav-devel] [PATCH 2/2] hls: Proceed to the next segment at any error code

2012-07-27 Thread Martin Storsjö
On Fri, 27 Jul 2012, Martin Storsjö wrote: Previously, we returned any error code except AVERROR_EOF to the caller - only if AVERROR_EOF or 0 was returned, we proceeded to the next segment. With some setups of web servers, using Connection: close in https and GnuTLS, we don't get a clean error

Re: [libav-devel] [PATCH] swscale: bury one more piece of inline asm under HAVE_INLINE_ASM.

2012-07-27 Thread Måns Rullgård
Ronald S. Bultje rsbul...@gmail.com writes: Hi, On Thu, Jul 26, 2012 at 9:05 AM, Måns Rullgård m...@mansr.com wrote: Ronald S. Bultje rsbul...@gmail.com writes: On Thu, Jul 26, 2012 at 7:30 AM, Martin Storsjö mar...@martin.st wrote: On Thu, 26 Jul 2012, Ronald S. Bultje wrote: On Thu, Jul

Re: [libav-devel] [PATCH] h264_idct_10bit: port x86 assembly to cpuflags.

2012-07-27 Thread Diego Biurrun
On Thu, Jul 26, 2012 at 08:54:30PM -0700, Ronald S. Bultje wrote: --- a/libavcodec/x86/h264_idct_10bit.asm +++ b/libavcodec/x86/h264_idct_10bit.asm @@ -72,25 +72,25 @@ SECTION .text ;;; NO FATE SAMPLES TRIGGER THIS -%macro ADD4x4IDCT 1 -add4x4_idct_%1: +%macro ADD4x4IDCT 0

Re: [libav-devel] [PATCH] rv34: port x86 SIMD to cpuflags.

2012-07-27 Thread Diego Biurrun
On Thu, Jul 26, 2012 at 10:19:19PM -0700, Ronald S. Bultje wrote: From: Ronald S. Bultje rsbul...@gmail.com --- libavcodec/x86/rv34dsp.asm | 11 ++- 1 file changed, 6 insertions(+), 5 deletions(-) LGTM Diego ___ libav-devel mailing

Re: [libav-devel] [PATCH 2/2] hls: Proceed to the next segment at any error code

2012-07-27 Thread Luca Barbato
On 07/27/2012 11:36 PM, Martin Storsjö wrote: On Fri, 27 Jul 2012, Martin Storsjö wrote: Previously, we returned any error code except AVERROR_EOF to the caller - only if AVERROR_EOF or 0 was returned, we proceeded to the next segment. With some setups of web servers, using Connection:

Re: [libav-devel] [PATCH] vp3: port x86 SIMD to cpuflags.

2012-07-27 Thread Diego Biurrun
On Thu, Jul 26, 2012 at 10:14:39PM -0700, Ronald S. Bultje wrote: --- a/libavcodec/x86/vp3dsp.asm +++ b/libavcodec/x86/vp3dsp.asm @@ -521,9 +521,17 @@ cglobal vp3_h_loop_filter_mmx2, 3, 4 -%macro vp3_idct_funcs 1 -cglobal vp3_idct_put_%1, 3, 4, 9 -VP3_IDCT_%1 r2 +%macro VP3_IDCT

[libav-devel] [PATCH] eamad: remove use of MpegEncContext

2012-07-27 Thread Mans Rullgard
There is no sense in pulling in this monster struct just for a handful of fields. The code does not call any functions expecting an MpegEncContext. Signed-off-by: Mans Rullgard m...@mansr.com --- libavcodec/eamad.c | 92 +++--- 1 file changed, 46

[libav-devel] [PATCH] h264_loopfilter: port x86 simd to cpuflags.

2012-07-27 Thread Ronald S. Bultje
From: Ronald S. Bultje rsbul...@gmail.com --- libavcodec/x86/h264_deblock.asm | 126 +++-- libavcodec/x86/h264_deblock_10bit.asm | 77 ++-- libavcodec/x86/h264dsp_mmx.c | 60 3 files changed, 141 insertions(+), 122

Re: [libav-devel] [PATCH] rtmp: add ff_amf_read_* functions

2012-07-27 Thread Jordi Ortiz
2012/7/27 Luca Barbato lu_z...@gentoo.org On 07/27/2012 08:51 PM, Martin Storsjö wrote: +int ff_amf_read_bool(const uint8_t **src, int src_size, int *val) +{ +GetByteContext bsc; +bytestream2_init(bsc, *src, src_size); +if (bytestream2_get_byte(bsc) != AMF_DATA_TYPE_BOOL)

[libav-devel] [PATCH] vp3: port x86 SIMD to cpuflags.

2012-07-27 Thread Ronald S. Bultje
From: Ronald S. Bultje rsbul...@gmail.com --- libavcodec/x86/vp3dsp.asm | 94 ++--- 1 file changed, 47 insertions(+), 47 deletions(-) diff --git a/libavcodec/x86/vp3dsp.asm b/libavcodec/x86/vp3dsp.asm index af2f60c..5877520 100644 ---

Re: [libav-devel] [PATCH] h264_idct_10bit: port x86 assembly to cpuflags.

2012-07-27 Thread Ronald S. Bultje
Hi, On Fri, Jul 27, 2012 at 2:49 PM, Diego Biurrun di...@biurrun.de wrote: On Thu, Jul 26, 2012 at 08:54:30PM -0700, Ronald S. Bultje wrote: --- a/libavcodec/x86/h264_idct_10bit.asm +++ b/libavcodec/x86/h264_idct_10bit.asm @@ -72,25 +72,25 @@ SECTION .text ;;; NO FATE SAMPLES TRIGGER

Re: [libav-devel] [PATCH] vp3: port x86 SIMD to cpuflags.

2012-07-27 Thread Diego Biurrun
On Fri, Jul 27, 2012 at 03:17:45PM -0700, Ronald S. Bultje wrote: From: Ronald S. Bultje rsbul...@gmail.com --- libavcodec/x86/vp3dsp.asm | 94 ++--- 1 file changed, 47 insertions(+), 47 deletions(-) LGTM Diego

Re: [libav-devel] [PATCH] h264_chromamc_10bit: port x86 simd to cpuflags.

2012-07-27 Thread Diego Biurrun
On Thu, Jul 26, 2012 at 08:43:50PM -0700, Ronald S. Bultje wrote: From: Ronald S. Bultje rsbul...@gmail.com --- libavcodec/x86/dsputil_mmx.c | 16 ++--- libavcodec/x86/h264_chromamc_10bit.asm | 40 2 files changed, 28 insertions(+),

Re: [libav-devel] [PATCH] h264_loopfilter: port x86 simd to cpuflags.

2012-07-27 Thread Diego Biurrun
On Fri, Jul 27, 2012 at 03:08:26PM -0700, Ronald S. Bultje wrote: --- a/libavcodec/x86/h264_deblock.asm +++ b/libavcodec/x86/h264_deblock.asm @@ -282,8 +282,8 @@ cextern pb_A1 ;- ; void deblock_v_luma( uint8_t

Re: [libav-devel] [PATCH] h264_loopfilter: port x86 simd to cpuflags.

2012-07-27 Thread Ronald S. Bultje
Hi, On Fri, Jul 27, 2012 at 4:45 PM, Diego Biurrun di...@biurrun.de wrote: On Fri, Jul 27, 2012 at 03:08:26PM -0700, Ronald S. Bultje wrote: --- a/libavcodec/x86/h264_deblock.asm +++ b/libavcodec/x86/h264_deblock.asm @@ -282,8 +282,8 @@ cextern pb_A1

Re: [libav-devel] [PATCH] h264_loopfilter: port x86 simd to cpuflags.

2012-07-27 Thread Diego Biurrun
On Fri, Jul 27, 2012 at 04:49:18PM -0700, Ronald S. Bultje wrote: On Fri, Jul 27, 2012 at 4:45 PM, Diego Biurrun di...@biurrun.de wrote: On Fri, Jul 27, 2012 at 03:08:26PM -0700, Ronald S. Bultje wrote: --- a/libavcodec/x86/h264_deblock.asm +++ b/libavcodec/x86/h264_deblock.asm @@

Re: [libav-devel] [PATCH 5/6] configure: Place non-user-selectable components in CONFIG_EXTRA

2012-07-27 Thread Diego Biurrun
On Fri, Jul 27, 2012 at 03:33:10PM +0100, Måns Rullgård wrote: Diego Biurrun di...@biurrun.de writes: On Fri, Jul 27, 2012 at 11:29:02AM +0100, Måns Rullgård wrote: Diego Biurrun di...@biurrun.de writes: --- a/configure +++ b/configure @@ -979,21 +976,16 @@ CONFIG_LIST=

Re: [libav-devel] [PATCH 5/6] configure: Place non-user-selectable components in CONFIG_EXTRA

2012-07-27 Thread Måns Rullgård
Diego Biurrun di...@biurrun.de writes: On Fri, Jul 27, 2012 at 03:33:10PM +0100, Måns Rullgård wrote: Diego Biurrun di...@biurrun.de writes: On Fri, Jul 27, 2012 at 11:29:02AM +0100, Måns Rullgård wrote: Diego Biurrun di...@biurrun.de writes: --- a/configure +++ b/configure @@

Re: [libav-devel] [PATCH] h264_loopfilter: port x86 simd to cpuflags.

2012-07-27 Thread Ronald S. Bultje
Hi, On Fri, Jul 27, 2012 at 5:04 PM, Diego Biurrun di...@biurrun.de wrote: On Fri, Jul 27, 2012 at 04:49:18PM -0700, Ronald S. Bultje wrote: On Fri, Jul 27, 2012 at 4:45 PM, Diego Biurrun di...@biurrun.de wrote: On Fri, Jul 27, 2012 at 03:08:26PM -0700, Ronald S. Bultje wrote: ---

  1   2   >