[cairo] Pushing vmx patches in pixman
Luca Barbato
lu_zero at gentoo.org
Wed Jun 4 12:46:42 PDT 2008
Soeren Sandmann wrote:
> The remaining comments I have:
>
> +pixman_bool_t pixman_have_vmx (void) {
> + if (!initialized) {
> + signal(SIGILL, vmx_test);
> + asm volatile ( "vor 0, 0, 0" );
> + signal(SIGILL, SIG_DFL);
> + initialized = TRUE;
> + }
> + return have_vmx;
> +}
>
> This will reset any signal handler the application itself may have
> set. It would be better to us sigaction() and restore the old handler
> instead of using SIG_DFL. Doing that still leaves a race where another
> thread could change the signal handler while pixman was initializing,
> but I think that's unlikely enough to be ok.
I'll do that soon
> Finally, I'd still like to see the fbByteMul() macros being generated
> into their own header file, and then #included in both
> pixman-combine{32,64}.c and pixman-vmx.c.
There are some definitions duplicated in the sources, should I replace
them as well?
lu
--
Luca Barbato
Gentoo Council Member
Gentoo/linux Gentoo/PPC
http://dev.gentoo.org/~lu_zero
More information about the cairo
mailing list