[illumos-Developer] Fwd: code review: 910 mountall thinks smbfs filesystems are local (and 911, 913)

[Robert Gordon]

it looks okay to me, i looked at this earlier and had comments 
which no-longer apply. 

Robert. 

On Apr 22, 2011, at 4:31 PM, Gordon Ross wrote:

> Would like one more reviewer on this stuff.
> Anyone have some time?
> 
> 
> 
> ---------- Forwarded message ----------
> From: Gordon Ross <gordon.w.ross at gmail.com>
> Date: Tue, Apr 19, 2011 at 3:34 PM
> Subject: code review: 910 mountall thinks smbfs filesystems are local
> (and 911, 913)
> To: illumos-dev <developer at lists.illumos.org>
> 
> 
> Some more fixes from way back.  Please review:
> 
> 910 mountall thinks smbfs filesystems are local
> http://www.yalms.org/cr/webrev-910/
> 
> 911 smbfs mount command fails during system boot
> http://www.yalms.org/cr/webrev-911/
> 
> 913 dfshares returning failure (1) even when it should return success
> http://www.yalms.org/cr/webrev-913/
> 
> See note 2 on issue 911 for how to test 910/911.
> 
> Thanks,
> Gordon
> 
> _______________________________________________
> Developer mailing list
> Developer at lists.illumos.org
> http://lists.illumos.org/m/listinfo/developer