Re: [edk2] [PATCH 5/6] MdeModulePkg/VariableRuntimeDxe: factor out boot service accesses

2019-01-09 Thread Zeng, Star
Hi Ard, Some minor feedback added inline. On 2019/1/4 2:28, Ard Biesheuvel wrote: In preparation of providing a standalone MM based variable runtime driver, move the existing SMM driver to the new MM services table, and factor out some pieces that are specific to the traditional driver, mainly

Re: [edk2] [PATCH 5/6] MdeModulePkg/VariableRuntimeDxe: factor out boot service accesses

2019-01-09 Thread Zeng, Star
On 2019/1/10 10:33, Wang, Jian J wrote: Laszlo, Regards, Jian -Original Message- From: Laszlo Ersek [mailto:ler...@redhat.com] Sent: Tuesday, January 08, 2019 11:38 PM To: Ard Biesheuvel ; edk2-devel@lists.01.org Cc: Leif Lindholm ; Kinney, Michael D ; Gao, Liming ; Wang, Jian J ; Wu,

Re: [edk2] [PATCH 5/6] MdeModulePkg/VariableRuntimeDxe: factor out boot service accesses

2019-01-09 Thread Wang, Jian J
Laszlo, Regards, Jian > -Original Message- > From: Laszlo Ersek [mailto:ler...@redhat.com] > Sent: Tuesday, January 08, 2019 11:38 PM > To: Ard Biesheuvel ; edk2-devel@lists.01.org > Cc: Leif Lindholm ; Kinney, Michael D > ; Gao, Liming ; Wang, > Jian J ; Wu, Hao A ; Jagadeesh > Ujja ;

Re: [edk2] [PATCH 5/6] MdeModulePkg/VariableRuntimeDxe: factor out boot service accesses

2019-01-08 Thread Laszlo Ersek
On 01/03/19 19:28, Ard Biesheuvel wrote: > In preparation of providing a standalone MM based variable runtime > driver, move the existing SMM driver to the new MM services table, > and factor out some pieces that are specific to the traditional > driver, mainly related to the use of UEFI boot

[edk2] [PATCH 5/6] MdeModulePkg/VariableRuntimeDxe: factor out boot service accesses

2019-01-03 Thread Ard Biesheuvel
In preparation of providing a standalone MM based variable runtime driver, move the existing SMM driver to the new MM services table, and factor out some pieces that are specific to the traditional driver, mainly related to the use of UEFI boot services, which are not accessible to standalone MM