svn commit: r269975 - in stable/10: . sys/dev/ixgbe

[Adam McDougall]

On 08/14/2014 10:07, Steven Hartland wrote:
> Author: smh
> Date: Thu Aug 14 14:07:05 2014
> New Revision: 269975
> URL: http://svnweb.freebsd.org/changeset/base/269975
> 
> Log:
>   Make the ixgbe tunables now match their sysctl counterparts.
>   
>   Previously the tunables and sysctls had different names for example:
>   hw.ixgbe.enable_aim => hw.ix.enable_aim
>   
>   Anyone using ixgbe tunables should ensure they update /boot/loader.conf.
>   
>   This is a direct commit to stable as the changes to sysctls in head
>   already fix this issue in a different way.
>   
>   Sponsored by:	Multiplay
> Modified: stable/10/sys/dev/ixgbe/ixgbe.c
> ==============================================================================
> --- stable/10/sys/dev/ixgbe/ixgbe.c	Thu Aug 14 13:57:17 2014	(r269974)
> +++ stable/10/sys/dev/ixgbe/ixgbe.c	Thu Aug 14 14:07:05 2014	(r269975)
> @@ -244,18 +244,18 @@ static SYSCTL_NODE(_hw, OID_AUTO, ix, CT
>  ** traffic for that interrupt vector
>  */
>  static int ixgbe_enable_aim = TRUE;
> -TUNABLE_INT("hw.ixgbe.enable_aim", &ixgbe_enable_aim);
> +TUNABLE_INT("hw.ix.enable_aim", &ixgbe_enable_aim);
>  SYSCTL_INT(_hw_ix, OID_AUTO, enable_aim, CTLFLAG_RW, &ixgbe_enable_aim, 0,
>      "Enable adaptive interrupt moderation");

I'm not using ix yet, but could you retain the old tunables in 10 as
hidden ones so configurations do not break?