Hi Ethan, The changes look fine to me.
One nit/idea on the comments though. Since we're doing this fix as a work around for 6176743 do we want to add a "TODO:" here to state that this needs to be revisited when 6176743 gets fixed? Thanks! -evan Ethan Quach wrote: > Can I get a review one liner (plus comments) > > Webrev: > ------------ > http://cr.opensolaris.org/~equach/webrev.4041 > > > Defect: > ---------- > 4041 Creating a BE with zones fails when an alternate BE is currently > mounted > http://defect.opensolaris.org/bz/show_bug.cgi?id=4041 > > > With this fix, new boot environments can be created while > some other boot environment is mounted with zones. > > > thanks, > -ethan > > _______________________________________________ > caiman-discuss mailing list > caiman-discuss at opensolaris.org > http://mail.opensolaris.org/mailman/listinfo/caiman-discuss
