From mboxrd@z Thu Jan 1 00:00:00 1970 From: Keir Fraser Subject: Re: [PATCH 1 of 3] Enable UEFI BIOS(OVMF) support in Xen-unstable HVM Date: Sat, 23 Jul 2011 17:50:32 +0100 Message-ID: References: Mime-Version: 1.0 Content-Type: text/plain; charset="ISO-8859-1" Content-Transfer-Encoding: quoted-printable Return-path: In-Reply-To: List-Unsubscribe: , List-Post: List-Help: List-Subscribe: , Sender: xen-devel-bounces@lists.xensource.com Errors-To: xen-devel-bounces@lists.xensource.com To: Bei Guan , Keir Fraser Cc: edk2-devel@lists.sourceforge.net, Xen Devel , Jordan Justen , Tim Deegan , Andrei Warkentin List-Id: xen-devel@lists.xenproject.org On 23/07/2011 16:18, "Bei Guan" wrote: > Do you mean that put the bios_relocate hook in the "else" statement? Just= like > this: >=20 > =A0 =A0 if (bios->load) { > =A0 =A0 =A0 =A0 bios->load(bios); > =A0 =A0 } else { > =A0 =A0 =A0 =A0 printf("Loading %s ...\n", bios->name); > =A0 =A0 =A0 =A0 memcpy((void *)bios->bios_address, bios->image, > =A0 =A0 =A0 =A0 =A0 =A0 =A0 =A0bios->image_size); >=20 > =A0 =A0 =A0 =A0 if (bios->bios_relocate) > =A0 =A0 =A0 =A0 =A0 =A0 bios->bios_relocate();=A0 > =A0=A0 } No I mean remove the bios_relocate hook entirely, and modify the rombios handler to use your new hook instead. It should be quite easy. -- Keir