Re: [PATCH 1/5] Move elfcorehdr_addr out of vmcore.c (Was: Re: [patch] crashdump: fix undefined reference to `elfcorehdr_addr')
[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
- To: Vivek Goyal <vgoyal@xxxxxxxxxx>
- Subject: Re: [PATCH 1/5] Move elfcorehdr_addr out of vmcore.c (Was: Re: [patch] crashdump: fix undefined reference to `elfcorehdr_addr')
- From: ebiederm@xxxxxxxxxxxx (Eric W. Biederman)
- Date: Mon, 28 Jul 2008 15:47:41 -0700
- Cc: Simon Horman <horms@xxxxxxxxxxxx>, Andrew Morton <akpm@xxxxxxxxxxxxxxxxxxxx>, Muli Ben-Yehuda <muli@xxxxxxxxxx>, Tony Luck <tony.luck@xxxxxxxxx>, linux-ia64@xxxxxxxxxxxxxxx, Chandru <chandru@xxxxxxxxxx>, kexec@xxxxxxxxxxxxxxxxxxx, linux-kernel@xxxxxxxxxxxxxxx, Terry Loftin <terry.loftin@xxxxxx>, Ingo Molnar <mingo@xxxxxxx>, Linus Torvalds <torvalds@xxxxxxxxxxxxxxxxxxxx>, Paul Mundt <lethal@xxxxxxxxxxxx>, Paul Mackerras <paulus@xxxxxxxxx>, linuxppc-dev@xxxxxxxxxx
- In-reply-to: <20080728211025.GA9985@xxxxxxxxxx> (Vivek Goyal's message of "Mon, 28 Jul 2008 17:10:25 -0400")
- List-id: <linux-ia64.vger.kernel.org>
- References: <20080727234529.GM6175@xxxxxxxxxxxx> <20080728015117.GA12055@xxxxxxxxxxxx> <20080728024526.GB3334@xxxxxxxxxxxx> <20080728034007.GA30450@xxxxxxxxxxxx> <20080728211025.GA9985@xxxxxxxxxx>
- User-agent: Gnus/5.110006 (No Gnus v0.6) Emacs/21.4 (gnu/linux)
Vivek Goyal <vgoyal@xxxxxxxxxx> writes:
> Hi All,
>
> How does following series of patches look like. I have moved
> elfcorehdr_addr out of vmcore.c and pushed it to arch dependent section
> of crash dump to make sure that it can be worked with even when
> CONFIG_PROC_VMCORE is disabled and CONFIG_CRASH_DUMP is enabled.
>
> I tested it on x86_64. Compile tested it on i386 and ppc64. ia64 and
> sh versions are completely untested.
Given the current state of the code:
Acked-by: "Eric W. Biederman" <ebiederm@xxxxxxxxxxxx>
To process a kernel crash dump we pass the kernel elfcorehdr option, so testing
to see if it was passed seems reasonable.
In general I think this method of handling the problems with kdump is
too brittle to live, but in the case of iommus we certainly need to do something
different, and unfortunately iommus were not common on x86 when the original code
was merged so we have not handled them well.
Eric
--
To unsubscribe from this list: send the line "unsubscribe linux-ia64" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
[Index of Archives]
[Linux Kernel]
[Sparc Linux]
[DCCP]
[Linux ARM]
[Yosemite News]
[Linux SCSI]
[Linux x86_64]
[Linux for Ham Radio]