All of lore.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: 77+ 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:41   ` Jiri Slaby
2010-10-21  7:49   ` Takashi Iwai
2010-10-21  7:49     ` Takashi Iwai
2010-10-21  7:57     ` Jiri Slaby
2010-10-21  7:57       ` Jiri Slaby
2010-10-21  8:05       ` Takashi Iwai
2010-10-21  8:05         ` Takashi Iwai
2010-10-21 18:27         ` Jiri Slaby
2010-10-21 18:27           ` Jiri Slaby
2010-10-21 19:20           ` Takashi Iwai
2010-10-21 19:20             ` Takashi Iwai
2010-10-21 19:21             ` Jiri Slaby
2010-10-21 19:21               ` Jiri Slaby
2010-10-21 19:22               ` Takashi Iwai
2010-10-21 19:22                 ` Takashi Iwai
2010-10-21 19:40               ` Takashi Iwai
2010-10-21 19:40                 ` Takashi Iwai
2010-10-21 20:29                 ` Valdis.Kletnieks
2010-10-21 20:29                   ` Valdis.Kletnieks
2010-10-21 20:48                   ` Takashi Iwai
2010-10-21 20:48                     ` Takashi Iwai
2010-10-21 21:24                     ` Valdis.Kletnieks
2010-10-21 21:24                       ` Valdis.Kletnieks
2010-10-21 21:31                       ` Takashi Iwai
2010-10-21 21:31                         ` Takashi Iwai
2010-10-21 21:42                         ` Jiri Slaby
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:02                               ` Takashi Iwai
2010-10-22  9:18                               ` Jiri Slaby
2010-10-22  9:18                                 ` Jiri Slaby
2010-10-22  9:40                                 ` Takashi Iwai
2010-10-22  9:40                                   ` Takashi Iwai
2010-10-22  9:45                                   ` Jiri Slaby
2010-10-22  9:45                                     ` Jiri Slaby
2010-10-22 10:26                                     ` Takashi Iwai
2010-10-22 10:26                                       ` Takashi Iwai
2010-10-22 11:10                                       ` Jiri Slaby
2010-10-22 11:10                                         ` Jiri Slaby
2010-10-22  9:41                               ` Colin Guthrie
2010-10-22 11:31           ` Clemens Ladisch
2010-10-22 11:31             ` [alsa-devel] " Clemens Ladisch
2010-10-22 11:51             ` Jiri Slaby
2010-10-22 11:51               ` [alsa-devel] " Jiri Slaby
2010-10-22 14:03             ` Takashi Iwai
2010-10-22 14:03               ` [alsa-devel] " Takashi Iwai
2010-10-22 19:53               ` Colin Guthrie
2010-10-21 15:22       ` Valdis.Kletnieks
2010-10-21 15:22         ` Valdis.Kletnieks
2010-10-21 17:59         ` Jiri Slaby
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 an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.