diff options
Diffstat (limited to 'src/event')
| -rw-r--r-- | src/event/modules/ngx_iocp_module.c | 17 | ||||
| -rw-r--r-- | src/event/modules/ngx_kqueue_module.c | 34 | ||||
| -rw-r--r-- | src/event/modules/ngx_select_module.c | 32 | ||||
| -rw-r--r-- | src/event/ngx_event.h | 9 | ||||
| -rw-r--r-- | src/event/ngx_event_accept.c | 12 |
5 files changed, 49 insertions, 55 deletions
diff --git a/src/event/modules/ngx_iocp_module.c b/src/event/modules/ngx_iocp_module.c index cd769141d..3a93825e8 100644 --- a/src/event/modules/ngx_iocp_module.c +++ b/src/event/modules/ngx_iocp_module.c @@ -14,7 +14,7 @@ static ngx_int_t ngx_iocp_init(ngx_cycle_t *cycle); static void ngx_iocp_done(ngx_cycle_t *cycle); static ngx_int_t ngx_iocp_add_event(ngx_event_t *ev, int event, u_int key); static ngx_int_t ngx_iocp_del_connection(ngx_connection_t *c, u_int flags); -static ngx_int_t ngx_iocp_process_events(ngx_log_t *log); +static ngx_int_t ngx_iocp_process_events(ngx_cycle_t *cycle); static void *ngx_iocp_create_conf(ngx_cycle_t *cycle); static char *ngx_iocp_init_conf(ngx_cycle_t *cycle, void *conf); @@ -160,7 +160,7 @@ static ngx_int_t ngx_iocp_del_connection(ngx_connection_t *c, u_int flags) } -static ngx_int_t ngx_iocp_process_events(ngx_log_t *log) +static ngx_int_t ngx_iocp_process_events(ngx_cycle_t *cycle) { int rc; u_int key; @@ -179,7 +179,7 @@ static ngx_int_t ngx_iocp_process_events(ngx_log_t *log) timer = INFINITE; } - ngx_log_debug1(NGX_LOG_DEBUG_EVENT, log, 0, "iocp timer: %d", timer); + ngx_log_debug1(NGX_LOG_DEBUG_EVENT, cycle->log, 0, "iocp timer: %d", timer); rc = GetQueuedCompletionStatus(iocp, &bytes, (LPDWORD) &key, (LPOVERLAPPED *) &ovlp, timer); @@ -193,7 +193,7 @@ static ngx_int_t ngx_iocp_process_events(ngx_log_t *log) ngx_gettimeofday(&tv); ngx_time_update(tv.tv_sec); - ngx_log_debug4(NGX_LOG_DEBUG_EVENT, log, 0, + ngx_log_debug4(NGX_LOG_DEBUG_EVENT, cycle->log, 0, "iocp: %d b:%d k:%d ov:" PTR_FMT, rc, bytes, key, ovlp); delta = ngx_elapsed_msec; @@ -202,7 +202,7 @@ static ngx_int_t ngx_iocp_process_events(ngx_log_t *log) if (err) { if (ovlp == NULL) { if (err != WAIT_TIMEOUT) { - ngx_log_error(NGX_LOG_ALERT, log, err, + ngx_log_error(NGX_LOG_ALERT, cycle->log, err, "GetQueuedCompletionStatus() failed"); return NGX_ERROR; @@ -216,14 +216,15 @@ static ngx_int_t ngx_iocp_process_events(ngx_log_t *log) if (timer != INFINITE) { delta = ngx_elapsed_msec - delta; - ngx_log_debug2(NGX_LOG_DEBUG_EVENT, log, 0, + ngx_log_debug2(NGX_LOG_DEBUG_EVENT, cycle->log, 0, "iocp timer: %d, delta: %d", timer, (int) delta); } if (ovlp) { ev = ovlp->event; - ngx_log_debug1(NGX_LOG_DEBUG_EVENT, log, 0, "iocp event:" PTR_FMT, ev); + ngx_log_debug1(NGX_LOG_DEBUG_EVENT, cycle->log, 0, + "iocp event:" PTR_FMT, ev); switch (key) { @@ -244,7 +245,7 @@ static ngx_int_t ngx_iocp_process_events(ngx_log_t *log) ev->available = bytes; - ngx_log_debug1(NGX_LOG_DEBUG_EVENT, log, 0, + ngx_log_debug1(NGX_LOG_DEBUG_EVENT, cycle->log, 0, "iocp event handler: " PTR_FMT, ev->event_handler); ev->event_handler(ev); diff --git a/src/event/modules/ngx_kqueue_module.c b/src/event/modules/ngx_kqueue_module.c index 7a6b5396f..8d37d0187 100644 --- a/src/event/modules/ngx_kqueue_module.c +++ b/src/event/modules/ngx_kqueue_module.c @@ -375,8 +375,8 @@ static ngx_int_t ngx_kqueue_process_events(ngx_cycle_t *cycle) return NGX_ERROR; } -#if 1 - if (ngx_accept_token == 0 && timer == 0) { +#if 0 + if (ngx_accept_mutex_held == 0 && timer == 0) { /* STUB */ timer = 500; } #endif @@ -416,11 +416,7 @@ static ngx_int_t ngx_kqueue_process_events(ngx_cycle_t *cycle) if (err) { ngx_log_error((err == NGX_EINTR) ? NGX_LOG_INFO : NGX_LOG_ALERT, cycle->log, err, "kevent() failed"); - - if (ngx_accept_token) { - *ngx_accept_mutex = 0; - } - + ngx_accept_mutex_unlock(); return NGX_ERROR; } @@ -434,21 +430,13 @@ static ngx_int_t ngx_kqueue_process_events(ngx_cycle_t *cycle) if (events == 0) { ngx_log_error(NGX_LOG_ALERT, cycle->log, 0, "kevent() returned no events without timeout"); - - if (ngx_accept_token) { - *ngx_accept_mutex = 0; - } - + ngx_accept_mutex_unlock(); return NGX_ERROR; } } if (ngx_mutex_lock(ngx_posted_events_mutex) == NGX_ERROR) { - - if (ngx_accept_token) { - *ngx_accept_mutex = 0; - } - + ngx_accept_mutex_unlock(); return NGX_ERROR; } @@ -532,7 +520,7 @@ static ngx_int_t ngx_kqueue_process_events(ngx_cycle_t *cycle) #if 0 if (ngx_threaded || ngx_accept_token) { #endif - if (ngx_accept_token) { + if (ngx_accept_mutex_held) { if (ev->accept) { ngx_mutex_unlock(ngx_posted_events_mutex); @@ -540,11 +528,7 @@ static ngx_int_t ngx_kqueue_process_events(ngx_cycle_t *cycle) ev->event_handler(ev); if (ngx_mutex_lock(ngx_posted_events_mutex) == NGX_ERROR) { - - if (ngx_accept_token) { - *ngx_accept_mutex = 0; - } - + ngx_accept_mutex_unlock(); return NGX_ERROR; } @@ -561,9 +545,7 @@ static ngx_int_t ngx_kqueue_process_events(ngx_cycle_t *cycle) ngx_mutex_unlock(ngx_posted_events_mutex); - if (ngx_accept_token) { - *ngx_accept_mutex = 0; - } + ngx_accept_mutex_unlock(); if (timer && delta) { ngx_event_expire_timers((ngx_msec_t) delta); diff --git a/src/event/modules/ngx_select_module.c b/src/event/modules/ngx_select_module.c index 8b675cd08..7fc79ba94 100644 --- a/src/event/modules/ngx_select_module.c +++ b/src/event/modules/ngx_select_module.c @@ -13,7 +13,7 @@ static int ngx_select_init(ngx_cycle_t *cycle); static void ngx_select_done(ngx_cycle_t *cycle); static int ngx_select_add_event(ngx_event_t *ev, int event, u_int flags); static int ngx_select_del_event(ngx_event_t *ev, int event, u_int flags); -static int ngx_select_process_events(ngx_log_t *log); +static int ngx_select_process_events(ngx_cycle_t *cycle); static char *ngx_select_init_conf(ngx_cycle_t *cycle, void *conf); @@ -237,7 +237,7 @@ static int ngx_select_del_event(ngx_event_t *ev, int event, u_int flags) } -static int ngx_select_process_events(ngx_log_t *log) +static int ngx_select_process_events(ngx_cycle_t *cycle) { int i, ready, nready,found; ngx_err_t err; @@ -274,7 +274,7 @@ static int ngx_select_process_events(ngx_log_t *log) } } - ngx_log_debug1(NGX_LOG_DEBUG_EVENT, log, 0, + ngx_log_debug1(NGX_LOG_DEBUG_EVENT, cycle->log, 0, "change max_fd: %d", max_fd); } #endif @@ -283,12 +283,13 @@ static int ngx_select_process_events(ngx_log_t *log) for (i = 0; i < nevents; i++) { ev = event_index[i]; c = ev->data; - ngx_log_debug2(NGX_LOG_DEBUG_EVENT, log, 0, + ngx_log_debug2(NGX_LOG_DEBUG_EVENT, cycle->log, 0, "select event: fd:%d wr:%d", c->fd,ev->write); } #endif - ngx_log_debug1(NGX_LOG_DEBUG_EVENT, log, 0, "select timer: %d", timer); + ngx_log_debug1(NGX_LOG_DEBUG_EVENT, cycle->log, 0, + "select timer: %d", timer); #if (WIN32) ready = select(0, &work_read_fd_set, &work_write_fd_set, NULL, tp); @@ -319,7 +320,7 @@ static int ngx_select_process_events(ngx_log_t *log) deltas = tv.tv_usec / 1000; } - ngx_log_debug2(NGX_LOG_DEBUG_EVENT, log, 0, + ngx_log_debug2(NGX_LOG_DEBUG_EVENT, cycle->log, 0, "select timer: %d, delta: %d", timer, (int) delta); } else { @@ -328,7 +329,7 @@ static int ngx_select_process_events(ngx_log_t *log) ngx_time_update(tv.tv_sec); if (ready == 0) { - ngx_log_error(NGX_LOG_ALERT, log, 0, + ngx_log_error(NGX_LOG_ALERT, cycle->log, 0, "select() returned no events without timeout"); return NGX_ERROR; } @@ -345,12 +346,12 @@ static int ngx_select_process_events(ngx_log_t *log) if (timer) { delta = ngx_elapsed_msec - delta; - ngx_log_debug2(NGX_LOG_DEBUG_EVENT, log, 0, + ngx_log_debug2(NGX_LOG_DEBUG_EVENT, cycle->log, 0, "select timer: %d, delta: %d", timer, (int) delta); } else { if (ready == 0) { - ngx_log_error(NGX_LOG_ALERT, log, 0, + ngx_log_error(NGX_LOG_ALERT, cycle->log, 0, "select() returned no events without timeout"); return NGX_ERROR; } @@ -358,14 +359,15 @@ static int ngx_select_process_events(ngx_log_t *log) #endif /* HAVE_SELECT_CHANGE_TIMEOUT */ - ngx_log_debug1(NGX_LOG_DEBUG_EVENT, log, 0, "select ready %d", ready); + ngx_log_debug1(NGX_LOG_DEBUG_EVENT, cycle->log, 0, + "select ready %d", ready); if (err) { #if (WIN32) - ngx_log_error(NGX_LOG_ALERT, log, err, "select() failed"); + ngx_log_error(NGX_LOG_ALERT, cycle->log, err, "select() failed"); #else ngx_log_error((err == NGX_EINTR) ? NGX_LOG_INFO : NGX_LOG_ALERT, - log, err, "select() failed"); + cycle->log, err, "select() failed"); #endif return NGX_ERROR; } @@ -380,14 +382,14 @@ static int ngx_select_process_events(ngx_log_t *log) if (ev->write) { if (FD_ISSET(c->fd, &work_write_fd_set)) { found = 1; - ngx_log_debug1(NGX_LOG_DEBUG_EVENT, log, 0, + ngx_log_debug1(NGX_LOG_DEBUG_EVENT, cycle->log, 0, "select write %d", c->fd); } } else { if (FD_ISSET(c->fd, &work_read_fd_set)) { found = 1; - ngx_log_debug1(NGX_LOG_DEBUG_EVENT, log, 0, + ngx_log_debug1(NGX_LOG_DEBUG_EVENT, cycle->log, 0, "select read %d", c->fd); } } @@ -423,7 +425,7 @@ static int ngx_select_process_events(ngx_log_t *log) } if (ready != 0) { - ngx_log_error(NGX_LOG_ALERT, log, 0, "select ready != events"); + ngx_log_error(NGX_LOG_ALERT, cycle->log, 0, "select ready != events"); } if (timer && delta) { diff --git a/src/event/ngx_event.h b/src/event/ngx_event.h index 049b7b7d5..69a08cd8a 100644 --- a/src/event/ngx_event.h +++ b/src/event/ngx_event.h @@ -396,8 +396,15 @@ extern ngx_thread_volatile ngx_event_t *ngx_posted_events; #if (NGX_THREADS) extern ngx_mutex_t *ngx_posted_events_mutex; #endif + +extern ngx_atomic_t *ngx_accept_mutex_ptr; extern ngx_atomic_t *ngx_accept_mutex; -extern ngx_uint_t ngx_accept_token; +extern ngx_uint_t ngx_accept_mutex_held; + +#define ngx_accept_mutex_unlock() \ + if (ngx_accept_mutex_held) { \ + *ngx_accept_mutex = 0; \ + } extern int ngx_event_flags; diff --git a/src/event/ngx_event_accept.c b/src/event/ngx_event_accept.c index 36f2cfe38..d1942d581 100644 --- a/src/event/ngx_event_accept.c +++ b/src/event/ngx_event_accept.c @@ -14,8 +14,9 @@ typedef struct { static size_t ngx_accept_log_error(void *data, char *buf, size_t len); +ngx_atomic_t *ngx_accept_mutex_ptr; ngx_atomic_t *ngx_accept_mutex; -ngx_uint_t ngx_accept_token; +ngx_uint_t ngx_accept_mutex_held; void ngx_event_accept(ngx_event_t *ev) @@ -311,23 +312,24 @@ ngx_int_t ngx_trylock_accept_mutex(ngx_cycle_t *cycle) ngx_log_debug0(NGX_LOG_DEBUG_EVENT, cycle->log, 0, "accept mutex locked"); - if (!ngx_accept_token) { + if (!ngx_accept_mutex_held) { if (ngx_enable_accept_events(cycle) == NGX_ERROR) { + *ngx_accept_mutex = 0; return NGX_ERROR; } - ngx_accept_token = 1; + ngx_accept_mutex_held = 1; } return NGX_OK; } - if (ngx_accept_token) { + if (ngx_accept_mutex_held) { if (ngx_disable_accept_events(cycle) == NGX_ERROR) { return NGX_ERROR; } - ngx_accept_token = 0; + ngx_accept_mutex_held = 0; } return NGX_OK; |
