Re: [Xen-devel] [edk2] OVMF broken under Xen (in PCI initialisation)

2016-04-25 Thread Laszlo Ersek
On 04/22/16 16:47, Anthony PERARD wrote: > Hi, > > Following the switch to MdeModulePkg/Bus/Pci/PciHostBridgeDxe, the pci root > bridge does not finish to initialize and breaks under Xen. (Adding Ray Ni) > There are several issue probably due to the use of > PcdPciDisableBusEnumeration=TRUE. >

Re: [Xen-devel] [edk2] OVMF broken under Xen (in PCI initialisation)

2016-04-27 Thread Laszlo Ersek
On 04/27/16 11:50, Ni, Ruiyu wrote: > Copying Mike. > > Regards, > Ray > >> -Original Message- >> From: Ni, Ruiyu >> Sent: Wednesday, April 27, 2016 5:49 PM >> To: 'Gary Lin' >> Cc: edk2-de...@lists.01.org; Xen Devel ; Laszlo >

Re: [Xen-devel] [edk2] OVMF broken under Xen (in PCI initialisation)

2016-04-28 Thread Laszlo Ersek
On 04/28/16 07:08, Ni, Ruiyu wrote: Do you know whether Xen passes the PCI device resource information to firmware? >> >> I don't think so, no. >> >> But, given that the previous PciHostBridgeDxe driver was working on Xen, >> can we perhaps emulate that behavior in >> "OvmfPkg/Library/P

Re: [Xen-devel] Is: Fix for 4MB BIOS payload in hvmloader. Was:Re: [edk2] [PATCH 0/5] OvmfPkg: complete the 4MB flash image support ("-bios" / emulated variables)

2017-05-23 Thread Laszlo Ersek
On 05/23/17 17:01, Jan Beulich wrote: >>>> On 23.05.17 at 16:12, wrote: >> On Thu, May 18, 2017 at 02:36:33PM +0200, Laszlo Ersek wrote: >>> The situation is further hampered by the fact that Xen is (apparently) >>> right at 4.9.0-rc5, so they likely won'

Re: [Xen-devel] Is: Fix for 4MB BIOS payload in hvmloader. Was:Re: [edk2] [PATCH 0/5] OvmfPkg: complete the 4MB flash image support ("-bios" / emulated variables)

2017-05-23 Thread Laszlo Ersek
On 05/23/17 17:02, Julien Grall wrote: > Hi, > > On 23/05/17 15:12, Konrad Rzeszutek Wilk wrote: The primary location for reporting bugs against the hypervisor and associated bundled tools [...] is by posting to the xen-devel mailing list (list info). Please tag your subject line wi

Re: [Xen-devel] [edk2] [PATCH] Maintainers.txt: update OvmfPkg maintainership

2017-08-23 Thread Laszlo Ersek
Hello Konrad, On 08/23/17 03:30, Konrad Rzeszutek Wilk wrote: > On Thu, Aug 17, 2017 at 01:47:59AM +0200, Laszlo Ersek wrote: >> On 08/17/17 00:37, Jordan Justen wrote: >>> On 2017-08-16 12:23:49, Leif Lindholm wrote: >> >> [snip] >> >>>> - the

Re: [Xen-devel] [ipxe-devel] Tips on how to debug EFI code (iPXE) from within KVM after ipxe.efi has crashed with #GP?

