2008-11-21 21:56:33

by Hannes Eder

Subject: [PATCH] x86: hypervisor - fix sparse warnings

Fix the following sparse warnings:

arch/x86/kernel/cpu/hypervisor.c:37:15: warning: symbol
'get_hypervisor_tsc_freq' was not declared. Should it be static?
arch/x86/kernel/cpu/hypervisor.c:53:16: warning: symbol
'init_hypervisor' was not declared. Should it be static?

Signed-off-by: Hannes Eder <[email protected]>
---
arch/x86/kernel/cpu/hypervisor.c | 1 +
1 files changed, 1 insertions(+), 0 deletions(-)

diff --git a/arch/x86/kernel/cpu/hypervisor.c b/arch/x86/kernel/cpu/hypervisor.c
index 35ae2b7..fb5b86a 100644
--- a/arch/x86/kernel/cpu/hypervisor.c
+++ b/arch/x86/kernel/cpu/hypervisor.c
@@ -23,6 +23,7 @@

#include <asm/processor.h>
#include <asm/vmware.h>
+#include <asm/hypervisor.h>

static inline void __cpuinit
detect_hypervisor_vendor(struct cpuinfo_x86 *c)
--
1.5.6.3

2008-11-23 10:12:32

by Ingo Molnar

Subject: Re: [PATCH] x86: hypervisor - fix sparse warnings

* Hannes Eder <[email protected]> wrote:

> Fix the following sparse warnings:
>
> arch/x86/kernel/cpu/hypervisor.c:37:15: warning: symbol
> 'get_hypervisor_tsc_freq' was not declared. Should it be static?
> arch/x86/kernel/cpu/hypervisor.c:53:16: warning: symbol
> 'init_hypervisor' was not declared. Should it be static?
>
> Signed-off-by: Hannes Eder <[email protected]>
> ---
> arch/x86/kernel/cpu/hypervisor.c | 1 +
> 1 files changed, 1 insertions(+), 0 deletions(-)

applied to tip/x86/detect-hyper, thanks Hannes!

Ingo