On Thu, Jan 24, 2013 at 01:15:21PM -0800, Greg Kroah-Hartman wrote:
> This is the start of the stable review cycle for the 3.4.28 release.
> There are 22 patches in this series, all will be posted as a response
> to this one.  If anyone has any issues with these being applied, please
> let me know.
> 
> Responses should be made by Sat Jan 26 21:13:38 UTC 2013.
> Anything received after that time might be too late.
> 
> The whole patch series can be found in one patch at:
>       kernel.org/pub/linux/kernel/v3.0/stable-review/patch-3.4.28-rc1.gz
> and the diffstat can be found below.
> 
> thanks,
> 
> greg k-h
> 
> -------------

3.4.28-rc1 seams to boot and run fine on my test environmenet (Debian Wheezy
VM with 2 vCPU, x86_64)

However, I've got a build failure with a .config having no PCI:

#
# Bus options (PCI etc.)
#
# CONFIG_PCI is not set
# CONFIG_ARCH_SUPPORTS_MSI is not set
CONFIG_PCI_LABEL=y
CONFIG_ISA_DMA_API=y
CONFIG_PCCARD=m
CONFIG_PCMCIA=m
CONFIG_PCMCIA_LOAD_CIS=y

The patch below fixes it


Thanks
-------------

From: Abdallah Chatila <[email protected]>
Date: Thu, 24 Jan 2013 20:30:18 -0500
Subject: [PATCH] arch/x86/kernel: Fix for a build failure

We need to have CONFIG_PCI for early_pci_allowed & read_pci_config_16.

patch based on commit e43b3cec711a61edf047adf6204d542f3a659ef8 upstream.

  LD      init/built-in.o
  LD      .tmp_vmlinux1
arch/x86/built-in.o: In function `setup_arch':
(.init.text+0x8df): undefined reference to `early_pci_allowed'
arch/x86/built-in.o: In function `setup_arch':
(.init.text+0x8f3): undefined reference to `read_pci_config_16'
arch/x86/built-in.o: In function `setup_arch':
(.init.text+0x90c): undefined reference to `read_pci_config_16'
make: *** [.tmp_vmlinux1] Error 1

Signed-off-by: Abdallah Chatila <[email protected]>
---
 arch/x86/kernel/setup.c |    2 ++
 1 file changed, 2 insertions(+)

diff --git a/arch/x86/kernel/setup.c b/arch/x86/kernel/setup.c
index d289ee7..b71e4a5 100644
--- a/arch/x86/kernel/setup.c
+++ b/arch/x86/kernel/setup.c
@@ -622,6 +622,7 @@ static unsigned reserve_low = CONFIG_X86_RESERVE_LOW << 10;
 
 static bool __init snb_gfx_workaround_needed(void)
 {
+#ifdef CONFIG_PCI
        int i;
        u16 vendor, devid;
        static const u16 snb_ids[] = {
@@ -646,6 +647,7 @@ static bool __init snb_gfx_workaround_needed(void)
        for (i = 0; i < ARRAY_SIZE(snb_ids); i++)
                if (devid == snb_ids[i])
                        return true;
+#endif
 
        return false;
 }
-- 
1.7.10.4

--
To unsubscribe from this list: send the line "unsubscribe stable" in
the body of a message to [email protected]
More majordomo info at  http://vger.kernel.org/majordomo-info.html

Reply via email to