Re: svn commit: r275136 - in head/sys: dev/e1000 dev/ixgbe kern sys

2014-12-01 Thread Steven Hartland
On 01/12/2014 19:17, Jack Vogel wrote: Not taking it personally, in this case I see some style things I don't like, and I'm not at all clear why this is even necessary, what the old way of doing queue config was missing for instance? Having asked the same question on the review I think I can ta

Re: svn commit: r275136 - in head/sys: dev/e1000 dev/ixgbe kern sys

2014-12-01 Thread Jack Vogel
Not taking it personally, in this case I see some style things I don't like, and I'm not at all clear why this is even necessary, what the old way of doing queue config was missing for instance? Thanks Steve, Jack On Mon, Dec 1, 2014 at 11:04 AM, Steven Hartland wrote: > > On 01/12/2014 16:46

Re: svn commit: r275136 - in head/sys: dev/e1000 dev/ixgbe kern sys

2014-12-01 Thread Steven Hartland
On 01/12/2014 16:46, Alfred Perlstein wrote: On Dec 1, 2014, at 8:37 AM, Hans Petter Selasky wrote: Hi, I think you maybe missed a point On 12/01/14 17:31, Alfred Perlstein wrote: Yes that is why it is being done by hand in the probe routine. I think proper thing might be a way to

Re: svn commit: r275136 - in head/sys: dev/e1000 dev/ixgbe kern sys

2014-12-01 Thread Alfred Perlstein
> On Dec 1, 2014, at 8:37 AM, Hans Petter Selasky wrote: > > Hi, > > I think you maybe missed a point > >> On 12/01/14 17:31, Alfred Perlstein wrote: >> >> Yes that is why it is being done by hand in the probe routine. I think >> proper thing might be a way to sort out how to get tunab

Re: svn commit: r275136 - in head/sys: dev/e1000 dev/ixgbe kern sys

2014-12-01 Thread John Baldwin
On Monday, December 01, 2014 07:55:57 AM Alfred Perlstein wrote: > > On Dec 1, 2014, at 7:49 AM, John Baldwin wrote: > >> On Monday, December 01, 2014 07:19:13 AM Alfred Perlstein wrote: > >> John, > >> > >> Will work on a new revision based on feedback. > >> > >> Two things to note however: > >

Re: svn commit: r275136 - in head/sys: dev/e1000 dev/ixgbe kern sys

2014-12-01 Thread Hans Petter Selasky
Hi, I think you maybe missed a point On 12/01/14 17:31, Alfred Perlstein wrote: Yes that is why it is being done by hand in the probe routine. I think proper thing might be a way to sort out how to get tunables to run at a driver load event? Is that possible? All sysctls are tried in

Re: svn commit: r275136 - in head/sys: dev/e1000 dev/ixgbe kern sys

2014-12-01 Thread Alfred Perlstein
> On Dec 1, 2014, at 8:28 AM, Hans Petter Selasky wrote: > >> On 12/01/14 16:56, Alfred Perlstein wrote: >> >> On Dec 1, 2014, at 7:49 AM, Hans Petter Selasky wrote: > On 12/01/14 16:45, Alfred Perlstein wrote: > > Hi, > > It is quite early, actually: >

Re: svn commit: r275136 - in head/sys: dev/e1000 dev/ixgbe kern sys

2014-12-01 Thread Hans Petter Selasky
On 12/01/14 16:56, Alfred Perlstein wrote: On Dec 1, 2014, at 7:49 AM, Hans Petter Selasky wrote: On 12/01/14 16:45, Alfred Perlstein wrote: Hi, It is quite early, actually: SYSINIT(sysctl, SI_SUB_KMEM, SI_ORDER_FIRST, sysctl_register_all, 0); In some parts of the machine independent,

Re: svn commit: r275136 - in head/sys: dev/e1000 dev/ixgbe kern sys

2014-12-01 Thread Jack Vogel
Really, and did I say that I approved, because I do not recall the event? Jack On Mon, Dec 1, 2014 at 8:08 AM, Alfred Perlstein wrote: > Jack you were asked. Please see the review system. > > Sent from my iPhone > > On Dec 1, 2014, at 8:05 AM, Jack Vogel wrote: > > There is no mystery about

Re: svn commit: r275136 - in head/sys: dev/e1000 dev/ixgbe kern sys

2014-12-01 Thread Alfred Perlstein
Jack you were asked. Please see the review system. Sent from my iPhone > On Dec 1, 2014, at 8:05 AM, Jack Vogel wrote: > > There is no mystery about who's drivers these are, and its not like it would > take a lot of effort to figure out ownership and ask us for review. > > Remove this comm

Re: svn commit: r275136 - in head/sys: dev/e1000 dev/ixgbe kern sys

2014-12-01 Thread Jack Vogel
There is no mystery about who's drivers these are, and its not like it would take a lot of effort to figure out ownership and ask us for review. Remove this commit until I have had time to look it over! Jack On Mon, Dec 1, 2014 at 7:56 AM, Alfred Perlstein wrote: > > > > On Dec 1, 2014, at 7:

Re: svn commit: r275136 - in head/sys: dev/e1000 dev/ixgbe kern sys

2014-12-01 Thread Alfred Perlstein
> On Dec 1, 2014, at 7:49 AM, Hans Petter Selasky wrote: > >> On 12/01/14 16:45, Alfred Perlstein wrote: >> >> > Hi, >>> >>> It is quite early, actually: >>> >>> SYSINIT(sysctl, SI_SUB_KMEM, SI_ORDER_FIRST, sysctl_register_all, 0); >>> >>> In some parts of the machine independent, MI, code

