diff options
| author | Frank Ch. Eigler <fche@elastic.org> | 2008-05-12 19:35:17 -0400 |
|---|---|---|
| committer | Frank Ch. Eigler <fche@elastic.org> | 2008-05-12 19:35:17 -0400 |
| commit | 49f426d9fd471bbdabb0e7a25525d0884b7e8831 (patch) | |
| tree | 7fc93232804de8f204058c9c0b19654090d0abd8 /tapsets.cxx | |
| parent | 77042bd72b98a29a89ccdb32c6eb69b3705664d4 (diff) | |
| download | systemtap-steved-49f426d9fd471bbdabb0e7a25525d0884b7e8831.tar.gz systemtap-steved-49f426d9fd471bbdabb0e7a25525d0884b7e8831.tar.xz systemtap-steved-49f426d9fd471bbdabb0e7a25525d0884b7e8831.zip | |
Revert "PR6487: extend blacklist with relay/timer subsystem"
This reverts commit e0d0dcf974e2d0898a7f5a05a2a3fdc93ba51fea.
It seems that blacklisting kernel/timer.c is overbroad.
Diffstat (limited to 'tapsets.cxx')
| -rw-r--r-- | tapsets.cxx | 12 |
1 files changed, 2 insertions, 10 deletions
diff --git a/tapsets.cxx b/tapsets.cxx index 67412396..7cdf3c32 100644 --- a/tapsets.cxx +++ b/tapsets.cxx @@ -2425,11 +2425,8 @@ dwarf_query::build_blacklist() // XXX: it would be nice if these blacklisted functions were pulled // in dynamically, instead of being statically defined here. - // Perhaps it could be populated from script files. A - // - // noprobe kernel.function("...")" - // - // construct might do the trick. + // Perhaps it could be populated from script files. A "noprobe + // kernel.function("...")" construct might do the trick. // Most of these are marked __kprobes in newer kernels. We list // them here (anyway) so the translator can block them on older @@ -2500,11 +2497,6 @@ dwarf_query::build_blacklist() blfn += "|.*preempt_count.*"; blfn += "|preempt_schedule"; - // PR 6487, relay/timer interactions - blfile += "|kernel/relay.c"; - blfile += "|kernel/timer.c"; - // blfile += "|kernel/workqueue.c"; - // These functions don't return, so return probes would never be recovered blfn_ret += "do_exit"; // no "|" blfn_ret += "|sys_exit"; |