linux-kernel - Re: [PATCH] Don't profile branches in arm's boot decompressor

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]

Message-ID: <alpine.DEB.2.00.0902271638510.13395@gandalf.stny.rr.com>
Date:	Fri, 27 Feb 2009 16:40:03 -0500 (EST)
From:	Steven Rostedt <rostedt@...dmis.org>
To:	Daniel Walker <dwalker@...o99.com>
cc:	Uwe Kleine-König 
	<u.kleine-koenig@...gutronix.de>, linux-kernel@...r.kernel.org,
	Steven Rostedt <rostedt@...e.goodmis.org>,
	Ingo Molnar <mingo@...e.hu>
Subject: Re: [PATCH] Don't profile branches in arm's boot decompressor


On Fri, 27 Feb 2009, Daniel Walker wrote:

> On Fri, 2009-02-27 at 22:09 +0100, Uwe Kleine-K?nig wrote:
> > This fixes many compile errors for arch/arm/boot/compressed/misc.c of
> > the type:
> > 
> > 	arch/arm/boot/compressed/misc.c:127: error: section attribute cannot be specified for local variables
> 
> Why do you think this error showed up just in that one file? It looks
> like the branch tracer puts section tags on non-globals all over the
> kernel..

The problem was with the -Dstatic= in the CFLAGS of the boot parameters. 
Since the branch profiler requires a static variable in the section, this 
would simply fail. That was the issue, and the solution was to disable the 
branch profiler for the boot code. No need for it there anyway.

-- 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/