]> git.kernelconcepts.de Git - karo-tx-linux.git/commitdiff
powerpc/prom: Remove limit on maximum size of properties
authorNishanth Aravamudan <nacc@linux.vnet.ibm.com>
Mon, 27 Feb 2012 08:55:15 +0000 (08:55 +0000)
committerBenjamin Herrenschmidt <benh@kernel.crashing.org>
Wed, 7 Mar 2012 06:06:10 +0000 (17:06 +1100)
On a 16TB system (using AMS/CMO), I get:

WARNING: ignoring large property [/ibm,dynamic-reconfiguration-memory] ibm,dynamic-memory length 0x000000000017ffec

and significantly less memory is thus shown to the partition. As far as
I can tell, the constant used is arbitrary. Ben Herrenschmidt provided
additional background that

> The limit was originally set because of Apple machines carrying ROM
> images in the device-tree, at a time where we were much more memory
> constrained than we are now.

and that it is likely not very useful any longer.

Signed-off-by: Nishanth Aravamudan <nacc@us.ibm.com>
Signed-off-by: Benjamin Herrenschmidt <benh@kernel.crashing.org>
arch/powerpc/kernel/prom_init.c

index eca626ea3f2393dcaf0cd6bfb61f6262a4443e7e..e2d599048142e2d4d7084f803b2b5abd5fa70f43 100644 (file)
 
 #include <linux/linux_logo.h>
 
-/*
- * Properties whose value is longer than this get excluded from our
- * copy of the device tree. This value does need to be big enough to
- * ensure that we don't lose things like the interrupt-map property
- * on a PCI-PCI bridge.
- */
-#define MAX_PROPERTY_LENGTH    (1UL * 1024 * 1024)
-
 /*
  * Eventually bump that one up
  */
@@ -2273,13 +2265,6 @@ static void __init scan_dt_build_struct(phandle node, unsigned long *mem_start,
                /* sanity checks */
                if (l == PROM_ERROR)
                        continue;
-               if (l > MAX_PROPERTY_LENGTH) {
-                       prom_printf("WARNING: ignoring large property ");
-                       /* It seems OF doesn't null-terminate the path :-( */
-                       prom_printf("[%s] ", path);
-                       prom_printf("%s length 0x%x\n", RELOC(pname), l);
-                       continue;
-               }
 
                /* push property head */
                dt_push_token(OF_DT_PROP, mem_start, mem_end);