[Libvir] [PATCH] Solaris dom0 support

[Daniel P. Berrange]

On Fri, Jun 15, 2007 at 02:26:36AM +0100, John Levon wrote:
> On Fri, Jun 15, 2007 at 01:02:16AM +0100, Daniel P. Berrange wrote:
> 
> > > > I'm curious as to what the changes for bootloader / kernel are for ?
> > > > Surely you always have either a bootloader, or a kenrel present in
> > > > the SEXPR ? So I'm not sure why its neccessary to disable the check
> > > 
> > > No, this is not true, and it's not true in Xen too. This is stuff that
> > > got merged up in my pygrub changes.
> > > 
> > > Basically the logic is something like:
> > > 
> > > if there is no kernel specified:
> > >     if there is no bootloader specified:
> > >         default to pygrub (for Solaris, this will fill in
> > >         kernel/ramdisk/extra automatically)
> > 
> > Ah ha - this is the key clause I was missing. I didn't realize that
> > XenD could now default to pygrub. The change in logic makes perfect
> > sense now. Though I wonder if we should add in an explicit element
> > for  <bootloader>/usr/bin/pygrub</bootloader> to reflect this default
> > done by XenD...
> 
> What would be the reason for this?

Well to give some form of indication as to how the guest is being booted.
Perhaps rather than making up a default path, just an empty <bootloader/>
element would work. The semantics being launch with the default bootloader
for the platform. That would avoid having to include any specific path
info

Regards,
Dan.
-- 
|=- Red Hat, Engineering, Emerging Technologies, Boston.  +1 978 392 2496 -=|
|=-           Perl modules: http://search.cpan.org/~danberr/              -=|
|=-               Projects: http://freshmeat.net/~danielpb/               -=|
|=-  GnuPG: 7D3B9505   F3C9 553F A1DA 4AC2 5648 23C1 B3DF F742 7D3B 9505  -=|