Hi, I need to get a review the following simple fix.
Some background: This bug was caused by the fix for 5221 which introduced the functionality to add a menu.lst file if it was found to be missing. While this is normally a good thing it was also being used by be_list when checking for the default BE in the grub menu, which really doesn't make sense anyway because there isn't a file so no grub default. For be_list when it can't get the default BE from the grub menu it defaults to getting this information from the zpool bootfs property. The problem with be_list creating the menu.lst file is that libtd uses be_list to check for existing BE's but does this before the ICT's run and create the menu.lst file. It was this that caused the problem... 6738 b108 generates the wrong grub menu for text boot and it fails to boot for x86 http://defect.opensolaris.org/bz/show_bug.cgi?id=6738 Webrev: http://cr.opensolaris.org/~evanl/6738 Thanks! -evan
