diff options
author | Martin Schwenke <martin@meltin.net> | 2012-06-07 14:20:13 +1000 |
---|---|---|
committer | Martin Schwenke <martin@meltin.net> | 2012-06-12 15:34:53 +1000 |
commit | 6edf0ecb4386bf5599aab3d9882accde24240ccb (patch) | |
tree | 0f338e71b6448af46b7298a176c9b032f4fc9b8a /ctdb/lib | |
parent | 6c83c0ae9c78f1cae46c682a3b00bac96febf2b9 (diff) | |
download | samba-6edf0ecb4386bf5599aab3d9882accde24240ccb.tar.gz samba-6edf0ecb4386bf5599aab3d9882accde24240ccb.tar.xz samba-6edf0ecb4386bf5599aab3d9882accde24240ccb.zip |
Revert "TEVENT: Add back tracking of long runnig events to the local copy of tevent library"
This reverts commit 5aba53e6adcfcd7edbdac9e30aa5fcba176aca00.
Do this using new tevent trace point callback.
(This used to be ctdb commit 88040778aace229d724de1ba7556aded12e22f86)
Diffstat (limited to 'ctdb/lib')
-rw-r--r-- | ctdb/lib/tevent/tevent_epoll.c | 4 | ||||
-rw-r--r-- | ctdb/lib/tevent/tevent_internal.h | 2 | ||||
-rw-r--r-- | ctdb/lib/tevent/tevent_select.c | 4 | ||||
-rw-r--r-- | ctdb/lib/tevent/tevent_standard.c | 4 | ||||
-rw-r--r-- | ctdb/lib/tevent/tevent_util.c | 52 |
5 files changed, 0 insertions, 66 deletions
diff --git a/ctdb/lib/tevent/tevent_epoll.c b/ctdb/lib/tevent/tevent_epoll.c index 7c09dd7449c..f5a69ddc50c 100644 --- a/ctdb/lib/tevent/tevent_epoll.c +++ b/ctdb/lib/tevent/tevent_epoll.c @@ -242,8 +242,6 @@ static void epoll_change_event(struct epoll_event_context *epoll_ev, struct teve } } -extern pid_t ctdbd_pid; - /* event loop handling using epoll */ @@ -266,9 +264,7 @@ static int epoll_event_loop(struct epoll_event_context *epoll_ev, struct timeval return 0; } - if (getpid() == ctdbd_pid) tevent_before_wait(epoll_ev->ev); ret = epoll_wait(epoll_ev->epoll_fd, events, MAXEVENTS, timeout); - if (getpid() == ctdbd_pid) tevent_after_wait(epoll_ev->ev); if (ret == -1 && errno == EINTR && epoll_ev->ev->signal_events) { if (tevent_common_check_signal(epoll_ev->ev)) { diff --git a/ctdb/lib/tevent/tevent_internal.h b/ctdb/lib/tevent/tevent_internal.h index 707bfab0540..7f1d8766ca9 100644 --- a/ctdb/lib/tevent/tevent_internal.h +++ b/ctdb/lib/tevent/tevent_internal.h @@ -313,5 +313,3 @@ bool tevent_poll_init(void); #ifdef HAVE_EPOLL bool tevent_epoll_init(void); #endif -void tevent_before_wait(struct tevent_context *ev); -void tevent_after_wait(struct tevent_context *ev); diff --git a/ctdb/lib/tevent/tevent_select.c b/ctdb/lib/tevent/tevent_select.c index 6fa38cebc04..51c1dec4a6f 100644 --- a/ctdb/lib/tevent/tevent_select.c +++ b/ctdb/lib/tevent/tevent_select.c @@ -130,8 +130,6 @@ static struct tevent_fd *select_event_add_fd(struct tevent_context *ev, TALLOC_C return fde; } -extern pid_t ctdbd_pid; - /* event loop handling using select() */ @@ -169,9 +167,7 @@ static int select_event_loop_select(struct select_event_context *select_ev, stru return 0; } - if (getpid() == ctdbd_pid) tevent_before_wait(select_ev->ev); selrtn = select(select_ev->maxfd+1, &r_fds, &w_fds, NULL, tvalp); - if (getpid() == ctdbd_pid) tevent_after_wait(select_ev->ev); if (selrtn == -1 && errno == EINTR && select_ev->ev->signal_events) { diff --git a/ctdb/lib/tevent/tevent_standard.c b/ctdb/lib/tevent/tevent_standard.c index 4041b6a9037..534576c108d 100644 --- a/ctdb/lib/tevent/tevent_standard.c +++ b/ctdb/lib/tevent/tevent_standard.c @@ -256,8 +256,6 @@ static void epoll_change_event(struct std_event_context *std_ev, struct tevent_f } } -extern pid_t ctdbd_pid; - /* event loop handling using epoll */ @@ -280,9 +278,7 @@ static int epoll_event_loop(struct std_event_context *std_ev, struct timeval *tv return 0; } - if (getpid() == ctdbd_pid) tevent_before_wait(std_ev->ev); ret = epoll_wait(std_ev->epoll_fd, events, MAXEVENTS, timeout); - if (getpid() == ctdbd_pid) tevent_after_wait(std_ev->ev); if (ret == -1 && errno == EINTR && std_ev->ev->signal_events) { if (tevent_common_check_signal(std_ev->ev)) { diff --git a/ctdb/lib/tevent/tevent_util.c b/ctdb/lib/tevent/tevent_util.c index e3a1605b263..16af8f3b908 100644 --- a/ctdb/lib/tevent/tevent_util.c +++ b/ctdb/lib/tevent/tevent_util.c @@ -105,55 +105,3 @@ bool ev_set_close_on_exec(int fd) #endif return false; } - - -static struct timeval tevent_before_wait_ts; -static struct timeval tevent_after_wait_ts; - -/* - * measure the time difference between multiple arrivals - * to the point where we wait for new events to come in - * - * allows to measure how long it takes to work on a - * event - */ -void tevent_before_wait(struct tevent_context *ev) { - - struct timeval diff; - struct timeval now = tevent_timeval_current(); - - if (!tevent_timeval_is_zero(&tevent_after_wait_ts)) { - diff = tevent_timeval_until(&tevent_after_wait_ts, &now); - if (diff.tv_sec > 3) { - tevent_debug(ev, TEVENT_DEBUG_ERROR, __location__ - " Handling event took %d seconds!", - (int) diff.tv_sec); - } - } - - tevent_before_wait_ts = tevent_timeval_current(); - -} - -/* - * measure how long the select()/epoll() call took - * - * allows to measure how long we are waiting for new events - */ -void tevent_after_wait(struct tevent_context *ev) { - - struct timeval diff; - struct timeval now = tevent_timeval_current(); - - if (!tevent_timeval_is_zero(&tevent_before_wait_ts)) { - diff = tevent_timeval_until(&tevent_before_wait_ts, &now); - if (diff.tv_sec > 3) { - tevent_debug(ev, TEVENT_DEBUG_FATAL, __location__ - " No event for %d seconds!", - (int) diff.tv_sec); - } - } - - tevent_after_wait_ts = tevent_timeval_current(); - -} |