Re: svn commit: r205104 - in head/sys: dev/xen/netback netinet netinet6

2010-03-16 Thread Michael Tuexen
Hi Randy, shouldn't be the right fix "for now" that we do not offload SCTP checksum computation for IPv6? The point is the semantic of the CSUM_SCTP flag. I think it means "will csum SCTP for IPv4" as all the other flags (CSUM_TCP, CSUM_UDP). With "for now" I mean "until there is a IPv6 checksum

Re: svn commit: r205104 - in head/sys: dev/xen/netback netinet netinet6

2010-03-13 Thread Randall Stewart
Hmm... M_PROTO* good idea ;-) On Mar 13, 2010, at 10:56 PM, Bruce Simpson wrote: On 03/13/10 13:53, Robert N. M. Watson wrote: On Mar 13, 2010, at 1:50 PM, Randall Stewart wrote: I could refactor that this way if you want... it would mean a few more de-ref's and looking to see if its a v4

Re: svn commit: r205104 - in head/sys: dev/xen/netback netinet netinet6

2010-03-13 Thread Bruce Simpson
On 03/13/10 13:53, Robert N. M. Watson wrote: On Mar 13, 2010, at 1:50 PM, Randall Stewart wrote: I could refactor that this way if you want... it would mean a few more de-ref's and looking to see if its a v4 or v6 packet and then doing the proper offset... This is the sort of thing

Re: svn commit: r205104 - in head/sys: dev/xen/netback netinet netinet6

2010-03-13 Thread Robert N. M. Watson
On Mar 13, 2010, at 1:50 PM, Randall Stewart wrote: > did not think of that.. we COULD possible do it another way.. a bit harder > but possible.. i.e. have the delayed sack code actually look into > the mbufs and see if its ipv4 or ipv6.. I thought about doing it > that way but it takes more cycl

Re: svn commit: r205104 - in head/sys: dev/xen/netback netinet netinet6

2010-03-13 Thread Randall Stewart
Hmm. did not think of that.. we COULD possible do it another way.. a bit harder but possible.. i.e. have the delayed sack code actually look into the mbufs and see if its ipv4 or ipv6.. I thought about doing it that way but it takes more cycles ;-o I could refactor that this way if you want..

Re: svn commit: r205104 - in head/sys: dev/xen/netback netinet netinet6

2010-03-13 Thread Robert Watson
On Fri, 12 Mar 2010, Randall Stewart wrote: The proper fix for the delayed SCTP checksum is to have the delayed function take an argument as to the offset to the SCTP header. This allows it to work for V4 and V6. This of course means changing all callers of the function to either pass the