2017-09-28 Thread Laszlo Ersek
On 09/28/17 20:04, Michael Brown wrote: > On 28/09/17 18:37, Konrad Rzeszutek Wilk wrote: >> !!! X64 Exception Type - 0D(#GP - General Protection)  CPU Apic ID - >> >> ExceptionData - >> RIP  - BEC2949C, CS  - 0038, RFLAGS - >> 00210216 >>

Re: [Xen-devel] OVMF for Xen PVH

2016-09-08 Thread Laszlo Ersek
On 09/08/16 11:38, Anthony PERARD wrote: > Hello, > > We are introducing a new virtualisation mode in Xen called PVHv2 (also > called hvmlite in the past). We would like to have a UEFI firmware > running on it to make it easier to start a guest. (Right now, I think it > involves supplying the gues

Re: [Xen-devel] [edk2] edk2 compile error

2016-09-19 Thread Laszlo Ersek
On 09/18/16 05:38, Chen, Farrah wrote: > Hi, > > When I compile xen with the latest commit in RHEL 6.7, it failed when make > tools. Errors showed when running edk2 build for OvmfPkgX64. > Bisected and this error occurred from commit > 8c8b6fb02342f7aa78e611a5f0f63dcf8fbf48f2. > > commit 8c8b6f

Re: [Xen-devel] [edk2] [PATCH RESEND] OvmfPkg/build.sh: Use GCC49 toolchain with GCC 6.*

2016-11-21 Thread Laszlo Ersek
On 11/21/16 17:20, Ard Biesheuvel wrote: > On 21 November 2016 at 15:56, Konrad Rzeszutek Wilk wrote: >> Without this I cannot build it under Fedora Core 25. >> >> Contributed-under: TianoCore Contribution Agreement 1.0 >> Signed-off-by: Konrad Rzeszutek Wilk >> --- >> OvmfPkg/build.sh | 2 +- >>

Re: [Xen-devel] [PATCH v2] OvmfPkg/build.sh: Make GCC5 the default toolchain, catch GCC43 and earlier

2016-11-23 Thread Laszlo Ersek
are you okay if I replace it with a plain 0 on commit? * Also, IIRC, Olaf considered pre-4.0 gcc releases as well (rejecting them of course), which is sort of meant as part of "gcc-4.3 and earlier". But, given your extensive testing with old distros (thanks for that!), I think it's safe

Re: [Xen-devel] [PATCH v2] OvmfPkg/build.sh: Make GCC5 the default toolchain, catch GCC43 and earlier

2016-11-23 Thread Laszlo Ersek
On 11/23/16 15:55, Laszlo Ersek wrote: > On 11/23/16 03:36, Konrad Rzeszutek Wilk wrote: >> From Laszlo: >> " change the catch-all (*) to GCC5, from GCC44 >> - remove the (5.*.*) pattern from GCC49 >> - add a branch (with multiple patterns if necessary) for gcc-4.3

Re: [Xen-devel] [PATCH v2] OvmfPkg/build.sh: Make GCC5 the default toolchain, catch GCC43 and earlier

2016-11-23 Thread Laszlo Ersek
On 11/23/16 16:01, Konrad Rzeszutek Wilk wrote: > On Wed, Nov 23, 2016 at 03:55:11PM +0100, Laszlo Ersek wrote: >> On 11/23/16 03:36, Konrad Rzeszutek Wilk wrote: >>> From Laszlo: >>> " change the catch-all (*) to GCC5, from GCC44 >>> - remove the (5.*

Re: [Xen-devel] [PATCH v3] OvmfPkg/build.sh: Make GCC5 the default toolchain, catch GCC43 and earlier

2016-11-24 Thread Laszlo Ersek
ly GCC < 4.4, like GCC 1. > > Reviewed-by: Laszlo Ersek > Reviewed-by: Jordan Justen > Bugzilla: https://bugzilla.tianocore.org/show_bug.cgi?id=62 > Contributed-under: TianoCore Contribution Agreement 1.0 > Signed-off-by: Konrad Rzeszutek Wilk > --- > v1: Initial > v2: Redo it

Re: [Xen-devel] [edk2] [PATCH RFC 01/14] OvmfPkg: Create platform XenOvmf

2017-01-04 Thread Laszlo Ersek
On 12/08/16 16:33, Anthony PERARD wrote: > This is a copy of OvmfX64, removing VirtIO and some SMM. (1) Please provide more details in the commit message: - changed: PLATFORM_GUID, OUTPUT_DIRECTORY, FLASH_DEFINITION - removed: VirtioLib class resolution - removed: DXE_SMM_DRIVER and SMM_CORE lib

Re: [Xen-devel] [edk2] [PATCH RFC 02/14] OvmfPkg/XenOvmf: Update debug IO port for Xen

2017-01-04 Thread Laszlo Ersek
ibIoPort > + gUefiOvmfPkgTokenSpaceGuid.PcdDebugIoPort|0xe9 > + > > > # > # Pcd Dynamic Section - list of all EDK II PCD Entries defined by this > Platform > This patch looks good to me: Reviewed-by: Laszlo Ersek But I ask that in the next po

Re: [Xen-devel] [edk2] [PATCH RFC 03/14] OvmfPkg/XenOvmf.dsc: Introduce XenResetVector

2017-01-04 Thread Laszlo Ersek
(1) I think the subject line should just say: OvmfPkg: Introduce XenResetVector (2) New files are added in this patch; you might want to tag them with a Citrix copyright notice. (3) When formatting the next version of this series for posting, please pass the "-C --find-copies-harder" options t

Re: [Xen-devel] [edk2] [PATCH RFC 00/14] Specific platform to run OVMF in Xen PVH and HVM guests

2017-01-04 Thread Laszlo Ersek
On 12/08/16 16:33, Anthony PERARD wrote: > Hi, > > I've started to create a Xen specifig plaform, in OvmfPkg/XenOvmf.dsc > with the goal to make it work on both Xen HVM and Xen PVHv2 Does this mean we can ultimately move all Xen roles from the current platform DSC files to the new Xen DSC file en

Re: [Xen-devel] [edk2] [PATCH RFC 04/14] OvmfPkg: Introduce XenPlatformPei

2017-01-05 Thread Laszlo Ersek
On 12/08/16 16:33, Anthony PERARD wrote: > A copy of OvmfPkg/PlatformPei without some of QEMU specific > initialization, Xen does not support QemuFwCfg. > > Contributed-under: TianoCore Contribution Agreement 1.0 > Signed-off-by: Anthony PERARD > --- > OvmfPkg/XenOvmf.dsc |

Re: [Xen-devel] [edk2] [PATCH RFC 05/14] OvmfPkg/Library: add XenPciHostBridgeLib

2017-01-05 Thread Laszlo Ersek
On 12/08/16 16:33, Anthony PERARD wrote: > A copy of OvmfPkg/Library/PciHostBridgeLib > > Removing support for pci bus enumeration, I think, and only use scan of > already enumerated pci bus. > > Contributed-under: TianoCore Contribution Agreement 1.0 > Signed-off-by: Anthony PERARD > --- > ...

Re: [Xen-devel] [edk2] [PATCH RFC 06/14] OvmfPkg/XenPlatformPei: Add xen PVH specific code

2017-01-05 Thread Laszlo Ersek
On 12/08/16 16:33, Anthony PERARD wrote: > - learn about memory size from the E820 > - ignore error if host bridge devid is 0x, PVH does not have PCI and > reading from unexisting device return -1. > > Contributed-under: TianoCore Contribution Agreement 1.0 > Signed-off-by: Anthony PERARD >

Re: [Xen-devel] [edk2] [PATCH RFC 07/14] OvmfPkg/XenResetVector: Add new entry point for Xen PVH

2017-01-05 Thread Laszlo Ersek
On 12/08/16 16:33, Anthony PERARD wrote: > This one enter directly in 32bits > > Contributed-under: TianoCore Contribution Agreement 1.0 > Signed-off-by: Anthony PERARD > --- > OvmfPkg/XenResetVector/Ia16/ResetVectorVtf0.asm | 79 > + > OvmfPkg/XenResetVector/Ia32/XenPVH

Re: [Xen-devel] [edk2] [PATCH RFC 08/14] OvmfPkg/PlatformBootManagerLib: Workaround missing PCI bus on Xen PVH

2017-01-05 Thread Laszlo Ersek
On 12/08/16 16:33, Anthony PERARD wrote: > Contributed-under: TianoCore Contribution Agreement 1.0 > Signed-off-by: Anthony PERARD > --- > OvmfPkg/Library/PlatformBootManagerLib/BdsPlatform.c | 2 ++ > 1 file changed, 2 insertions(+) > > diff --git a/OvmfPkg/Library/PlatformBootManagerLib/BdsPla

Re: [Xen-devel] [edk2] [PATCH RFC 09/14] OvmfPkg/ResetSystemLib: Add missing dependency on PciLib

2017-01-05 Thread Laszlo Ersek
On 12/08/16 16:33, Anthony PERARD wrote: > Contributed-under: TianoCore Contribution Agreement 1.0 > Signed-off-by: Anthony PERARD > --- > OvmfPkg/Library/ResetSystemLib/ResetSystemLib.inf | 1 + > 1 file changed, 1 insertion(+) > > diff --git a/OvmfPkg/Library/ResetSystemLib/ResetSystemLib.inf

Re: [Xen-devel] [edk2] [PATCH RFC 11/14] OvmfPkg/XenOvmf: Adding XenTimerLocalApic

2017-01-05 Thread Laszlo Ersek
On 12/08/16 16:33, Anthony PERARD wrote: > And replacing the ACPI Timer by this one based on the local APIC. > > ACPI Timer does not work in a PVH guest, but local APIC works on both > PVH and HVM. > > Contributed-under: TianoCore Contribution Agreement 1.0 > Signed-off-by: Anthony PERARD > ---

Re: [Xen-devel] [edk2] [PATCH RFC 12/14] OvmfPkg/PlatformBootManagerLib: Use a Xen console for ConOut/ConIn

2017-01-05 Thread Laszlo Ersek
On 12/08/16 16:33, Anthony PERARD wrote: > and add OvmfPkg/XenConsoleIo/XenConsoleIo to XenOvmf platform. > > It actually look for gEfiSerialIoProtocolGuid. > --- > .../Library/PlatformBootManagerLib/BdsPlatform.c | 33 > ++ > .../PlatformBootManagerLib.inf

Re: [Xen-devel] [edk2] [PATCH RFC 13/14] OvmfPkg: Introduce XenIoPvhDxe to initialize Grant Tables

2017-01-05 Thread Laszlo Ersek
On 12/08/16 16:33, Anthony PERARD wrote: > This "device" use XenIoMmioLib to reserve some space to be use by grant > tables. It's use 0xfc00, which might not be a good choice... > > There is probably a better way that using a device for this. > --- > OvmfPkg/XenIoPvhDxe/XenIoPvhDxe.c | 263

Re: [Xen-devel] [edk2] [PATCH RFC 14/14] XenOvmf: Use a different RTC

2017-01-05 Thread Laszlo Ersek
On 12/08/16 16:33, Anthony PERARD wrote: > --- > OvmfPkg/XenOvmf.dsc | 5 - > OvmfPkg/XenOvmf.fdf | 2 +- > 2 files changed, 5 insertions(+), 2 deletions(-) > > diff --git a/OvmfPkg/XenOvmf.dsc b/OvmfPkg/XenOvmf.dsc > index a7a884e..345157b 100644 > --- a/OvmfPkg/XenOvmf.dsc > +++ b/OvmfPkg/X

Re: [Xen-devel] [edk2] [PATCH RFC 06/14] OvmfPkg/XenPlatformPei: Add xen PVH specific code

2017-01-10 Thread Laszlo Ersek
On 01/10/17 17:18, Anthony PERARD wrote: > On Thu, Jan 05, 2017 at 11:30:32AM +0100, Laszlo Ersek wrote: >> On 12/08/16 16:33, Anthony PERARD wrote: >>> - learn about memory size from the E820 >>> - ignore error if host bridge devid is 0x, PVH does not have

Re: [Xen-devel] [edk2] A problem about the memory size of virtual machine(VM) when using OVMF

2015-06-04 Thread Laszlo Ersek
I have the impression that this is HTML mail. Please don't post HTML mail; it falls apart when it is quoted. That said, On 06/04/15 16:07, Maoming wrote: > Hi all: > >I started a virtual machine(VM) (redhat6.3_64bit) using OVMF in XEN4.6. > > And the memory of the VM is 64G. > >But I o

Re: [Xen-devel] [edk2] The size of memory is wrong inside of virtual machine(VM) when using OVMF

2015-06-08 Thread Laszlo Ersek
On 06/07/15 14:29, Wei Liu wrote: > On Mon, Jun 01, 2015 at 09:13:08AM +, Maoming wrote: >> Hi all: >>I encountered a troublesome problem about OVMF. >>I used OVMF.fd as a BIOS of virtual machine(VM). >> >>1、my environment: >>xen_version: 4.6-unstable

Re: [Xen-devel] [PATCH] OvmfPkg: Add ACPI support for Virt Xen ARM

2016-05-31 Thread Laszlo Ersek
On 05/31/16 06:59, Shannon Zhao wrote: > From: Shannon Zhao > > Add ACPI support for Virt Xen ARM and it gets the ACPI tables through > Xen ARM multiboot protocol. > > Contributed-under: TianoCore Contribution Agreement 1.0 > Signed-off-by: Shannon Zhao > --- > The corresponding Xen patches can

Re: [Xen-devel] [edk2] 答复: Windows 2008 r2 smp guest booting hang with viridian=true on ovmf(xen latest version 4.5.1-rc1 + latest edk2)

2015-05-20 Thread Laszlo Ersek
On 05/20/15 09:24, Fanhenglong wrote: > Can anyone have idea about how to boot window 2008 r2 smp guest in ovmf > with viridian flag is true? I had to confirm first that "viridian" means "Hyper-V", : "Hyper-V, codenamed Viridian, ..." With that out of the

Re: [Xen-devel] [edk2] Question about PEX boot on Xen with OVMF as bios

2015-05-26 Thread Laszlo Ersek
On 05/25/15 03:50, lidonglin wrote: > Hi all: > Recentlly, I want to use PXE boot on Xen with OVMF as bios. At > beginning, I just add rtl8139 as guest nic device, and I compile a > release ovmf. When I enter into uefi, I can't find network boot menu. > According to edk2/OvmfPkg/README file, I kno

Re: [Xen-devel] [Qemu-devel] [iGVT-g] [vfio-users] [PATCH v3 00/11] igd passthrough chipset tweaks

2016-02-02 Thread Laszlo Ersek
Including Igor & MST Thanks Laszlo On 02/02/16 17:31, Kevin O'Connor wrote: > On Tue, Feb 02, 2016 at 09:56:20AM +0100, Gerd Hoffmann wrote: >> Hi, >> I'd have qemu copy the data on 0xfc write then, so things continue to work without updating seabios. So, the firmware has to allocate

Re: [Xen-devel] [SeaBIOS] [SEABIOS] Plans for either 1.9.1 or 1.10.0?

2016-01-14 Thread Laszlo Ersek
On 01/14/16 17:36, Gerd Hoffmann wrote: > On Do, 2016-01-14 at 14:50 +, Ian Campbell wrote: >> Hello, >> >> The xen.git development branch currently points to SeaBIOS rel-1.9.0, but >> Roger has tripped over a build issue which is fixed by 3b8c5378dfe2 "build: >> fix typo in buildversion.py". >

Re: [Xen-devel] [PATCH v3 25/27] ArmVirtualizationPkg: add XenIoMmioLib

2015-02-10 Thread Laszlo Ersek
ypo "if if" in this patch. In case you send a v4, please fix that. Reviewed-by: Laszlo Ersek I'm done with the v3 review; you addressed all my v2 comments. Thanks! Laszlo ___ Xen-devel mailing list Xen-devel@lists.xen.org http://lists.xen.org/xen-devel

Re: [Xen-devel] [PATCH] MdeModulePkg: mark completion of PCI enumeration in PciEnumeratorLight

2015-02-12 Thread Laszlo Ersek
is, in > combination with 66b280df2, makes AcpiPlatformDxe not able to be loaded, > resulting in guest crash. > > The fix is to install gEfiPciEnumerationCompleteProtocolGuid in > PciEnumeratorLight. > > Contributed-under: TianoCore Contribution Agreement 1.0 > Signed-off-by: Wei Li

[Xen-devel] [PATCH] OvmfPkg: AcpiPlatformDxe: PCI enumeration may be disabled

2015-02-12 Thread Laszlo Ersek
case. When PCI enumeration is disabled (ie. when running on Xen), AcpiPlatformDxe doesn't wait for EFI_PCI_ENUMERATION_COMPLETE_PROTOCOL. Contributed-under: TianoCore Contribution Agreement 1.0 Signed-off-by: Laszlo Ersek --- OvmfPkg/AcpiPlatformDxe/AcpiPlatformDxe.inf | 4 +- OvmfPkg/AcpiP

Re: [Xen-devel] [PATCH] OvmfPkg: AcpiPlatformDxe: PCI enumeration may be disabled

2015-02-12 Thread Laszlo Ersek
On 02/12/15 13:29, Wei Liu wrote: > On Thu, Feb 12, 2015 at 01:16:07PM +0100, Laszlo Ersek wrote: >> SVN r16411 delayed ACPI table installation until PCI enumeration was >> complete, because on QEMU the ACPI-related fw_cfg files should only be >> downloaded after PCI enume

Re: [Xen-devel] [edk2] [PATCH] OvmfPkg: AcpiPlatformDxe: PCI enumeration may be disabled

2015-02-14 Thread Laszlo Ersek
steer clear of drivers that are central to edk2, like PciBusDxe. What do you guys think? Thanks! Laszlo > > -Jordan > > On 2015-02-12 04:16:07, Laszlo Ersek wrote: >> SVN r16411 delayed ACPI table installation until PCI enumeration was >> complete, because on QEMU the AC

Re: [Xen-devel] [edk2] [PATCH] OvmfPkg: AcpiPlatformDxe: PCI enumeration may be disabled

2015-02-14 Thread Laszlo Ersek
On 02/14/15 21:03, Jordan Justen wrote: > On 2015-02-14 08:38:37, Laszlo Ersek wrote: >> On 02/12/15 21:53, Jordan Justen wrote: >>> I think gEfiPciEnumerationCompleteProtocolGuid should be installed by >>> MdeModulePkg/Bus/Pci/PciBusDxe, even when PcdPciDisab

Re: [Xen-devel] [PATCH v5 00/29] Xen/ARM guest support

2015-02-24 Thread Laszlo Ersek
On 02/24/15 19:02, Ard Biesheuvel wrote: > Changes since v4: > - rename InterlockedCompareExchange16 () patch as suggested by Jordan, and > added > his ack > - fix a bug spotted by Anthony in the TestAndClearBit () implementation > - added more acks and R-b's - Are there any patches missing re

Re: [Xen-devel] [PATCH v5 00/29] Xen/ARM guest support

2015-02-24 Thread Laszlo Ersek
On 02/24/15 19:37, Ard Biesheuvel wrote: > On 24 February 2015 at 18:34, Laszlo Ersek wrote: >> On 02/24/15 19:02, Ard Biesheuvel wrote: >> >>> Changes since v4: >>> - rename InterlockedCompareExchange16 () patch as suggested by Jordan, and >>> adde

Re: [Xen-devel] [PATCH v5 00/29] Xen/ARM guest support

2015-02-24 Thread Laszlo Ersek
On 02/24/15 20:01, Ard Biesheuvel wrote: > On 24 February 2015 at 18:41, Laszlo Ersek wrote: >> On 02/24/15 19:37, Ard Biesheuvel wrote: >>> On 24 February 2015 at 18:34, Laszlo Ersek wrote: >>>> On 02/24/15 19:02, Ard Biesheuvel wrote: >>>&

Re: [Xen-devel] OVMF/Xen, Debian wheezy can't boot with NX on stack (Was: Re: [edk2] [PATCH] OvmfPkg: prevent code execution from DXE stack)

2015-09-08 Thread Laszlo Ersek
On 09/08/15 19:26, Anthony PERARD wrote: > On Fri, Aug 28, 2015 at 10:17:28AM +0200, Laszlo Ersek wrote: >> On 08/08/15 02:02, Zeng, Star wrote: >>>> -Original Message- >>>> From: edk2-devel [mailto:edk2-devel-boun...@lists.01.org] On Behalf Of >>>

Re: [Xen-devel] OVMF BoF @ KVM Forum 2015

2015-09-09 Thread Laszlo Ersek
On 08/10/15 18:24, Laszlo Ersek wrote: > Hi. > > Let's do an OVMF BoF at this year's KVM Forum too. Here's a preliminary task list, after some off-list discussion (I tried to incorporate comments): - create GPL'd fork called "ovmf" for expediting

Re: [Xen-devel] OVMF/Xen, Debian wheezy can't boot with NX on stack (Was: Re: [edk2] [PATCH] OvmfPkg: prevent code execution from DXE stack)

2015-09-09 Thread Laszlo Ersek
On 09/09/15 09:06, Jan Beulich wrote: On 09.09.15 at 00:23, wrote: >> On 09/08/15 19:26, Anthony PERARD wrote: >>> And I get this on the console: >>> Welcome to GRUB! >>> >>> X64 Exception Type - 0E(#PF - Page-Fault) CPU Apic ID - >>> RIP - 0F5F8918, CS - 000

Re: [Xen-devel] OVMF/Xen, Debian wheezy can't boot with NX on stack (Was: Re: [edk2] [PATCH] OvmfPkg: prevent code execution from DXE stack)

2015-09-09 Thread Laszlo Ersek
On 09/09/15 11:37, Ian Campbell wrote: > On Wed, 2015-09-09 at 01:06 -0600, Jan Beulich wrote: > On 09.09.15 at 00:23, wrote: >>> On 09/08/15 19:26, Anthony PERARD wrote: And I get this on the console: Welcome to GRUB! X64 Exception Type - 0E(#PF - Page-Fault) CPU Api

Re: [Xen-devel] OVMF/Xen, Debian wheezy can't boot with NX on stack (Was: Re: [edk2] [PATCH] OvmfPkg: prevent code execution from DXE stack)

2015-09-09 Thread Laszlo Ersek
On 09/09/15 13:07, Ian Campbell wrote: > On Wed, 2015-09-09 at 12:48 +0200, Laszlo Ersek wrote: > > Thanks for all the info, I think I get it (although its not clear to me > whether how an app can claim to be UEFI 2.5 capable and what the transition > plan for legacy applications

Re: [Xen-devel] OVMF/Xen, Debian wheezy can't boot with NX on stack (Was: Re: [edk2] [PATCH] OvmfPkg: prevent code execution from DXE stack)

2015-09-09 Thread Laszlo Ersek
On 09/09/15 13:30, Paolo Bonzini wrote: > > > On 09/09/2015 13:07, Ian Campbell wrote: >> I have a question: What attack vector is setting the stack as Nx in OVMF >> (or even UEFI generally) trying to protect against? Or is this being done >> for a reason other than security? >> >> I understand w

Re: [Xen-devel] OVMF/Xen, Debian wheezy can't boot with NX on stack (Was: Re: [edk2] [PATCH] OvmfPkg: prevent code execution from DXE stack)

2015-09-09 Thread Laszlo Ersek
On 09/09/15 14:08, Jan Beulich wrote: On 09.09.15 at 12:48, wrote: >> Personally I think that this dynamic approach is overkill (mainly >> because I'm fine with being unable to install Debian Wheezy guests, both >> wearing and not wearing my red fedora; and because the properties table >> fea

Re: [Xen-devel] EDK II & GPL - Re: [edk2] OVMF BoF @ KVM Forum 2015

2015-09-09 Thread Laszlo Ersek
On 09/09/15 18:17, Jordan Justen wrote: > On 2015-09-09 01:57:51, Laszlo Ersek wrote: >> On 08/10/15 18:24, Laszlo Ersek wrote: >>> Hi. >>> >>> Let's do an OVMF BoF at this year's KVM Forum too. >> >> Here's a preliminary task

Re: [Xen-devel] OVMF BoF @ KVM Forum 2015

2015-09-09 Thread Laszlo Ersek
On 09/09/15 18:34, Ian Campbell wrote: > On Wed, 2015-09-09 at 10:57 +0200, Laszlo Ersek wrote: >> On 08/10/15 18:24, Laszlo Ersek wrote: >>> Hi. >>> >>> Let's do an OVMF BoF at this year's KVM Forum too. >> >> Here's a preliminary ta

Re: [Xen-devel] OVMF BoF @ KVM Forum 2015

2015-09-09 Thread Laszlo Ersek
On 09/09/15 18:34, Ian Campbell wrote: > On Wed, 2015-09-09 at 10:57 +0200, Laszlo Ersek wrote: >> On 08/10/15 18:24, Laszlo Ersek wrote: >>> Hi. >>> >>> Let's do an OVMF BoF at this year's KVM Forum too. >> >> Here's a preliminary ta

Re: [Xen-devel] OVMF/Xen, Debian wheezy can't boot with NX on stack (Was: Re: [edk2] [PATCH] OvmfPkg: prevent code execution from DXE stack)

2015-09-10 Thread Laszlo Ersek
On 09/10/15 05:05, Zeng, Star wrote: > On 2015/9/9 18:48, Laszlo Ersek wrote: >> me neither :) >> >>> but if this (executable code on stack) is >>> happening in grub is there something which is explicitly forbidden to >>> UEFI >>> apps by the

Re: [Xen-devel] [edk2] EDK II & GPL - Re: OVMF BoF @ KVM Forum 2015

2015-09-10 Thread Laszlo Ersek
On 09/10/15 08:19, Alexander Graf wrote: > > >> Am 10.09.2015 um 07:32 schrieb Jordan Justen : >> Laszlo's email raised the GPL question, but I was not sure what the >> EDK II community would accept with regards to GPL. Thus ... I asked. I >> guess I'm getting a better idea with regards to Apple

Re: [Xen-devel] OVMF/Xen, Debian wheezy can't boot with NX on stack (Was: Re: [edk2] [PATCH] OvmfPkg: prevent code execution from DXE stack)

2015-09-11 Thread Laszlo Ersek
On 09/09/15 12:48, Laszlo Ersek wrote: > On 09/09/15 11:37, Ian Campbell wrote: >> On Wed, 2015-09-09 at 01:06 -0600, Jan Beulich wrote: >>>>>> On 09.09.15 at 00:23, wrote: >>>> On 09/08/15 19:26, Anthony PERARD wrote: >>>>>

Re: [Xen-devel] OVMF/Xen, Debian wheezy can't boot with NX on stack (Was: Re: [edk2] [PATCH] OvmfPkg: prevent code execution from DXE stack)

2015-09-11 Thread Laszlo Ersek
On 09/11/15 16:10, Josh Triplett wrote: > On Fri, Sep 11, 2015 at 01:43:53PM +0200, Laszlo Ersek wrote: >> On 09/09/15 12:48, Laszlo Ersek wrote: >>> On 09/09/15 11:37, Ian Campbell wrote: >>>> On Wed, 2015-09-09 at 01:06 -0600, Jan Beulich wrote: >>>>&g

Re: [Xen-devel] OVMF/Xen, Debian wheezy can't boot with NX on stack (Was: Re: [edk2] [PATCH] OvmfPkg: prevent code execution from DXE stack)

2015-09-11 Thread Laszlo Ersek
On 09/11/15 21:30, Josh Triplett wrote: > On Fri, Sep 11, 2015 at 05:28:06PM +0200, Laszlo Ersek wrote: >> On 09/11/15 16:10, Josh Triplett wrote: >>> On Fri, Sep 11, 2015 at 01:43:53PM +0200, Laszlo Ersek wrote: >>>> On 09/09/15 12:48, Laszlo Ersek wrote: >>

Re: [Xen-devel] OVMF/Xen, Debian wheezy can't boot with NX on stack (Was: Re: [edk2] [PATCH] OvmfPkg: prevent code execution from DXE stack)

2015-09-14 Thread Laszlo Ersek
On 09/12/15 01:06, Josh Triplett wrote: > On Fri, Sep 11, 2015 at 11:27:32PM +0200, Laszlo Ersek wrote: >> On 09/11/15 21:30, Josh Triplett wrote: >>> On Fri, Sep 11, 2015 at 05:28:06PM +0200, Laszlo Ersek wrote: >>>> Breaking Debian Wheezy's and BITS&#x

Re: [Xen-devel] OVMF/Xen, Debian wheezy can't boot with NX on stack (Was: Re: [edk2] [PATCH] OvmfPkg: prevent code execution from DXE stack)

2015-09-14 Thread Laszlo Ersek
On 09/14/15 11:22, Ian Campbell wrote: > On Fri, 2015-09-11 at 17:28 +0200, Laszlo Ersek wrote: >> > [...] >> For me that's not so clear-cut. OVMF is frequently used as a UEFI >> development environment (it's better to brick a virtual machine than >> your

[Xen-devel] OVMF BoF @ KVM Forum 2015

2015-08-10 Thread Laszlo Ersek
Hi. Let's do an OVMF BoF at this year's KVM Forum too. Paolo will present Securing secure boot: system management mode in KVM and Tiano Core on Thursday, August 20, in the 5:00pm - 5:30pm time slot. Right after that, the BoF section starts at 5:30pm: http://events.linuxfoundation.org/even

Re: [Xen-devel] [PATCH] x86/xen: avoid freeing static 'name' when kasprintf() fails

2015-01-05 Thread Laszlo Ersek
(), kernel is supposed to crash with >> 'kernel BUG at mm/slub.c:3341!' >> >> Solve the issue by making name a fixed length string inside struct >> xen_clock_event_device. 16 bytes should be enough. >> >> The issue was discovered by Laszlo Ersek. > >

Re: [Xen-devel] [PATCH v2] x86/xen: avoid freeing static 'name' when kasprintf() fails

2015-01-05 Thread Laszlo Ersek
G at mm/slub.c:3341!' > > Solve the issue by making name a fixed length string inside struct > xen_clock_event_device. 16 bytes should be enough. > > Suggested-by: Laszlo Ersek > Signed-off-by: Vitaly Kuznetsov > --- > Changes from v1 (David Vrabel): > - add 

Re: [Xen-devel] [Qemu-devel] Question about xen disk unplug support for ahci missed in qemu

2015-10-15 Thread Laszlo Ersek
CC'ing Kevin O'Connor On 10/14/15 13:27, Ian Campbell wrote: > On Wed, 2015-10-14 at 12:06 +0100, Stefano Stabellini wrote: >>> Can't you just teach SeaBIOS how to deal with your PV disks and then >>> only add that to your VM and forget about IDE/AHCI? I mean, that's how >>> it's done for virtio-b

Re: [Xen-devel] [Qemu-devel] Question about xen disk unplug support for ahci missed in qemu

2015-10-15 Thread Laszlo Ersek
On 10/14/15 14:48, Paul Durrant wrote: >> -Original Message- >> From: Fabio Fantoni [mailto:fabio.fant...@m2r.biz] >> Sent: 14 October 2015 12:12 >> To: Kevin Wolf; Stefano Stabellini >> Cc: John Snow; Anthony Perard; qemu-de...@nongnu.org; xen- >> de...@lists.xen.org; qemu-bl...@nongnu.org

Re: [Xen-devel] [Qemu-devel] Question about xen disk unplug support for ahci missed in qemu

2015-10-16 Thread Laszlo Ersek
On 10/16/15 04:38, Kevin O'Connor wrote: > On Fri, Oct 16, 2015 at 01:10:54AM +0200, Laszlo Ersek wrote: >> On 10/14/15 13:27, Ian Campbell wrote: >>> On Wed, 2015-10-14 at 12:06 +0100, Stefano Stabellini wrote: >>>>> Can't you just teach SeaBIOS how to

Re: [Xen-devel] [Qemu-devel] Question about xen disk unplug support for ahci missed in qemu

2015-10-16 Thread Laszlo Ersek
On 10/16/15 11:06, Stefano Stabellini wrote: > On Thu, 15 Oct 2015, Kevin O'Connor wrote: >> On Fri, Oct 16, 2015 at 01:10:54AM +0200, Laszlo Ersek wrote: >>> On 10/14/15 13:27, Ian Campbell wrote: >>>> On Wed, 2015-10-14 at 12:06 +0100, Stefano Stabellini

Re: [Xen-devel] [Qemu-devel] Question about xen disk unplug support for ahci missed in qemu

2015-10-16 Thread Laszlo Ersek
On 10/16/15 13:34, Fabio Fantoni wrote: > Il 16/10/2015 12:47, Stefano Stabellini ha scritto: >> On Fri, 16 Oct 2015, Fabio Fantoni wrote: >>> Il 16/10/2015 12:13, Anthony PERARD ha scritto: On Fri, Oct 16, 2015 at 10:32:44AM +0200, Fabio Fantoni wrote: > Il 15/10/2015 20:02, Anthony PERAR

Re: [Xen-devel] [Qemu-devel] Question about xen disk unplug support for ahci missed in qemu

2015-10-19 Thread Laszlo Ersek
On 10/19/15 20:29, Fabio Fantoni wrote: > 2015-10-19 18:57 GMT+02:00 Stefano Stabellini > >: > > On Mon, 19 Oct 2015, John Snow wrote: > > On 10/19/2015 07:44 AM, Stefano Stabellini wrote: > > > On Mon, 19 Oct 2015, Gerd Hoffmann wrote: > >

Re: [Xen-devel] [Qemu-devel] Question about xen disk unplug support for ahci missed in qemu

2015-10-19 Thread Laszlo Ersek
On 10/16/15 21:09, Laszlo Ersek wrote: > On 10/16/15 13:34, Fabio Fantoni wrote: >> Il 16/10/2015 12:47, Stefano Stabellini ha scritto: >>> On Fri, 16 Oct 2015, Fabio Fantoni wrote: >>>> Il 16/10/2015 12:13, Anthony PERARD ha scritto: >>>>> On Fri, Oc

Re: [Xen-devel] [Qemu-devel] Question about xen disk unplug support for ahci missed in qemu

2015-10-20 Thread Laszlo Ersek
On 10/20/15 13:59, Stefano Stabellini wrote: > On Mon, 19 Oct 2015, Laszlo Ersek wrote: >> On 10/16/15 21:09, Laszlo Ersek wrote: >>> On 10/16/15 13:34, Fabio Fantoni wrote: >>>> Il 16/10/2015 12:47, Stefano Stabellini ha scritto: >>>>> On Fri, 16 O

Re: [Xen-devel] [PATCH] XenPvBlk: handle empty cdrom drives

2015-10-20 Thread Laszlo Ersek
(1) Please make the subject line say: OvmfPkg: XenPvBlkDxe: handle empty cdrom drives On 10/20/15 17:03, Stefano Stabellini wrote: > Empty cdroms are not going to connect, avoid waiting for the backend to > switch to state 4, which is never going to happen, and return > EFI_NO_MEDIA instead. (

Re: [Xen-devel] [edk2] [PATCH v2] OvmfPkg: XenPvBlkDxe: handle empty cdrom drives

2015-10-21 Thread Laszlo Ersek
FreePool (Params); > + DEBUG ((EFI_D_INFO, "%a: Empty cdrom\n", __FUNCTION__)); > + goto Error; > +} > +FreePool (Params); > + } > + > Status = XenBusReadUint64 (XenBusIo, "backend-id", FALSE, &Value); >if (St

Re: [Xen-devel] [PATCH v1 00/21] Xen/ARM guest support

2015-01-23 Thread Laszlo Ersek
On 01/23/15 16:02, Ard Biesheuvel wrote: > ArmPkg/Drivers/TimerDxe/TimerDxe.c | 14 +- > .../ArmVirtualizationPkg/ArmVirtualizationPkg.dec | 3 +- > .../ArmVirtualizationPkg/ArmVirtualizationQemu.dsc | 3 - > .../ArmVirtualizationPkg/ArmVirtualizationXen.dsc | 274 +

Re: [Xen-devel] [PATCH v1 01/21] ArmPkg: allow HYP timer interrupt to be omitted

2015-01-23 Thread Laszlo Ersek
On 01/23/15 16:02, Ard Biesheuvel wrote: > The DT binding for the ARM generic timer describes the secure, > non-secure, virtual and hypervisor timer interrupts, respectively. > However, under virtualization, only the virtual timer is usable, and > the device tree may omit the hypervisor timer inter

Re: [Xen-devel] [PATCH v1 02/21] ArmVirtualizationPkg: add GICv3 detection to VirtFdtDxe

2015-01-23 Thread Laszlo Ersek
PcdSet32 (PcdGicDistributorBase, (UINT32)DistBase); > + > + DEBUG ((EFI_D_INFO, "Found GIC v3 distributor @ 0x%Lx\n", DistBase)); > + break; > + > case PropertyTypeRtc: >ASSERT (Len == 16); > > Acked-by: Laszlo Ersek ___ Xen-devel mailing list Xen-devel@lists.xen.org http://lists.xen.org/xen-devel

Re: [Xen-devel] [PATCH v1 03/21] ArmVirtualizationPkg: replace instance of FixedPcdGet()

2015-01-23 Thread Laszlo Ersek
ere, so it bears a bit more explanation IMO :) Also, after the relocation, FixedPcdGet64() and PcdGet64() would not return the same value for the same PCD (despite it being a fixed PCD), which is presumably a "first" in edk2. I can't recommend an alternative, but please put a warning com

Re: [Xen-devel] [PATCH v1 04/21] ArmVirtualizationPkg: move early UART discovery to PlatformPeim

2015-01-23 Thread Laszlo Ersek
re Contribution Agreement 1.0 > Reviewed-by: Laszlo Ersek > Signed-off-by: Ard Biesheuvel > --- > .../Library/ArmVirtualizationPlatformLib/Virt.c| 46 + > .../Library/PlatformPeiLib/PlatformPeiLib.c| 48 > ++ > 2 files cha

Re: [Xen-devel] [PATCH v1 05/21] ArmVirtualizationPkg: use a HOB to store device tree blob

2015-01-23 Thread Laszlo Ersek
I reviewed the discussion under http://lists.linaro.org/pipermail/linaro-uefi/2014-December/000601.html and I can see that you addressed all points there. I have some new comments: On 01/23/15 16:02, Ard Biesheuvel wrote: > Instead of using a dynamic PCD, store the device tree address in a HOB >

Re: [Xen-devel] [PATCH v1 09/21] ArmVirtualizationPkg: implement custom MemoryInitPeiLib

2015-01-23 Thread Laszlo Ersek
some notes On 01/23/15 16:02, Ard Biesheuvel wrote: > This implements a MemoryInitPeiLib instance that differs from the > stock ArmPlatformPkg version only in the fact that it does not remove > the memory used by the flash device (FD). The reason is that, when using > PrePi, the DXE core is starte

Re: [Xen-devel] [PATCH v1 10/21] ArmVirtualizationPkg: Xen/PV relocatable platformlib instance

2015-01-23 Thread Laszlo Ersek
On 01/23/15 16:02, Ard Biesheuvel wrote: > Add a ArmPlatformLib instance that can deal with the self relocation > and truly dynamic discovery of system RAM base and size. > > Signed-off-by: Ard Biesheuvel > --- > .../AARCH64/MemnodeParser.S| 232 > + >

Re: [Xen-devel] [PATCH v1 06/21] ArmVirtualizationPkg: add padding to FDT allocation

2015-01-23 Thread Laszlo Ersek
On 01/23/15 16:02, Ard Biesheuvel wrote: > Our primary user QEMU/mach-virt presents us with a FDT blob padded > to 64 KB with plenty of room to set additional properties. However, > in the general case, we should only add properties after making sure > there is enough room available. > > Contribut

Re: [Xen-devel] [PATCH v1 11/21] Ovmf/Xen: move Xen interface version to

2015-01-23 Thread Laszlo Ersek
fPkg/XenPvBlkDxe/XenPvBlkDxe.h > @@ -18,10 +18,6 @@ > > #include > > -// > -// Xen interface version used > -// > -#define __XEN_INTERFACE_VERSION__ 0x00040400 > #define xen_mb() MemoryFence() > #define xen_rmb() MemoryFence() > #define xen_wmb() MemoryFence() > Acked-by: Laszlo Ersek ___ Xen-devel mailing list Xen-devel@lists.xen.org http://lists.xen.org/xen-devel

Re: [Xen-devel] [PATCH v1 12/21] Ovmf/Xen: fix pointer to int cast in XenBusDxe

2015-01-23 Thread Laszlo Ersek
On 01/23/15 16:03, Ard Biesheuvel wrote: > On ARM, xen_pfn_t is 64 bits but the size of a pointer is only > 32 bits, so casting between them needs to go via (UINTN) > > Contributed-under: TianoCore Contribution Agreement 1.0 > Signed-off-by: Ard Biesheuvel > --- > OvmfPkg/XenBusDxe/GrantTable.c

Re: [Xen-devel] [PATCH v1 13/21] Ovmf/Xen: move arch specific hypercall implementation to XenHypercallLib

2015-01-23 Thread Laszlo Ersek
On 01/23/15 19:24, Stefano Stabellini wrote: > On Fri, 23 Jan 2015, Ard Biesheuvel wrote: >> For future ARM/AArch64 support in the XenBus code, move the implementation >> of hypercall invocation to a dedicated library. The use of a library rather >> than just an arch specific source in XenBusDxe.in

Re: [Xen-devel] [PATCH v1 14/21] Ovmf/Xen: allow non-PCI usage of XenBusDxe

2015-01-26 Thread Laszlo Ersek
On 01/23/15 16:03, Ard Biesheuvel wrote: > While Xen on Intel uses a virtual PCI device to communicate the > base address of the grant table, the ARM implementation uses a DT > node, which is fundamentally incompatible with the way XenBusDxe is > implemented, i.e., as a UEFI Driver Model implementa

Re: [Xen-devel] [PATCH v1 14/21] Ovmf/Xen: allow non-PCI usage of XenBusDxe

2015-01-26 Thread Laszlo Ersek
On 01/26/15 10:46, Ard Biesheuvel wrote: > So it would be sufficient to install the XENIO_PROTOCOL on the > existing ControllerHandle containing the EFI_PCI_IO_PROTOCOL? Yes. Because there would be only one PCI (b,d,f) that would qualify (you'd write up the Supported() check appropriately), ther

Re: [Xen-devel] [PATCH v1 03/21] ArmVirtualizationPkg: replace instance of FixedPcdGet()

2015-01-26 Thread Laszlo Ersek
On 01/26/15 11:57, Ard Biesheuvel wrote: > On 23 January 2015 at 19:38, Laszlo Ersek wrote: >> On 01/23/15 16:02, Ard Biesheuvel wrote: >>> This removes an instance of FixedPcdGet () so that the self relocating >>> PrePi instance can poke another value into it

Re: [Xen-devel] [PATCH v1 14/21] Ovmf/Xen: allow non-PCI usage of XenBusDxe

2015-01-26 Thread Laszlo Ersek
On 01/26/15 14:52, Ard Biesheuvel wrote: > Well, the problem is that the XenConsoleSerialPortLib implementation > also needs to issue Xen hypercalls, and needs to do so very early. In general virtual serial consoles, be they Xen or virtio, are a huge "impedance mismatch" (is that the right term?)

Re: [Xen-devel] [PATCH v2 01/29] ArmPkg: allow HYP timer interrupt to be omitted

2015-01-28 Thread Laszlo Ersek
pe ? 16 : 0); >VirtIntrNum = fdt32_to_cpu (InterruptProp[2].Number) > + (InterruptProp[2].Type ? 16 : 0); > - HypIntrNum = fdt32_to_cpu (InterruptProp[3].Number) > - + (InterruptProp[3].Type ? 16 : 0); > + HypIntrNum = Len < 48 ? 0 : f

Re: [Xen-devel] [PATCH v2 02/29] ArmPkg: allow patchable PCDs for memory, FD and FV addresses

2015-01-28 Thread Laszlo Ersek
l be > declared ># to UEFI by ArmPlatformLib >gArmTokenSpaceGuid.PcdSystemMemoryBase|0|UINT64|0x0029 >gArmTokenSpaceGuid.PcdSystemMemorySize|0|UINT64|0x002A > > +[PcdsFixedAtBuild.common, PcdsDynamic.common] ># ># ARM Architectural Timer ># > Acked-by: Laszlo Ersek ___ Xen-devel mailing list Xen-devel@lists.xen.org http://lists.xen.org/xen-devel

Re: [Xen-devel] [PATCH v2 05/29] ArmVirtualizationPkg: allow patchable PCD for device tree base address

2015-01-30 Thread Laszlo Ersek
ArmVirtualizationPkg/Library/ArmVirtualizationPlatformLib/Virt.c > @@ -96,7 +96,7 @@ ArmPlatformInitializeSystemMemory ( >ASSERT (HobData != NULL); >*HobData = 0; > > - DeviceTreeBase = (VOID *)(UINTN)FixedPcdGet64 > (PcdDeviceTreeInitialBaseAddress); > + De

Re: [Xen-devel] [PATCH v2 07/29] ArmVirtualizationPkg: use a HOB to store device tree blob

2015-01-30 Thread Laszlo Ersek
>> armlink : error L6218: Undefined symbol AllocatePages (referred from >> ArmVirtualizationPlatformLib.lib). >> armlink : Not enough information to list image symbols. >> armlink : Finished: 1 information, 0 warning and 1 error messages. >> > > Probably just

Re: [Xen-devel] [PATCH v2 08/29] ArmVirtualizationPkg: add padding to FDT allocation

2015-01-30 Thread Laszlo Ersek
> Some platforms might want to have a larger FDT padding. >> > > Agreed. Will add it to v3 I think using a FixedPcd will be okay. With that change, you can add Reviewed-by: Laszlo Ersek Thanks Laszlo > > >> >>> -Original Message- >>> From

Re: [Xen-devel] [PATCH v2 12/29] ArmVirtualizationPkg: implement custom MemoryInitPeiLib

2015-01-30 Thread Laszlo Ersek
uid.PcdMemoryTypeEfiReservedMemoryType > + gEmbeddedTokenSpaceGuid.PcdMemoryTypeEfiRuntimeServicesData > + gEmbeddedTokenSpaceGuid.PcdMemoryTypeEfiRuntimeServicesCode > + gEmbeddedTokenSpaceGuid.PcdMemoryTypeEfiBootServicesCode > + gEmbeddedTokenSpaceGuid.PcdMemoryTypeEfiBootServicesD

Re: [Xen-devel] [PATCH v2 13/29] ArmVirtualizationPkg: allow patchable PCD for FV base address

2015-01-30 Thread Laszlo Ersek
dress > > [Guids] >gEarlyPL011BaseAddressGuid > It also seems to change the PCD type of PcdDeviceTreeInitialBaseAddress. Care to mention that too in the commit message, just for completeness? Reviewed-by: Laszlo Ersek Thanks Laszlo ___ Xen-devel mailing list Xen-devel@lists.xen.org http://lists.xen.org/xen-devel

Re: [Xen-devel] [PATCH v2 14/29] ArmVirtualizationPkg: Xen/PV relocatable platformlib instance

2015-01-30 Thread Laszlo Ersek
Lib/XenVirtMem.c > | 83 + > 5 files changed, 606 insertions(+) Five new files; it appears impossible that this patch regress QEMU VMs. Hence Acked-by: Laszlo Ersek ___ Xen-devel mailing list Xen-devel@lists.xen.org http://lists.xen.org/xen-devel

Re: [Xen-devel] [PATCH v2 16/29] Ovmf/Xen: fix pointer to int cast in XenBusDxe

2015-01-30 Thread Laszlo Ersek
bindings. Most of the time, the above means ((type)expr1) expr2 and not (type)(expr1 expr2) as the space would incorrectly suggest. Your patch is fine, but I had to think thrice. :) Reviewed-by: Laszlo Ersek ___ Xen-devel mailing list Xen-devel@lists.xen.org http://lists.xen.org/xen-devel

  1   2   >