Subject: Re: CVS commit: src/sys/netinet
To: None <matt@netbsd.org>
From: YAMAMOTO Takashi <yamt@mwd.biglobe.ne.jp>
List: source-changes
Date: 03/12/2005 01:35:28
hi,

> Module Name:	src
> Committed By:	matt
> Date:		Wed Mar  9 03:39:28 UTC 2005
> 
> Modified Files:
> 	src/sys/netinet: ip_output.c
> 
> Log Message:
> Move all the hardware-assisted checksum/segment offload code together.
> 
> 
> To generate a diff of this commit:
> cvs rdiff -r1.146 -r1.147 src/sys/netinet/ip_output.c

> 			/*
> 			 * The len field is meaningless at this point.
> 			 * Clear it now, since some HW requires this.
> 			 */
> 			ip->ip_len = 0;

what's "some HW"?  wm(4)?

i think it's better to move this into the driver of the "some HW"
because at least rtl8169 seems to need this "meaningless" value and
it's easier to clear the field than calculating it.

YAMAMOTO Takashi