Re: [PATCH] ftrace: Allow section alignment

From: Steven Rostedt
Date: Fri Nov 07 2008 - 08:29:44 EST



On Fri, 7 Nov 2008, Peter Zijlstra wrote:

> On Fri, 2008-11-07 at 13:12 +0000, Matt Fleming wrote:
> > Provide a means of aligning the __mcount_loc section.
>
> Not that I object to the patch, but this changelog needs work.
>
> Its wrong, it doesn't provide means, it plain does.
> It doesn't tell us why.

Also, please tell us what broke. Did you have a compiler that did
not align it properly? As long as the alignment is less than or equal to
the word size, it is fine. If the alignment is larger than the word size,
then there will be holes in the mcount_loc section and that will break
ftrace. Heck, an alignment of 1 should work for everyone ;-) All the
sections are packed together in an aligned space when it is pulled in by
the linker. If that is not true, then the linker script needs to be fixed.

-- Steve
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/