cvs commit: src/sys/dev/bfe if_bfe.c if_bfereg.h

John Baldwin jhb at freebsd.org
Fri Aug 22 19:36:56 UTC 2008


On Friday 22 August 2008 02:46:55 am Pyun YongHyeon wrote:
> yongari     2008-08-22 06:46:55 UTC
> 
>   FreeBSD src repository
> 
>   Modified files:
>     sys/dev/bfe          if_bfe.c if_bfereg.h 
>   Log:
>   SVN rev 181994 on 2008-08-22 06:46:55Z by yongari
>   
>   Add a new sysctl node 'dev.bfe.N.stats' that shows various MAC
>   counters for Rx/Tx statistics. Various counters in ifnet is also
>   updated with these hardware counters.
>   
>   Tested by:      kib, Gleb Kurtsou gleb.kurtsou at gmail dot com
>                   Ulrich Spoerlein uspoerlein at gmail dot com

I think it's kind of wrong to have sysctl's that print stats in dmesg.  You 
can either have a sysctl that returns a string and uses sbuf's (e.g. 
vm.phys_free) if you want formatting or even better, you can give each stat 
it's own sysctl.  This is more friendly for people to use when they aren't on 
the console (e.g. when you ssh into a box).

-- 
John Baldwin


More information about the cvs-all mailing list