[Ffmpeg-devel] [PATCH] h264 *_pixels16_l2_altivec

Luca Barbato lu_zero
Wed Mar 8 10:12:38 CET 2006


Loren Merritt wrote:
> On Tue, 7 Mar 2006, Luca Barbato wrote:
>> Romain Dolbeau wrote:
>>>
>>> I think the fact that src_stride2 is hardwired to 16
>>> should be mentioned above the functions, should it
>>> be called from someplace that doesn't know about
>>> the restriction.
>>
>> The previous code assumed the same, so I just removed the define and
>> hardwired it there.
>>
>>> Doesn't H264 requires image_stride % 16 == 0 ? Because
>>> if it can be assumed src_stride1 % 16 == 0, then the
>>> vec_lvsl can be moved out of the loop.
>>
>> Not sure I check this evening
>>
>>> (I don't think that src1/src2/dst are always properly
>>> aligned ? that could solve the performance problem)
>>
>> sadly they are all unaligned.
> 
> dst and one of the src can be made aligned.
> 

about src probably is feasible, but I wonder how you can align dst since
it could be a buffer provided by uncheckable means (like X), probably
I'm missing something

lu





More information about the ffmpeg-devel mailing list