cvs commit: src/sys/dev/netif/nfe if_nfe.c if_nfevar.h

Sepherosa Ziehau sepherosa at gmail.com
Wed Jul 9 05:20:52 PDT 2008


On Wed, Jul 9, 2008 at 8:08 PM, Thomas Nikolajsen
<thomas.nikolajsen at mail.dk> wrote:
> There is a minor inconsistency in nfe:
>  - sysctl node is named 'imtimer' (Interrupt moderation time)
>  - loader tunable is named 'imtime'
>  - program variable is named 'nfe_imtime'
>
> I think the normal pattern is that sysctl node and loader tunable has same name.
>
> Name of sysctl node is user visible, so maybe this should be new name for the two other.
>
> If no objection I will make change;
> will also update nfe.4.

Please do :)

Thanks,
sephe

-- 
Live Free or Die





More information about the Commits mailing list