Bug#597069: linux-source-2.6: missing cmov support in arch/x86/Kconfig.cpu for AMD geode LX
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.
|