From mboxrd@z Thu Jan 1 00:00:00 1970 From: Zhigang Wang Subject: Re: [PATCH] add new bootloader xenpvnetboot for pv guest [and 1 more messages] Date: Sat, 03 Mar 2012 17:46:25 -0500 Message-ID: <4F529F41.2050506@oracle.com> References: <6770476b814532bb36a3.1329919602@zhigang.us.oracle.com> <1330081468.8557.176.camel@zakaz.uk.xensource.com> <20303.50752.554261.804604@mariner.uk.xensource.com> Mime-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit Return-path: In-Reply-To: <20303.50752.554261.804604@mariner.uk.xensource.com> List-Unsubscribe: , List-Post: List-Help: List-Subscribe: , Sender: xen-devel-bounces@lists.xen.org Errors-To: xen-devel-bounces@lists.xen.org To: Ian Jackson Cc: Ian Campbell , xen-devel List-Id: xen-devel@lists.xenproject.org On 03/01/2012 01:56 PM, Ian Jackson wrote: > Zhigang Wang writes ("[Xen-devel] [PATCH] add new bootloader xenpvnetboot for pv guest"): >> add new bootloader xenpvnetboot for pv guest > > Great, thanks. I agree with Ian's comments but this is a clear > improvement so I have applied it. But, improving the documentation is > something we should definitely do. > > Ian writes: >> Since you have, I guess, reverse engineered the PV bootloader protocol >> >from pygrub in order to implement this I wonder if you would consider >> writing up a specification for the required interface or such >> bootloaders? e.g. docs/misc/pvbootloader.markdown. > > That is also an excellent suggestion. > > Ian. Thanks Ian C and Ian J. Sorry I missed Ian C's mail a few days ago. I add what you suggested to my TODO list. Another thing you may help about this is: PV guest diskless boot using this boot loader. Current for PV guest, we require at least one disk. I will work on it and probably ask for help later. Thanks, Zhigang