Re: passive references

[Kengo NAKAHARA <k-nakahara%iij.ad.jp@localhost>]

Hi riastradh@n.o

On 2016/02/11 3:01, Taylor R Campbell wrote:
>    Date: Wed, 10 Feb 2016 17:20:15 +0900
>    From: Kengo NAKAHARA <k-nakahara%iij.ad.jp@localhost>
> 
>    I tried to use passive reference for a encaptab list itself instead of
>    pserialize_read_enter().
>    # As you know, packet processing path encap4_lookup() can sleep at
>    # ep->func.
> 
>    I try to reuse psref for the encaptab list itself, however that must not
>    be a normal usage, I see.
> 
> I think you can interleave pserialize read sections and passive
> references like so:

Thank you very match for your design.

I implement gif(4) softint fix code suggested by your design using
passive reference. Here is the patch include your passive reference
implement(with tiny fixes).
    http://netbsd.org/~knakahara/fix-gif-softint/fix-gif-softint-using-psref.patch

Could you comment this patch?


BTW, I think psref utilities to use KASSERT(like mutex_owned) would be
useful for debug. However, I have no idea to implement it lightly...


Thanks,

-- 
//////////////////////////////////////////////////////////////////////
Internet Initiative Japan Inc.

Device Engineering Section,
Core Product Development Department,
Product Division,
Technology Unit

Kengo NAKAHARA <k-nakahara%iij.ad.jp@localhost>