On Thu, Jan 24, 2013 at 11:16:08PM -0500, Abdallah Chatila wrote:
> On Thu, Jan 24, 2013 at 01:12:38PM -0800, Greg Kroah-Hartman wrote:
> > This is the start of the stable review cycle for the 3.7.5 release.
> > There are 46 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:09:21 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.7.5-rc1.gz
> > and the diffstat can be found below.
> > 
> > thanks,
> > 
> > greg k-h
> > 
> > -------------
> 
> 3.7.5-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:

Is this something new that is not a problem in 3.7.4?

> #
> # 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 22:28:03 -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
> arch/x86/built-in.o: In function `setup_arch':
> (.init.text+0xb3c): undefined reference to `early_pci_allowed'
> arch/x86/built-in.o: In function `setup_arch':
> (.init.text+0xb50): undefined reference to `read_pci_config_16'
> arch/x86/built-in.o: In function `setup_arch':
> (.init.text+0xb69): undefined reference to `read_pci_config_16'
> make: *** [vmlinux] 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 86c524c..7e07344 100644
> --- a/arch/x86/kernel/setup.c
> +++ b/arch/x86/kernel/setup.c
> @@ -616,6 +616,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[] = {
> @@ -640,6 +641,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

Can't I just take the original e43b3cec711a61edf047adf6204d542f3a659ef8
patch, or have you modified it here, and for the other -stable releases?

thanks,

greg k-h
--
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