Re: [PATCH] x86/microcode/intel: make pointer intel_ucode_patch static
On Thu, Jun 22, 2017 at 09:49:40AM +0100, Colin King wrote: > From: Colin Ian King> > The pointer intel_ucode_patch does not need to be in global scope, > so make it static. > > Cleans up sparse warning: > "symbol 'intel_ucode_patch' was not declared. Should it be static?" > > Signed-off-by: Colin Ian King > --- > arch/x86/kernel/cpu/microcode/intel.c | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > > diff --git a/arch/x86/kernel/cpu/microcode/intel.c > b/arch/x86/kernel/cpu/microcode/intel.c > index d525a0bd7d28..59edbe9d4ccb 100644 > --- a/arch/x86/kernel/cpu/microcode/intel.c > +++ b/arch/x86/kernel/cpu/microcode/intel.c > @@ -42,7 +42,7 @@ > static const char ucode_path[] = "kernel/x86/microcode/GenuineIntel.bin"; > > /* Current microcode patch used in early patching on the APs. */ > -struct microcode_intel *intel_ucode_patch; > +static struct microcode_intel *intel_ucode_patch; > > static inline bool cpu_signatures_match(unsigned int s1, unsigned int p1, > unsigned int s2, unsigned int p2) > -- Acked-by: Borislav Petkov -- Regards/Gruss, Boris. Good mailing practices for 400: avoid top-posting and trim the reply.
Re: [PATCH] x86/microcode/intel: make pointer intel_ucode_patch static
On Thu, Jun 22, 2017 at 09:49:40AM +0100, Colin King wrote: > From: Colin Ian King > > The pointer intel_ucode_patch does not need to be in global scope, > so make it static. > > Cleans up sparse warning: > "symbol 'intel_ucode_patch' was not declared. Should it be static?" > > Signed-off-by: Colin Ian King > --- > arch/x86/kernel/cpu/microcode/intel.c | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > > diff --git a/arch/x86/kernel/cpu/microcode/intel.c > b/arch/x86/kernel/cpu/microcode/intel.c > index d525a0bd7d28..59edbe9d4ccb 100644 > --- a/arch/x86/kernel/cpu/microcode/intel.c > +++ b/arch/x86/kernel/cpu/microcode/intel.c > @@ -42,7 +42,7 @@ > static const char ucode_path[] = "kernel/x86/microcode/GenuineIntel.bin"; > > /* Current microcode patch used in early patching on the APs. */ > -struct microcode_intel *intel_ucode_patch; > +static struct microcode_intel *intel_ucode_patch; > > static inline bool cpu_signatures_match(unsigned int s1, unsigned int p1, > unsigned int s2, unsigned int p2) > -- Acked-by: Borislav Petkov -- Regards/Gruss, Boris. Good mailing practices for 400: avoid top-posting and trim the reply.
[PATCH] x86/microcode/intel: make pointer intel_ucode_patch static
From: Colin Ian KingThe pointer intel_ucode_patch does not need to be in global scope, so make it static. Cleans up sparse warning: "symbol 'intel_ucode_patch' was not declared. Should it be static?" Signed-off-by: Colin Ian King --- arch/x86/kernel/cpu/microcode/intel.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/arch/x86/kernel/cpu/microcode/intel.c b/arch/x86/kernel/cpu/microcode/intel.c index d525a0bd7d28..59edbe9d4ccb 100644 --- a/arch/x86/kernel/cpu/microcode/intel.c +++ b/arch/x86/kernel/cpu/microcode/intel.c @@ -42,7 +42,7 @@ static const char ucode_path[] = "kernel/x86/microcode/GenuineIntel.bin"; /* Current microcode patch used in early patching on the APs. */ -struct microcode_intel *intel_ucode_patch; +static struct microcode_intel *intel_ucode_patch; static inline bool cpu_signatures_match(unsigned int s1, unsigned int p1, unsigned int s2, unsigned int p2) -- 2.11.0
[PATCH] x86/microcode/intel: make pointer intel_ucode_patch static
From: Colin Ian King The pointer intel_ucode_patch does not need to be in global scope, so make it static. Cleans up sparse warning: "symbol 'intel_ucode_patch' was not declared. Should it be static?" Signed-off-by: Colin Ian King --- arch/x86/kernel/cpu/microcode/intel.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/arch/x86/kernel/cpu/microcode/intel.c b/arch/x86/kernel/cpu/microcode/intel.c index d525a0bd7d28..59edbe9d4ccb 100644 --- a/arch/x86/kernel/cpu/microcode/intel.c +++ b/arch/x86/kernel/cpu/microcode/intel.c @@ -42,7 +42,7 @@ static const char ucode_path[] = "kernel/x86/microcode/GenuineIntel.bin"; /* Current microcode patch used in early patching on the APs. */ -struct microcode_intel *intel_ucode_patch; +static struct microcode_intel *intel_ucode_patch; static inline bool cpu_signatures_match(unsigned int s1, unsigned int p1, unsigned int s2, unsigned int p2) -- 2.11.0