* Re: [PATCH 0/3] vmcoreinfo support for dump filtering
[not found] <20070822210838oomichi@mail.jp.nec.com>
@ 2007-10-17 17:19 ` Tony Luck
2007-10-17 20:20 ` Andrew Morton
0 siblings, 1 reply; 3+ messages in thread
From: Tony Luck @ 2007-10-17 17:19 UTC (permalink / raw)
To: Ken'ichi Ohmichi
Cc: Andrew Morton, lkml, kexec-ml, Neil Horman, Vivek Goyal,
Bernhard Walle, Don Zickus, Dan Aloni, linux-ia64@vger.kernel.org
> This patch set frees the restriction that makedumpfile users should
> install a vmlinux file (including the debugging information) into
> each system.
This patch went from Andrew to Linus last night, and looks at first
glance to be the reason why the ia64 build is broken this morning
when using the arch/ia64/configs/tiger_defconfig configuration.
Here are the error messages:
arch/ia64/kernel/machine_kexec.c: In function `arch_crash_save_vmcoreinfo':
arch/ia64/kernel/machine_kexec.c:131: error: `pgdat_list' undeclared
(first use in this function)
arch/ia64/kernel/machine_kexec.c:131: error: (Each undeclared
identifier is reported only once
arch/ia64/kernel/machine_kexec.c:131: error: for each function it appears in.)
arch/ia64/kernel/machine_kexec.c:134: error: `node_memblk' undeclared
(first use in this function)
arch/ia64/kernel/machine_kexec.c:135: error: `NR_NODE_MEMBLKS'
undeclared (first use in this function)
arch/ia64/kernel/machine_kexec.c:136: error: invalid application of
`sizeof' to incomplete type `node_memblk_s'
arch/ia64/kernel/machine_kexec.c:137: error: dereferencing pointer to
incomplete type
arch/ia64/kernel/machine_kexec.c:138: error: dereferencing pointer to
incomplete type
make[1]: *** [arch/ia64/kernel/machine_kexec.o] Error 1
-Tony
^ permalink raw reply [flat|nested] 3+ messages in thread
* Re: [PATCH 0/3] vmcoreinfo support for dump filtering
2007-10-17 17:19 ` [PATCH 0/3] vmcoreinfo support for dump filtering Tony Luck
@ 2007-10-17 20:20 ` Andrew Morton
2007-10-17 21:32 ` Luck, Tony
0 siblings, 1 reply; 3+ messages in thread
From: Andrew Morton @ 2007-10-17 20:20 UTC (permalink / raw)
To: Tony Luck
Cc: oomichi, linux-kernel, kexec, nhorman, vgoyal, bwalle, dzickus,
da-x, linux-ia64
On Wed, 17 Oct 2007 10:19:18 -0700
"Tony Luck" <tony.luck@intel.com> wrote:
> > This patch set frees the restriction that makedumpfile users should
> > install a vmlinux file (including the debugging information) into
> > each system.
>
> This patch went from Andrew to Linus last night, and looks at first
> glance to be the reason why the ia64 build is broken this morning
> when using the arch/ia64/configs/tiger_defconfig configuration.
>
> Here are the error messages:
> arch/ia64/kernel/machine_kexec.c: In function `arch_crash_save_vmcoreinfo':
> arch/ia64/kernel/machine_kexec.c:131: error: `pgdat_list' undeclared
> (first use in this function)
> arch/ia64/kernel/machine_kexec.c:131: error: (Each undeclared
> identifier is reported only once
> arch/ia64/kernel/machine_kexec.c:131: error: for each function it appears in.)
> arch/ia64/kernel/machine_kexec.c:134: error: `node_memblk' undeclared
> (first use in this function)
> arch/ia64/kernel/machine_kexec.c:135: error: `NR_NODE_MEMBLKS'
> undeclared (first use in this function)
> arch/ia64/kernel/machine_kexec.c:136: error: invalid application of
> `sizeof' to incomplete type `node_memblk_s'
> arch/ia64/kernel/machine_kexec.c:137: error: dereferencing pointer to
> incomplete type
> arch/ia64/kernel/machine_kexec.c:138: error: dereferencing pointer to
> incomplete type
> make[1]: *** [arch/ia64/kernel/machine_kexec.o] Error 1
>
This?
--- a/arch/ia64/kernel/machine_kexec.c~ia64-fix-non-numa-build
+++ a/arch/ia64/kernel/machine_kexec.c
@@ -17,6 +17,8 @@
#include <linux/efi.h>
#include <linux/numa.h>
#include <linux/mmzone.h>
+
+#include <asm/numa.h>
#include <asm/mmu_context.h>
#include <asm/setup.h>
#include <asm/delay.h>
@@ -127,7 +129,7 @@ void machine_kexec(struct kimage *image)
void arch_crash_save_vmcoreinfo(void)
{
-#ifdef CONFIG_ARCH_DISCONTIGMEM_ENABLE
+#if defined(CONFIG_ARCH_DISCONTIGMEM_ENABLE) && defined(CONFIG_NUMA)
VMCOREINFO_SYMBOL(pgdat_list);
VMCOREINFO_LENGTH(pgdat_list, MAX_NUMNODES);
_
(the asm/numa.h include isn't needed by this patch, but strictly should be
there, as this file refers to things which are defined in asm/numa.h)
(it's weird that linux/numa.h doesn't include asm/numa.h)
(please tell me if you want me to send this to Linus)
^ permalink raw reply [flat|nested] 3+ messages in thread
* RE: [PATCH 0/3] vmcoreinfo support for dump filtering
2007-10-17 20:20 ` Andrew Morton
@ 2007-10-17 21:32 ` Luck, Tony
0 siblings, 0 replies; 3+ messages in thread
From: Luck, Tony @ 2007-10-17 21:32 UTC (permalink / raw)
To: Andrew Morton
Cc: oomichi, linux-kernel, kexec, nhorman, vgoyal, bwalle, dzickus,
da-x, linux-ia64
> This?
That does the trick, yes.
> (please tell me if you want me to send this to Linus)
I've put it in my tree now ... so I'll ask Linus to pull
it from there.
-Tony
^ permalink raw reply [flat|nested] 3+ messages in thread
end of thread, other threads:[~2007-10-17 21:32 UTC | newest]
Thread overview: 3+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
[not found] <20070822210838oomichi@mail.jp.nec.com>
2007-10-17 17:19 ` [PATCH 0/3] vmcoreinfo support for dump filtering Tony Luck
2007-10-17 20:20 ` Andrew Morton
2007-10-17 21:32 ` Luck, Tony
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox