[FFmpeg-devel] [PATCH] Re: Color corruption and seeking errors with H264 disc sources

Andreas Öman andreas
Tue Jul 3 13:13:49 CEST 2007


Hi

Michael Niedermayer wrote:
> Hi
> 
> On Wed, Jun 27, 2007 at 11:20:28AM +0200, Andreas ?man wrote:
>> Hi (again)
>>
>> Andreas ?man wrote:
>>> Hi,
>>>
>>> I no one else does it, i might take a look at this tomorrow if i can
>>> find some time. But please, since i'm no h264 expert, if someone with
>>> more expertise takes a look at it, drop me a mail or something so
>>> i dont wast my time.
>>>
>> The following patch fixes the issue (at least it looks much better,
>> but i have no reference decoder to make a binary verification)
>> Can you please try it?
>>
>> There is a slight slowdown though, (see qp-timings.txt)
>> I'll look if some parts can be rewritten or optimized in some way.
>>
>> Comments welcome..
> 
> well as its a bugfix its needed evem if it slows the code down, but of
> course applying a bugfix which slows the code down is a last resort
> 
> ...  [snip]

I basically did two things.

a) compute a new qptable for each pps, see build_qp_table()

b) keep qmul argument around calls to residual decoding.
This one seems to make most of the difference.

These two changes will be separate patches.

Here are the results from timing with

time ./ffmpeg -i $FILE -f null /deV/null 2>/dev/null

Aladin.mpg

svn:     (3.248+3.220+3.224+3.248+3.248)/5
           3.238

patched: (3.236+3.236+3.208+3.248+3.264)/5
           3.238

SVT.ts

svn:     (5.384+5.356+5.304+5.340+5.372)/5
           5.351

patched: (5.336+5.316+5.332+5.320+5.440)/5
           5.349


ffh264_issue3.h264

svn:     (25.962+26.170+25.974+26.022+25.998)/5
           26.025

patched: (26.050+26.106+26.050+26.106+26.074)/5
           26.077




More information about the ffmpeg-devel mailing list