On Thu, 2010-09-16 at 11:14 +0100, Stephan Goll wrote:
> The /proc/cpuinfo shows that the geode lx has cmov, so I changed the sources 
> to compile a kernel with cmov support.
> 
> diff arch/x86/Kconfig.cpu arch/x86/Kconfig.cpu.bak 
> 409c409
> <     depends on (MK7 || MPENTIUM4 || MPENTIUMM || MPENTIUMIII || MPENTIUMII 
> || M686 || MVIAC3_2 || MVIAC7 || MGEODE_LX || X86_64)
> ---
> >     depends on (MK7 || MPENTIUM4 || MPENTIUMM || MPENTIUMIII || MPENTIUMII 
> > || M686 || MVIAC3_2 || MVIAC7 || X86_64)
> 
> Because I'm running an alix board with geode cpu I was able to verify that 
> this does noch break the kernel. It seems that all geode do have cmov so
> I think this is a save bet. This wish also applies to the 2.6.32 kernel in 
> testing aka squeeze.
> There is also a related discussion on the LKML on this topic.

We know Geode LX supports CMOV; that's why it's included in the list of
suitable processors in thedescription of the '686' flavour package.
However, the configuration option MGEODE_LX is described as 'Geode
GX/LX' and the Geode GX does *not* support CMOV, so your proposed change
is incorrect.

Ben.

-- 
Ben Hutchings
Once a job is fouled up, anything done to improve it makes it worse.

Attachment: signature.asc
Description: This is a digitally signed message part

Reply via email to