Re: [Xen-devel] [PATCH] Guest boot loader support

[aq <aquynh@xxxxxxxxx>]

On 4/26/05, aq <aquynh@xxxxxxxxx> wrote:
> On 4/26/05, Jeremy Katz <katzj@xxxxxxxxxx> wrote:
> > Attached is an updated version of the patch to add boot loader support
> > for guest domains.
> >
> > Changes from the initial set of patches:
> > * Per Ian's request, adds an option to specify booting a specific kernel
> > from the host domain (you can specify it either with the title of the
> > boot loader config entry or with the grub 0-based index)
> > * Reduces some of the code duplication for finding out what the virtual
> > disk to boot off of is
> > * Improved error handling to use the standard Xen logging and error
> > facilities
> >

Jeremy, I looked at your code, and find few lines like this:

   exec("%s = r\"%s\"" %(self.commands[com], arg.strip()))

Looks like you dont need the above "r". Then the code should be:

   exec("%s = \"%s\"" %(self.commands[com], arg.strip()))

Here is a small patch (against your patch) to remove those typos.

$ diffstat booloader1.patch
 xen-bootloader2.patch |    4 ++--
 1 files changed, 2 insertions(+), 2 deletions(-)

Signed-off-by: Nguyen Anh Quynh <aquynh@xxxxxxxxx>

Attachment: booloader1.patch
Description: Binary data

_______________________________________________
Xen-devel mailing list
Xen-devel@xxxxxxxxxxxxxxxxxxx
http://lists.xensource.com/xen-devel