Ah, indeed! Cool; all's well. I read the code-review bottom to top so read
that as (re)boot command.
Happy pushing!
Thank you,
Clay
On Mon, 11 Oct 2010, Dave Miner wrote:
On 10/11/10 04:48 PM, [email protected] wrote:
Hi Dave,
As per my #caiman-discuss question: since the final command is reboot -f
-- -Z $dset for SPARC (and since reboot(1M) doesn't mention the -Z
kernel flag requirement), could you comment around line 182 that the -Z
is a boot(1M) flag, not a reboot(1M) flag? Otherwise, things look
reasonable to me.
The comment at 180 already says -Z is on the boot command, not reboot.
Dave
Thank you,
Clay
On Mon, 11 Oct 2010, Dave Miner wrote:
I need a couple of reviewers for a brief fix for
6990345 failed on first reboot after snv_149 installation
webrev is at
http://cr.opensolaris.org/~dminer/slim_reboot/
Tested on x86 and SPARC (4u and 4v).
Dave
_______________________________________________
caiman-discuss mailing list
[email protected]
http://mail.opensolaris.org/mailman/listinfo/caiman-discuss
_______________________________________________
caiman-discuss mailing list
[email protected]
http://mail.opensolaris.org/mailman/listinfo/caiman-discuss
