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

[Steven Hartland]

On 01/12/2014 16:46, Alfred Perlstein wrote:
>
>> On Dec 1, 2014, at 8:37 AM, Hans Petter Selasky <hps at selasky.org> 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 tunables to run at a driver load event?  Is that possible?
>> All sysctls are tried init when they are created, both so-called "static" and "dynamic" ones.
>>
>> If the sysctl is created inside the probe routine and has the tunable flag set, it will get init before the creation is complete, if present in the boot environment.
>>
>> If the sysctl is of a "static" kind, it will be created and initialized when SI_SUB_KMEM is executing!
> I totally understand this. It is in the phabricator review. :)
>
As a more general comment, my personal preference when I ask for review 
is that at least one of the reviewers accepts the final revision before 
I commit, but preferably all that have taken part in the discussion. 
This often takes a bit longer and some times takes a little prodding but 
should be worth it in the long run.

I know I commented on this one but I unfortunately didn't get chance to 
look after changes where made and hence never accepted the revision. Had 
I done so I would have caveat-ed it with it being accepted by Jack or 
other Intel delegate in his absence, so sorry about that Jack.

No one should take this personally, as know this is still new to 
everyone, but it does raise the wider question of who should be counted 
as a "reviewer" from phabric and do we need some additional guidelines 
on this, or even better can it be automated?

     Regards
     Steve