[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-Id: <1232128377.27278.69.camel@nimitz>
Date: Fri, 16 Jan 2009 09:52:57 -0800
From: Dave Hansen <dave@...ux.vnet.ibm.com>
To: Chandru <chandru@...ibm.com>
Cc: Paul Mackerras <paulus@...ba.org>,
Andrew Morton <akpm@...ux-foundation.org>,
linux-kernel@...r.kernel.org, linuxppc-dev@...abs.org,
Benjamin Herrenschmidt <benh@...nel.crashing.org>
Subject: Re: 2.6.28-rc9 panics with crashkernel=256M while booting
On Fri, 2009-01-16 at 17:46 +0530, Chandru wrote:
> On Thursday 15 January 2009 13:35:27 Chandru wrote:
> > Hello Dave, From the debug console output, if there is anything you can add
> > here, pls let me know.
>
> As we can see from the console output here, physbase isn't page aligned when
> the panic occurs. So we could as well send (start_pfn << PAGE_SHIFT) to
> reserve_bootmem_node() instead of physbase. your thoughts ?.
>
> Also end_pfn in mark_reserved_region_for_nid() is defined as
>
> unsigned long end_pfn = ((physbase + size) >> PAGE_SHIFT);
>
> Does this refer to the pfn after the area that we are interested in ?. We have
> atleast two fixes here,
> 1. Limit start and end to bdata->node_min_pfn and bdata->node_low_pfn in
> reserve_bootmem_node() and add comments out in there that the caller of the
> funtion should be aware of how much are they reserving.
> 2. send (start_pfn << PAGE_SHIFT) to reserve_bootmem_node() instead of
> physbase.
Just looking at it, that calculation is OK. But, there was one in your
dmesg that looked a page too long, like page 0x1001 instead of 0x1000.
I'd find out how that happened.
-- Dave
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@...r.kernel.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/
Powered by blists - more mailing lists