diff options
| author | Paul Mundt <[email protected]> | 2009-09-28 16:55:40 +0900 |
|---|---|---|
| committer | Ingo Molnar <[email protected]> | 2009-10-01 11:53:44 +0200 |
| commit | f9ac5a69edee0ee7e06a05727226e3f275306c8d (patch) | |
| tree | 76e93260f97e7f810faa811965032814ec36d678 | |
| parent | 33974093c024f08caadd2fc71a83bd811ed1831d (diff) | |
kmemtrace: Fix up tracer registration
Commit ddc1637af217dbd8bc51f30e6d24e84476a869a6 ("kmemtrace: Print
binary output only if 'bin' option is set") ended up inverting the
error detection logic. register_tracer() returns 0 on success,
which this change caused to treat as an error, resulting in:
[ 0.132000] Warning: could not register the kmem tracer
as well as bailing out of the initcall with an error value. This
restores the old logic.
Signed-off-by: Paul Mundt <[email protected]>
Acked-by: Pekka Enberg <[email protected]>
Acked-by: Frederic Weisbecker <[email protected]>
Cc: Eduard - Gabriel Munteanu <[email protected]>
Cc: Steven Rostedt <[email protected]>
Cc: Li Zefan <[email protected]>
LKML-Reference: <[email protected]>
Signed-off-by: Ingo Molnar <[email protected]>
| -rw-r--r-- | kernel/trace/kmemtrace.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/kernel/trace/kmemtrace.c b/kernel/trace/kmemtrace.c index 81b1645c8549..a91da69f153a 100644 --- a/kernel/trace/kmemtrace.c +++ b/kernel/trace/kmemtrace.c @@ -501,7 +501,7 @@ static int __init init_kmem_tracer(void) return 1; } - if (!register_tracer(&kmem_tracer)) { + if (register_tracer(&kmem_tracer) != 0) { pr_warning("Warning: could not register the kmem tracer\n"); return 1; } |