[FFmpeg-devel] [PATCH] RV40 Decoder - 1/3 - Loop Filter

[Michael Niedermayer]

On Fri, Nov 28, 2008 at 08:17:17PM +0200, Kostya wrote:
> On Fri, Nov 28, 2008 at 09:19:19AM +0100, Michael Niedermayer wrote:
> > On Fri, Nov 28, 2008 at 09:03:04AM +0200, Kostya wrote:
> > > On Thu, Nov 27, 2008 at 10:04:40PM +0100, Michael Niedermayer wrote:
> > > > On Thu, Nov 27, 2008 at 09:23:21AM +0200, Kostya wrote:
> > > > > $subj
> > > > > 
> > > > > Fixed several bugs in chroma loop filtering introduced during
> > > > > refactoring, now it's bitexact again.
> > > > 
> > > > [...]
> > > > 
> > > > > +    for(s->mb_y = 0; s->mb_y < s->mb_height; s->mb_y++){
> > > > > +        mb_pos = s->mb_y * s->mb_stride;
> > > > > +        for(s->mb_x = 0; s->mb_x < s->mb_width; s->mb_x++, mb_pos++){
> > > > > +            int mbtype = s->current_picture_ptr->mb_type[mb_pos];
> > > > > +            if(IS_INTRA(mbtype) || IS_SEPARATE_DC(mbtype))
> > > > > +                r->cbp_luma  [mb_pos] = 0xFFFF;
> > > > > +            if(IS_INTRA(mbtype))
> > > > > +                r->cbp_chroma[mb_pos] = 0xFF;
> > > > > +        }
> > > > > +    }
> > > > > +    for(s->mb_y = 0; s->mb_y < s->mb_height; s->mb_y++){
> > > > > +        mb_pos = s->mb_y * s->mb_stride;
> > > > > +        for(s->mb_x = 0; s->mb_x < s->mb_width; s->mb_x++, mb_pos++){
> > > > 
> > > > loop filtering should be done after each block or row of blocks has been
> > > > decoded to maximize the chance that things still are in the cache
> > > 
> > > I'd rather leave it as is since loop filter affects bottom macroblock edge
> > > too so filtering the row above just decoded one is a bit strange.
> > 
> > please post benchmark scores for both, if theres no speed gain i dont mind
> > leaving it
>  
> Hmm, there is ~1% gain, so row-based deblocking it is.

good
patch ok unless you want to clean it up further ...

[...]
-- 
Michael     GnuPG fingerprint: 9FF2128B147EF6730BADF133611EC787040B0FAB

There will always be a question for which you do not know the correct awnser.
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 189 bytes
Desc: Digital signature
URL: <http://lists.mplayerhq.hu/pipermail/ffmpeg-devel/attachments/20081128/2517ab19/attachment.pgp>