[MPlayer-dev-eng] [PATCH] Change >8 memalign() to av_malloc() under libmpcodecs, avoiding crash under Win32

Zuxy Meng zuxy.meng at gmail.com
Tue Apr 13 20:13:23 CEST 2010


2010/4/14 Uoti Urpala <uoti.urpala at pp1.inet.fi>:
> On Wed, 2010-04-14 at 01:47 +0800, Zuxy Meng wrote:
>> As $title, pairing free() changed to av_free() too.
>
> av_malloc only gives 16-byte alignment, so you're reducing the alignment
> of the 64-byte aligned ones (presumably meant to be cache line
> aligned?).

Indeed. But are there any real concerns here? If they are all aligned
then accesses will never cross the cache line.

-- 
Zuxy
Beauty is truth,
While truth is beauty.
PGP KeyID: E8555ED6



More information about the MPlayer-dev-eng mailing list