Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1759557Ab0FJQ3S (ORCPT ); Thu, 10 Jun 2010 12:29:18 -0400 Received: from hrndva-omtalb.mail.rr.com ([71.74.56.124]:56030 "EHLO hrndva-omtalb.mail.rr.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1750989Ab0FJQ3P (ORCPT ); Thu, 10 Jun 2010 12:29:15 -0400 X-Authority-Analysis: v=1.1 cv=qXMh5QPBcdVGPlSuovMI3YqffpUTMWPwMqUnXLzRgMM= c=1 sm=0 a=i7LXe8TlZakA:10 a=ood2b7iyd8MA:10 a=7U3hwN5JcxgA:10 a=Q9fys5e9bTEA:10 a=gMqfjgEr1zLu/65IO0LwxA==:17 a=JfrnYn6hAAAA:8 a=5oc08QPB5tXWEBcBArMA:9 a=2C-QW0ii_ew1GB5NkZoA:7 a=nQ4Ecfa-YrMH0i1NFpt_tug74QkA:4 a=PUjeQqilurYA:10 a=3Rfx1nUSh_UA:10 a=Qw7BfDJW9mHjxGDW:21 a=oBMinR2w_07PqoqG:21 a=gMqfjgEr1zLu/65IO0LwxA==:117 X-Cloudmark-Score: 0 X-Originating-IP: 74.67.89.75 Subject: Re: [PATCH 03/13] jump label v9: x86 support From: Steven Rostedt Reply-To: rostedt@goodmis.org To: Andi Kleen Cc: Ingo Molnar , Peter Zijlstra , Jason Baron , linux-kernel@vger.kernel.org, mathieu.desnoyers@polymtl.ca, hpa@zytor.com, tglx@linutronix.de, roland@redhat.com, rth@redhat.com, mhiramat@redhat.com, fweisbec@gmail.com, avi@redhat.com, davem@davemloft.net, vgoyal@redhat.com, sam@ravnborg.org In-Reply-To: <20100610132616.GA19561@basil.fritz.box> References: <1276171930.2077.200.camel@twins> <20100610121440.GA20130@elte.hu> <20100610132616.GA19561@basil.fritz.box> Content-Type: text/plain; charset="ISO-8859-15" Organization: Kihon Technologies Inc. Date: Thu, 10 Jun 2010 12:29:11 -0400 Message-ID: <1276187351.12038.24.camel@gandalf.stny.rr.com> Mime-Version: 1.0 X-Mailer: Evolution 2.28.3 Content-Transfer-Encoding: 7bit Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Content-Length: 1531 Lines: 37 On Thu, 2010-06-10 at 15:26 +0200, Andi Kleen wrote: > On Thu, Jun 10, 2010 at 02:14:40PM +0200, Ingo Molnar wrote: > > > > * Peter Zijlstra wrote: > > > > > On Wed, 2010-06-09 at 17:39 -0400, Jason Baron wrote: > > > > + select HAVE_ARCH_JUMP_LABEL if !CC_OPTIMIZE_FOR_SIZE > > > > > > That deserves a comment somewhere, it basically makes OPTIMIZE_FOR_SIZE > > > useless... > > > > Hm, we need more than a comment for that - distros enable CC_OPTIMIZE_FOR_SIZE > > all the time, for the massive kernel image (and hotpath cache footprint) > > savings. Is this fixable? > > Actually the big distros (RHEL, SLES) disable it all the time now. It costs you > in some benchmarks. The code generated by -Os is often terrible. > > Nearly everytime I investigate some bad asm code being generated by gcc > it goes away when that flag is disabled. I agree to the above statement. It was this option that first caused the crazy mcount prefix code that totally broke the function tracer on i386. With this option, some functions would send a copy of the return address to mcount and not an actual pointer to the return address on stack. This would confuse the function tracer and we ended up with a kernel panic. -Os really needs to be fixed. -- Steve -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/