I need to get a code review for the following fixes.

webrev:
------------
http://cr.opensolaris.org/~evanl/9594


Bugids:
-----------
http://defect.opensolaris.org/bz/show_bug.cgi?id=8638
http://defect.opensolaris.org/bz/show_bug.cgi?id=9051
http://defect.opensolaris.org/bz/show_bug.cgi?id=9594

Some back ground:
The main bug is 9594 which was an oversight in the original design
where we did not take into account the possibility that the root
pool dataset (/rpool) was not mounted. To fix this we attempt to
mount it at it's normal mount point, if that mount fails we attempt
to mount it on a temporary mount point.

We also discovered we were erroneously checking for error conditions
from libzfs property checking functions such as zfs_prop_get_int().
These functions are expected to allows succeed and do not set the
libzfs error codes. The incorrect error checking was also fixed as
part of this fix.


Thanks,
-evan

Reply via email to