Repo: https://github.com/lersek/edk2.git Branch: sev_debug_messages
I've now looked at quite a bit of SEV-related DEBUG messages, and I find them really hard to read, and to process with command line utilities like grep, sort, uniq. For example, shell pipelines should be possible to construct with these utilities to collect decrypt/encrypt operations, in chronological order, grouped by address ever decrypted. The log should also be human-readable as much as possible -- it shouldn't be redundant, but all information for forward- and back-referencing should be available. I propose this series for the above improvements. Cc: Brijesh Singh <brijesh.si...@amd.com> Cc: Jordan Justen <jordan.l.jus...@intel.com> Thanks, Laszlo Laszlo Ersek (11): OvmfPkg/BaseMemEncryptSevLib: unify encrypt/decrypt DEBUG messages OvmfPkg/BaseMemEncryptSevLib: break DEBUG calls to multiple lines OvmfPkg/BaseMemEncryptSevLib: clean up DEBUG prefixes OvmfPkg/BaseMemEncryptSevLib: clean up debug logging of PhysicalAddress OvmfPkg/BaseMemEncryptSevLib: promote DEBUG_WARN levels to DEBUG_ERROR OvmfPkg/BaseMemEncryptSevLib: clean up upper-case / lower-case in DEBUGs OvmfPkg/BaseMemEncryptSevLib: fix typos in DEBUG messages OvmfPkg/IoMmuDxe: IoMmuMap(): log nicer and more informative DEBUG msgs OvmfPkg/IoMmuDxe: IoMmuUnmap(): clean up DEBUG message OvmfPkg/IoMmuDxe: IoMmuAllocateBuffer(): nicer and more informative DEBUGs OvmfPkg/IoMmuDxe: IoMmuFreeBuffer(): clean up DEBUG message OvmfPkg/IoMmuDxe/AmdSevIoMmu.c | 72 +++++++++++----- OvmfPkg/Library/BaseMemEncryptSevLib/X64/VirtualMemory.c | 89 ++++++++++++++------ 2 files changed, 110 insertions(+), 51 deletions(-) -- 2.14.1.3.gb7cf6e02401b _______________________________________________ edk2-devel mailing list edk2-devel@lists.01.org https://lists.01.org/mailman/listinfo/edk2-devel