Messages in this thread Patch in this message | | | Date | Fri, 9 Oct 2009 08:30:28 -0700 | From | Randy Dunlap <> | Subject | Re: [was: mmotm 2009-10-09-01-07 uploaded] |
| |
On Fri, 09 Oct 2009 12:00:55 +0200 Jiri Slaby wrote:
> On 10/09/2009 10:07 AM, akpm@linux-foundation.org wrote: > > The mm-of-the-moment snapshot 2009-10-09-01-07 has been uploaded to > > Hi, build fails with: > ERROR: "cpufreq_get" [arch/x86/kvm/kvm.ko] undefined! > because of > CONFIG_KVM=m > ... > # CONFIG_CPU_FREQ is not set > > Should cpufreq_get be defined as inline return 0 the same as > cpufreq_quick_get on !CONFIG_CPU_FREQ?
I submitted a patch for that several days ago, against linux-next-20091006. (below again)
--- From: Randy Dunlap <randy.dunlap@oracle.com>
When CONFIG_CPU_FREQ is disabled, cpufreq_get() needs a stub. Used by kvm (although it looks like a bit of the kvm code could be omitted when CONFIG_CPU_FREQ is disabled).
arch/x86/built-in.o: In function `kvm_arch_init': (.text+0x10de7): undefined reference to `cpufreq_get' Signed-off-by: Randy Dunlap <randy.dunlap@oracle.com> Tested-by: Eric Paris <eparis@redhat.com> --- include/linux/cpufreq.h | 7 +++++++ 1 file changed, 7 insertions(+)
--- linux-next-20091006.orig/include/linux/cpufreq.h +++ linux-next-20091006/include/linux/cpufreq.h @@ -291,8 +291,15 @@ struct global_attr { int cpufreq_get_policy(struct cpufreq_policy *policy, unsigned int cpu); int cpufreq_update_policy(unsigned int cpu); +#ifdef CONFIG_CPU_FREQ /* query the current CPU frequency (in kHz). If zero, cpufreq couldn't detect it */ unsigned int cpufreq_get(unsigned int cpu); +#else +static inline unsigned int cpufreq_get(unsigned int cpu) +{ + return 0; +} +#endif /* query the last known CPU freq (in kHz). If zero, cpufreq couldn't detect it */ #ifdef CONFIG_CPU_FREQ
| |