From mboxrd@z Thu Jan 1 00:00:00 1970 From: Ian Campbell Subject: Re: [PATCH RFC 0/4] hvmloader: make OVMF work with Xen Date: Tue, 19 Nov 2013 17:47:36 +0000 Message-ID: <1384883256.16252.62.camel@hastur.hellion.org.uk> References: <1384531165-3600-1-git-send-email-wei.liu2@citrix.com> Mime-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit Return-path: Received: from mail6.bemta14.messagelabs.com ([193.109.254.103]) by lists.xen.org with esmtp (Exim 4.72) (envelope-from ) id 1VipOn-00074H-2e for xen-devel@lists.xenproject.org; Tue, 19 Nov 2013 17:47:41 +0000 In-Reply-To: <1384531165-3600-1-git-send-email-wei.liu2@citrix.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: Wei Liu Cc: xen-devel@lists.xenproject.org List-Id: xen-devel@lists.xenproject.org On Fri, 2013-11-15 at 15:59 +0000, Wei Liu wrote: > This is the Xen side patches. > > The important bits are setting up ovmf_info and E820 map. I'm particular > interested what to be included in ovmf_info. Follow the lead of seabios_info? In any case you should design things to be extensible in the future, since even if we think really hard now inevitably something else will come along. That way you can add exactly what is needed today and add other stuff when it becomes necessary. I'm going to review what is here, but I think we should wait until agreement is reached with the OVMF folks about that side of thing before committing anything. > > Wei. > > Wei Liu (4): > hvmloader/ovmf: increase blob size to 2MB > hvmloader/ovmf: show OVMF_BEGIN as bios address > hvmloader/ovmf: setup ovmf_info > hvmloader:ovmf: setup E820 map > > tools/firmware/hvmloader/ovmf.c | 58 +++++++++++++++++++++++++++++++++++---- > 1 file changed, 52 insertions(+), 6 deletions(-) >