From mboxrd@z Thu Jan 1 00:00:00 1970 Authentication-Results: mx.groups.io; dkim=missing; spf=pass (domain: redhat.com, ip: 209.132.183.28, mailfrom: lersek@redhat.com) Received: from mx1.redhat.com (mx1.redhat.com [209.132.183.28]) by groups.io with SMTP; Wed, 10 Jul 2019 02:42:17 -0700 Received: from smtp.corp.redhat.com (int-mx08.intmail.prod.int.phx2.redhat.com [10.5.11.23]) (using TLSv1.2 with cipher AECDH-AES256-SHA (256/256 bits)) (No client certificate requested) by mx1.redhat.com (Postfix) with ESMTPS id A722830C1CD6; Wed, 10 Jul 2019 09:42:16 +0000 (UTC) Received: from lacos-laptop-7.usersys.redhat.com (ovpn-117-153.ams2.redhat.com [10.36.117.153]) by smtp.corp.redhat.com (Postfix) with ESMTP id 236F22B593; Wed, 10 Jul 2019 09:42:14 +0000 (UTC) Subject: Re: [PATCH v3 14/35] OvmfPkg/AcpiPlatformDxe: Use XenPlatformLib From: "Laszlo Ersek" To: Anthony PERARD , devel@edk2.groups.io Cc: xen-devel@lists.xenproject.org, Ard Biesheuvel , Jordan Justen , Julien Grall References: <20190704144233.27968-1-anthony.perard@citrix.com> <20190704144233.27968-15-anthony.perard@citrix.com> Message-ID: <8fc58d46-3621-8ac7-e7e1-74cdc290f0f6@redhat.com> Date: Wed, 10 Jul 2019 11:42:14 +0200 User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:52.0) Gecko/20100101 Thunderbird/52.9.1 MIME-Version: 1.0 In-Reply-To: X-Scanned-By: MIMEDefang 2.84 on 10.5.11.23 X-Greylist: Sender IP whitelisted, not delayed by milter-greylist-4.5.16 (mx1.redhat.com [10.5.110.46]); Wed, 10 Jul 2019 09:42:16 +0000 (UTC) Content-Type: text/plain; charset=utf-8 Content-Language: en-US Content-Transfer-Encoding: 7bit On 07/08/19 16:38, Laszlo Ersek wrote: > On 07/04/19 16:42, Anthony PERARD wrote: >> This patch replace the XenDetected() function by the one in >> XenPlatformLib. >> >> Ref: https://bugzilla.tianocore.org/show_bug.cgi?id=1689 >> Signed-off-by: Anthony PERARD >> --- >> >> Notes: >> v3: >> - new patch, splited from the next patch >> (which was OvmfPkg/AcpiPlatformDxe: Use PVH RSDP if exist) > > Reviewed-by: Laszlo Ersek > Sorry, I missed a cleanup option: please remove "gEfiXenInfoGuid" from [Guids] in "AcpiPlatformDxe.inf". (You can pick up the R-b with that.) Thanks Laszlo