Re: [PATCH] Do not inline putprops function
From: M. Mohan Kumar <hidden>
Date: 2009-08-07 14:54:33
Also in:
kexec
On Fri, Aug 07, 2009 at 08:05:49PM +0530, M. Mohan Kumar wrote:
Hi, After enabling EARLY_DEBUG (and DEBUG in some of the files in arch/powerpc/kernel directory), without forcing the dtstruct variable to 8 byte alignment: # ./kexec -e Starting new kernel console [udbg0] enabled -> early_setup(), dt_ptr: 0x7723000 -> early_init_devtree(c000000007723000) Invalid tag 5 scanning flattened device tree ! search "chosen", depth: 0, uname: Invalid tag 5 scanning flattened device tree ! dt_root_size_cells = 2 dt_root_addr_cells = 2 Invalid tag 5 scanning flattened device tree ! reserving: 128c000 -> 5ec1f7 reserving: 7734000 -> 8cc000 reserving: 7723000 -> f698 Phys. mem: 0 -> move_device_tree <- move_device_tree Scanning CPUs ... Invalid tag 5 scanning flattened device tree ! <- early_init_devtree() Probing machine type ... pSeries ... No suitable machine found ! So device-tree is getting corrupted when dtstruct variable is not aligned to 8 byte variable. This problem is not seen with gcc-3.4. Is it compiler issue? or bug in the code.
I tried writing the device tree to a binary file with and without dt_len and compared the files after hexdump: 0000 0001 2f00 0000 0000 0003 0000 0004 0000 0001 2f00 0000 0000 0003 0000 0004 0000 0000 0000 0002 0000 0003 0000 0004 0000 0000 0000 0002 0000 0003 0000 0004 0000 000f 0000 0002 0000 0003 0000 0004 0000 000f 0000 0002 0000 0003 0000 0004 0000 001b 3ef1 4800 0000 0003 0000 000d 0000 001b 3ef1 4800 0000 0003 0000 000d 0000 002b 0000 0000 4942 4d2c 3931 3135 | 0000 002b 4942 4d2c 3931 3135 2d35 3035 2d35 3035 0000 0000 0000 0003 0000 0005 | 0000 0000 0000 0003 0000 0005 0000 0036 Regards, M. Mohan Kumar
On Fri, Aug 07, 2009 at 12:24:20AM +1000, Michael Ellerman wrote:quoted
On Wed, 2009-08-05 at 22:19 +0530, M. Mohan Kumar wrote:quoted
Hi, When I align the dtstruct variable to 8 bytes, I am able to invoke kdump. When the line static unsigned dtstruct[TREEWORDS], *dt; changed to static unsigned dtstruct[TREEWORDS] __attribute__ ((aligned (8))), *dt; kexec-tool works.Hmm, odd. Can you check how it's aligned without your change? ie. in the original binary, is it 4 byte aligned? When you make the change, is the only thing that changes in the binary the alignedness of dtstruct, or does it cause other things to move around? I don't think an unaligned dt blob should have any effect on the kernel, ie. it should copy it in fine, but I'd have to look at the code. cheers_______________________________________________ kexec mailing list kexec@lists.infradead.org http://lists.infradead.org/mailman/listinfo/kexec