More 16 vs 24 bpp profiling

[Marco Pesenti Gritti]

On 9/12/07, Jordan Crouse <jordan.crouse at amd.com> wrote:
> On 12/09/07 17:21 +0200, Marco Pesenti Gritti wrote:
> > On 9/12/07, Dan Williams <dcbw at redhat.com> 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