Re: Compile Error on 2.6.17-mm6

[Randy.Dunlap]

On Sat, 8 Jul 2006 20:44:48 -0700 Andrew Morton wrote:

> On Sat, 8 Jul 2006 20:34:24 -0700
> "Randy.Dunlap" <rdunlap@xxxxxxxxxxxx> wrote:
> 
> > On Sat, 8 Jul 2006 17:43:47 -0700 Andrew Morton wrote:
> > 
> > > On Fri, 7 Jul 2006 22:22:16 -0400
> > > Matt Reuther <mreuther@xxxxxxxxx> wrote:
> > > 
> > > > Here is the error:
> > > >   CHK     include/linux/compile.h
> > > >   UPD     include/linux/compile.h
> > > >   CC      init/version.o
> > > >   LD      init/built-in.o
> > > >   LD      .tmp_vmlinux1
> > > > arch/i386/kernel/built-in.o(.text+0xe282): In function 
> > > > `cpu_request_microcode':
> > > > arch/i386/kernel/microcode.c:544: undefined reference to `request_firmware'
> > > > arch/i386/kernel/built-in.o(.text+0xe304):arch/i386/kernel/microcode.c:573: 
> > > > undefined reference to `release_firmware'
> > > 
> > > CONFIG_FW_LOADER=m
> > > CONFIG_MICROCODE=y
> > > 
> > > So
> > > 
> > > config MICROCODE
> > > 	tristate "/dev/cpu/microcode - Intel IA32 CPU microcode support"
> > > 	depends on FW_LOADER
> > > 
> > > is not sufficient.  There's a fix for this, but I cannot remember what it
> > > is.  Help.
> > 
> > That 1-line depends patch fixes the problem for me (on x86-64,
> > but they are the same in this area).
> > 
> 
> What patch is that?

My -mm6 does not have "depends on FW_LOADER" like you wrote above,
so I thought that you had added that 1-line as a patch.



From: Randy Dunlap <rdunlap@xxxxxxxxxxxx>

MICROCODE needs FW_LOADER functions.

Signed-off-by: Randy Dunlap <rdunlap@xxxxxxxxxxxx>
---
 arch/i386/Kconfig   |    1 +
 arch/x86_64/Kconfig |    1 +
 2 files changed, 2 insertions(+)

--- linux-2617-mm6.orig/arch/x86_64/Kconfig
+++ linux-2617-mm6/arch/x86_64/Kconfig
@@ -159,6 +159,7 @@ config X86_GOOD_APIC
 
 config MICROCODE
 	tristate "/dev/cpu/microcode - Intel CPU microcode support"
+	depends on FW_LOADER
 	---help---
 	  If you say Y here the 'File systems' section, you will be
 	  able to update the microcode on Intel processors. You will
--- linux-2617-mm6.orig/arch/i386/Kconfig
+++ linux-2617-mm6/arch/i386/Kconfig
@@ -399,6 +399,7 @@ config X86_REBOOTFIXUPS
 
 config MICROCODE
 	tristate "/dev/cpu/microcode - Intel IA32 CPU microcode support"
+	depends on FW_LOADER
 	---help---
 	  If you say Y here and also to "/dev file system support" in the
 	  'File systems' section, you will be able to update the microcode on



---
-
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/