[Pixman] [cairo] [PATCH/RFC][pixman] More ARM NEON performance updates

Benjamin Otte otte at redhat.com
Fri Feb 19 10:48:13 PST 2010


On Fri, 2010-02-19 at 18:37 +0100, Soeren Sandmann wrote:
> Add some new functions to the implementation struct. Something like
> 
>         fetch_scanline_t (* get_scanline_fetcher_32) (...)
>         fetch_scanline_t (* get_scanline_fetcher_64 (...)
>         store_scanline_t (* get_scanline_storer_32) (...)
>         store_scanline_t (* get_scanline_storer_64) (...)
> 
> By default these just delegate, and the general implementation will
> return the fetchers that are now being set up in the various
> property_changed() functions. The property_changed() functions are
> extended to also take an implementation argument, so that they can
> call the new functions to get the fetchers and storers.
> 
I like this, a) because store functions are showing up on my profiles
(not sure if that's just the kernel page faulting though) and b) because
it could be used to extend it to do color space conversion, too.

Though I'd need to think more about how the API should look.

Benjamin



More information about the Pixman mailing list