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

Diego Biurrun diego
Thu Mar 9 09:06:54 CET 2006


On Wed, Mar 08, 2006 at 10:17:57PM -0800, David S. Miller wrote:
> 
> > On Mon, Dec 26, 2005 at 03:39:25PM -0800, David S. Miller wrote:
> > > 
> > > > 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.

Compiles and passes regression tests on my machine, applied.

Diego





More information about the ffmpeg-devel mailing list