Ethan Quach wrote: > Evan, > > be_utils.c > > 588 - Doesn't seem like you need to check found_default here. You > wouldn't be here unless it was True because of the check on 577. > > 590,594 - Does this assume that the bootfs line is always the next > line under title? If so, needs to be fixed. >
There was a problem in this area however that could have caused a problem. I've address this as well and updated the webrev. -evan > > -ethan > > > Evan Layton wrote: >> I need a review of >> >> http://cr.opensolaris.org/~evanl/snap_945/ >> >> which addresses >> >> 945 libbe: be_list needs better active on reboot checking >> http://defect.opensolaris.org/bz/show_bug.cgi?id=945 >> >> Thanks! >> -evan >> _______________________________________________ >> caiman-discuss mailing list >> caiman-discuss at opensolaris.org >> http://mail.opensolaris.org/mailman/listinfo/caiman-discuss
