On 06/17/10 04:19 PM, Dave Miner wrote:
I'd appreciate a review of the fix at

http://cr.opensolaris.org/~dminer/slim_6936371/

6936371 install fails if OBP lists multiple paths for boot device

Testing on the originally broken system is not entirely complete due to some configuration issues that caused the install to fail later in the ICT's (this one completed without the error seen in the CR) and it won't be pushed until that's been verified as fixed. I've already verified that it doesn't regress on other SPARC systems, at least.

Dave
_______________________________________________
caiman-discuss mailing list
[email protected]
http://mail.opensolaris.org/mailman/listinfo/caiman-discuss


Hey Dave,

One comment.

1917             new_oprom_size, new_dev = struct.unpack('I%ds' % oprommaxparam,

Looks as if new_oprom_size is no longer needed. Please confirm and if it is not 
it could be removed.

Joe


_______________________________________________
caiman-discuss mailing list
[email protected]
http://mail.opensolaris.org/mailman/listinfo/caiman-discuss

Reply via email to