Re: sendfile in darwin

2008-05-14 Thread William A. Rowe, Jr.
Jim Jagielski wrote: Trying to recreate so far, no luck. This was at today's trunk. Again I'd hint, try across the wire with ./sendfile server sitting on yet another box. It's really effective at making these errors pop out at us.

Re: sendfile in darwin

2008-05-14 Thread Jim Jagielski
Trying to recreate so far, no luck. On Wed, May 14, 2008 at 04:06:34PM -0500, William A. Rowe, Jr. wrote: > sendfile client timeout now works locally. > > sendfile client nonblocking test now fails. > > server dies at apr_socket_recv()->70014/End of file found > > client dies at apr_socket_

Re: sendfile in darwin

2008-05-14 Thread William A. Rowe, Jr.
sendfile client timeout now works locally. sendfile client nonblocking test now fails. server dies at apr_socket_recv()->70014/End of file found client dies at apr_socket_sendfile()->70014, sent 0 bytes client problem: sent 227008 of 370049 bytes Jim Jagielski wrote: GOT IT! That was it. Pat

Re: sendfile in darwin

2008-05-14 Thread Jim Jagielski
GOT IT! That was it. Patching apr-trunk and apr-1.3 now. On May 14, 2008, at 2:35 PM, Jim Jagielski wrote: Hmmm I'm thinking it has to do when the value passed to sendfile for *len is 0... doing some checking now.

Re: sendfile in darwin

2008-05-14 Thread Jim Jagielski
Hmmm I'm thinking it has to do when the value passed to sendfile for *len is 0... doing some checking now.

Re: sendfile in darwin

2008-05-14 Thread William A. Rowe, Jr.
Corrupt patch, would you mind resending at a text/plain attach? Jim Jagielski wrote: On May 14, 2008, at 1:52 PM, William A. Rowe, Jr. wrote: Jim Jagielski wrote: I'll have to try that... in the meantime, does re-adding the 'rv=0' line fix it? With flying colors. Yes. But I'm guessing

Re: sendfile in darwin

2008-05-14 Thread Jim Jagielski
On May 14, 2008, at 1:52 PM, William A. Rowe, Jr. wrote: Jim Jagielski wrote: I'll have to try that... in the meantime, does re-adding the 'rv=0' line fix it? With flying colors. Yes. But I'm guessing it dies with the httpd test framework. Can you try with the below patch: Index: netwo

Re: sendfile in darwin

2008-05-14 Thread William A. Rowe, Jr.
Jim Jagielski wrote: I'll have to try that... in the meantime, does re-adding the 'rv=0' line fix it? With flying colors. Yes. Bill

Re: sendfile in darwin

2008-05-14 Thread Jim Jagielski
On May 14, 2008, at 1:12 PM, William A. Rowe, Jr. wrote: Jim Jagielski wrote: I wonder if you are seeing timing issues rather than "real" errors... Confirmed this is not so, and that you can reproduce it. Although your machine is too fast to observe the error (parallel CPU's), ./ sendfil

Re: sendfile in darwin

2008-05-14 Thread William A. Rowe, Jr.
Jim Jagielski wrote: I wonder if you are seeing timing issues rather than "real" errors... Confirmed this is not so, and that you can reproduce it. Although your machine is too fast to observe the error (parallel CPU's), ./sendfile's authors already anticipated that. From another box; ./sen