diff options
| author | ddomingo <ddomingo@redhat.com> | 2009-03-17 15:31:54 +1000 |
|---|---|---|
| committer | ddomingo <ddomingo@redhat.com> | 2009-03-17 15:31:54 +1000 |
| commit | b9315d945fa0700f9a1d8cf9185c66388d29023c (patch) | |
| tree | 8965c368ed7ef5ff8bc835a76fa61b241133f933 /tapsets.cxx | |
| parent | 4e7c048f5184ffcb06bc64d8be6e859156c8af97 (diff) | |
| parent | bdca08879745471fdb86991a8e7276900aaaf066 (diff) | |
| download | systemtap-steved-b9315d945fa0700f9a1d8cf9185c66388d29023c.tar.gz systemtap-steved-b9315d945fa0700f9a1d8cf9185c66388d29023c.tar.xz systemtap-steved-b9315d945fa0700f9a1d8cf9185c66388d29023c.zip | |
Merge branch 'master' of ssh://sources.redhat.com/git/systemtap
Diffstat (limited to 'tapsets.cxx')
| -rw-r--r-- | tapsets.cxx | 11 |
1 files changed, 8 insertions, 3 deletions
diff --git a/tapsets.cxx b/tapsets.cxx index 6efcb3af..2f940b29 100644 --- a/tapsets.cxx +++ b/tapsets.cxx @@ -9770,8 +9770,13 @@ tracepoint_derived_probe_group::emit_module_decls (systemtap_session& s) s.op->newline(1) << "return register_trace_" << p->tracepoint_name << "(enter_tracepoint_probe_" << i << ");"; s.op->newline(-1) << "}"; - s.op->newline() << "static int unregister_tracepoint_probe_" << i << "(void) {"; - s.op->newline(1) << "return unregister_trace_" << p->tracepoint_name + + // NB: we're not prepared to deal with unreg failures. However, failures + // can only occur if the tracepoint doesn't exist (yet?), or if we + // weren't even registered. The former should be OKed by the initial + // registration call, and the latter is safe to ignore. + s.op->newline() << "static void unregister_tracepoint_probe_" << i << "(void) {"; + s.op->newline(1) << "(void) unregister_trace_" << p->tracepoint_name << "(enter_tracepoint_probe_" << i << ");"; s.op->newline(-1) << "}"; s.op->newline(); @@ -9780,7 +9785,7 @@ tracepoint_derived_probe_group::emit_module_decls (systemtap_session& s) // emit an array of registration functions for easy init/shutdown s.op->newline() << "static struct stap_tracepoint_probe {"; s.op->newline(1) << "int (*reg)(void);"; - s.op->newline(0) << "int (*unreg)(void);"; + s.op->newline(0) << "void (*unreg)(void);"; s.op->newline(-1) << "} stap_tracepoint_probes[] = {"; s.op->indent(1); for (unsigned i = 0; i < probes.size(); ++i) |