linux-kernel - Re: [PATCH] ftrace: Allow section alignment

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]

Date:	Fri, 7 Nov 2008 08:29:31 -0500 (EST)
From:	Steven Rostedt <rostedt@...dmis.org>
To:	Peter Zijlstra <peterz@...radead.org>
cc:	Matt Fleming <mjf@...too.org>, LKML <linux-kernel@...r.kernel.org>,
	mingo@...e.hu
Subject: Re: [PATCH] ftrace: Allow section alignment

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@...r.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/