Re: [patch 0/2] Report error status when fail to load/start boot option

[Laszlo Ersek <lersek@redhat.com>]

On 02/15/19 09:51, Dandan Bi wrote:
> REF: https://bugzilla.tianocore.org/show_bug.cgi?id=1398
> 
> According to PI1.7, this patch series is to add the support that
> report extended data describing an EFI_STATUS return value along
> with  EFI_SW_DXE_BS_EC_BOOT_OPTION_LOAD_ERROR and
> EFI_SW_DXE_BS_EC_BOOT_OPTION_FAILED status code when fail to load
> or start boot option image.
> 
> These two commit are also available at:
> https://github.com/dandanbi/edk2/commits/ExtendedStatusCodeV1
> 
> PI1.7 is at
> https://uefi.org/sites/default/files/resources/PI_Spec_1_7_final_Jan_2019.pdf
> 
> Cc: Jian J Wang <jian.j.wang@intel.com>
> Cc: Hao Wu <hao.a.wu@intel.com>
> Cc: Ruiyu Ni <ruiyu.ni@intel.com>
> Cc: Michael D Kinney <michael.d.kinney@intel.com>
> Cc: Liming Gao <liming.gao@intel.com>
> Cc: Laszlo Ersek <lersek@redhat.com>
> Cc: Sean Brogan <sean.brogan@microsoft.com>
> Dandan Bi (2):
>   MdePkg/StatusCodeDataTypeId.h: Add new definition per PI1.7 Spec
>   MdeModulePkg/BmBoot: Report status when fail to load/start boot option
> 
>  .../Library/UefiBootManagerLib/BmBoot.c       | 22 ++++++++++++++-----
>  MdePkg/Include/Guid/StatusCodeDataTypeId.h    | 22 ++++++++++++++++++-
>  2 files changed, 37 insertions(+), 7 deletions(-)
> 

Looks good to me.

Reviewed-by: Laszlo Ersek <lersek@redhat.com>

Thank you,
Laszlo