> From: Olof Johansson [mailto:olof@xxxxxxxxx] > Sent: Friday, December 21, 2012 9:36 AM > > On Thu, Dec 20, 2012 at 4:32 PM, Kukjin Kim <kgene.kim@xxxxxxxxxxx> wrote: > > Russell King - ARM Linux wrote: > >> > > > > [...] > > > >> > > >> > > What's the actual bug caused by this? The description is vague. > >> > > >> > The kernel panics early on NULL pointer dereference in memory > >> > initialization. > >> > > > > (Cc'ed KyongHo Cho) > > > > Yeah, correct. The size of memory bank should be under section size and > > current section size is 256MiB on EXYNOS. So if we don't change the section > > size, each memory bank should be under 256MiB. If not, as Tomasz said, > > kernel panic happens in mem_init(). > > That's a bug somewhere, the device tree should describe the hardware, > and not necessarily the massaged format that the kernel wants it in to > not trigger a bug. > > So I don't want to see this patch merged, I want to see a proper fix > instead, please. > > There are also already other exynos platforms and device trees with > more than 256MB in the memory node. Changing all of them this way > seems unreasonable. > As Kukjin said earlier, the kernel panic may happen while gathering memory statistics in mem_init()(arch/arm/mm/init.c). Russell King does not agreed to modify mem_init() instead he told that a memory bank must not cross section boundaries. -- To unsubscribe from this list: send the line "unsubscribe linux-samsung-soc" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html
 |