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

2020-04-24 Thread Laszlo Ersek
On 04/22/20 18:48, Rebecca Cran wrote: > On 4/22/20 9:21 AM, Laszlo Ersek wrote: > >> OK, that makes sense -- but, without the ResetSystemLib instance, are >> you able to boot the BhyvePkg platform firmware in a bhyve guest? Does >> (for example) the "reset -c" UEFI Shell command work? > >

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

2020-04-22 Thread Rebecca Cran
On 4/22/20 9:21 AM, Laszlo Ersek wrote: OK, that makes sense -- but, without the ResetSystemLib instance, are you able to boot the BhyvePkg platform firmware in a bhyve guest? Does (for example) the "reset -c" UEFI Shell command work? "reset -c" works, but "reset -s" doesn't. But that's a

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

2020-04-22 Thread Laszlo Ersek
On 04/21/20 17:38, Rebecca Cran wrote: > On 4/21/20 9:27 AM, Laszlo Ersek wrote: > >> >> (1) For example, I can't find the patch that adds >> "BhyvePkg/BhyvePkgX64.dsc", in spite of the file being listed in the >> cumulative diffstat below. > > That's in the "Add BhyvePkg" patch. I'm very

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

2020-04-21 Thread Rebecca Cran
On 4/21/20 9:27 AM, Laszlo Ersek wrote: (1) For example, I can't find the patch that adds "BhyvePkg/BhyvePkgX64.dsc", in spite of the file being listed in the cumulative diffstat below. That's in the "Add BhyvePkg" patch. (2) I also don't really understand why v2 / v3 have been posted,

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

2020-04-21 Thread Laszlo Ersek
Hi Rebecca, On 04/21/20 05:09, Rebecca Cran wrote: > I noticed a few things wrong with the v2 series: > > o BhyveFwCtlLibNull is no longer used, and could be deleted. > o Some changes from the v1 AcpiPlatformDxe patch were left in OvmfPkg. > o BhyvePkg/License.txt still referred to OvmfPkg. I've

[edk2-devel] [PATCH v3 0/6] Add BhyvePkg, to support the bhyve hypervisor

2020-04-20 Thread Rebecca Cran
I noticed a few things wrong with the v2 series: o BhyveFwCtlLibNull is no longer used, and could be deleted. o Some changes from the v1 AcpiPlatformDxe patch were left in OvmfPkg. o BhyvePkg/License.txt still referred to OvmfPkg. I've updated it so everything under BhyvePkg should be