]> git.kaiwu.me - nginx.git/commit
Fixed timeouts with threaded sendfile() and subrequests.
authorMaxim Dounin <mdounin@mdounin.ru>
Fri, 18 Mar 2016 02:04:45 +0000 (05:04 +0300)
committerMaxim Dounin <mdounin@mdounin.ru>
Fri, 18 Mar 2016 02:04:45 +0000 (05:04 +0300)
commit931ce7f02a9c55987ef9a7128882097b89a36540
treea7f7debd059a6608c4731eaa842fac683ff7e3d5
parent07fc7dc7194e9928bf7e2b9dc66bb51ab9563cf3
Fixed timeouts with threaded sendfile() and subrequests.

If a write event happens after sendfile() but before we've got the
sendfile results in the main thread, this write event will be ignored.
And if no more events will happen, the connection will hang.

Removing the events works in the simple cases, but not always, as
in some cases events are added back by an unrelated code.  E.g.,
the upstream module adds write event in the ngx_http_upstream_init()
to track client aborts.

Fix is to use wev->complete instead.  It is now set to 0 before
a sendfile() task is posted, and it is set to 1 once a write event
happens.  If on completion of the sendfile() task wev->complete is 1,
we know that an event happened while we were executing sendfile(), and
the socket is still ready for writing even if sendfile() did not sent
all the data or returned EAGAIN.
src/event/modules/ngx_epoll_module.c
src/os/unix/ngx_linux_sendfile_chain.c