Re: svn commit: r275136 - in head/sys: dev/e1000 dev/ixgbe kern sys

2014-12-01 Thread Alfred Perlstein
> On Dec 1, 2014, at 7:49 AM, John Baldwin wrote: > >> On Monday, December 01, 2014 07:19:13 AM Alfred Perlstein wrote: >> John, >> >> Will work on a new revision based on feedback. >> >> Two things to note however: >> >> Already explored the idea of using kernel_sysctlbyname but rejected du

Re: svn commit: r275136 - in head/sys: dev/e1000 dev/ixgbe kern sys

2014-12-01 Thread John Baldwin
On Monday, December 01, 2014 07:19:13 AM Alfred Perlstein wrote: > John, > > Will work on a new revision based on feedback. > > Two things to note however: > > Already explored the idea of using kernel_sysctlbyname but rejected due to > following: > > It makes little sense to have a rw sysctl t

Re: svn commit: r275136 - in head/sys: dev/e1000 dev/ixgbe kern sys

2014-12-01 Thread Hans Petter Selasky
On 12/01/14 16:45, Alfred Perlstein wrote: > Hi, It is quite early, actually: SYSINIT(sysctl, SI_SUB_KMEM, SI_ORDER_FIRST, sysctl_register_all, 0); In some parts of the machine independent, MI, code you neee to keep the TUNABLE_FETCH'es, because its run before SI_SUB_KMEM ! Then it will n

Re: svn commit: r275136 - in head/sys: dev/e1000 dev/ixgbe kern sys

2014-12-01 Thread Alfred Perlstein
> On Dec 1, 2014, at 7:43 AM, Hans Petter Selasky wrote: > >> On 12/01/14 16:39, Alfred Perlstein wrote: >> >> On Dec 1, 2014, at 7:34 AM, Hans Petter Selasky wrote: On 12/01/14 16:29, Alfred Perlstein wrote: >> On Dec 1, 2014, at 7:26 AM, Hans Petter Selasky

Re: svn commit: r275136 - in head/sys: dev/e1000 dev/ixgbe kern sys

2014-12-01 Thread Hans Petter Selasky
On 12/01/14 16:39, Alfred Perlstein wrote: On Dec 1, 2014, at 7:34 AM, Hans Petter Selasky wrote: On 12/01/14 16:29, Alfred Perlstein wrote: On Dec 1, 2014, at 7:26 AM, Hans Petter Selasky wrote: On 12/01/14 16:19, Alfred Perlstein wrote: It makes little sense to have a rw sysctl that

Re: svn commit: r275136 - in head/sys: dev/e1000 dev/ixgbe kern sys

2014-12-01 Thread Alfred Perlstein
> On Dec 1, 2014, at 7:34 AM, Hans Petter Selasky wrote: > >> On 12/01/14 16:29, Alfred Perlstein wrote: >> >> On Dec 1, 2014, at 7:26 AM, Hans Petter Selasky wrote: On 12/01/14 16:19, Alfred Perlstein wrote: It makes little sense to have a rw sysctl that only takes effe

Re: svn commit: r275136 - in head/sys: dev/e1000 dev/ixgbe kern sys

2014-12-01 Thread Hans Petter Selasky
On 12/01/14 16:29, Alfred Perlstein wrote: On Dec 1, 2014, at 7:26 AM, Hans Petter Selasky wrote: On 12/01/14 16:19, Alfred Perlstein wrote: It makes little sense to have a rw sysctl that only takes effect "some times". This violates POLA at the expense of making code appear cleaner. Expec

Re: svn commit: r275136 - in head/sys: dev/e1000 dev/ixgbe kern sys

2014-12-01 Thread Alfred Perlstein
> On Dec 1, 2014, at 7:26 AM, Hans Petter Selasky wrote: > >> On 12/01/14 16:19, Alfred Perlstein wrote: >> It makes little sense to have a rw sysctl that only takes effect "some >> times". This violates POLA at the expense of making code appear cleaner. >> Expectation is that writable sysctl

Re: svn commit: r275136 - in head/sys: dev/e1000 dev/ixgbe kern sys

2014-12-01 Thread Hans Petter Selasky
On 12/01/14 16:19, Alfred Perlstein wrote: It makes little sense to have a rw sysctl that only takes effect "some times". This violates POLA at the expense of making code appear cleaner. Expectation is that writable sysctls take Hi, I think you are missing a new feature in 11-current, that i

Re: svn commit: r275136 - in head/sys: dev/e1000 dev/ixgbe kern sys

2014-12-01 Thread Alfred Perlstein
John, Will work on a new revision based on feedback. Two things to note however: Already explored the idea of using kernel_sysctlbyname but rejected due to following: It makes little sense to have a rw sysctl that only takes effect "some times". This violates POLA at the expense of making co

Re: svn commit: r275136 - in head/sys: dev/e1000 dev/ixgbe kern sys

2014-12-01 Thread John Baldwin
On Wednesday, November 26, 2014 08:19:36 PM Alfred Perlstein wrote: > Author: alfred > Date: Wed Nov 26 20:19:36 2014 > New Revision: 275136 > URL: https://svnweb.freebsd.org/changeset/base/275136 > > Log: > Make igb and ixgbe check tunables at probe time. > > This allows one to make a kernel

svn commit: r275136 - in head/sys: dev/e1000 dev/ixgbe kern sys

2014-11-26 Thread Alfred Perlstein
Author: alfred Date: Wed Nov 26 20:19:36 2014 New Revision: 275136 URL: https://svnweb.freebsd.org/changeset/base/275136 Log: Make igb and ixgbe check tunables at probe time. This allows one to make a kernel module to tune the number of queues before the driver loads. This is needed