]> git.kaiwu.me - nginx.git/commit
Fixed sendfile in threads (or with aio preload) and subrequests.
authorMaxim Dounin <mdounin@mdounin.ru>
Thu, 3 Mar 2016 18:14:12 +0000 (21:14 +0300)
committerMaxim Dounin <mdounin@mdounin.ru>
Thu, 3 Mar 2016 18:14:12 +0000 (21:14 +0300)
commita6fe512df6111ad79cc04d315dc4bb5b6ef909cb
treec703fccc66fabdb6b2f6df3a06d644d542962b07
parente243a2d6c29c618722d8a49e376ce3e96815821b
Fixed sendfile in threads (or with aio preload) and subrequests.

If sendfile in threads is used, it is possible that multiple
subrequests will trigger multiple ngx_linux_sendfile_thread() calls,
as operations are only serialized in output chain based on r->aio,
that is, on subrequest level.

This resulted in "task #N already active" alerts, in particular, when
running proxy_store.t with "aio threads; sendfile on;".

Fix is to tolerate duplicate calls, with an additional safety check
that the file is the same as previously used.

The same problem also affects "aio on; sendfile on;" on FreeBSD
(previously known as "aio sendfile;"), where aio->preload_handler()
could be called multiple times due to similar reasons, resulting in
"second aio post" alerts.  Fix is the same as well.

It is also believed that similar problems can arise if a filter
calls the next body filter multiple times for some reason.  These are
mostly theoretical though.
src/os/unix/ngx_freebsd_sendfile_chain.c
src/os/unix/ngx_linux_sendfile_chain.c