[Ffmpeg-devel] [PATCH]: Too much alignment assumed by H264 decoder

David S. Miller davem
Thu Mar 9 07:17:57 CET 2006


From: Michael Niedermayer <michaelni at gmx.at>
Date: Mon, 30 Jan 2006 11:41:37 +0100

> On Mon, Dec 26, 2005 at 03:39:25PM -0800, David S. Miller wrote:
> > From: Loren Merritt <lorenm at u.washington.edu>
> > Date: Mon, 26 Dec 2005 00:37:42 -0800 (PST)
> > 
> > > int16_t mv_cache[2][5*8][2] __align8;
> > > 
> > > h->mv_cache[i][j] is size 4, so j only needs to be a multiple of 2, which 
> > > it is.
> > 
> > You're right, thanks everyone for steering me in the right direction.
> > 
> > The following works for me, but I can't vouch for it's correctness.
> > I can't say why the "+ 1" was there in the first place.
> 
> if all reference bitstreams are decoded with same md5 after the change
> then this is ok and can be applied

I finally got around to testing this and things look good.
Sorry for taking so long.





More information about the ffmpeg-devel mailing list