On 9/12/07, Jordan Crouse <[EMAIL PROTECTED]> wrote: > On 12/09/07 17:21 +0200, Marco Pesenti Gritti wrote: > > On 9/12/07, Dan Williams <[EMAIL PROTECTED]> wrote: > > > On Tue, 2007-09-11 at 14:19 -0400, Bernardo Innocenti wrote: > > > > On 09/11/2007 01:32 PM, Bernardo Innocenti wrote: > > > > > > > > > The 16bpp codepath has to be broken somewhere if > > > > > it takes twice the time to copy half the bits :-) > > > > > > > > It strikes me that we don't see any time spent in > > > > pixman_fill_mmx(), even though it's not inlinable. > > > > > > > > For some reason, pixman thinks it cannot accelerate > > > > 16bpp fills with MMX, at least on the Geode. > > > > > > > > Might be worth investigating... > > > > > > We did have to patch the MMX check in pixman long ago, maybe that got > > > broken somehow? There were actually two, an update to the cpu flags and > > > also a strcmp() on the processor ID that had to be fixed to get pixman > > > to detect MMX capability on the geode. > > > > > > > Yeah this the current check: > > > > (strcmp(vendor, "AuthenticAMD") == 0 || > > strcmp(vendor, "Geode by NSC") == 0)) > > > I think Jordan mentioned that the LX is not using "Geode by NSC" anymore. > > We should be using AuthenticAMD now. Check /proc/cpuinfo to make sure. >
Yeah, that's correct. I guess it would still be worth to make sure mmx is actually enabled in cairo, other parts of that check might be failing... Marco _______________________________________________ Devel mailing list [email protected] http://lists.laptop.org/listinfo/devel
