Hi Ethan,

On Fri, 25 Sep 2009, Ethan Quach wrote:

> Alok,
>
> Just a couple of minor comments ...
>
>
> usr/src/cmd/auto-install/svc/manifest-locator
> ---------------------------------------------
> 242 - Why is the value aimanifest=true to not start auto-installer?
>
>      Can I suggest aimanifest=none instead?

If "aimanifest=" is specified on the grub line, this
property shows up in the device tree as having a value
of "true". That is why you see the check for 
aimanifest=true.

> usr/src/cmd/slim-install/svc/live-fs-root-minimal
> -------------------------------------------------
> 90 - Can we explicitly require $MEDIA_BOOT = true here to
>     do this?

Will do.

Thanks for looking again,
Alok

Reply via email to