Skip site navigation (1)Skip section navigation (2)
Date:      Wed, 02 Jun 2004 17:04:04 -0600
From:      Scott Long <scottl@freebsd.org>
To:        Maxime Henrion <mux@freebsd.org>
Cc:        cvs-all@freebsd.org
Subject:   Re: cvs commit: src/sys/dev/fxp if_fxp.c if_fxpvar.h
Message-ID:  <40BE5CE4.60304@freebsd.org>
In-Reply-To: <200406022252.i52MqJFp094240@repoman.freebsd.org>
References:  <200406022252.i52MqJFp094240@repoman.freebsd.org>

next in thread | previous in thread | raw e-mail | index | archive | help
Maxime Henrion wrote:
> mux         2004/06/02 15:52:18 PDT
> 
>   FreeBSD src repository
> 
>   Modified files:
>     sys/dev/fxp          if_fxp.c if_fxpvar.h 
>   Log:
>   Use the device sysctl tree instead of rolling our own.  Some of the
>   sysctls were global (hw.fxp_rnr and hw.fxp_noflow), all of them are
>   now per-device.  Sample output of "sysctl dev.fxp0" with this patch,
>   with the standard %foo nodes removed :
>   
>   dev.fxp0.int_delay: 1000
>   dev.fxp0.bundle_max: 6
>   dev.fxp0.rnr: 0
>   dev.fxp0.noflow: 0
>   
>   Revision  Changes    Path
>   1.213     +18 -24    src/sys/dev/fxp/if_fxp.c
>   1.31      +2 -2      src/sys/dev/fxp/if_fxpvar.h

Not having the ability to deal with global settings is a bit
of a pain.  I had envisioned that something like this would
be:

dev.fxp.N.instance_variable

dev.fxp.global_variable

Anyways, this work still looks very nice =-)

Scott



Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?40BE5CE4.60304>