[illumos-Developer] webrev(b): 1282 nsmb_open has a race

Gordon Ross gordon.w.ross at gmail.com
Fri Jul 29 15:19:52 PDT 2011


>
> The parts related to ensuring that only one instance attaches is good.
>
> The way that minor numbers are being handled seems to work and should be
> fine. You might want to consider using a vmem arena instead. This is what
> I've been using for pseudo-drivers with equivalent semantics and what
> dtrace(7D) does.
> http://src.illumos.org/source/xref/illumos-gate/usr/src/uts/common/dtrace/dtrace.c#12222
> for reference.
>
> Robert

Thanks.  I'll consider that in the future.  BTW, the current code is from:
http://bugs.ec.illumos.org:8080/source/xref/illumos-gate/usr/src/uts/common/fs/zfs/zfs_ioctl.c#4775

Gordon



More information about the Developer mailing list