Re: MP-safe ifnet with psz & psref

[Ryota Ozaki <ozaki-r%netbsd.org@localhost>]

Sorry for consecutive patches but one more patch is here:
  http://www.netbsd.org/~ozaki-r/if_get_byindex.diff

The patch implements if_get_byindex that is if_byindex
with psref. Same as if_get it's used together with
if_put. The patch also replaces some if_byindex with it
while some if_byindex are kept because just adding
pserialize is enough.

Any comments?

  ozaki-r


On Wed, May 11, 2016 at 6:53 PM, Ryota Ozaki <ozaki-r%netbsd.org@localhost> wrote:
> http://www.netbsd.org/~ozaki-r/psref-ifnet.diff
>
> One more update. Use IFNET_LOCK to protect whole if_getindex
> that updates global shared objects and retire index_gen_mtx
> that is used only for index_gen that is also updated in
> if_getindex.
>
>   ozaki-r
>
>
> On Wed, May 11, 2016 at 10:03 AM, Ryota Ozaki <ozaki-r%netbsd.org@localhost> wrote:
>> On Wed, May 11, 2016 at 4:18 AM, matthew green <mrg%eterna.com.au@localhost> wrote:
>>> Thor Lancelot Simon writes:
>>>> I do not think you should do any extra work to support kvm grovelling.
>>
>> Actually I do nothing other than leaving the original list
>> as it is. I thinks it's reasonable as what we do now.
>>
>>>
>>> existing tool functionality should not be broken, however.  the
>>> netstat -i groveller code should be either updated to the new
>>> method or converted to a real sysctl, though the current work
>>> around would be ok for now.  simply breaking netstat -i is not
>>> an acceptable solution.
>>
>> Yeah, someone should sweep kvm(3) users in the future...
>>
>> BTW netstat already has a feature to retrieve information via
>> sysctl but its man page warns that it lacks some information
>> compared to kvm(3). Do anyone know what's lost?
>>
>>   ozaki-r