Re: [edk2-devel] [Patch] UefiCpuPkg/CpuCommonFeaturesLib: Fix spelling mistake

2020-08-01 Thread Dong, Eric
Reviewed-by: Eric Dong > -Original Message- > From: Kinney, Michael D > Sent: Saturday, August 1, 2020 8:28 AM > To: devel@edk2.groups.io > Cc: Dong, Eric ; Ni, Ray ; Laszlo > Ersek ; Kumar, Rahul1 > Subject: [Patch] UefiCpuPkg/CpuCommonFeaturesLib: Fix spelling mistake > > REF: https:

Re: [edk2-devel] [PATCH] Check return status on calls to GetTime()

2020-08-01 Thread Heinrich Schuchardt
On 7/31/20 7:19 PM, Grant Likely wrote: > Not all platforms implement GetTime(), but the SCT just assumes calls to > GetTime will be successful. If GetTime() doesn't return EFI_SUCCESS, > then the EFI_TIME value will be uninitialized data. > > Fix by checking the GetTime() return code. If it doesn'

[edk2-devel] [PATCH] OvmfPkg: fix DEC spec violation introduced by Bhyve addition

2020-08-01 Thread Laszlo Ersek
Sean reports that having two DEC files under OvmfPkg violates the DEC spec: > An EDK II Package (directory) is a directory that contains an EDK II > package declaration (DEC) file. Only one DEC file is permitted per > directory. EDK II Packages cannot be nested within other EDK II > Packages. Thi

Re: [edk2-devel] [PATCH 1/1] Add BhyvePkg, to support the bhyve hypervisor

2020-08-01 Thread Laszlo Ersek
On 07/31/20 23:14, Laszlo Ersek wrote: > Regarding actual actions: I'm going to be away for a short while now. > Plus, I'm not entirely sure what exactly is being prevented by the > current state of the tree (i.e., how grave the regression is). > > (1) If the current issue interferes with work on

Re: [edk2-devel] [PATCH] OvmfPkg: fix DEC spec violation introduced by Bhyve addition

2020-08-01 Thread Philippe Mathieu-Daudé
On 8/1/20 5:50 PM, Laszlo Ersek wrote: > Sean reports that having two DEC files under OvmfPkg violates the DEC > spec: > >> An EDK II Package (directory) is a directory that contains an EDK II >> package declaration (DEC) file. Only one DEC file is permitted per >> directory. EDK II Packages canno

Re: [edk2-devel] [PATCH v13 45/46] UefiCpuPkg/MpInitLib: Prepare SEV-ES guest APs for OS use

2020-08-01 Thread Laszlo Ersek
On 07/31/20 23:38, Laszlo Ersek wrote: > On 07/31/20 16:47, Tom Lendacky wrote: >> On 7/31/20 9:44 AM, Tom Lendacky wrote: >>> On 7/31/20 8:36 AM, Tom Lendacky wrote: On 7/31/20 7:43 AM, Laszlo Ersek wrote: > Hi Tom, Hi Laszlo, >>> >>> Hi Laszlo, >>> >>> Can you try this incremen

[edk2-devel] please review the pending MdeModulePkg patches

2020-08-01 Thread Laszlo Ersek
Hi Jian, Hao, the following patches have been on the list for several *months* now (multiple versons): - MdeModulePkg: Create PCDs to be used in support of SEV-ES - MdeModulePkg/DxeIplPeim: Support GHCB pages when creating page tables The latest posted version of the containing series is v13. D

Re: [edk2-devel] [PATCH] OvmfPkg: fix DEC spec violation introduced by Bhyve addition

2020-08-01 Thread Rebecca Cran
On 8/1/20 9:50 AM, Laszlo Ersek wrote: - Repo: https://github.com/lersek/edk2.git Branch: bhyve_dec_fix - Personal CI build: https://github.com/tianocore/edk2/pull/846 - Because the DEC spec violation is breaking CI, I'm going to push this as soon as R

