Re: [Patch v5 48/48] UefiCpuPkg/PiSmmCpuDxeSmm: Add gEfiVariableArchProtocolGuid dependency

[Laszlo Ersek <lersek@redhat.com>]

On 08/19/16 04:26, Laszlo Ersek wrote:
> On 08/19/16 04:00, Fan, Jeff wrote:
>> Laszlo,
>>
>> I could revert this patch firstly.
> 
> Thank you, that would be very kind.
> 
>> Could you please dig out the OVMF to address the potential issue, then I could re-commit it for those platforms required this patch.
> 
> The problem is that this week (what remains of it) and the next week I
> won't really have time for this -- tomorrow I'm hoping to finish up
> something else in a mortal hurry. It was actually in preparation for
> rebasing / pushing that work that I pulled "master", and found out about
> the regression.
> 
> Can we perhaps get help from the variable stack maintainers? What's the
> design of the (lack of) depexes on those drivers?
> 
> Anyway, if you could live with the patch reverted for one or two weeks,
> then reverting it would be best, IMO. It results in a regression, even
> if ultimately it might only expose a bug in something else.

I can see the revert (commit eadf70bdfbc1cb14a1d52b4840ca831d172d0399),
thank you! As I said, I hope I can look into this (as well...) the week
after next, or maybe a bit later... Sorry about the inconvenience :(

Laszlo