Re: [patch 0/2] MdeModulePkg/HiiDatabaseDxe: Make sure database update behaviors are atomic

["Gao, Liming" <liming.gao@intel.com>]

Reviewed-by: Liming Gao <liming.gao@intel.com>

> -----Original Message-----
> From: Bi, Dandan
> Sent: Friday, October 12, 2018 7:26 PM
> To: edk2-devel@lists.01.org
> Cc: Gao, Liming <liming.gao@intel.com>; Dong, Eric <eric.dong@intel.com>
> Subject: [patch 0/2] MdeModulePkg/HiiDatabaseDxe: Make sure database update behaviors are atomic
> 
> The main purpose of this task is to make sure the operations
> that update the HiiDatabase atomic the avoid the potential
> risk that the one update operation with higher TPL may interrupt
> another.
> 
> Patch 1 is to reorgnize the existing code logic and make it's
> easy to add EfiAcquireLock/EfiReleaseLock function in patch 2.
> 
> Patch 2 is to add EfiAcquireLock/EfiReleaseLock function to
> make sure the HiiDatabse update operations atomic.
> 
> Cc: Liming Gao <liming.gao@intel.com>
> Cc: Eric Dong <eric.dong@intel.com>
> Dandan Bi (2):
>   MdeModulePkg/HiiDB: Reorganize codes of exporting HII settings
>   MdeModulePkg/HiiDB: Make sure database update behaviors are atomic
> 
>  .../Universal/HiiDatabaseDxe/Database.c       | 128 ++++++++++++------
>  .../Universal/HiiDatabaseDxe/HiiDatabase.h    |   8 +-
>  .../HiiDatabaseDxe/HiiDatabaseEntry.c         |   3 +-
>  MdeModulePkg/Universal/HiiDatabaseDxe/Image.c |  12 ++
>  .../Universal/HiiDatabaseDxe/String.c         |   9 ++
>  5 files changed, 112 insertions(+), 48 deletions(-)
> 
> --
> 2.18.0.windows.1