annotate src/os/unix/ngx_linux_aio_read.c @ 640:eb208e0cf44d NGINX_1_1_4

nginx 1.1.4 *) Feature: the ngx_http_upstream_keepalive module. *) Feature: the "proxy_http_version" directive. *) Feature: the "fastcgi_keep_conn" directive. *) Feature: the "worker_aio_requests" directive. *) Bugfix: if nginx was built --with-file-aio it could not be run on Linux kernel which did not support AIO. *) Bugfix: in Linux AIO error processing. Thanks to Hagai Avrahami. *) Bugfix: reduced memory consumption for long-lived requests. *) Bugfix: the module ngx_http_mp4_module did not support 64-bit MP4 "co64" atom.
author Igor Sysoev <http://sysoev.ru>
date Tue, 20 Sep 2011 00:00:00 +0400
parents f5a8cf31a203
children d0f7a625f27c
Ignore whitespace changes - Everywhere: Within whitespace: At end of lines:
rev   line source
518
86dad910eeb6 nginx 0.8.11
Igor Sysoev <http://sysoev.ru>
parents:
diff changeset
1
86dad910eeb6 nginx 0.8.11
Igor Sysoev <http://sysoev.ru>
parents:
diff changeset
2 /*
86dad910eeb6 nginx 0.8.11
Igor Sysoev <http://sysoev.ru>
parents:
diff changeset
3 * Copyright (C) Igor Sysoev
86dad910eeb6 nginx 0.8.11
Igor Sysoev <http://sysoev.ru>
parents:
diff changeset
4 */
86dad910eeb6 nginx 0.8.11
Igor Sysoev <http://sysoev.ru>
parents:
diff changeset
5
86dad910eeb6 nginx 0.8.11
Igor Sysoev <http://sysoev.ru>
parents:
diff changeset
6
86dad910eeb6 nginx 0.8.11
Igor Sysoev <http://sysoev.ru>
parents:
diff changeset
7 #include <ngx_config.h>
86dad910eeb6 nginx 0.8.11
Igor Sysoev <http://sysoev.ru>
parents:
diff changeset
8 #include <ngx_core.h>
86dad910eeb6 nginx 0.8.11
Igor Sysoev <http://sysoev.ru>
parents:
diff changeset
9 #include <ngx_event.h>
86dad910eeb6 nginx 0.8.11
Igor Sysoev <http://sysoev.ru>
parents:
diff changeset
10
86dad910eeb6 nginx 0.8.11
Igor Sysoev <http://sysoev.ru>
parents:
diff changeset
11
86dad910eeb6 nginx 0.8.11
Igor Sysoev <http://sysoev.ru>
parents:
diff changeset
12 extern int ngx_eventfd;
86dad910eeb6 nginx 0.8.11
Igor Sysoev <http://sysoev.ru>
parents:
diff changeset
13 extern aio_context_t ngx_aio_ctx;
86dad910eeb6 nginx 0.8.11
Igor Sysoev <http://sysoev.ru>
parents:
diff changeset
14
86dad910eeb6 nginx 0.8.11
Igor Sysoev <http://sysoev.ru>
parents:
diff changeset
15
86dad910eeb6 nginx 0.8.11
Igor Sysoev <http://sysoev.ru>
parents:
diff changeset
16 static void ngx_file_aio_event_handler(ngx_event_t *ev);
86dad910eeb6 nginx 0.8.11
Igor Sysoev <http://sysoev.ru>
parents:
diff changeset
17
86dad910eeb6 nginx 0.8.11
Igor Sysoev <http://sysoev.ru>
parents:
diff changeset
18
640
eb208e0cf44d nginx 1.1.4
Igor Sysoev <http://sysoev.ru>
parents: 638
diff changeset
19 static int
518
86dad910eeb6 nginx 0.8.11
Igor Sysoev <http://sysoev.ru>
parents:
diff changeset
20 io_submit(aio_context_t ctx, long n, struct iocb **paiocb)
86dad910eeb6 nginx 0.8.11
Igor Sysoev <http://sysoev.ru>
parents:
diff changeset
21 {
86dad910eeb6 nginx 0.8.11
Igor Sysoev <http://sysoev.ru>
parents:
diff changeset
22 return syscall(SYS_io_submit, ctx, n, paiocb);
86dad910eeb6 nginx 0.8.11
Igor Sysoev <http://sysoev.ru>
parents:
diff changeset
23 }
86dad910eeb6 nginx 0.8.11
Igor Sysoev <http://sysoev.ru>
parents:
diff changeset
24
86dad910eeb6 nginx 0.8.11
Igor Sysoev <http://sysoev.ru>
parents:
diff changeset
25
86dad910eeb6 nginx 0.8.11
Igor Sysoev <http://sysoev.ru>
parents:
diff changeset
26 ssize_t
86dad910eeb6 nginx 0.8.11
Igor Sysoev <http://sysoev.ru>
parents:
diff changeset
27 ngx_file_aio_read(ngx_file_t *file, u_char *buf, size_t size, off_t offset,
86dad910eeb6 nginx 0.8.11
Igor Sysoev <http://sysoev.ru>
parents:
diff changeset
28 ngx_pool_t *pool)
86dad910eeb6 nginx 0.8.11
Igor Sysoev <http://sysoev.ru>
parents:
diff changeset
29 {
640
eb208e0cf44d nginx 1.1.4
Igor Sysoev <http://sysoev.ru>
parents: 638
diff changeset
30 ngx_err_t err;
542
2b9e388c61f1 nginx 0.8.23
Igor Sysoev <http://sysoev.ru>
parents: 518
diff changeset
31 struct iocb *piocb[1];
2b9e388c61f1 nginx 0.8.23
Igor Sysoev <http://sysoev.ru>
parents: 518
diff changeset
32 ngx_event_t *ev;
2b9e388c61f1 nginx 0.8.23
Igor Sysoev <http://sysoev.ru>
parents: 518
diff changeset
33 ngx_event_aio_t *aio;
518
86dad910eeb6 nginx 0.8.11
Igor Sysoev <http://sysoev.ru>
parents:
diff changeset
34
542
2b9e388c61f1 nginx 0.8.23
Igor Sysoev <http://sysoev.ru>
parents: 518
diff changeset
35 if (!ngx_file_aio) {
518
86dad910eeb6 nginx 0.8.11
Igor Sysoev <http://sysoev.ru>
parents:
diff changeset
36 return ngx_read_file(file, buf, size, offset);
86dad910eeb6 nginx 0.8.11
Igor Sysoev <http://sysoev.ru>
parents:
diff changeset
37 }
86dad910eeb6 nginx 0.8.11
Igor Sysoev <http://sysoev.ru>
parents:
diff changeset
38
86dad910eeb6 nginx 0.8.11
Igor Sysoev <http://sysoev.ru>
parents:
diff changeset
39 aio = file->aio;
86dad910eeb6 nginx 0.8.11
Igor Sysoev <http://sysoev.ru>
parents:
diff changeset
40
86dad910eeb6 nginx 0.8.11
Igor Sysoev <http://sysoev.ru>
parents:
diff changeset
41 if (aio == NULL) {
86dad910eeb6 nginx 0.8.11
Igor Sysoev <http://sysoev.ru>
parents:
diff changeset
42 aio = ngx_pcalloc(pool, sizeof(ngx_event_aio_t));
86dad910eeb6 nginx 0.8.11
Igor Sysoev <http://sysoev.ru>
parents:
diff changeset
43 if (aio == NULL) {
86dad910eeb6 nginx 0.8.11
Igor Sysoev <http://sysoev.ru>
parents:
diff changeset
44 return NGX_ERROR;
86dad910eeb6 nginx 0.8.11
Igor Sysoev <http://sysoev.ru>
parents:
diff changeset
45 }
86dad910eeb6 nginx 0.8.11
Igor Sysoev <http://sysoev.ru>
parents:
diff changeset
46
86dad910eeb6 nginx 0.8.11
Igor Sysoev <http://sysoev.ru>
parents:
diff changeset
47 aio->file = file;
86dad910eeb6 nginx 0.8.11
Igor Sysoev <http://sysoev.ru>
parents:
diff changeset
48 aio->fd = file->fd;
86dad910eeb6 nginx 0.8.11
Igor Sysoev <http://sysoev.ru>
parents:
diff changeset
49 aio->event.data = aio;
86dad910eeb6 nginx 0.8.11
Igor Sysoev <http://sysoev.ru>
parents:
diff changeset
50 aio->event.ready = 1;
86dad910eeb6 nginx 0.8.11
Igor Sysoev <http://sysoev.ru>
parents:
diff changeset
51 aio->event.log = file->log;
86dad910eeb6 nginx 0.8.11
Igor Sysoev <http://sysoev.ru>
parents:
diff changeset
52 file->aio = aio;
86dad910eeb6 nginx 0.8.11
Igor Sysoev <http://sysoev.ru>
parents:
diff changeset
53 }
86dad910eeb6 nginx 0.8.11
Igor Sysoev <http://sysoev.ru>
parents:
diff changeset
54
86dad910eeb6 nginx 0.8.11
Igor Sysoev <http://sysoev.ru>
parents:
diff changeset
55 ev = &aio->event;
86dad910eeb6 nginx 0.8.11
Igor Sysoev <http://sysoev.ru>
parents:
diff changeset
56
86dad910eeb6 nginx 0.8.11
Igor Sysoev <http://sysoev.ru>
parents:
diff changeset
57 if (!ev->ready) {
86dad910eeb6 nginx 0.8.11
Igor Sysoev <http://sysoev.ru>
parents:
diff changeset
58 ngx_log_error(NGX_LOG_ALERT, file->log, 0,
86dad910eeb6 nginx 0.8.11
Igor Sysoev <http://sysoev.ru>
parents:
diff changeset
59 "second aio post for \"%V\"", &file->name);
86dad910eeb6 nginx 0.8.11
Igor Sysoev <http://sysoev.ru>
parents:
diff changeset
60 return NGX_AGAIN;
86dad910eeb6 nginx 0.8.11
Igor Sysoev <http://sysoev.ru>
parents:
diff changeset
61 }
86dad910eeb6 nginx 0.8.11
Igor Sysoev <http://sysoev.ru>
parents:
diff changeset
62
86dad910eeb6 nginx 0.8.11
Igor Sysoev <http://sysoev.ru>
parents:
diff changeset
63 ngx_log_debug4(NGX_LOG_DEBUG_CORE, file->log, 0,
86dad910eeb6 nginx 0.8.11
Igor Sysoev <http://sysoev.ru>
parents:
diff changeset
64 "aio complete:%d @%O:%z %V",
86dad910eeb6 nginx 0.8.11
Igor Sysoev <http://sysoev.ru>
parents:
diff changeset
65 ev->complete, offset, size, &file->name);
86dad910eeb6 nginx 0.8.11
Igor Sysoev <http://sysoev.ru>
parents:
diff changeset
66
86dad910eeb6 nginx 0.8.11
Igor Sysoev <http://sysoev.ru>
parents:
diff changeset
67 if (ev->complete) {
86dad910eeb6 nginx 0.8.11
Igor Sysoev <http://sysoev.ru>
parents:
diff changeset
68 ev->active = 0;
86dad910eeb6 nginx 0.8.11
Igor Sysoev <http://sysoev.ru>
parents:
diff changeset
69 ev->complete = 0;
86dad910eeb6 nginx 0.8.11
Igor Sysoev <http://sysoev.ru>
parents:
diff changeset
70
86dad910eeb6 nginx 0.8.11
Igor Sysoev <http://sysoev.ru>
parents:
diff changeset
71 if (aio->res >= 0) {
86dad910eeb6 nginx 0.8.11
Igor Sysoev <http://sysoev.ru>
parents:
diff changeset
72 ngx_set_errno(0);
86dad910eeb6 nginx 0.8.11
Igor Sysoev <http://sysoev.ru>
parents:
diff changeset
73 return aio->res;
86dad910eeb6 nginx 0.8.11
Igor Sysoev <http://sysoev.ru>
parents:
diff changeset
74 }
86dad910eeb6 nginx 0.8.11
Igor Sysoev <http://sysoev.ru>
parents:
diff changeset
75
86dad910eeb6 nginx 0.8.11
Igor Sysoev <http://sysoev.ru>
parents:
diff changeset
76 ngx_set_errno(-aio->res);
638
f5a8cf31a203 nginx 1.1.3
Igor Sysoev <http://sysoev.ru>
parents: 602
diff changeset
77
f5a8cf31a203 nginx 1.1.3
Igor Sysoev <http://sysoev.ru>
parents: 602
diff changeset
78 ngx_log_error(NGX_LOG_CRIT, file->log, ngx_errno,
f5a8cf31a203 nginx 1.1.3
Igor Sysoev <http://sysoev.ru>
parents: 602
diff changeset
79 "aio read \"%s\" failed", file->name.data);
f5a8cf31a203 nginx 1.1.3
Igor Sysoev <http://sysoev.ru>
parents: 602
diff changeset
80
518
86dad910eeb6 nginx 0.8.11
Igor Sysoev <http://sysoev.ru>
parents:
diff changeset
81 return NGX_ERROR;
86dad910eeb6 nginx 0.8.11
Igor Sysoev <http://sysoev.ru>
parents:
diff changeset
82 }
86dad910eeb6 nginx 0.8.11
Igor Sysoev <http://sysoev.ru>
parents:
diff changeset
83
86dad910eeb6 nginx 0.8.11
Igor Sysoev <http://sysoev.ru>
parents:
diff changeset
84 ngx_memzero(&aio->aiocb, sizeof(struct iocb));
86dad910eeb6 nginx 0.8.11
Igor Sysoev <http://sysoev.ru>
parents:
diff changeset
85
86dad910eeb6 nginx 0.8.11
Igor Sysoev <http://sysoev.ru>
parents:
diff changeset
86 aio->aiocb.aio_data = (uint64_t) (uintptr_t) ev;
86dad910eeb6 nginx 0.8.11
Igor Sysoev <http://sysoev.ru>
parents:
diff changeset
87 aio->aiocb.aio_lio_opcode = IOCB_CMD_PREAD;
86dad910eeb6 nginx 0.8.11
Igor Sysoev <http://sysoev.ru>
parents:
diff changeset
88 aio->aiocb.aio_fildes = file->fd;
86dad910eeb6 nginx 0.8.11
Igor Sysoev <http://sysoev.ru>
parents:
diff changeset
89 aio->aiocb.aio_buf = (uint64_t) (uintptr_t) buf;
86dad910eeb6 nginx 0.8.11
Igor Sysoev <http://sysoev.ru>
parents:
diff changeset
90 aio->aiocb.aio_nbytes = size;
86dad910eeb6 nginx 0.8.11
Igor Sysoev <http://sysoev.ru>
parents:
diff changeset
91 aio->aiocb.aio_offset = offset;
86dad910eeb6 nginx 0.8.11
Igor Sysoev <http://sysoev.ru>
parents:
diff changeset
92 aio->aiocb.aio_flags = IOCB_FLAG_RESFD;
86dad910eeb6 nginx 0.8.11
Igor Sysoev <http://sysoev.ru>
parents:
diff changeset
93 aio->aiocb.aio_resfd = ngx_eventfd;
86dad910eeb6 nginx 0.8.11
Igor Sysoev <http://sysoev.ru>
parents:
diff changeset
94
86dad910eeb6 nginx 0.8.11
Igor Sysoev <http://sysoev.ru>
parents:
diff changeset
95 ev->handler = ngx_file_aio_event_handler;
86dad910eeb6 nginx 0.8.11
Igor Sysoev <http://sysoev.ru>
parents:
diff changeset
96
86dad910eeb6 nginx 0.8.11
Igor Sysoev <http://sysoev.ru>
parents:
diff changeset
97 piocb[0] = &aio->aiocb;
86dad910eeb6 nginx 0.8.11
Igor Sysoev <http://sysoev.ru>
parents:
diff changeset
98
640
eb208e0cf44d nginx 1.1.4
Igor Sysoev <http://sysoev.ru>
parents: 638
diff changeset
99 if (io_submit(ngx_aio_ctx, 1, piocb) == 1) {
602
c5122335e41d nginx 0.8.53
Igor Sysoev <http://sysoev.ru>
parents: 542
diff changeset
100 ev->active = 1;
c5122335e41d nginx 0.8.53
Igor Sysoev <http://sysoev.ru>
parents: 542
diff changeset
101 ev->ready = 0;
c5122335e41d nginx 0.8.53
Igor Sysoev <http://sysoev.ru>
parents: 542
diff changeset
102 ev->complete = 0;
c5122335e41d nginx 0.8.53
Igor Sysoev <http://sysoev.ru>
parents: 542
diff changeset
103
518
86dad910eeb6 nginx 0.8.11
Igor Sysoev <http://sysoev.ru>
parents:
diff changeset
104 return NGX_AGAIN;
86dad910eeb6 nginx 0.8.11
Igor Sysoev <http://sysoev.ru>
parents:
diff changeset
105 }
86dad910eeb6 nginx 0.8.11
Igor Sysoev <http://sysoev.ru>
parents:
diff changeset
106
640
eb208e0cf44d nginx 1.1.4
Igor Sysoev <http://sysoev.ru>
parents: 638
diff changeset
107 err = ngx_errno;
518
86dad910eeb6 nginx 0.8.11
Igor Sysoev <http://sysoev.ru>
parents:
diff changeset
108
640
eb208e0cf44d nginx 1.1.4
Igor Sysoev <http://sysoev.ru>
parents: 638
diff changeset
109 if (err == NGX_EAGAIN) {
518
86dad910eeb6 nginx 0.8.11
Igor Sysoev <http://sysoev.ru>
parents:
diff changeset
110 return ngx_read_file(file, buf, size, offset);
86dad910eeb6 nginx 0.8.11
Igor Sysoev <http://sysoev.ru>
parents:
diff changeset
111 }
86dad910eeb6 nginx 0.8.11
Igor Sysoev <http://sysoev.ru>
parents:
diff changeset
112
640
eb208e0cf44d nginx 1.1.4
Igor Sysoev <http://sysoev.ru>
parents: 638
diff changeset
113 ngx_log_error(NGX_LOG_CRIT, file->log, err,
518
86dad910eeb6 nginx 0.8.11
Igor Sysoev <http://sysoev.ru>
parents:
diff changeset
114 "io_submit(\"%V\") failed", &file->name);
86dad910eeb6 nginx 0.8.11
Igor Sysoev <http://sysoev.ru>
parents:
diff changeset
115
640
eb208e0cf44d nginx 1.1.4
Igor Sysoev <http://sysoev.ru>
parents: 638
diff changeset
116 if (err == NGX_ENOSYS) {
542
2b9e388c61f1 nginx 0.8.23
Igor Sysoev <http://sysoev.ru>
parents: 518
diff changeset
117 ngx_file_aio = 0;
518
86dad910eeb6 nginx 0.8.11
Igor Sysoev <http://sysoev.ru>
parents:
diff changeset
118 return ngx_read_file(file, buf, size, offset);
86dad910eeb6 nginx 0.8.11
Igor Sysoev <http://sysoev.ru>
parents:
diff changeset
119 }
86dad910eeb6 nginx 0.8.11
Igor Sysoev <http://sysoev.ru>
parents:
diff changeset
120
86dad910eeb6 nginx 0.8.11
Igor Sysoev <http://sysoev.ru>
parents:
diff changeset
121 return NGX_ERROR;
86dad910eeb6 nginx 0.8.11
Igor Sysoev <http://sysoev.ru>
parents:
diff changeset
122 }
86dad910eeb6 nginx 0.8.11
Igor Sysoev <http://sysoev.ru>
parents:
diff changeset
123
86dad910eeb6 nginx 0.8.11
Igor Sysoev <http://sysoev.ru>
parents:
diff changeset
124
86dad910eeb6 nginx 0.8.11
Igor Sysoev <http://sysoev.ru>
parents:
diff changeset
125 static void
86dad910eeb6 nginx 0.8.11
Igor Sysoev <http://sysoev.ru>
parents:
diff changeset
126 ngx_file_aio_event_handler(ngx_event_t *ev)
86dad910eeb6 nginx 0.8.11
Igor Sysoev <http://sysoev.ru>
parents:
diff changeset
127 {
86dad910eeb6 nginx 0.8.11
Igor Sysoev <http://sysoev.ru>
parents:
diff changeset
128 ngx_event_aio_t *aio;
86dad910eeb6 nginx 0.8.11
Igor Sysoev <http://sysoev.ru>
parents:
diff changeset
129
86dad910eeb6 nginx 0.8.11
Igor Sysoev <http://sysoev.ru>
parents:
diff changeset
130 aio = ev->data;
86dad910eeb6 nginx 0.8.11
Igor Sysoev <http://sysoev.ru>
parents:
diff changeset
131
86dad910eeb6 nginx 0.8.11
Igor Sysoev <http://sysoev.ru>
parents:
diff changeset
132 ngx_log_debug2(NGX_LOG_DEBUG_CORE, ev->log, 0,
86dad910eeb6 nginx 0.8.11
Igor Sysoev <http://sysoev.ru>
parents:
diff changeset
133 "aio event handler fd:%d %V", aio->fd, &aio->file->name);
86dad910eeb6 nginx 0.8.11
Igor Sysoev <http://sysoev.ru>
parents:
diff changeset
134
86dad910eeb6 nginx 0.8.11
Igor Sysoev <http://sysoev.ru>
parents:
diff changeset
135 aio->handler(ev);
86dad910eeb6 nginx 0.8.11
Igor Sysoev <http://sysoev.ru>
parents:
diff changeset
136 }