WARNING - OLD ARCHIVES

This is an archived copy of the Xen.org mailing list, which we have preserved to ensure that existing links to archives are not broken. The live archive, which contains the latest emails, can be found at http://lists.xen.org/
   
 
 
Xen 
 
Home Products Support Community News
 
   
 

xen-devel

Re: [Xen-devel] [PATCH] pygrub: parse xen module option in grub

To: John Levon <levon@xxxxxxxxxxxxxxxxx>
Subject: Re: [Xen-devel] [PATCH] pygrub: parse xen module option in grub
From: Wei Kong <weikong.cn@xxxxxxxxx>
Date: Fri, 20 Feb 2009 11:20:38 +0800
Cc: xen-devel@xxxxxxxxxxxxxxxxxxx
Delivery-date: Thu, 19 Feb 2009 19:21:04 -0800
Dkim-signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=gmail.com; s=gamma; h=domainkey-signature:mime-version:received:in-reply-to:references :date:message-id:subject:from:to:cc:content-type; bh=0LJkAiCMf4YrbH+MEca46skSKK16CEh0hPcrYWL9GB8=; b=cDTRIToevYRE6ksDotcozNNbk63nwowH9jM/m1ROeXWRmAEB2fh3OTnK2rhk/1iN/m MLCxKXovrNF/EpOoGhwMyurrrw3Jp98t8Vobf22kbDZMUZf1nmscQw/qpCodxitxhMaK 9FWuAqMo46C+fOSwXP9eQTE3gPhYLdnZ8aqJI=
Domainkey-signature: a=rsa-sha1; c=nofws; d=gmail.com; s=gamma; h=mime-version:in-reply-to:references:date:message-id:subject:from:to :cc:content-type; b=bEs0AYQTK4FjbsKf+T5ndP9GaWNTtR5SUlDC6ZJOqJKIH4u/CdeAMJKX40QfmsfrCd ihH7E9CDIxDaqPEWK09nqEPS7gXL0qTkWqq8xadikntBNbE7WHl5IF0kBk6wwXKPP7UY x2eop3vRj3E588Ul7TYBgqCMLk/QddC0OUIco=
Envelope-to: www-data@xxxxxxxxxxxxxxxxxxx
In-reply-to: <20090220025505.GA5452@xxxxxxxxxxxxxxxxx>
List-help: <mailto:xen-devel-request@lists.xensource.com?subject=help>
List-id: Xen developer discussion <xen-devel.lists.xensource.com>
List-post: <mailto:xen-devel@lists.xensource.com>
List-subscribe: <http://lists.xensource.com/mailman/listinfo/xen-devel>, <mailto:xen-devel-request@lists.xensource.com?subject=subscribe>
List-unsubscribe: <http://lists.xensource.com/mailman/listinfo/xen-devel>, <mailto:xen-devel-request@lists.xensource.com?subject=unsubscribe>
References: <fac556d20902191840j79bcb2ccwcb32e80b4369cf08@xxxxxxxxxxxxxx> <20090220025505.GA5452@xxxxxxxxxxxxxxxxx>
Sender: xen-devel-bounces@xxxxxxxxxxxxxxxxxxx


2009/2/20 John Levon <levon@xxxxxxxxxxxxxxxxx>
On Fri, Feb 20, 2009 at 10:40:14AM +0800, Wei Kong wrote:

> Add function for GrubConf to parse xen module option in grub.

Why?
thanks, John
 
Maybe that "guest" doesn't have right kernel pattern as xen module.
o, the "guest" means an os on a phy partition which never be a guest before.
Due to sometimes I have many os on different partitions, but I haven't any guest and I don't want install one,
so just let xen boot any exist os as guest.  

 
> Pygrub should pass args as domUloader did.

domUloader was never in the tree.

> -            sxp += "(args \"%s\")" % chosencfg["args"]
> +            sxp += "(args \"%s %s\")" % (chosencfg["args"], incfg["args"])

Are you trying to let the user add arguments to the ones specified in
grub? Why?
Yes, this args is the one user want to pass to guest kernel, 
domUloader real did this, and sorry for I don't know domUloader isn't in this tree.
 

The patch is wrong anyway, you can't just add on the incoming args for
Solaris at least. At the very least you need to add the args only when
run_grub() is called.

No, I didn't add any args for solaris.
Because I don't want to add args for Solaris, so I add args here, not in run_grub().

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