[FFmpeg-devel] [PATCH 1/3] lavd/xv: speed up yuv420p write packet

[Stefano Sabatini]

On date Friday 2013-11-15 00:32:21 +0100, Lukasz M encoded:
> On 14 November 2013 12:15, Stefano Sabatini <stefasab at gmail.com> wrote:
[...]
> I used  av_image_copy() in new patch. I did again performance tests and
> real increase is only for small dimensions.
> For a movie 640x352 is is around 0.5%. For HD movies it will be even
> smaller probably.
> For a movie 320x480 it was around 5%, but small frames are copied fast
> anyway just because they are small so optimization for them is less
> important.
> Yesterday I tested with gcc nationalizations turned off and got higher
> values, but it's not reliable.
> On other hand with av_image_copy it is slower by 0.5% for 640x352 comparing
> to original code.
> New patch is then 1% slower than previous for medium size frames.
> I think it is not worth anyway, cleaner code I better.

> From c734b91e7d36153eac39b917f129014e5ac5973a Mon Sep 17 00:00:00 2001
> From: Lukasz Marek <lukasz.m.luki at gmail.com>
> Date: Thu, 14 Nov 2013 21:17:00 +0100
> Subject: [PATCH 1/3] lavd/xv: simplify write_packet
> 
> Signed-off-by: Lukasz Marek <lukasz.m.luki at gmail.com>
> ---
>  libavdevice/xv.c |   22 ++++++++--------------
>  1 file changed, 8 insertions(+), 14 deletions(-)

LGTM, I'll push it soon.
-- 
FFmpeg = Friendly and Frenzy Multimedia Pitiless Earthshaking Game