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

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

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 cycles

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

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