Date: Tue, 22 Aug 2006 14:36:05 -0600 (MDT) From: "M. Warner Losh" <imp@bsdimp.com> To: ru@freebsd.org Cc: pyunyh@gmail.com, src-committers@freebsd.org, cvs-src@freebsd.org, cvs-all@freebsd.org, glebius@freebsd.org, yongari@freebsd.org Subject: Re: cvs commit: src/sys/dev/em if_em.c Message-ID: <20060822.143605.1102534622.imp@bsdimp.com> In-Reply-To: <20060822101012.GC43494@rambler-co.ru> References: <20060822082237.GC41304@rambler-co.ru> <20060822094452.GJ12848@cdnetworks.co.kr> <20060822101012.GC43494@rambler-co.ru>
next in thread | previous in thread | raw e-mail | index | archive | help
In message: <20060822101012.GC43494@rambler-co.ru>
Ruslan Ermilov <ru@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...
Warner
Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?20060822.143605.1102534622.imp>