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-stable
mailing list