On Sat, Dec 22, 2007 at 09:37:17AM +0100, Sam Ravnborg wrote: > Hi Jesper. > > > __initramfs_end = .; > > - /* We fill to the next page, so we can discard all init > > - pages without needing to consider what payload might be > > - appended to the kernel image. */ > > - FILL (0); > > - . = ALIGN (8192); > > } > > #endif > > - > > __vmlinux_end = .; /* last address of the physical file */ > > - __init_end = .; > > + > > + /* > > + * We fill to the next page, so we can discard all init > > + * pages without needing to consider what payload might be > > + * appended to the kernel image. > > + */ > > + .init.fill : { > > + FILL (0); > > + . = ALIGN (8192); > > + } > > + > > + __init_end = .; > > Can we please have the hardcoded 8192 replaced with a proper constant. > I assume this is PAGESIZE?
Yes, quite so. > See hwat other archs does to make PAGESIZE available for assembly > files and ldscripts. The macro AC is the key point here. Ok, will do. > And then please explain why it make a diffrence to put the > alignment in a section like you do. > I do not see it used in the other archs. I'm not an expert on ld-scripts, so I only know that the linker didn't respect the other way of doing the alignment, but did when added in a section. Perhaps Mikael has a better explanation when he's back from vacation. > Sam Best regards, /^JN - Jesper Nilsson -- Jesper Nilsson -- [EMAIL PROTECTED] -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to [EMAIL PROTECTED] More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/