From 6a1cc903bf2db47b7061791b9e878ededcba1aa4 Mon Sep 17 00:00:00 2001 From: Igor Sysoev Date: Thu, 22 May 2003 15:23:47 +0000 Subject: nginx-0.0.1-2003-05-22-19:23:47 import --- src/os/unix/ngx_freebsd_config.h | 10 +- src/os/unix/ngx_freebsd_init.c | 4 +- src/os/unix/ngx_freebsd_sendfile_chain.c | 182 +++++++++++++++++++++++++++++++ src/os/unix/ngx_freebsd_write_chain.c | 182 ------------------------------- src/os/unix/ngx_linux_config.h | 22 +++- src/os/unix/ngx_solaris_config.h | 13 +-- src/os/unix/ngx_writev_chain.c | 6 + 7 files changed, 212 insertions(+), 207 deletions(-) create mode 100644 src/os/unix/ngx_freebsd_sendfile_chain.c delete mode 100644 src/os/unix/ngx_freebsd_write_chain.c (limited to 'src/os') diff --git a/src/os/unix/ngx_freebsd_config.h b/src/os/unix/ngx_freebsd_config.h index f8066e561..73646fa0a 100644 --- a/src/os/unix/ngx_freebsd_config.h +++ b/src/os/unix/ngx_freebsd_config.h @@ -6,22 +6,16 @@ #include /* offsetof */ #include #include -#include #include #include -#include -#include #include -#include -#include -#include +#include #include #include #include +#include #include #include -#include - #include diff --git a/src/os/unix/ngx_freebsd_init.c b/src/os/unix/ngx_freebsd_init.c index aac8ed988..100dfc475 100644 --- a/src/os/unix/ngx_freebsd_init.c +++ b/src/os/unix/ngx_freebsd_init.c @@ -14,7 +14,7 @@ ngx_os_io_t ngx_os_io = { ngx_unix_recv, ngx_readv_chain, NULL, - ngx_freebsd_write_chain, + ngx_freebsd_sendfile_chain, NGX_HAVE_SENDFILE|NGX_HAVE_ZEROCOPY }; @@ -56,7 +56,7 @@ int ngx_os_init(ngx_log_t *log) ngx_freebsd_kern_osreldate, __FreeBSD_version); -#if HAVE_FREEBSD_SENDFILE +#if (HAVE_FREEBSD_SENDFILE) /* The determination of the sendfile() nbytes bug is complex enough. There're two sendfile() syscalls: a new 393 has no bug while diff --git a/src/os/unix/ngx_freebsd_sendfile_chain.c b/src/os/unix/ngx_freebsd_sendfile_chain.c new file mode 100644 index 000000000..fc510b72f --- /dev/null +++ b/src/os/unix/ngx_freebsd_sendfile_chain.c @@ -0,0 +1,182 @@ + +#include + +#include +#include +#include +#include +#include +#include +#include +#include +#include + + +ngx_chain_t *ngx_freebsd_sendfile_chain(ngx_connection_t *c, ngx_chain_t *in) +{ + int rc; + char *prev; + size_t hsize, size; + off_t sent; + struct iovec *iov; + struct sf_hdtr hdtr; + ngx_err_t err; + ngx_array_t header, trailer; + ngx_hunk_t *file; + ngx_chain_t *ce; + + ce = in; + file = NULL; + hsize = 0; + + ngx_init_array(header, c->pool, 10, sizeof(struct iovec), NGX_CHAIN_ERROR); + ngx_init_array(trailer, c->pool, 10, sizeof(struct iovec), NGX_CHAIN_ERROR); + + /* create the header iovec */ + if (ngx_hunk_in_memory_only(ce->hunk)) { + prev = NULL; + iov = NULL; + + /* create the iovec and coalesce the neighbouring chain entries */ + while (ce && ngx_hunk_in_memory_only(ce->hunk)) { + + if (prev == ce->hunk->pos) { + iov->iov_len += ce->hunk->last - ce->hunk->pos; + prev = ce->hunk->last; + + } else { + ngx_test_null(iov, ngx_push_array(&header), NGX_CHAIN_ERROR); + iov->iov_base = ce->hunk->pos; + iov->iov_len = ce->hunk->last - ce->hunk->pos; + prev = ce->hunk->last; + } + + if (ngx_freebsd_sendfile_nbytes_bug) { + hsize += ce->hunk->last - ce->hunk->pos; + } + + ce = ce->next; + } + } + + /* TODO: coalesce the neighbouring file hunks */ + if (ce && (ce->hunk->type & NGX_HUNK_FILE)) { + file = ce->hunk; + ce = ce->next; + } + + /* create the trailer iovec */ + if (ce && ngx_hunk_in_memory_only(ce->hunk)) { + prev = NULL; + iov = NULL; + + /* create the iovec and coalesce the neighbouring chain entries */ + while (ce && ngx_hunk_in_memory_only(ce->hunk)) { + + if (prev == ce->hunk->pos) { + iov->iov_len += ce->hunk->last - ce->hunk->pos; + prev = ce->hunk->last; + + } else { + ngx_test_null(iov, ngx_push_array(&trailer), NGX_CHAIN_ERROR); + iov->iov_base = ce->hunk->pos; + iov->iov_len = ce->hunk->last - ce->hunk->pos; + prev = ce->hunk->last; + } + + ce = ce->next; + } + } + + if (file) { + hdtr.headers = (struct iovec *) header.elts; + hdtr.hdr_cnt = header.nelts; + hdtr.trailers = (struct iovec *) trailer.elts; + hdtr.trl_cnt = trailer.nelts; + + rc = sendfile(file->file->fd, c->fd, file->file_pos, + (size_t) (file->file_last - file->file_pos) + hsize, + &hdtr, &sent, 0); + + if (rc == -1) { + err = ngx_errno; + if (err == NGX_EAGAIN || err == NGX_EINTR) { + ngx_log_error(NGX_LOG_INFO, c->log, err, + "sendfile() sent only %qd bytes", sent); + + } else { + ngx_log_error(NGX_LOG_CRIT, c->log, err, "sendfile() failed"); + return NGX_CHAIN_ERROR; + } + } + +#if (NGX_DEBUG_WRITE_CHAIN) + ngx_log_debug(c->log, "sendfile: %d, @%qd %qd:%d" _ + rc _ file->file_pos _ sent _ + (size_t) (file->file_last - file->file_pos) + hsize); +#endif + + } else { + rc = writev(c->fd, (struct iovec *) header.elts, header.nelts); + + if (rc == -1) { + err = ngx_errno; + if (err == NGX_EAGAIN) { + ngx_log_error(NGX_LOG_INFO, c->log, err, "writev() EAGAIN"); + + } else if (err == NGX_EINTR) { + ngx_log_error(NGX_LOG_INFO, c->log, err, "writev() EINTR"); + + } else { + ngx_log_error(NGX_LOG_CRIT, c->log, err, "writev() failed"); + return NGX_CHAIN_ERROR; + } + } + + sent = rc > 0 ? rc : 0; + } + +#if (NGX_DEBUG_WRITE_CHAIN) + ngx_log_debug(c->log, "sendv: %qd" _ sent); +#endif + + c->sent += sent; + + for (ce = in; ce && sent > 0; ce = ce->next) { + + if (ce->hunk->type & NGX_HUNK_IN_MEMORY) { + size = ce->hunk->last - ce->hunk->pos; + } else { + size = ce->hunk->file_last - ce->hunk->file_pos; + } + + if (sent >= size) { + sent -= size; + + if (ce->hunk->type & NGX_HUNK_IN_MEMORY) { + ce->hunk->pos = ce->hunk->last; + } + + if (ce->hunk->type & NGX_HUNK_FILE) { + ce->hunk->file_pos = ce->hunk->file_last; + } + + continue; + } + + if (ce->hunk->type & NGX_HUNK_IN_MEMORY) { + ce->hunk->pos += sent; + } + + if (ce->hunk->type & NGX_HUNK_FILE) { + ce->hunk->file_pos += sent; + } + + break; + } + + ngx_destroy_array(&trailer); + ngx_destroy_array(&header); + + return ce; +} diff --git a/src/os/unix/ngx_freebsd_write_chain.c b/src/os/unix/ngx_freebsd_write_chain.c deleted file mode 100644 index 0317ade2c..000000000 --- a/src/os/unix/ngx_freebsd_write_chain.c +++ /dev/null @@ -1,182 +0,0 @@ - -#include - -#include -#include -#include -#include -#include -#include -#include -#include -#include - - -ngx_chain_t *ngx_freebsd_write_chain(ngx_connection_t *c, ngx_chain_t *in) -{ - int rc; - char *prev; - size_t hsize, size; - off_t sent; - struct iovec *iov; - struct sf_hdtr hdtr; - ngx_err_t err; - ngx_array_t header, trailer; - ngx_hunk_t *file; - ngx_chain_t *ce; - - ce = in; - file = NULL; - hsize = 0; - - ngx_init_array(header, c->pool, 10, sizeof(struct iovec), NGX_CHAIN_ERROR); - ngx_init_array(trailer, c->pool, 10, sizeof(struct iovec), NGX_CHAIN_ERROR); - - /* create the header iovec */ - if (ngx_hunk_in_memory_only(ce->hunk)) { - prev = NULL; - iov = NULL; - - /* create the iovec and coalesce the neighbouring chain entries */ - while (ce && ngx_hunk_in_memory_only(ce->hunk)) { - - if (prev == ce->hunk->pos) { - iov->iov_len += ce->hunk->last - ce->hunk->pos; - prev = ce->hunk->last; - - } else { - ngx_test_null(iov, ngx_push_array(&header), NGX_CHAIN_ERROR); - iov->iov_base = ce->hunk->pos; - iov->iov_len = ce->hunk->last - ce->hunk->pos; - prev = ce->hunk->last; - } - - if (ngx_freebsd_sendfile_nbytes_bug) { - hsize += ce->hunk->last - ce->hunk->pos; - } - - ce = ce->next; - } - } - - /* TODO: coalesce the neighbouring file hunks */ - if (ce && (ce->hunk->type & NGX_HUNK_FILE)) { - file = ce->hunk; - ce = ce->next; - } - - /* create the trailer iovec */ - if (ce && ngx_hunk_in_memory_only(ce->hunk)) { - prev = NULL; - iov = NULL; - - /* create the iovec and coalesce the neighbouring chain entries */ - while (ce && ngx_hunk_in_memory_only(ce->hunk)) { - - if (prev == ce->hunk->pos) { - iov->iov_len += ce->hunk->last - ce->hunk->pos; - prev = ce->hunk->last; - - } else { - ngx_test_null(iov, ngx_push_array(&trailer), NGX_CHAIN_ERROR); - iov->iov_base = ce->hunk->pos; - iov->iov_len = ce->hunk->last - ce->hunk->pos; - prev = ce->hunk->last; - } - - ce = ce->next; - } - } - - if (file) { - hdtr.headers = (struct iovec *) header.elts; - hdtr.hdr_cnt = header.nelts; - hdtr.trailers = (struct iovec *) trailer.elts; - hdtr.trl_cnt = trailer.nelts; - - rc = sendfile(file->file->fd, c->fd, file->file_pos, - (size_t) (file->file_last - file->file_pos) + hsize, - &hdtr, &sent, 0); - - if (rc == -1) { - err = ngx_errno; - if (err == NGX_EAGAIN || err == NGX_EINTR) { - ngx_log_error(NGX_LOG_INFO, c->log, err, - "sendfile() sent only %qd bytes", sent); - - } else { - ngx_log_error(NGX_LOG_CRIT, c->log, err, "sendfile() failed"); - return NGX_CHAIN_ERROR; - } - } - -#if (NGX_DEBUG_WRITE_CHAIN) - ngx_log_debug(c->log, "sendfile: %d, @%qd %qd:%d" _ - rc _ file->file_pos _ sent _ - (size_t) (file->file_last - file->file_pos) + hsize); -#endif - - } else { - rc = writev(c->fd, (struct iovec *) header.elts, header.nelts); - - if (rc == -1) { - err = ngx_errno; - if (err == NGX_EAGAIN) { - ngx_log_error(NGX_LOG_INFO, c->log, err, "writev() EAGAIN"); - - } else if (err == NGX_EINTR) { - ngx_log_error(NGX_LOG_INFO, c->log, err, "writev() EINTR"); - - } else { - ngx_log_error(NGX_LOG_CRIT, c->log, err, "writev() failed"); - return NGX_CHAIN_ERROR; - } - } - - sent = rc > 0 ? rc : 0; - } - -#if (NGX_DEBUG_WRITE_CHAIN) - ngx_log_debug(c->log, "sendv: %qd" _ sent); -#endif - - c->sent += sent; - - for (ce = in; ce && sent > 0; ce = ce->next) { - - if (ce->hunk->type & NGX_HUNK_IN_MEMORY) { - size = ce->hunk->last - ce->hunk->pos; - } else { - size = ce->hunk->file_last - ce->hunk->file_pos; - } - - if (sent >= size) { - sent -= size; - - if (ce->hunk->type & NGX_HUNK_IN_MEMORY) { - ce->hunk->pos = ce->hunk->last; - } - - if (ce->hunk->type & NGX_HUNK_FILE) { - ce->hunk->file_pos = ce->hunk->file_last; - } - - continue; - } - - if (ce->hunk->type & NGX_HUNK_IN_MEMORY) { - ce->hunk->pos += sent; - } - - if (ce->hunk->type & NGX_HUNK_FILE) { - ce->hunk->file_pos += sent; - } - - break; - } - - ngx_destroy_array(&trailer); - ngx_destroy_array(&header); - - return ce; -} diff --git a/src/os/unix/ngx_linux_config.h b/src/os/unix/ngx_linux_config.h index 69a607ed5..e4c5dd0a3 100644 --- a/src/os/unix/ngx_linux_config.h +++ b/src/os/unix/ngx_linux_config.h @@ -2,24 +2,34 @@ #define _NGX_LINUX_CONFIG_H_INCLUDED_ +#define _XOPEN_SOURCE 500 + + #include #include /* offsetof */ #include #include -#include #include #include -#include #include + +#define __USE_BSD +#include +#undef __USE_BSD + #include -#include -#include -#include +#include +#include #include +#include #include +#include #include #include -#include + + +typedef unsigned int u_int; +typedef unsigned char u_char; #ifndef HAVE_SELECT diff --git a/src/os/unix/ngx_solaris_config.h b/src/os/unix/ngx_solaris_config.h index 1545fc914..786cf0b67 100644 --- a/src/os/unix/ngx_solaris_config.h +++ b/src/os/unix/ngx_solaris_config.h @@ -4,29 +4,24 @@ #define SOLARIS 1 +#define _REENTRANT #define _FILE_OFFSET_BITS 64 /* must be before sys/types.h */ + #include #include /* offsetof */ #include #include -#include #include -#include #include -#include -#include /* bzero() */ +#include + #include #include /* FIONBIO */ #include /* INFTIM */ -#include -#include #include -#include -#include #include #include -#include typedef uint32_t u_int32_t; diff --git a/src/os/unix/ngx_writev_chain.c b/src/os/unix/ngx_writev_chain.c index fdaa52a48..b667bc507 100644 --- a/src/os/unix/ngx_writev_chain.c +++ b/src/os/unix/ngx_writev_chain.c @@ -62,6 +62,8 @@ ngx_chain_t *ngx_writev_chain(ngx_connection_t *c, ngx_chain_t *in) size = ce->hunk->last - ce->hunk->pos; +ngx_log_debug(c->log, "SIZE: %d" _ size); + if (sent >= size) { sent -= size; @@ -69,9 +71,11 @@ ngx_chain_t *ngx_writev_chain(ngx_connection_t *c, ngx_chain_t *in) ce->hunk->pos = ce->hunk->last; } +#if 0 if (ce->hunk->type & NGX_HUNK_FILE) { ce->hunk->file_pos = ce->hunk->file_last; } +#endif continue; } @@ -80,9 +84,11 @@ ngx_chain_t *ngx_writev_chain(ngx_connection_t *c, ngx_chain_t *in) ce->hunk->pos += sent; } +#if 0 if (ce->hunk->type & NGX_HUNK_FILE) { ce->hunk->file_pos += sent; } +#endif break; } -- cgit