comparison src/os/unix/ngx_linux_aio_read.c @ 4129:6903dac6ad19

Fixing Linux AIO syscalls return value handling: syscall(2) uses usual libc convention, it returns -1 on error and sets errno. Obsolete _syscall(2) returns negative value of error. Thanks to Hagai Avrahami.
author Igor Sysoev <igor@sysoev.ru>
date Fri, 16 Sep 2011 12:43:16 +0000
parents 37da005a5808
children d620f497c50f
comparison
equal deleted inserted replaced
4128:4b0b0e77dc3d 4129:6903dac6ad19
14 14
15 15
16 static void ngx_file_aio_event_handler(ngx_event_t *ev); 16 static void ngx_file_aio_event_handler(ngx_event_t *ev);
17 17
18 18
19 static long 19 static int
20 io_submit(aio_context_t ctx, long n, struct iocb **paiocb) 20 io_submit(aio_context_t ctx, long n, struct iocb **paiocb)
21 { 21 {
22 return syscall(SYS_io_submit, ctx, n, paiocb); 22 return syscall(SYS_io_submit, ctx, n, paiocb);
23 } 23 }
24 24
25 25
26 ssize_t 26 ssize_t
27 ngx_file_aio_read(ngx_file_t *file, u_char *buf, size_t size, off_t offset, 27 ngx_file_aio_read(ngx_file_t *file, u_char *buf, size_t size, off_t offset,
28 ngx_pool_t *pool) 28 ngx_pool_t *pool)
29 { 29 {
30 long n; 30 ngx_err_t err;
31 struct iocb *piocb[1]; 31 struct iocb *piocb[1];
32 ngx_event_t *ev; 32 ngx_event_t *ev;
33 ngx_event_aio_t *aio; 33 ngx_event_aio_t *aio;
34 34
35 if (!ngx_file_aio) { 35 if (!ngx_file_aio) {
94 94
95 ev->handler = ngx_file_aio_event_handler; 95 ev->handler = ngx_file_aio_event_handler;
96 96
97 piocb[0] = &aio->aiocb; 97 piocb[0] = &aio->aiocb;
98 98
99 n = io_submit(ngx_aio_ctx, 1, piocb); 99 if (io_submit(ngx_aio_ctx, 1, piocb) == 1) {
100
101 if (n == 1) {
102 ev->active = 1; 100 ev->active = 1;
103 ev->ready = 0; 101 ev->ready = 0;
104 ev->complete = 0; 102 ev->complete = 0;
105 103
106 return NGX_AGAIN; 104 return NGX_AGAIN;
107 } 105 }
108 106
109 n = -n; 107 err = ngx_errno;
110 108
111 if (n == NGX_EAGAIN) { 109 if (err == NGX_EAGAIN) {
112 return ngx_read_file(file, buf, size, offset); 110 return ngx_read_file(file, buf, size, offset);
113 } 111 }
114 112
115 ngx_log_error(NGX_LOG_CRIT, file->log, n, 113 ngx_log_error(NGX_LOG_CRIT, file->log, err,
116 "io_submit(\"%V\") failed", &file->name); 114 "io_submit(\"%V\") failed", &file->name);
117 115
118 if (n == NGX_ENOSYS) { 116 if (err == NGX_ENOSYS) {
119 ngx_file_aio = 0; 117 ngx_file_aio = 0;
120 return ngx_read_file(file, buf, size, offset); 118 return ngx_read_file(file, buf, size, offset);
121 } 119 }
122 120
123 return NGX_ERROR; 121 return NGX_ERROR;