cvs commit: src/sys/dev/em if_em.c

John Baldwin jhb at freebsd.org
Tue Aug 22 21:14:30 UTC 2006


On Tuesday 22 August 2006 16:36, M. Warner Losh wrote:
> In message: <20060822101012.GC43494 at rambler-co.ru>
>             Ruslan Ermilov <ru at FreeBSD.org> writes:
> : It's still a watchdog event.  We can make it a separate counter,
> : like watchdog_tx_event, and not add it to oerrors, but still show
> : it in em_print_hw_stats().  It'd be useful to have this statistics
> : available.
> 
> I've noticed some drivers have started to export stats via sysctls in
> the dev tree.  Maybe that's where it should go?
> 
> I've used it for cbb, for example, to report things that are usually
> hidden behind bootverbose (more could be useful here):
> 
> dev.cbb.1.pribus: 6
> dev.cbb.1.secbus: 7
> dev.cbb.1.subbus: 7
> 
> the code is real simple in the init routine to hook up these sysctl to
> a softc variable...

em already has several:

dev.em.0.debug_info: -1
dev.em.0.stats: -1
dev.em.0.rx_int_delay: 0
dev.em.0.tx_int_delay: 66
dev.em.0.rx_abs_int_delay: 66
dev.em.0.tx_abs_int_delay: 66

-- 
John Baldwin


More information about the cvs-src mailing list