From: Andi Kleen <andi@firstfloor.org>
To: Ingo Molnar <mingo@elte.hu>
Cc: Andi Kleen <andi@firstfloor.org>,
Peter Zijlstra <peterz@infradead.org>,
Jason Baron <jbaron@redhat.com>,
linux-kernel@vger.kernel.org, mathieu.desnoyers@polymtl.ca,
hpa@zytor.com, tglx@linutronix.de, rostedt@goodmis.org,
roland@redhat.com, rth@redhat.com, mhiramat@redhat.com,
fweisbec@gmail.com, avi@redhat.com, davem@davemloft.net,
vgoyal@redhat.com, sam@ravnborg.org,
Linus Torvalds <torvalds@linux-foundation.org>,
Arjan van de Ven <arjan@infradead.org>
Subject: Re: [PATCH 03/13] jump label v9: x86 support
Date: Fri, 11 Jun 2010 10:30:28 +0200 [thread overview]
Message-ID: <20100611083028.GE6864@basil.fritz.box> (raw)
In-Reply-To: <20100611081256.GA20217@elte.hu>
> What you are arguing for is a 'bloaty code generator by default' model and
> that model sucks.
I am arguing for a "non sucky code by default" model.
It is widely known that "sucky code by default" sucks already,
that is why the big distros made their choice.
Anyways luckily the default is all config options so we don't need
to agree on this (and the best choice likely varies by workload
too)
> Possibly, but not without substantiating the rather vague statements you have
> made so far.
Yes, more data with recent builds is needed for concrete changes.
BTW afaik the "icache over everything" model was never really
substantiated by all that much data either, just somehow
it became dogma.
I must say I was a bit burned by doing annotations -- i added
unlikely() originally and as far as I can see most unlikely()s
are quite useless today because they do nothing the compiler
doesn't do already so I would prefer to not repeat that.
So my personal preference is actually less annotations over more.
-Andi
--
ak@linux.intel.com -- Speaking for myself only.
next prev parent reply other threads:[~2010-06-11 8:30 UTC|newest]
Thread overview: 54+ messages / expand[flat|nested] mbox.gz Atom feed top
2010-06-09 21:38 [PATCH 00/13] jump label v9 Jason Baron
2010-06-09 21:38 ` [PATCH 01/13] jump label v9: notifier atomic call chain notrace Jason Baron
2010-06-09 21:58 ` Frederic Weisbecker
2010-06-10 15:34 ` Jason Baron
2010-06-09 21:38 ` [PATCH 02/13] jump label v9: base patch Jason Baron
2010-06-09 22:35 ` Frederic Weisbecker
2010-06-10 15:44 ` Jason Baron
2010-06-10 16:22 ` Ingo Molnar
2010-06-10 17:11 ` Frederic Weisbecker
2010-06-09 22:36 ` Frederic Weisbecker
2010-06-10 12:06 ` Peter Zijlstra
2010-06-10 12:18 ` Peter Zijlstra
2010-06-09 21:39 ` [PATCH 03/13] jump label v9: x86 support Jason Baron
2010-06-10 12:12 ` Peter Zijlstra
2010-06-10 12:14 ` Ingo Molnar
2010-06-10 13:26 ` Andi Kleen
2010-06-10 14:12 ` Peter Zijlstra
2010-06-10 14:28 ` Andi Kleen
2010-06-10 15:37 ` Ingo Molnar
2010-06-10 16:24 ` Andi Kleen
2010-06-11 8:12 ` Ingo Molnar
2010-06-11 8:30 ` Andi Kleen [this message]
2010-06-10 16:29 ` Steven Rostedt
2010-06-10 15:04 ` Jason Baron
2010-06-10 16:13 ` Mathieu Desnoyers
2010-06-11 0:52 ` Jason Baron
2010-06-11 6:18 ` H. Peter Anvin
2010-06-11 7:58 ` Ingo Molnar
2010-06-10 12:15 ` Peter Zijlstra
2010-06-10 12:33 ` Peter Zijlstra
2010-06-09 21:39 ` [PATCH 04/13] jump label v9: tracepoint support Jason Baron
2010-06-10 12:22 ` Peter Zijlstra
2010-06-09 21:39 ` [PATCH 05/13] jump label v9: add module support Jason Baron
2010-06-09 21:39 ` [PATCH 06/13] jump label v9: move ftrace_dyn_arch_init to common code Jason Baron
2010-06-19 3:24 ` Steven Rostedt
2010-06-09 21:39 ` [PATCH 07/13] jump label v9: sort jump table at build-time Jason Baron
2010-06-09 21:39 ` [PATCH 08/13] jump label v9: initialize workqueue tracepoints *before* they are registered Jason Baron
2010-06-09 21:39 ` [PATCH 09/13] jump label v9: jump_label_text_reserved() to reserve our jump points Jason Baron
2010-06-09 21:39 ` [PATCH 10/13] jump label v9: convert jump label to use a key Jason Baron
2010-06-10 12:38 ` Peter Zijlstra
2010-06-10 12:43 ` Peter Zijlstra
2010-06-10 13:57 ` Jason Baron
2010-06-10 14:46 ` Peter Zijlstra
2010-06-09 21:39 ` [PATCH 11/13] jump label v9: convert dynamic debug to use jump labels Jason Baron
2010-06-09 21:39 ` [PATCH 12/13] jump label v9: sparc64 add jump_label support Jason Baron
2010-06-09 21:39 ` [PATCH 13/13] jump label v9: add docs Jason Baron
2010-06-10 12:49 ` Peter Zijlstra
2010-06-15 3:47 ` [PATCH 00/13] jump label v9 David Miller
2010-06-15 14:28 ` Jason Baron
2010-06-15 15:44 ` Mathieu Desnoyers
2010-06-18 3:45 ` Tony Breeds
2010-06-18 15:18 ` Mathieu Desnoyers
2010-06-15 17:13 ` David Miller
2010-06-15 17:28 ` H. Peter Anvin
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=20100611083028.GE6864@basil.fritz.box \
--to=andi@firstfloor.org \
--cc=arjan@infradead.org \
--cc=avi@redhat.com \
--cc=davem@davemloft.net \
--cc=fweisbec@gmail.com \
--cc=hpa@zytor.com \
--cc=jbaron@redhat.com \
--cc=linux-kernel@vger.kernel.org \
--cc=mathieu.desnoyers@polymtl.ca \
--cc=mhiramat@redhat.com \
--cc=mingo@elte.hu \
--cc=peterz@infradead.org \
--cc=roland@redhat.com \
--cc=rostedt@goodmis.org \
--cc=rth@redhat.com \
--cc=sam@ravnborg.org \
--cc=tglx@linutronix.de \
--cc=torvalds@linux-foundation.org \
--cc=vgoyal@redhat.com \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.