diff options
author | Linus Torvalds <[email protected]> | 2013-12-27 15:30:58 -0800 |
---|---|---|
committer | H. Peter Anvin <[email protected]> | 2013-12-27 16:58:17 -0800 |
commit | c5fe5d80680e2949ffe102180f5fc6cefc0d145f (patch) | |
tree | ca8b287deb7c2fc0167e31dd08de2f5397a33728 /tools/perf/util/trace-event-scripting.c | |
parent | 661c80192d21269c7fc566f1d547510b0c867677 (diff) |
x86: Replace assembly access_ok() with a C variant
It turns out that the assembly variant doesn't actually produce that
good code, presumably partly because it creates a long dependency
chain with no scheduling, and partly because we cannot get a flags
result out of gcc (which could be fixed with asm goto, but it turns
out not to be worth it.)
The C code allows gcc to schedule and generate multiple (easily
predictable) branches, and as a side benefit we can really optimize
the case where the size is constant.
Link: http://lkml.kernel.org/r/CA%2B55aFzPBdbfKovMT8Edr4SmE2_=%[email protected]
Signed-off-by: H. Peter Anvin <[email protected]>
Diffstat (limited to 'tools/perf/util/trace-event-scripting.c')
0 files changed, 0 insertions, 0 deletions