|Summary:||AAVMF: output improvements|
|Product:||Red Hat Enterprise Linux 7||Reporter:||Andrew Jones <drjones>|
|Component:||OVMF||Assignee:||Laszlo Ersek <lersek>|
|Status:||CLOSED CURRENTRELEASE||QA Contact:||Virtualization Bugs <virt-bugs>|
|Severity:||unspecified||Docs Contact:||Jiri Herrmann <jherrman>|
|Version:||7.1||CC:||chayang, juzhang, lmiksik, michen, mrezanin, qizhu|
|Target Milestone:||rc||Keywords:||OtherQA, Rebase|
|Fixed In Version:||ovmf-20151104-1.gitb9ffeab.el7||Doc Type:||No Doc Update|
no doc update
|Last Closed:||2017-10-10 10:58:37 UTC||Type:||Bug|
|oVirt Team:||---||RHEL 7.3 requirements from Atomic Host:|
Description Andrew Jones 2015-10-09 13:32:45 UTC
Two suggestions for output improvements 1) When creating a brand new guest there's typically no nvram yet. AAVMF reports this as an error, but then corrects it and moves on fine. I.e. ValidateFvHeader: Variable Store Guid non-compatible NorFlashFvbInitialize: ERROR - The FVB Header is not valid. Installing a correct one for this volume. Ftw: Work block header check error Ftw: Work block header check error Ftw: Both are invalid, init workspace Status messages that don't say "error" might be nicer, as this is standard new guest behavior. 2) I once requested quieting AAVMF's output messages, so we now have the default (quiet) and also a verbose version that can be used. Now I'd like to suggest that the quiet one output at least one message though, something like "Starting UEFI...", because currently several seconds go by with no output - long enough to start suspecting a hang.
Comment 4 Laszlo Ersek 2015-10-13 12:03:24 UTC
I posted a patch set [edk2] [PATCH 0/3] soften some (non-)ERROR messages http://thread.gmane.org/gmane.comp.bios.edk2.devel/2902 that should take care of the annoying messages in the current state of upstream. With regard to the "Starting UEFI..." message, to be printed early in SEC, I think we should do that in downstream only.
Comment 5 Laszlo Ersek 2015-10-13 15:50:11 UTC
Based on further discussion, I'll try to log a "Starting UEFI..."-like message in upstream too, on DEBUG_INFO level, and in downstream we could flip that to DEBUG_ERROR.
Comment 6 Laszlo Ersek 2015-10-14 12:31:52 UTC
Comment 7 Laszlo Ersek 2015-10-26 14:24:49 UTC
The first four patches of v2 have been committed. The second half seems to be generally ignored, so we should carry those three of said four patches that we actually need in downstream only.
Comment 13 Laszlo Ersek 2017-10-10 10:58:37 UTC
Fixed in RHEL-ALT-7.3.