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@xxxxxxxxxxxx>
> Thanks
> Zou Nan hai
>
> Signed-off-by: Zou Nan hai <nanhai.zou@xxxxxxxxx>
>
> --- 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@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
|