On 01/31/18 06:54, Ni, Ruiyu wrote:
> Laszlo, Mike,
> Considering this patch doesn't make the code worse,
> actually improved a tiny bit, can we firstly check in the three patches?
I agree; the PatchAssembly() discussion is taking quite a bit of
thought, meanwhile IA32 SMM is broken on KVM -- and even on QEMU! (Paolo
helped me test that, and yes, QEMU/TCG is affected the exact same way.)
I will go ahead and push the patches with the reviews from Paolo and
Ray, with the following small modifications:
* In the commit message of the first patch, I'll change
we can never remove
to
we can't yet remove
in the commit message.
* In the third patch, I will change the commit message from
SMM emulation under KVM
to
SMM emulation under both KVM and QEMU (TCG)
* In the third patch, I will update the code comments as requested by
Ray.
> Reviewed-by: Ruiyu Ni <ruiyu...@intel.com>
Thanks!
Laszlo
_______________________________________________
edk2-devel mailing list
edk2-devel@lists.01.org
https://lists.01.org/mailman/listinfo/edk2-devel
