# HG changeset patch # User Igor Sysoev # Date 1040713797 0 # Node ID 2ffaa35fba425e7130946ed67b4a099851f971dc # Parent 9fe40e51d4a3bdad61bb4a996c4182645de6ca13 nginx-0.0.1-2002-12-24-10:09:57 import diff --git a/src/event/modules/ngx_kqueue_module.c b/src/event/modules/ngx_kqueue_module.c --- a/src/event/modules/ngx_kqueue_module.c +++ b/src/event/modules/ngx_kqueue_module.c @@ -61,7 +61,6 @@ int ngx_kqueue_add_event(ngx_event_t *ev { ev->oneshot = (flags & NGX_ONESHOT_EVENT) ? 1: 0; -#if 1 if (nchanges > 0 && ev->index < nchanges && change_list[ev->index].udata == ev) @@ -75,7 +74,6 @@ int ngx_kqueue_add_event(ngx_event_t *ev return NGX_OK; } -#endif return ngx_kqueue_set_event(ev, event, EV_ADD | flags); } @@ -140,10 +138,7 @@ int ngx_kqueue_set_event(ngx_event_t *ev change_list[nchanges].data = 0; change_list[nchanges].udata = ev; -#if 0 - if (flags == EV_ADD) -#endif - ev->index = nchanges; + ev->index = nchanges; nchanges++; @@ -264,8 +259,10 @@ void ngx_kqueue_add_timer(ngx_event_t *e { ngx_event_t *e; - ngx_log_debug(ev->log, "set timer: %d" _ timer); - +#if (NGX_DEBUG) + ngx_connection_t *c = (ngx_connection_t *) ev->data; + ngx_log_debug(ev->log, "set timer: %d:%d" _ c->fd _ timer); +#endif ngx_assert((!ev->timer_next && !ev->timer_prev), return, ev->log, "timer already set"); diff --git a/src/event/modules/ngx_poll_module.c b/src/event/modules/ngx_poll_module.c --- a/src/event/modules/ngx_poll_module.c +++ b/src/event/modules/ngx_poll_module.c @@ -190,7 +190,7 @@ int ngx_poll_process_events(ngx_log_t *l } } - for (i = 0; ready; i++) { + for (i = 0; i < nevents && ready; i++) { c = &ngx_connections[event_list[i].fd]; ngx_log_debug(log, "poll: fd:%d, ev:%d, rev:%d" _ @@ -245,6 +245,8 @@ int ngx_poll_process_events(ngx_log_t *l } } + ngx_assert((ready == 0), /* void */ ; , log, "poll ready != nevents"); + return NGX_OK; } diff --git a/src/event/modules/ngx_select_module.c b/src/event/modules/ngx_select_module.c --- a/src/event/modules/ngx_select_module.c +++ b/src/event/modules/ngx_select_module.c @@ -81,6 +81,12 @@ int ngx_select_add_event(ngx_event_t *ev ngx_log_debug(ev->log, "select fd:%d event:%d" _ c->fd _ event); + if (ev->index != NGX_INVALID_INDEX) { + ngx_log_error(NGX_LOG_ALERT, ev->log, 0, + "%d:%d is already set", c->fd, event); + return NGX_OK; + } + #if (WIN32) if ((event == NGX_READ_EVENT) && (max_read >= FD_SETSIZE) || (event == NGX_WRITE_EVENT) && (max_write >= FD_SETSIZE)) @@ -202,7 +208,7 @@ int ngx_select_process_events(ngx_log_t for (i = 0; i < nevents; i++) { ev = event_index[i]; c = (ngx_connection_t *) ev->data; - ngx_log_debug(log, "select: %d" _ c->fd); + ngx_log_debug(log, "select: %d:%d" _ c->fd _ ev->write); } #endif @@ -305,6 +311,13 @@ void ngx_select_add_timer(ngx_event_t *e { ngx_event_t *e; +#if (NGX_DEBUG) + ngx_connection_t *c = (ngx_connection_t *) ev->data; + ngx_log_debug(ev->log, "set timer: %d:%d" _ c->fd _ timer); +#endif + ngx_assert((!ev->timer_next && !ev->timer_prev), return, ev->log, + "timer already set"); + for (e = timer_queue.timer_next; e != &timer_queue && timer > e->timer_delta; e = e->timer_next) diff --git a/src/event/ngx_event.c b/src/event/ngx_event.c --- a/src/event/ngx_event.c +++ b/src/event/ngx_event.c @@ -116,6 +116,7 @@ void ngx_pre_thread(ngx_array_t *ls, ngx ngx_read_events[fd].data = &ngx_connections[fd]; ngx_read_events[fd].event_handler = &ngx_event_accept; ngx_read_events[fd].listening = 1; + ev->index = NGX_INVALID_INDEX; ngx_read_events[fd].available = 0; diff --git a/src/event/ngx_event.h b/src/event/ngx_event.h --- a/src/event/ngx_event.h +++ b/src/event/ngx_event.h @@ -161,16 +161,23 @@ NGX_CLOSE_EVENT kqueue: kqueu static void ngx_inline ngx_del_timer(ngx_event_t *ev) { - if (ev->timer_prev) +#if (NGX_DEBUG) + ngx_log_debug(ev->log, "del timer: %d" _ *(int *)(ev->data)); +#endif + + if (ev->timer_prev) { ev->timer_prev->timer_next = ev->timer_next; + } if (ev->timer_next) { + ev->timer_next->timer_delta += ev->timer_delta; ev->timer_next->timer_prev = ev->timer_prev; ev->timer_next = NULL; } - if (ev->timer_prev) + if (ev->timer_prev) { ev->timer_prev = NULL; + } } diff --git a/src/event/ngx_event_close.c b/src/event/ngx_event_close.c --- a/src/event/ngx_event_close.c +++ b/src/event/ngx_event_close.c @@ -18,6 +18,9 @@ int ngx_event_close_connection(ngx_event ngx_destroy_pool(c->pool); + ngx_del_timer(c->read); + ngx_del_timer(c->write); + ngx_del_event(c->read, NGX_READ_EVENT, NGX_CLOSE_EVENT); ngx_del_event(c->write, NGX_WRITE_EVENT, NGX_CLOSE_EVENT); diff --git a/src/http/ngx_http.c b/src/http/ngx_http.c --- a/src/http/ngx_http.c +++ b/src/http/ngx_http.c @@ -33,6 +33,7 @@ int ngx_http_init(ngx_pool_t *pool, ngx_ #else ngx_http_server.doc_root = "/home/is/dox/"; ngx_http_server.doc_root = "/home/is/work/xml/site-1.0.0/html"; + ngx_http_server.doc_root = "/spool/test/lperltk"; #endif ngx_http_server.doc_root_len = strlen(ngx_http_server.doc_root) + 1; diff --git a/src/http/ngx_http_core.c b/src/http/ngx_http_core.c --- a/src/http/ngx_http_core.c +++ b/src/http/ngx_http_core.c @@ -50,7 +50,7 @@ int ngx_http_handler(ngx_http_request_t r->connection->unexpected_eof = 0; r->lingering_close = 1; - r->keepalive = 1; + r->keepalive = 0; #if 1 r->filter = NGX_HTTP_FILTER_NEED_IN_MEMORY; diff --git a/src/http/ngx_http_event.c b/src/http/ngx_http_event.c --- a/src/http/ngx_http_event.c +++ b/src/http/ngx_http_event.c @@ -695,14 +695,16 @@ static int ngx_http_set_lingering_close( ngx_del_timer(r->connection->read); ngx_add_timer(r->connection->read, r->server->lingering_timeout); + if (r->connection->read->blocked) { + if (ngx_add_event(r->connection->read, NGX_READ_EVENT, #if (HAVE_CLEAR_EVENT) - if (ngx_add_event(r->connection->read, NGX_READ_EVENT, - NGX_CLEAR_EVENT) == NGX_ERROR) { + NGX_CLEAR_EVENT) == NGX_ERROR) #else - if (ngx_add_event(r->connection->read, NGX_READ_EVENT, - NGX_ONESHOT_EVENT) == NGX_ERROR) { + NGX_ONESHOT_EVENT) == NGX_ERROR) #endif - return ngx_http_close_request(r); + { + return ngx_http_close_request(r); + } } if (ngx_shutdown_socket(r->connection->fd, NGX_WRITE_SHUTDOWN) == -1) @@ -728,12 +730,14 @@ static int ngx_http_lingering_close_hand ngx_log_debug(ev->log, "http lingering close handler"); - if (ev->timedout) - return NGX_DONE; + if (ev->timedout) { + return ngx_http_close_request(r); + } timer = r->lingering_time - ngx_time(); - if (timer <= 0) - return NGX_DONE; + if (timer <= 0) { + return ngx_http_close_request(r); + } if (r->discarded_buffer == NULL) { if (r->header_in->end - r->header_in->last.mem @@ -743,22 +747,23 @@ static int ngx_http_lingering_close_hand } else { ngx_test_null(r->discarded_buffer, ngx_palloc(c->pool, r->server->discarded_buffer_size), - NGX_ERROR); + ngx_http_close_request(r)); } } n = ngx_event_recv(c, r->discarded_buffer, r->server->discarded_buffer_size); - if (n == NGX_ERROR) - return NGX_ERROR; + ngx_log_debug(ev->log, "lingering read: %d" _ n); - if (n == 0) - return NGX_DONE; + if (n == NGX_ERROR || n == 0) { + return ngx_http_close_request(r); + } timer *= 1000; - if (timer > r->server->lingering_timeout) + if (timer > r->server->lingering_timeout) { timer = r->server->lingering_timeout; + } ngx_del_timer(ev); ngx_add_timer(ev, timer);