On 4/17/20 5:59 PM, Ard Biesheuvel wrote:
On 4/17/20 5:37 PM, Laszlo Ersek wrote:
Ref: https://bugzilla.tianocore.org/show_bug.cgi?id=2675
Repo: https://pagure.io/lersek/edk2.git
Branch: rsl_cleanup
Rebecca's
[PATCH 02/13] OvmfPkg: support powering off bhyve guests
at
https://edk2.groups.io/g/devel/message/57450
e4e9b29189b83076e1d1a0b9c989938f5226cab6.1586991816.git.rebecca@bsdio.com">http://mid.mail-archive.com/e4e9b29189b83076e1d1a0b9c989938f5226cab6.1586991816.git.rebecca@bsdio.com
made me realize OvmfPkg/ResetSystemLib should be refreshed. This will
let us add a simple bhyve-specific instance (later), and also allows us
to fix a long time dormant bug (now).
Cc: Anthony Perard <anthony.per...@citrix.com>
Cc: Ard Biesheuvel <ard.biesheu...@arm.com>
Cc: Jordan Justen <jordan.l.jus...@intel.com>
Cc: Julien Grall <jul...@xen.org>
Cc: Philippe Mathieu-Daudé <phi...@redhat.com>
Cc: Rebecca Cran <rebe...@bsdio.com>
Thanks,
Laszlo
Laszlo Ersek (6):
OvmfPkg/ResetSystemLib: wrap long lines
OvmfPkg/ResetSystemLib: clean up library dependencies
OvmfPkg/ResetSystemLib: improve coding style in ResetSystem()
OvmfPkg/ResetSystemLib: factor out ResetShutdown()
OvmfPkg/ResetSystemLib: rename to BaseResetSystemLib
OvmfPkg/ResetSystemLib: introduce the DxeResetSystemLib instance
For the series,
Reviewed-by: Ard Biesheuvel <ard.biesheu...@arm.com>
One nit: putting a diff block inside the commit log [6/6] doesn't help
legibility a lot, and the issue of not being able to access memory that
is not mapped for runtime is so basic that it doesn't require that level
of detail to describe a reproducer and the Linux kernel log output when
the issue is triggered.
Personally I find the kernel log relevant, it will help to understand th
e patch when looking at it in >5years from now.
OvmfPkg/OvmfPkgIa32.dsc
| 8 +-
OvmfPkg/OvmfPkgIa32X64.dsc
| 8 +-
OvmfPkg/OvmfPkgX64.dsc
| 8 +-
OvmfPkg/OvmfXen.dsc
| 6 +-
OvmfPkg/Library/ResetSystemLib/{ResetSystemLib.inf =>
BaseResetSystemLib.inf} | 12 ++-
OvmfPkg/Library/ResetSystemLib/DxeResetSystemLib.inf
| 43 ++++++++++
OvmfPkg/Library/ResetSystemLib/BaseResetShutdown.c
| 51 ++++++++++++
OvmfPkg/Library/ResetSystemLib/DxeResetShutdown.c
| 62 +++++++++++++++
OvmfPkg/Library/ResetSystemLib/ResetSystemLib.c
| 84 +++++---------------
9 files changed, 209 insertions(+), 73 deletions(-)
rename OvmfPkg/Library/ResetSystemLib/{ResetSystemLib.inf =>
BaseResetSystemLib.inf} (65%)
create mode 100644 OvmfPkg/Library/ResetSystemLib/DxeResetSystemLib.inf
create mode 100644 OvmfPkg/Library/ResetSystemLib/BaseResetShutdown.c
create mode 100644 OvmfPkg/Library/ResetSystemLib/DxeResetShutdown.c
-=-=-=-=-=-=-=-=-=-=-=-
Groups.io Links: You receive all messages sent to this group.
View/Reply Online (#57530): https://edk2.groups.io/g/devel/message/57530
Mute This Topic: https://groups.io/mt/73085501/21656
Group Owner: devel+ow...@edk2.groups.io
Unsubscribe: https://edk2.groups.io/g/devel/unsub [arch...@mail-archive.com]
-=-=-=-=-=-=-=-=-=-=-=-