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 >

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

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

2020-07-31 Thread Laszlo Ersek
Hi Sean, thank you for reporting this. I apologize for the breakage. Please see my comments below. (Rebecca and the stewards should read on as well, please.) On 07/31/20 19:32, Sean Brogan wrote: > This patch as committed is breaking CI.  It was not captured in PR > because the PR optimizes to

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

2020-07-31 Thread Rebecca Cran
On 7/31/20 11:32 AM, Sean Brogan wrote: This patch as committed is breaking CI.  It was not captured in PR because the PR optimizes to detect packages impacted by the commits and the BhyvePkg addition is not depended on by other packages (that are in CI).  BhyvePkg which is nested inside

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

2020-07-31 Thread Sean
This patch as committed is breaking CI. It was not captured in PR because the PR optimizes to detect packages impacted by the commits and the BhyvePkg addition is not depended on by other packages (that are in CI). BhyvePkg which is nested inside OvmfPkg ( a violation of DEC spec: see

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

2020-07-31 Thread Laszlo Ersek
On 07/31/20 15:07, Laszlo Ersek wrote: > On 07/31/20 07:17, Rebecca Cran wrote: >> On 7/13/20 2:25 PM, Rebecca Cran wrote: >> >>> On 7/13/20 12:09 PM, Laszlo Ersek wrote: >>> Can you please ping me when the PatchCheck.py problem has been solved, so I can go ahead with the merge? >>> >>>

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

2020-07-31 Thread Laszlo Ersek
On 07/31/20 07:17, Rebecca Cran wrote: > On 7/13/20 2:25 PM, Rebecca Cran wrote: > >> On 7/13/20 12:09 PM, Laszlo Ersek wrote: >> >>> Can you please ping me when the PatchCheck.py problem has been solved, >>> so I can go ahead with the merge? >> >> Will do. > > > Hi Laszlo, > > > I just

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

2020-07-30 Thread Rebecca Cran
On 7/13/20 2:25 PM, Rebecca Cran wrote: On 7/13/20 12:09 PM, Laszlo Ersek wrote: Can you please ping me when the PatchCheck.py problem has been solved, so I can go ahead with the merge? Will do. Hi Laszlo, I just rebased the bhyve commit against the latest edk2 master with no

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

2020-07-13 Thread Rebecca Cran
On 7/13/20 12:09 PM, Laszlo Ersek wrote: Can you please ping me when the PatchCheck.py problem has been solved, so I can go ahead with the merge? Will do. -- Rebecca Cran -=-=-=-=-=-=-=-=-=-=-=- Groups.io Links: You receive all messages sent to this group. View/Reply Online (#62455):

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

2020-07-13 Thread Rebecca Cran
On 7/13/20 10:16 AM, Laszlo Ersek wrote: My understanding has been that you and Peter Grehan would be added as "R"s to Maintainers.txt, covering "OvmfPkg/Bhyve/". Has anything changed in that regard? No, that's still the plan. I'll update Maintainers.txt in a follow-up patch. -- Rebecca

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

2020-07-13 Thread Laszlo Ersek
Hi Rebecca, On 07/13/20 18:55, Laszlo Ersek wrote: > On 07/13/20 07:41, Rebecca Cran wrote: >> BhyvePkg supports the bhyve hypervisor, which is a hypervisor/virtual >> machine manager available on FreeBSD, macOS and Illumos. >> >> Signed-off-by: Rebecca Cran >> --- >>

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

2020-07-13 Thread Laszlo Ersek
On 07/13/20 07:41, Rebecca Cran wrote: > BhyvePkg supports the bhyve hypervisor, which is a hypervisor/virtual > machine manager available on FreeBSD, macOS and Illumos. > > Signed-off-by: Rebecca Cran > --- > OvmfPkg/Bhyve/AcpiPlatformDxe/AcpiPlatform.c | 251 +++ >

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

2020-07-13 Thread Laszlo Ersek
On 07/13/20 07:41, Rebecca Cran wrote: > BhyvePkg supports the bhyve hypervisor, which is a hypervisor/virtual > machine manager available on FreeBSD, macOS and Illumos. > > Signed-off-by: Rebecca Cran > --- > OvmfPkg/Bhyve/AcpiPlatformDxe/AcpiPlatform.c | 251 +++ >