Re: Linux + TCP_CORK + IPv6 = Broken [PATCH]

2002-12-05 Thread Colm MacCárthaigh
On Wed, Dec 04, 2002 at 11:05:21PM +, Joe Orton wrote: On Wed, Dec 04, 2002 at 08:21:36AM -0500, Jeff Trawick wrote: Colm MacCárthaigh [EMAIL PROTECTED] writes: On Wed, Dec 04, 2002 at 12:25:49PM +, Colm MacCárthaigh wrote: My tests (and patch) were based on apr and apr-util

Re: Linux + TCP_CORK + IPv6 = Broken [PATCH]

2002-12-05 Thread Colm MacCárthaigh
On Thu, Dec 05, 2002 at 07:16:02AM +, Joe Orton wrote: On Wed, Dec 04, 2002 at 11:38:06PM +, Colm MacCárthaigh wrote: On Wed, Dec 04, 2002 at 11:05:21PM +, Joe Orton wrote: Colm, can you try running the apr/test/sendfile binary with your machines? It was one of the first

Re: Linux + TCP_CORK + IPv6 = Broken [PATCH]

2002-12-05 Thread Colm MacCárthaigh
On Thu, Dec 05, 2002 at 11:30:29AM +, Joe Orton wrote: On Thu, Dec 05, 2002 at 10:54:53AM +, Colm MacCárthaigh wrote: .. telnet [v6addr] 80 GET / HTTP/1.1 Host: madeup.tld observe lack of response Have you verified with tcpdump/ethereal etc that this hang is because

Re: Linux + TCP_CORK + IPv6 = Broken [PATCH]

2002-12-05 Thread Colm MacCárthaigh
On Thu, Dec 05, 2002 at 04:17:53PM +, Joe Orton wrote: To summarize some off-list dicussion - the kernel guys have said that using sendfile on IPv6 sockets may trigger bugs in cards which do hardware TCP checksumming for card/driver/OS combinations which support that. (since the cards have

Re: Linux + TCP_CORK + IPv6 = Broken [PATCH]

2002-12-04 Thread Colm MacCárthaigh
On Wed, Dec 04, 2002 at 12:25:49PM +, Colm MacCárthaigh wrote: My tests (and patch) were based on apr and apr-util from CVS , with the 2.0.43 codebase, because CVS seems broken right now. stupid pre-test patch, here's the real one: Index: configure.in

Re: Linux + TCP_CORK + IPv6 = Broken [PATCH]

2002-12-04 Thread Colm MacCárthaigh
On Wed, Dec 04, 2002 at 08:21:36AM -0500, Jeff Trawick wrote: Colm MacCárthaigh [EMAIL PROTECTED] writes: On Wed, Dec 04, 2002 at 12:25:49PM +, Colm MacCárthaigh wrote: My tests (and patch) were based on apr and apr-util from CVS , with the 2.0.43 codebase, because CVS seems broken