Re: [edk2-devel] [Patch 4/6] OvmfPkg: Fix spelling mistake for occurred

2020-08-01 Thread Laszlo Ersek
On 08/01/20 03:04, Michael D Kinney wrote: > REF: https://bugzilla.tianocore.org/show_bug.cgi?id=2361 > > Cc: Jordan Justen > Cc: Laszlo Ersek > Cc: Ard Biesheuvel > Signed-off-by: Michael D Kinney > --- > OvmfPkg/Bhyve/BhyveRfbDxe/GopScreen.c | 2 +- > OvmfPkg/Include/Protocol/Legacy8259.h |

Re: [edk2-devel] [Patch] UefiCpuPkg/CpuCommonFeaturesLib: Fix spelling mistake

2020-08-01 Thread Laszlo Ersek
On 08/01/20 02:28, Michael D Kinney wrote: > REF: https://bugzilla.tianocore.org/show_bug.cgi?id=2357 > > Cc: Eric Dong > Cc: Ray Ni > Cc: Laszlo Ersek > Cc: Rahul Kumar > Signed-off-by: Michael D Kinney > --- > UefiCpuPkg/Library/CpuCommonFeaturesLib/MachineCheck.c | 2 +- > 1 file changed,

Re: [edk2-devel] [PATCH] OvmfPkg: fix DEC spec violation introduced by Bhyve addition

2020-08-01 Thread Laszlo Ersek
On 08/01/20 17:50, Laszlo Ersek wrote: > Sean reports that having two DEC files under OvmfPkg violates the DEC > spec: > >> An EDK II Package (directory) is a directory that contains an EDK II >> package declaration (DEC) file. Only one DEC file is permitted per >> directory. EDK II Packages canno

Re: [edk2-devel] [PATCH 1/1] Add BhyvePkg, to support the bhyve hypervisor

2020-08-01 Thread Laszlo Ersek
On 07/31/20 19:32, Sean Brogan wrote: > You can see the ReadMe badge showing the broken state of edk2 master. > The build with logs can be seen here > https://dev.azure.com/tianocore/edk2-ci/_build/results?buildId=10494&view=logs&j=ec42d809-3c3b-54a9-276c-e54a8b9aaee9&t=596e0656-4def-5804-b10b-158

[edk2-devel] [PATCH] [EmbeddedPkg]:Update PrePiLib to return DxeCoreEntrypoint

2020-08-01 Thread Guo Dong
Added LoadDxeCore() API to return DxeCore entry point after loading DxeCore from FV, and del LoadDxeCoreFromFfsFile() as it is replaced by LoadDxeCore(). Update LoadDxeCoreFromFv() to use LoadDxeCore() to reduce code, and its behavior is same. Updated FfsProcessSection() to support both IA32 and X6

[edk2-devel] TianoCore Design Meeting - APAC/NAMO - Fri, 08/07/2020 9:30am-10:30am #cal-reminder

2020-08-01 Thread devel@edk2.groups.io Calendar
*Reminder:* TianoCore Design Meeting - APAC/NAMO *When:* Friday, 7 August 2020, 9:30am to 10:30am, (GMT+08:00) Asia/Chongqing *Where:* https://zoom.us/j/299494771 View Event ( https://edk2.groups.io/g/devel/viewevent?eventid=805409 ) *Organizer:* Ray Ni ray...@intel.com ( ray...@intel.com?subj

[edk2-devel] [PATCH 1/1] Maintainers.txt: Add bhyve reviewers

2020-08-01 Thread Rebecca Cran
Bhyve files are under OvmfPkg, in OvmfPkg/Bhyve and various files in OvmfPkg/Library and OvmfPkg/Include. Update Maintainers.txt to indicate reviewers for all bhyve-specific files. Signed-off-by: Rebecca Cran Cc: Andrew Fish Cc: Laszlo Ersek Cc: Leif Lindholm Cc: Michael D Kinney Cc: Jordan