On Sat, Mar 26, 2011 at 04:10:35PM +0000, Jason McIntyre wrote: > On Wed, Mar 23, 2011 at 10:51:34AM +1300, [email protected] wrote: > > > > # cvs -d [email protected]:/cvs diff -u bge.4 > > Index: bge.4 > > =================================================================== > > RCS file: /cvs/src/share/man/man4/bge.4,v > > retrieving revision 1.50 > > diff -u -r1.50 bge.4 > > --- bge.4 9 Jul 2010 07:10:55 -0000 1.50 > > +++ bge.4 22 Mar 2011 18:57:36 -0000 > > @@ -193,10 +193,7 @@ > > A fatal initialization error has occurred. > > .It "bge%d: couldn't map interrupt" > > A fatal initialization error has occurred. > > -.It "bge%d: no memory for jumbo buffer queue!" > > -The driver failed to allocate memory for jumbo frames during > > -initialization. > > -.It "bge%d: watchdog timeout" > > +.It "bge%d: watchdog timeout -- resetting" > > The device has stopped responding to the network, or there is a problem > > with > > the network connection (cable). > > .El > > > > stuart has already removed the jumbo buffer diagnostic. that leaves the > watchdog timeout... > > digging further in src/sys: > > /usr/src/sys/dev/ic/ti.c: printf("%s: watchdog timeout -- resetting\n", > sc ->sc_dv.dv_xname); > /usr/src/sys/dev/pci/if_ix.c: printf("%s: Watchdog timeout -- resetting\n", > if p->if_xname); > /usr/src/sys/dev/pci/if_bge.c: printf("%s: watchdog timeout -- resetting\n", > sc ->bge_dev.dv_xname); > /usr/src/sys/dev/pci/if_em.c: printf("%s: watchdog timeout -- resetting\n", > sc ->sc_dv.dv_xname); > /usr/src/sys/dev/pci/if_ixgb.c: printf("%s: watchdog timeout -- resetting\n", > sc ->sc_dv.dv_xname); > > of these devices, only ti(4) and bge(4) have DIAGNOSTICS sections. both > omit "-- resetting". > > so, would any developer like to comment on whether there is a reason for > it being documented this way, or whether we need to change the docs or > not? > > jmc
no one commented, so i went ahead and added "-- resetting" to both the ti(4) and bge(4) manuals. thanks for your mail, jmc
