On Tue, Mar 06, 2007 at 10:32:09AM +0800, Zou Nan hai wrote: > On Tue, 2007-03-06 at 09:56, Horms wrote: > > Hi, > > > > I am currently looking over the code that places the crashdump > > region into /proc/iomem, and the code that determines its base > > address if it is not passed on the kernel comamnd. It seems to me that > > the current code allows the crashkernel to be placed incide a > > /proc/iomem region of any type. Is this behaviour correct? > > If not, should it be restricted to "System RAM" regions? > > Please review the patch. This looks good to me. Acked-by: Simon Horman <horms@verge.net.au> > Thanks > Zou Nan hai > > Signed-off-by: Zou Nan hai <nanhai.zou@intel.com> > > --- a/arch/ia64/kernel/efi.c 2007-02-27 00:42:06.000000000 -0500 > +++ b/arch/ia64/kernel/efi.c 2007-03-05 22:09:40.000000000 -0500 > @@ -1154,7 +1154,7 @@ kdump_find_rsvd_region (unsigned long si > > for (p = efi_map_start; p < efi_map_end; p += efi_desc_size) { > md = p; > - if (!efi_wb(md)) > + if (!is_memory_available(md)) > continue; > start = ALIGN(md->phys_addr, alignment); > end = efi_md_end(md); > > -- Horms H: http://www.vergenet.net/~horms/ W: http://www.valinux.co.jp/en/ - To unsubscribe from this list: send the line "unsubscribe linux-ia64" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.htmlReceived on Tue Mar 06 18:59:49 2007
This archive was generated by hypermail 2.1.8 : 2007-03-06 19:04:32 EST