diff options
author | Ben Pfaff <blp@nicira.com> | 2013-07-11 17:02:12 -0700 |
---|---|---|
committer | Ben Pfaff <blp@nicira.com> | 2013-07-11 17:02:12 -0700 |
commit | 94f3e5269e93e85001b078fd475217c997bbba70 (patch) | |
tree | d5f2ce407cc50458ca69bf885ff21969b0a1987d /lib/timeval.h | |
parent | bc7ad7d4740b10d08249b2e8017c766396ff53ce (diff) | |
download | openvswitch-94f3e5269e93e85001b078fd475217c997bbba70.tar.gz |
timeval: Remove backtrace feature.
The backtrace feature of timeval is useful because it provides a "poor
man's profile" view of Open vSwitch. But it is not likely to be useful in
a multithreaded process, because signal delivery doesn't necessarily follow
the profile when there is more than one thread. (A signal in a
multithreaded process are delivered to an arbitrary thread.)
Another problem with the backtrace feature is that it is difficult for
format_backtraces() to synchronize properly with the signal handler in a
multithreaded process. In a single-threaded process, it can just block
the signal handler, but in a multithreaded process this does not prevent
signal delivery to threads other than the one running format_backtrace().
Signed-off-by: Ben Pfaff <blp@nicira.com>
Acked-by: Ed Maste <emaste@freebsd.org>
Diffstat (limited to 'lib/timeval.h')
-rw-r--r-- | lib/timeval.h | 1 |
1 files changed, 0 insertions, 1 deletions
diff --git a/lib/timeval.h b/lib/timeval.h index 7bf8d1f77..8dd2e2b06 100644 --- a/lib/timeval.h +++ b/lib/timeval.h @@ -76,7 +76,6 @@ void xgettimeofday(struct timeval *); void xclock_gettime(clock_t, struct timespec *); int get_cpu_usage(void); -void format_backtraces(struct ds *, size_t min_count); long long int time_boot_msec(void); |