[Pixman] [PATCH] Add a lowlevel blitter benchmark

Soeren Sandmann sandmann at daimi.au.dk
Wed Sep 15 07:06:27 PDT 2010


Dmitri Vorobiev <dmitri.vorobiev at movial.com> writes:

> diff --git a/test/Makefile.am b/test/Makefile.am
> index 3d98e17..8f3fdaf 100644
> --- a/test/Makefile.am
> +++ b/test/Makefile.am
> @@ -18,7 +18,8 @@ TESTPROGRAMS =			\
>  	scaling-crash-test	\
>  	blitters-test		\
>  	scaling-test		\
> -	composite
> +	composite		\
> +	lowlevel-blt-bench

As Siarhei mentioned, this shouldn't go in TESTPROGRAMS. Eventually,
I'd like to have a separate performance test suite, perhaps even in
its own directory next to test, but for now, let's just add it in the
test directory.

> +++ b/test/lowlevel-blt-bench.c
> @@ -0,0 +1,713 @@
> +/*
> + * Copyright © 2010 Movial Creative Technologies Oy
> + *
> + * Permission to use, copy, modify, distribute, and sell this software and its
> + * documentation for any purpose is hereby granted without fee, provided that
> + * the above copyright notice appear in all copies and that both that
> + * copyright notice and this permission notice appear in supporting
> + * documentation, and that the name of ARM Ltd not be used in
> + * advertising or publicity pertaining to distribution of the software without
> + * specific, written prior permission.  ARM Ltd makes no
> + * representations about the suitability of this software for any purpose.  It
> + * is provided "as is" without express or implied warranty.
> + *
> + * THE COPYRIGHT HOLDERS DISCLAIM ALL WARRANTIES WITH REGARD TO THIS
> + * SOFTWARE, INCLUDING ALL IMPLIED WARRANTIES OF MERCHANTABILITY AND
> + * FITNESS, IN NO EVENT SHALL THE COPYRIGHT HOLDERS BE LIABLE FOR ANY
> + * SPECIAL, INDIRECT OR CONSEQUENTIAL DAMAGES OR ANY DAMAGES
> + * WHATSOEVER RESULTING FROM LOSS OF USE, DATA OR PROFITS, WHETHER IN
> + * AN ACTION OF CONTRACT, NEGLIGENCE OR OTHER TORTIOUS ACTION, ARISING
> + * OUT OF OR IN CONNECTION WITH THE USE OR PERFORMANCE OF THIS
> + * SOFTWARE.
> + *
> + * Author:  Jonathan Morton (jonathan.morton at movial.com)
> + *
> + * Based on the code by Siarhei Siamashka <siarhei.siamashka at gmail.com>
> + *
> + */

This says copyright Movial, but mentions ARM Ltd in the text. 

Unless you have a reason to not use it, the preferred license is the
one in COPYING. Its main advantage is that it avoids this kind of
cut-and-paste issue.

Also, is this code mainly written by Siarhei? If so, presumably either
Nokia or Siarhei are also copyright holders.

> +tests_tbl[] =
> +{
>      ....
> +    { "src_n_2222",            PIXMAN_a8r8g8b8,    1, PIXMAN_OP_SRC,     PIXMAN_null,     0, PIXMAN_a1r5g5b5 },

This one looks like it should have a2r2g2b2 in the destination

> +    { "src_n_4444",            PIXMAN_a8r8g8b8,    1, PIXMAN_OP_SRC,     PIXMAN_null,     0, PIXMAN_a1r5g5b5 },

This one looks wrong too.

> +/* Return time since the Epoch in seconds */
> +double
> +gettime (void);

Like the malloc() changes, this could usefully be done in its own patch.



Thanks,
Soren


More information about the Pixman mailing list