Bug 7354

Summary:	CONFIG_PCI_GOANY enabled in config but not enabled in build
Product:	[Build System, Metadata & Runtime] OE-Core	Reporter:	Ross Burton <ross.burton>
Component:	kernel	Assignee:	Bruce Ashfield <bruce.ashfield>
Status:	VERIFIED FIXED	QA Contact:	Alexandru Georgescu <alexandru.c.georgescu>
Severity:	normal
Priority:	Medium+	CC:	alexandru.c.georgescu, bruce.ashfield, sgw, tom.zanussi
Version:	1.8
Target Milestone:	1.8
Hardware:	x86
OS:	x86_64
Whiteboard:
OS type for building Yocto:	---	Type of Regression:	Unknown (Hard to categorize)
Verified:		Documentation change:	No (bug/feature does not impact docs)

Description Ross Burton 2015-02-24 15:49:43 UTC

With Poky:

$ MACHINE=qemux86-64 bitbake virtual/kernel
...
Value requested for CONFIG_PCI_GOANY not in final ".config"
Requested value: "CONFIG_PCI_GOANY=y"
Actual value set: ""

Comment 1 Bruce Ashfield 2015-02-24 15:59:57 UTC

I'm not seeing this in my builds. Let me root cause the difference.

./bsp/common-pc/common-pc-drivers.cfg:CONFIG_PCI_GOANY=y

Is what contains the option, so I should be seeing the same warning if it is being dropped.

Comment 2 Bruce Ashfield 2015-02-24 16:01:55 UTC

I'll take this one, since it is qemu related.

Comment 3 Bruce Ashfield 2015-02-24 18:24:35 UTC

I have the fix, and will send the update to 3.19 shortly.

commit f7a81274b461d0c748964d75c24bea8574d350fb
Author: Bruce Ashfield <bruce.ashfield@windriver.com>
Date:   Tue Feb 24 13:19:13 2015 -0500

    meta: common-pc: split 32 bit-only drivers
    
    64 bit BSPs also use the common-pc drivers. To avoid warnings when these
    options are dropped, we split the config into 32 bit only and common.
    
    Signed-off-by: Bruce Ashfield <bruce.ashfield@windriver.com>

:000000 100644 000000000000... 35bc92b397ef... A        meta/cfg/kernel-cache/bsp/common-pc/common-pc-drivers-32.cfg
:100644 100644 28d1e73b5270... cf7ae1b80cbc... M        meta/cfg/kernel-cache/bsp/common-pc/common-pc-drivers.cfg
:100644 100644 f8abc4de443c... e0b2cc932874... M        meta/cfg/kernel-cache/bsp/common-pc/common-pc.scc

Comment 4 Alexandru Georgescu 2015-03-26 13:54:00 UTC

To test this:

$ MACHINE=qemux86-64 bitbake virtual/kernel

clean the sttate: 
$ MACHINE=qemux86-64 bitbake virtual/kernel -c cleansstate

then try again
$ MACHINE=qemux86-64 bitbake virtual/kernel

Comment 5 Alexandru Georgescu 2015-03-26 13:57:50 UTC

verified with fido:8b3d3e7c957ad06c390886487f69aeb8e145da6a