svn commit: r295525 - stable/10/sys/dev/ixgbe

Steven Hartland steven.hartland at multiplay.co.uk
Fri Feb 12 11:27:20 UTC 2016


There's a typo here and the unreverted version is still present in head, 
was this the intention?

Clearly the code does limit it to 8, so the old description is more 
correct, just needs the typo fixing, but I would have expected this to 
have been corrected in head first and then MFC'ed

This was reported to me by Dmitry Luhtionov.

     Regards
     Steve
On 11/02/2016 16:35, Sean Bruno wrote:
> Author: sbruno
> Date: Thu Feb 11 16:35:16 2016
> New Revision: 295525
> URL: https://svnweb.freebsd.org/changeset/base/295525
>
> Log:
>    Revert sysctl description change introduced in r295008 to correctly
>    reflect the maximum number of queues supported (8) by this driver.
>    
>    Submitted by:	jwd
>    Reviewed by:	erj
>    Approved by:	re (gjb)
>    Sponsored by:	Intel Corporation and Limelight Networks
>
> Modified:
>    stable/10/sys/dev/ixgbe/if_ix.c
>
> Modified: stable/10/sys/dev/ixgbe/if_ix.c
> ==============================================================================
> --- stable/10/sys/dev/ixgbe/if_ix.c	Thu Feb 11 16:16:10 2016	(r295524)
> +++ stable/10/sys/dev/ixgbe/if_ix.c	Thu Feb 11 16:35:16 2016	(r295525)
> @@ -320,7 +320,8 @@ SYSCTL_INT(_hw_ix, OID_AUTO, enable_msix
>   static int ixgbe_num_queues = 0;
>   TUNABLE_INT("hw.ix.num_queues", &ixgbe_num_queues);
>   SYSCTL_INT(_hw_ix, OID_AUTO, num_queues, CTLFLAG_RDTUN, &ixgbe_num_queues, 0,
> -    "Number of queues to configure, 0 indicates autoconfigure");
> +    "Number of queues to configure up to a mximum of 8,"
> +    "0 indicates autoconfigure");
>   
>   /*
>   ** Number of TX descriptors per ring,
>



More information about the svn-src-all mailing list