I've updated the webrev with a TODO. To track this, I can file another defect to track removal of this code. That defect will hinge on 617643.
-ethan Virginia Wray wrote: > Hi Ethan - > > The changes look ok to me also. Evan echoed my thought, which > was...how will the fix for 617643 be tracked and then implemented. > g > > On 10/27/08 11:07, Evan Layton wrote: >> 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 >>> >> >> _______________________________________________ >> caiman-discuss mailing list >> caiman-discuss at opensolaris.org >> http://mail.opensolaris.org/mailman/listinfo/caiman-discuss >> > > > -- > > Ginnie > > > > > > >
