On Thu, Jun 23, 2011 at 02:46:20PM +0100, Stuart Henderson wrote: > On 2011/06/23 11:59, Rechner-Tester wrote: > > Hi > > > > > > The vr(4) man page is missing a note about vlan support. > > Vlan support was added as of OpenBSD 4.3: > > > > Add VLAN reception support to vr(4). > > <http://openbsd.org/plus43.html> > > We don't list vlan support on the manpage for each driver. > However there are other changes that should be mentioned. > Rx checksum offload support was added, and the longword-alignment > mentioned in BUGS is no longer required on the current chips, > it seems simplest to remove BUGS completely but we could > alternatively keep it and discuss which chips are affected. > > comments/oks?
I think it looks better sticking the checksum bit a bit further down. Index: vr.4 =================================================================== RCS file: /home/cvs/src/share/man/man4/vr.4,v retrieving revision 1.25 diff -u -p -r1.25 vr.4 --- vr.4 13 Mar 2011 21:32:29 -0000 1.25 +++ vr.4 23 Jun 2011 20:37:40 -0000 @@ -64,6 +64,10 @@ They support both 10 and 100Mbps speeds .Pp The .Nm +driver for the VT6105M controller supports IPv4 IP/TCP/UDP receive +checksum offload. +The +.Nm driver additionally supports Wake on LAN (WoL). See .Xr arp 8 @@ -178,14 +182,3 @@ The .Nm driver was written by .An Bill Paul Aq [email protected] . -.Sh BUGS -The -.Nm -driver always copies transmit mbuf chains into longword-aligned -buffers prior to transmission in order to pacify the Rhine chips. -If buffers are not aligned correctly, the chip will round the -supplied buffer address and begin DMAing from the wrong location. -This buffer copying impairs transmit performance on slower systems but can't -be avoided. -On faster machines (e.g., a Pentium II), the performance -impact is much less noticeable. -- This message has been scanned for viruses and dangerous content by MailScanner, and is believed to be clean.
