public inbox for linux-kernel@vger.kernel.org
 help / color / mirror / Atom feed
From: Konrad Rzeszutek Wilk <konrad.wilk@oracle.com>
To: Li Zefan <lizf@cn.fujitsu.com>
Cc: akpm@linux-foundation.org, mm-commits@vger.kernel.org,
	linux-kernel@vger.kernel.org
Subject: Re: mmotm 2010-10-20-15-01 uploaded
Date: Thu, 21 Oct 2010 10:28:27 -0400	[thread overview]
Message-ID: <20101021142827.GA31420@dumpdata.com> (raw)
In-Reply-To: <4CBFB70C.1030009@cn.fujitsu.com>

> >>> Got this:
> >>>
> >>> arch/x86/pci/xen.c: In function 'pci_xen_init':
> >>> arch/x86/pci/xen.c:138: error: 'isapnp_disable' undeclared (first use in this function)
> >>> arch/x86/pci/xen.c:138: error: (Each undeclared identifier is reported only once
> >>> arch/x86/pci/xen.c:138: error: for each function it appears in.)
> > 
> > Ok. Will fix this.
> >>> # quilt patches arch/x86/pci/xen.c
> >>> linux-next.patch
> >> I just commented out isapnp_disable, and after boot I saw this warning:

I got this patch queued up. You mentioned that commenting it out did fix
the problem, so I stuck a Tested-by tag from you on it.

>From 5bba6c56dc99ff88f79a79572e29ecf445710878 Mon Sep 17 00:00:00 2001
From: Konrad Rzeszutek Wilk <konrad.wilk@oracle.com>
Date: Thu, 21 Oct 2010 09:36:07 -0400
Subject: [PATCH 27/27] X86/PCI: Remove the dependency on isapnp_disable.

This looks to be vestigial dependency that had never been used even
in the original code base (2.6.18) from which this driver
was up-ported. Without this fix, with the CONFIG_ISAPNP, we get this
compile failure:

arch/x86/pci/xen.c: In function 'pci_xen_init':
arch/x86/pci/xen.c:138: error: 'isapnp_disable' undeclared (first use in this function)
arch/x86/pci/xen.c:138: error: (Each undeclared identifier is reported only once
arch/x86/pci/xen.c:138: error: for each function it appears in.)

Reported-by: Li Zefan <lizf@cn.fujitsu.com>
Tested-by: Li Zefan <lizf@cn.fujitsu.com>
Signed-off-by: Konrad Rzeszutek Wilk <konrad.wilk@oracle.com>
---
 arch/x86/pci/xen.c |    5 -----
 1 files changed, 0 insertions(+), 5 deletions(-)

diff --git a/arch/x86/pci/xen.c b/arch/x86/pci/xen.c
index b19c873..4e37106 100644
--- a/arch/x86/pci/xen.c
+++ b/arch/x86/pci/xen.c
@@ -133,11 +133,6 @@ int __init pci_xen_init(void)
 	acpi_noirq = 1;
 #endif
 
-#ifdef CONFIG_ISAPNP
-	/* Stop isapnp from probing */
-	isapnp_disable = 1;
-#endif
-
 #ifdef CONFIG_PCI_MSI
 	x86_msi.setup_msi_irqs = xen_setup_msi_irqs;
 	x86_msi.teardown_msi_irq = xen_teardown_msi_irq;
-- 
1.7.1


  reply	other threads:[~2010-10-21 14:29 UTC|newest]

Thread overview: 52+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2010-10-20 22:01 mmotm 2010-10-20-15-01 uploaded akpm
2010-10-21  2:54 ` Li Zefan
2010-10-21  3:15   ` Li Zefan
2010-10-21  3:38     ` Konrad Rzeszutek Wilk
2010-10-21  3:44       ` Li Zefan
2010-10-21 14:28         ` Konrad Rzeszutek Wilk [this message]
2010-10-21  3:46     ` Andrew Morton
2010-10-21  7:17       ` Peter Zijlstra
2010-10-21  7:30         ` Andrew Morton
2010-10-21  9:52           ` Peter Zijlstra
2010-10-21  9:54           ` Peter Zijlstra
2010-10-21  7:41 ` HDA: no sound [was: mmotm 2010-10-20-15-01 uploaded] Jiri Slaby
2010-10-21  7:49   ` Takashi Iwai
2010-10-21  7:57     ` Jiri Slaby
2010-10-21  8:05       ` Takashi Iwai
2010-10-21 18:27         ` Jiri Slaby
2010-10-21 19:20           ` Takashi Iwai
2010-10-21 19:21             ` Jiri Slaby
2010-10-21 19:22               ` Takashi Iwai
2010-10-21 19:40               ` Takashi Iwai
2010-10-21 20:29                 ` Valdis.Kletnieks
2010-10-21 20:48                   ` Takashi Iwai
2010-10-21 21:24                     ` Valdis.Kletnieks
2010-10-21 21:31                       ` Takashi Iwai
2010-10-21 21:42                         ` Jiri Slaby
2010-10-22  8:46                           ` Colin Guthrie
2010-10-22  9:02                             ` Takashi Iwai
2010-10-22  9:18                               ` Jiri Slaby
2010-10-22  9:40                                 ` Takashi Iwai
2010-10-22  9:45                                   ` Jiri Slaby
2010-10-22 10:26                                     ` Takashi Iwai
2010-10-22 11:10                                       ` Jiri Slaby
2010-10-22  9:41                               ` Colin Guthrie
2010-10-22 11:31           ` [alsa-devel] " Clemens Ladisch
2010-10-22 11:51             ` Jiri Slaby
2010-10-22 14:03             ` Takashi Iwai
2010-10-22 19:53               ` Colin Guthrie
2010-10-21 15:22       ` Valdis.Kletnieks
2010-10-21 17:59         ` Jiri Slaby
2010-10-21 11:10 ` mmotm 2010-10-20-15-01 uploaded Zimny Lech
2010-10-21 11:14 ` mmotm 2010-10-20 - netfilter Kconfig whinge Valdis.Kletnieks
2010-10-29 19:11   ` Patrick McHardy
2010-10-21 17:51 ` DVB: af9015 defunct [was: mmotm 2010-10-20-15-01 uploaded] Jiri Slaby
2010-10-21 22:24   ` Jiri Slaby
2010-10-22  9:21   ` Jiri Slaby
2010-10-22 13:56     ` Mauro Carvalho Chehab
2010-10-22 14:06       ` Jiri Slaby
2010-10-27 16:12         ` Mauro Carvalho Chehab
2010-10-27 16:36           ` Jiri Slaby
2010-10-27 17:05             ` Mauro Carvalho Chehab
2010-10-27 16:41           ` Antti Palosaari
2010-10-27 17:11             ` Mauro Carvalho Chehab

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=20101021142827.GA31420@dumpdata.com \
    --to=konrad.wilk@oracle.com \
    --cc=akpm@linux-foundation.org \
    --cc=linux-kernel@vger.kernel.org \
    --cc=lizf@cn.fujitsu.com \
    --cc=mm-commits@vger.kernel.org \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox