Re: [edk2] [Patch 0/3] Improve Fixed-MTRR programming performance

2016-04-27 Thread Fan, Jeff
Feng, Thanks your comments. I will update the parameter name to LastMsrNum to match its meaning more exactly. @param LastMsrNum On input, the last index of the fixed MTRR MSR to program. On return, the current fixed MTRR MSR to program. Thanks! Jeff

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

2016-04-27 Thread Ni, Ruiyu
Regards, Ray >-Original Message- >From: Laszlo Ersek [mailto:ler...@redhat.com] >Sent: Wednesday, April 27, 2016 6:44 PM >To: Ni, Ruiyu ; Gary Lin >Cc: edk2-devel@lists.01.org ; Xen Devel >; Kinney, Michael D > >Subject: Re: [edk2] OVMF broken under Xen (in PCI initialisation) > >On 04

[edk2] [PATCH v3 0/2] Report ACPI NFIT for reserved memory RAM disks

2016-04-27 Thread Hao Wu
Changes compared with V2: 1. RamDiskDxe driver now will register an EFI event in the Ready To Boot Event Group to a). detect whether EFI_ACPI_TABLE_PROTOCOL and EFI_ACPI_SDT_PROTOCOL are both produced; b). publish all the reserved memory type RAM disks registered at this point to the NFIT

[edk2] [PATCH v3 2/2] OvmfPkg: Modify fdf files due to RamDiskDxe driver's adding ASL code

2016-04-27 Thread Hao Wu
The RamDiskDxe driver in MdeModulePkg now will sometimes report a NVDIMM Root Device using ASL code which is put in a Secondary System Description Table (SSDT) according to the ACPI 6.1 spec. Locating the SSDT requires modifying the fdf files in OvmfPkg. Cc: Laszlo Ersek Cc: Jordan Justen Contr

[edk2] [PATCH v3 1/2] MdeModulePkg RamDiskDxe: Report ACPI NFIT for reserved memory RAM disks

2016-04-27 Thread Hao Wu
The RamDiskDxe now will report RAM disks with reserved memory type to NFIT in the ACPI table. This commit will also make sure that an NVDIMM root device exists in the \SB scope before reporting any RAM disk to NFIT. To properly report the NVDIMM root device, one will need to append the following

Re: [edk2] [PATCH 1/3] MdeModulePkg: PiDxeS3BootScriptLib: honor PcdAcpiS3Enable

2016-04-27 Thread Yao, Jiewen
Reviewed by: jiewen@intel.com > -Original Message- > From: Laszlo Ersek [mailto:ler...@redhat.com] > Sent: Thursday, April 28, 2016 3:21 AM > To: edk2-devel-01 > Cc: Tian, Feng ; Yao, Jiewen ; > Justen, Jordan L ; Ni, Ruiyu > ; Zeng, Star > Subject: [PATCH 1/3] MdeModulePkg: PiDxeS3

Re: [edk2] [Patch 0/3] Improve Fixed-MTRR programming performance

2016-04-27 Thread Tian, Feng
For patch #1, I have comment: 1. the parameter name is StartMsrNum, but the parameter comment is MsrNum. 2. the algorithm is calculating the MsrNum from *StartMsrNum + 1, it seems not match the comment "The start index of the fixed MTRR MSR to program". Other looks good to me. Reviewed-by: Feng

Re: [edk2] [PATCH 0/3] OvmfPkg: tighten the SMM lockdown

2016-04-27 Thread Zeng, Star
On 2016/4/28 3:20, Laszlo Ersek wrote: The first patch (for MdeModulePkg) fixes a bug that is exposed (triggered) by the third patch. The second and third patches fix a security vulnerability in OVMF that I reported to the UEFI SRT more than three weeks ago: To: secur...@uefi.org, secal...@r

Re: [edk2] [PATCH] MdeModulePkg/TerminalDxe: Set polling rate by serial IO mode

2016-04-27 Thread Andrew Fish
> On Apr 27, 2016, at 8:10 PM, Ni, Ruiyu wrote: > >> >> -Original Message- >> From: edk2-devel [mailto:edk2-devel-boun...@lists.01.org] On Behalf Of Heyi >> Guo >> Sent: Saturday, April 23, 2016 4:54 PM >> To: edk2-devel@lists.01.org >> Cc: Kinney, Michael D ; Heyi Guo >> ; Tian, Feng

Re: [edk2] [PATCH] MdeModulePkg/TerminalDxe: Set polling rate by serial IO mode

2016-04-27 Thread Ni, Ruiyu
>-Original Message- >From: edk2-devel [mailto:edk2-devel-boun...@lists.01.org] On Behalf Of Heyi Guo >Sent: Saturday, April 23, 2016 4:54 PM >To: edk2-devel@lists.01.org >Cc: Kinney, Michael D ; Heyi Guo >; Tian, Feng ; >Zeng, Star >Subject: [edk2] [PATCH] MdeModulePkg/TerminalDxe: Set p

Re: [edk2] [Patch] ShellPkg: Enhance ping to select the interface automatically

2016-04-27 Thread Fu, Siyuan
Hi, Jiaxin A interface with only link local address should be allowed as the src if the dest is also an link local address, right? And the 2 if conditions in line 1052 is better to change to a "if (UnspecifiedSrc) else ..." so it's more readable. If (UnspecifiedSrc && !NetIp6IsUnspecif

Re: [edk2] [Patch] NetworkPkg: Avoid the indefinite wait case in HttpDxe

2016-04-27 Thread Fu, Siyuan
Reviewed-by: Fu Siyuan > -Original Message- > From: edk2-devel [mailto:edk2-devel-boun...@lists.01.org] On Behalf Of > Jiaxin Wu > Sent: Tuesday, April 26, 2016 6:39 PM > To: edk2-devel@lists.01.org > Cc: Ye, Ting ; Zhang, Lubo ; Fu, > Siyuan > Subject: [edk2] [Patch] NetworkPkg: Avoid t

Re: [edk2] [Patch v3 04/23] OvmfPkg/QemuNewBootOrderLib: Build with UefiBootManagerLib

2016-04-27 Thread Ni, Ruiyu
Regards, Ray >-Original Message- >From: Laszlo Ersek [mailto:ler...@redhat.com] >Sent: Tuesday, April 26, 2016 11:02 PM >To: Ni, Ruiyu ; edk2-de...@ml01.01.org >Cc: Justen, Jordan L >Subject: Re: [edk2] [Patch v3 04/23] OvmfPkg/QemuNewBootOrderLib: Build with >UefiBootManagerLib > >Thi

Re: [edk2] [Patch] MdeModulePkg: DxeCore MemoryPool Algorithm Update

2016-04-27 Thread Kinney, Michael D
With that update, Reviewed-by: Michael Kinney Mike From: Gao, Liming Sent: Wednesday, April 27, 2016 5:51 PM To: Ard Biesheuvel Cc: edk2-devel-01 ; Kinney, Michael D Subject: RE: [Patch] MdeModulePkg: DxeCore MemoryPool Algorithm Update Good catch. I will update it. From: Ard Biesheuvel

Re: [edk2] [Patch v3 02/23] OvmfPkg/PlatformPei: Add memory above 4GB as tested

2016-04-27 Thread Ni, Ruiyu
Regards, Ray >-Original Message- >From: Laszlo Ersek [mailto:ler...@redhat.com] >Sent: Tuesday, April 26, 2016 5:27 AM >To: Ni, Ruiyu >Cc: edk2-de...@ml01.01.org; Justen, Jordan L >Subject: Re: [edk2] [Patch v3 02/23] OvmfPkg/PlatformPei: Add memory above 4GB >as tested > >On 04/21/16

Re: [edk2] [Patch] MdeModulePkg: DxeCore MemoryPool Algorithm Update

2016-04-27 Thread Gao, Liming
Good catch. I will update it. From: Ard Biesheuvel [mailto:ard.biesheu...@linaro.org] Sent: Thursday, April 28, 2016 12:05 AM To: Gao, Liming Cc: edk2-devel-01 ; Kinney, Michael D Subject: Re: [Patch] MdeModulePkg: DxeCore MemoryPool Algorithm Update On 27 April 2016 at 08:40, Liming Gao wrote

Re: [edk2] [PATCH 1/3] MdeModulePkg: PiDxeS3BootScriptLib: honor PcdAcpiS3Enable

2016-04-27 Thread Carsey, Jaben
thanks for the explanation. I agree that libraries must be designed for this feature from the beginning. I also know that people use libraries for purposes other than their original intent. I think it would be best to plan for this by having properly coded destructors in libraries whenever po

Re: [edk2] [PATCH] ShellPkg: Fix pci command for '_e' option

2016-04-27 Thread Carsey, Jaben
Done. > -Original Message- > From: edk2-devel [mailto:edk2-devel-boun...@lists.01.org] On Behalf Of > Attar, Abdul Lateef > Sent: Tuesday, April 26, 2016 3:42 AM > To: Qiu, Shumin ; Carsey, Jaben > ; Ni, Ruiyu ; edk2- > de...@lists.01.org > Cc: Parthasarathy, Mohan (HPE Servers) ; > Lakshm

[edk2] [PATCH] ShellPkg: Update smbiosview for latest Type 17 devices

2016-04-27 Thread Samer El-Haj-Mahmoud
Update smbiosview to understand latest SMBIOS Type 17 devices from SMBIOS 3.0.0 spec Contributed-under: TianoCore Contribution Agreement 1.0 Signed-off-by: Samer El-Haj-Mahmoud --- .../SmbiosView/QueryTable.c | 21 + 1 file changed, 21 insertions(+) d

[edk2] [PATCH] [BaseTools]/Build: Better DSC arch filtering

2016-04-27 Thread Thomas Palmer
Description: When building for any specific architecture, the build script today is loading DSC sections for other architectures not in the build. The build process should disregard DSC sections that are not relevant to the build. My previous patch only fixed issue for one section type (Components

Re: [edk2] [PATCH 1/3] MdeModulePkg: PiDxeS3BootScriptLib: honor PcdAcpiS3Enable

2016-04-27 Thread Laszlo Ersek
On 04/27/16 21:45, Carsey, Jaben wrote: > Laszlo, > > Does the library destructor not get called? Shouldn't that destructor > unregister the protocol notify and leave the callback pointer in DXE core > correct? PiDxeS3BootScriptLib doesn't have a DESTRUCTOR at the moment. I would prefer not a

Re: [edk2] [PATCH 1/3] MdeModulePkg: PiDxeS3BootScriptLib: honor PcdAcpiS3Enable

2016-04-27 Thread Carsey, Jaben
Laszlo, Does the library destructor not get called? Shouldn't that destructor unregister the protocol notify and leave the callback pointer in DXE core correct? -Jaben > -Original Message- > From: edk2-devel [mailto:edk2-devel-boun...@lists.01.org] On Behalf Of > Laszlo Ersek > Sent:

[edk2] [PATCH 2/3] OvmfPkg: PlatformBdsLib: lock down SMM in PlatformBdsInit()

2016-04-27 Thread Laszlo Ersek
OVMF's PlatformBdsLib currently makes SMM vulnerable to the following attack: (1) a malicious guest OS copies a UEFI driver module to the EFI system partition, (2) the OS adds the driver as a Driver option, and references it from DriverOrder, (3) at next boot, the BdsEntry() function

[edk2] [PATCH 3/3] OvmfPkg: PlatformBdsLib: lock down SMM regardless of S3

2016-04-27 Thread Laszlo Ersek
At the moment, the EFI_DXE_SMM_READY_TO_LOCK_PROTOCOL is only installed if S3 is enabled -- at the end of SaveS3BootScript(). While a runtime OS is never booted with SMM unlocked (because the SMM IPL locks down SMM as a last resort: > SMM IPL! DXE SMM Ready To Lock Protocol not installed before

[edk2] [PATCH 1/3] MdeModulePkg: PiDxeS3BootScriptLib: honor PcdAcpiS3Enable

2016-04-27 Thread Laszlo Ersek
In the edk2 tree, there are currently four drivers that consume PcdAcpiS3Enable: IntelFrameworkModulePkg/Universal/Acpi/AcpiS3SaveDxe/AcpiS3SaveDxe.inf MdeModulePkg/Universal/Acpi/BootScriptExecutorDxe/BootScriptExecutorDxe.inf MdeModulePkg/Universal/Acpi/S3SaveStateDxe/S3SaveStateDxe.inf

[edk2] [PATCH 0/3] OvmfPkg: tighten the SMM lockdown

2016-04-27 Thread Laszlo Ersek
The first patch (for MdeModulePkg) fixes a bug that is exposed (triggered) by the third patch. The second and third patches fix a security vulnerability in OVMF that I reported to the UEFI SRT more than three weeks ago: To: secur...@uefi.org, secal...@redhat.com [...] From: Laszlo Ersek Su

Re: [edk2] Minor problems with dualboot VM. EFIVars?

2016-04-27 Thread Laszlo Ersek
aside: On 04/27/16 18:40, Blank Field wrote: >>> Your NvVars file could be very old, maybe a remnant of your earlier >> Will check the date, but i doubt about this. Maybe there was no problem at >> all and i just need to create a normal BootOrder entry. > Oct 9 2015. Old. Okay. >> Separate VARS...

Re: [edk2] Minor problems with dualboot VM. EFIVars?

2016-04-27 Thread Blank Field
>> Your NvVars file could be very old, maybe a remnant of your earlier > Will check the date, but i doubt about this. Maybe there was no problem at > all and i just need to create a normal BootOrder entry. Oct 9 2015. Old. Okay. > Separate VARS... hvm /usr/share/edk2.git/ovmf-x64/OVMF_CO

Re: [edk2] HMAC calculation clarification for TPM 1.2

2016-04-27 Thread David Van Arnem
On 04/26/2016 09:46 PM, Long, Qin wrote: > David, > > I think your original understanding should be correct. (I am not 100% sure. > Still need to double-confirm.) > For one OSAP session, > sharedSecret = HMAC(key.usageAuth, nonceEvenOSAP, nonceOddOSAP) > The shared secret will be calculate

Re: [edk2] [PATCH] ShellPkg: Fix typos and EDK2 coding style issues

2016-04-27 Thread Carsey, Jaben
Reviewed-by: Jaben Carsey > -Original Message- > From: edk2-devel [mailto:edk2-devel-boun...@lists.01.org] On Behalf Of El- > Haj-Mahmoud, Samer > Sent: Wednesday, April 27, 2016 8:11 AM > To: Shah, Tapan ; edk2-devel@lists.01.org > Cc: Carsey, Jaben > Subject: Re: [edk2] [PATCH] ShellPk

Re: [edk2] [PATCH] ArmPlatformPkg: implement CpuIo2 protocol driver specific for PCI

2016-04-27 Thread Leif Lindholm
On Wed, Apr 27, 2016 at 05:48:55PM +0200, Ard Biesheuvel wrote: > >> So we can implement this protocol using MMIO operations only, and take the > >> PCI I/O translation offset into account when performing I/O port accesses. > > > > Some of these lines look a bit long? > > Will fix that Thanks. >

Re: [edk2] [Patch] MdeModulePkg: DxeCore MemoryPool Algorithm Update

2016-04-27 Thread Ard Biesheuvel
On 27 April 2016 at 08:40, Liming Gao wrote: > Use 128 bytes as the start size region to be same to previous one. > > 64 bytes is small as the first range. On X64 arch, POOL_OVERHEAD > takes 40 bytes, the pool data less than 24 bytes can be fit into > it. But, the real allocation is few that can't

Re: [edk2] [PATCH] MdeModulePkg/TerminalDxe: Set polling rate by serial IO mode

2016-04-27 Thread Kinney, Michael D
Heyi, 1) A couple ways to measure. a) Loop calling gBS->Stall() and counting number of stall calls between 2 periodic timer event notifications. The accuracy of the measurement depends on the granularity and accuracy of the Stall service. For your use case, using a Stall() per

Re: [edk2] [PATCH] ArmPlatformPkg: implement CpuIo2 protocol driver specific for PCI

2016-04-27 Thread Ard Biesheuvel
On 27 April 2016 at 17:44, Leif Lindholm wrote: > On Wed, Apr 27, 2016 at 02:58:29PM +0200, Ard Biesheuvel wrote: >> The CpuIo2 protocol is required by the generic PciHostBridgeDxe driver, >> which relies on it to back its own I/O and MMIO operations. >> >> Since ARM has no native I/O port equival

Re: [edk2] [PATCH] ArmPlatformPkg: implement CpuIo2 protocol driver specific for PCI

2016-04-27 Thread Laszlo Ersek
On 04/27/16 17:44, Leif Lindholm wrote: > On Wed, Apr 27, 2016 at 02:58:29PM +0200, Ard Biesheuvel wrote: >> The CpuIo2 protocol is required by the generic PciHostBridgeDxe driver, >> which relies on it to back its own I/O and MMIO operations. >> >> Since ARM has no native I/O port equivalent, such

Re: [edk2] [PATCH] ArmPlatformPkg: implement CpuIo2 protocol driver specific for PCI

2016-04-27 Thread Leif Lindholm
On Wed, Apr 27, 2016 at 02:58:29PM +0200, Ard Biesheuvel wrote: > The CpuIo2 protocol is required by the generic PciHostBridgeDxe driver, > which relies on it to back its own I/O and MMIO operations. > > Since ARM has no native I/O port equivalent, such accesses can only originate > from PCI drive

Re: [edk2] [PATCH] MdeModulePkg/TerminalDxe: Set polling rate by serial IO mode

2016-04-27 Thread Heyi Guo
Hi Michael, I still have questions about measuring timer tick by event: 1. How can we measure the time elapsed? TimerLib is not in UEFI spec, and shall we use gBS->Stall in a loop to measure the time? 2. By using a periodic timer event, I think we need to drop the 1st trigger as it is random a

Re: [edk2] [PATCH] ShellPkg: Fix typos and EDK2 coding style issues

2016-04-27 Thread El-Haj-Mahmoud, Samer
Reviewed-by: Samer El-Haj-Mahmoud -Original Message- From: edk2-devel [mailto:edk2-devel-boun...@lists.01.org] On Behalf Of Shah, Tapan Sent: Wednesday, April 27, 2016 9:56 AM To: edk2-devel@lists.01.org Cc: jaben.car...@intel.com Subject: [edk2] [PATCH] ShellPkg: Fix typos and EDK2 cod

Re: [edk2] [PATCH] ArmPlatformPkg: implement CpuIo2 protocol driver specific for PCI

2016-04-27 Thread Laszlo Ersek
On 04/27/16 14:58, Ard Biesheuvel wrote: > The CpuIo2 protocol is required by the generic PciHostBridgeDxe driver, > which relies on it to back its own I/O and MMIO operations. > > Since ARM has no native I/O port equivalent, such accesses can only originate > from PCI drivers, and the PCI I/O spa

Re: [edk2] [Patch] ShellPkg: Enhance ping6 to select the interface automatically

2016-04-27 Thread Carsey, Jaben
Reviewed-by: Jaben Carsey > -Original Message- > From: Wu, Jiaxin > Sent: Tuesday, April 26, 2016 6:21 PM > To: Bhupesh Sharma ; edk2-devel@lists.01.org > Cc: Carsey, Jaben ; Ye, Ting ; > Fu, Siyuan > Subject: RE: [edk2] [Patch] ShellPkg: Enhance ping6 to select the interface > automatic

Re: [edk2] [Patch v3 11/23] OvmfPkg/PlatformBds: Initialize console variables in *BeforeConsole()

2016-04-27 Thread Laszlo Ersek
On 04/21/16 08:58, Ruiyu Ni wrote: > The major difference between IntelFrameworkModulePkg/BDS and > MdeModulePkg/BDS is the latter connects the consoles in core > code while the former connects in platform code. > The change initializes the console variables in > PlatformBootManagerBeforeConsole()

[edk2] [PATCH] ShellPkg: Fix typos and EDK2 coding style issues

2016-04-27 Thread Tapan Shah
Fixing typos and EDK2 coding style issues found from previous submit Contributed-under: TianoCore Contribution Agreement 1.0 Signed-off-by: Tapan Shah --- .../UefiHandleParsingLib/UefiHandleParsingLib.c | 20 ++-- 1 file changed, 10 insertions(+), 10 deletions(-) diff --git

Re: [edk2] [Patch v3 10/23] OvmfPkg/PlatformBds: Use ConvertDevicePathToText in DevicePathLib

2016-04-27 Thread Laszlo Ersek
On 04/21/16 08:58, Ruiyu Ni wrote: > Contributed-under: TianoCore Contribution Agreement 1.0 > Signed-off-by: Ruiyu Ni > Cc: Jordan Justen > Cc: Laszlo Ersek > --- > OvmfPkg/Library/PlatformBootManagerLib/BdsPlatform.c | 6 +++--- > OvmfPkg/Library/PlatformBootManagerLib/PlatformBo

Re: [edk2] [Patch v3 09/23] OvmfPkg/PlatformBds: link to UefiBootManagerLib

2016-04-27 Thread Laszlo Ersek
On 04/21/16 08:57, Ruiyu Ni wrote: > Contributed-under: TianoCore Contribution Agreement 1.0 > Signed-off-by: Ruiyu Ni > Cc: Jordan Justen > Cc: Laszlo Ersek > --- > OvmfPkg/Library/PlatformBootManagerLib/BdsPlatform.h | 2 +- > OvmfPkg/Library/PlatformBootManagerLib/PlatformBootMa

Re: [edk2] [Patch v3 08/23] OvmfPkg/PlatformBds: use EfiBootManagerUpdateConsoleVariable

2016-04-27 Thread Laszlo Ersek
On 04/21/16 08:57, Ruiyu Ni wrote: > Call EfiBootManagerUpdateConsoleVariable in UefiBootManagerLib > instead of BdsLibUpdateConsoleVariable in GenericBdsLib. > > Still cannot pass build. > > Contributed-under: TianoCore Contribution Agreement 1.0 > Signed-off-by: Ruiyu Ni > Cc: Jordan Justen >

[edk2] [PATCH] ArmPlatformPkg: implement CpuIo2 protocol driver specific for PCI

2016-04-27 Thread Ard Biesheuvel
The CpuIo2 protocol is required by the generic PciHostBridgeDxe driver, which relies on it to back its own I/O and MMIO operations. Since ARM has no native I/O port equivalent, such accesses can only originate from PCI drivers, and the PCI I/O space is translated to MMIO in this case. So we can i

Re: [edk2] [Patch v3 07/23] OvmfPkg/PlatformBds: Follow PlatformBootManagerLib interfaces

2016-04-27 Thread Laszlo Ersek
On 04/21/16 08:57, Ruiyu Ni wrote: > Change the function name to follow new library class > PlatformBootManagerLib interfaces. > > Contributed-under: TianoCore Contribution Agreement 1.0 > Signed-off-by: Ruiyu Ni > Cc: Jordan Justen > Cc: Laszlo Ersek > --- > .../Library/PlatformBootManagerLib

Re: [edk2] Minor problems with dualboot VM. EFIVars?

2016-04-27 Thread Blank Field
> Yes, they are /var/lib/libvirt/qemu/nvram/${vmname}_VARS.fd. So the VM is(should be) configured properly - one r/o .fd with the OVMF_CODE, one r/w .fd with vars per VM. > Your NvVars file could be very old, maybe a remnant of your earlier Will check the date, but i doubt about this. Maybe there

Re: [edk2] Minor problems with dualboot VM. EFIVars?

2016-04-27 Thread Laszlo Ersek
On 04/27/16 12:43, Blank Field wrote: >> Because your VM configuration is incorrect (you didn't configure two >> pflash chips, one for storing the UEFI variables, another (r/o) for >> presenting the firmware binary). Hence OVMF fell back to the historical >> hack where it stored (some of) the UEFI

Re: [edk2] Minor problems with dualboot VM. EFIVars?

2016-04-27 Thread Blank Field
> Because your VM configuration is incorrect (you didn't configure two > pflash chips, one for storing the UEFI variables, another (r/o) for > presenting the firmware binary). Hence OVMF fell back to the historical > hack where it stored (some of) the UEFI variables in a file called > NvVars on the

Re: [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-devel@lists.01.org; Xen Devel ; Laszlo >> Ersek >> Subject: RE: [edk2] OVMF broken under Xen (in PC

Re: [edk2] Minor problems with dualboot VM. EFIVars?

2016-04-27 Thread Laszlo Ersek
On 04/27/16 11:52, Blank Field wrote: > And i've thought that efivars on a physical platform are located > somewhere in a separate chip on the motherboard. Why i've found it on > the ESP? Because your VM configuration is incorrect (you didn't configure two pflash chips, one for storing the UEFI v

Re: [edk2] [Patch v3 00/23] Use MdeModulePkg/BDS in OVMF platform

2016-04-27 Thread Laszlo Ersek
On 04/27/16 11:53, Michael Zimmermann wrote: > well all ARM targets are still using Intel's BDS and iirc this even is > the "new" one for them because previously they were using the ARM bds. Once this series converges for OvmfPkg, we intend to port ArmVirtPkg as well, using the lessons learned her

Re: [edk2] Minor problems with dualboot VM. EFIVars?

2016-04-27 Thread Blank Field
>Since the ESP and the main partitions are also accessible from the host(which would be almost impossible to achieve if the VM would be stored in a file)... ...i was able to boot it bare-metal. Sorry, lost a bit of text due to small screen size on a mobile client. __

Re: [edk2] [Patch v3 00/23] Use MdeModulePkg/BDS in OVMF platform

2016-04-27 Thread Michael Zimmermann
well all ARM targets are still using Intel's BDS and iirc this even is the "new" one for them because previously they were using the ARM bds. Michael On Wed, Apr 27, 2016 at 3:32 AM, Andrew Fish wrote: > > > On Apr 26, 2016, at 8:58 AM, Laszlo Ersek wrote: > > > > On 04/26/16 17:38, Michael Zi

Re: [edk2] Minor problems with dualboot VM. EFIVars?

2016-04-27 Thread Blank Field
> I don't know what "windows exVM" means. It is my fault not giving enough backstory, sorry. The windows system was previously installed in a VM using host disk as a storage. Since the ESP and the main partitions are also accessible from the host(which would be almost impossible to achieve if the V

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

2016-04-27 Thread Ni, Ruiyu
Copying Mike. Regards, Ray >-Original Message- >From: Ni, Ruiyu >Sent: Wednesday, April 27, 2016 5:49 PM >To: 'Gary Lin' >Cc: edk2-devel@lists.01.org; Xen Devel ; Laszlo Ersek > >Subject: RE: [edk2] OVMF broken under Xen (in PCI initialisation) > >Gary, >Thanks for the try. > >I now und

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

2016-04-27 Thread Ni, Ruiyu
Gary, Thanks for the try. I now understand why the issue happens. 1. PciEnumeratorLight() depends on the MinBus returned from PciRootBridgeIo.Configuration(). 2. PciRootBridgeIo.Configuration() returns the resource descriptors initialized by PciHostBridge.NotifyPhase(EfiPciHostBridgeAllocateResour

Re: [edk2] Minor problems with dualboot VM. EFIVars?

2016-04-27 Thread Laszlo Ersek
On 04/27/16 11:13, Laszlo Ersek wrote: > On 04/27/16 01:51, Blank Field wrote: >> Hi. I am attempting to rebuild my system with a host EFI bootloader, >> namely GRUB. >> I did a fresh install of F23(i've found that way easier than figuring >> out how to migrate to EFI), turned off my CSM in host UE

Re: [edk2] Minor problems with dualboot VM. EFIVars?

2016-04-27 Thread Laszlo Ersek
On 04/27/16 01:51, Blank Field wrote: > Hi. I am attempting to rebuild my system with a host EFI bootloader, > namely GRUB. > I did a fresh install of F23(i've found that way easier than figuring > out how to migrate to EFI), turned off my CSM in host UEFI setup, got > my windows exVM booting in pu

Re: [edk2] [Patch] NetworkPkg: Fix incorrect buffer free in HttpDxe

2016-04-27 Thread Ye, Ting
Reviewed-by: Ye Ting -Original Message- From: Wu, Jiaxin Sent: Wednesday, April 27, 2016 3:08 PM To: edk2-devel@lists.01.org Cc: Ye, Ting ; Fu, Siyuan ; Zhang, Lubo Subject: [Patch] NetworkPkg: Fix incorrect buffer free in HttpDxe FragmentBuffer of each TcpWrap in HttpDxe should not

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

2016-04-27 Thread Gary Lin
On Wed, Apr 27, 2016 at 07:18:21AM +, Ni, Ruiyu wrote: > Gary, > In PciEnumeratorLight(), please directly assign 0 to MinBus when > Configuration() returns error, instead of calling PciGetBusRange() to > extract MinBus from the descriptors returned from Configuration(). > OK, I ignored PciGetB

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

2016-04-27 Thread Ni, Ruiyu
Gary, In PciEnumeratorLight(), please directly assign 0 to MinBus when Configuration() returns error, instead of calling PciGetBusRange() to extract MinBus from the descriptors returned from Configuration(). Regards, Ray >-Original Message- >From: Gary Lin [mailto:g...@suse.com] >Sent: We

[edk2] [Patch] NetworkPkg: Fix incorrect buffer free in HttpDxe

2016-04-27 Thread Jiaxin Wu
FragmentBuffer of each TcpWrap in HttpDxe should not be freed in HttpTcpTokenCleanup(). This buffer points to HttpMsg body actually, which is the responsibility of the caller to allocate a buffer for Body. Cc: Ye Ting Cc: Fu Siyuan Cc: Zhang Lubo Contributed-under: TianoCore Contribution Agreem