[FFmpeg-devel] [PATCH] VP56: 32-byte align picture width

Måns Rullgård mans
Sun Jul 13 16:11:56 CEST 2008


Michael Niedermayer <michaelni at gmx.at> writes:

> On Sun, Jul 13, 2008 at 02:13:39PM +0100, Mans Rullgard wrote:
>> The VP5/6 decoder calls put_pixels16() on the chroma planes, so these
>> need 16-byte alignment.  Setting luma alignment to 32 achieves this,
>> although alignment of 16 is sufficient for the luma plane.
>
> Hmm, isnt STRIDE_ALIGN 16 ?

Apparently not on ARM.  Changing this makes the problem go away.

> Is some code ignoring STRIDE_ALIGN ?

No, I wasn't aware of it.  Nevertheless, dsputil.h promises that the
destination for put_pixels16() will be 16-byte aligned, and this is
simply not true.

-- 
M?ns Rullg?rd
mans at mansr.com




More information about the ffmpeg-devel mailing list