From mboxrd@z Thu Jan  1 00:00:00 1970
Received: from mail.byosoft.com.cn (mail.byosoft.com.cn [58.240.74.242])
 by mx.groups.io with SMTP id smtpd.web12.4693.1603865396296698353
 for <devel@edk2.groups.io>;
 Tue, 27 Oct 2020 23:09:58 -0700
Authentication-Results: mx.groups.io;
 dkim=missing; spf=none, err=permanent DNS error (domain: byosoft.com.cn, ip: 58.240.74.242, mailfrom: gaoliming@byosoft.com.cn)
Received: from DESKTOPS6D0PVI ([58.246.60.130])
	(envelope-sender <gaoliming@byosoft.com.cn>)
	by 192.168.6.13 with ESMTP
	for <dandan.bi@intel.com>; Wed, 28 Oct 2020 14:09:47 +0800
X-WM-Sender: gaoliming@byosoft.com.cn
X-WM-AuthFlag: YES
X-WM-AuthUser: gaoliming@byosoft.com.cn
From: "gaoliming" <gaoliming@byosoft.com.cn>
To: "'Siyuan Fu'" <siyuan.fu@intel.com>,
	<devel@edk2.groups.io>
Cc: "'Dandan Bi'" <dandan.bi@intel.com>
References: <38c94c28e001783a852c990481d46a0937874f39.1603853801.git.siyuan.fu@intel.com>
In-Reply-To: <38c94c28e001783a852c990481d46a0937874f39.1603853801.git.siyuan.fu@intel.com>
Subject: =?UTF-8?B?5Zue5aSNOiBbUGF0Y2hdIE1kZU1vZHVsZVBrZy9Db3JlOiBTdXBwb3J0IHN0YW5kYWxvbmUgTU0gaW4gRlYyIHByb3RvY29sIEdldE5leHRGaWxlKCku?=
Date: Wed, 28 Oct 2020 14:09:54 +0800
Message-ID: <003301d6acf0$f4666c90$dd3345b0$@byosoft.com.cn>
MIME-Version: 1.0
X-Mailer: Microsoft Outlook 16.0
Thread-Index: AQLQApDVmjVu+zt3kGwJG/y+Wstk5qe53MRQ
Content-Type: text/plain;
	charset="gb2312"
Content-Transfer-Encoding: quoted-printable
Content-Language: zh-cn

Reviewed-by: Liming Gao <gaoliming@byosoft.com.cn>

> -----=D3=CA=BC=FE=D4=AD=BC=FE-----
> =B7=A2=BC=FE=C8=CB: Siyuan Fu <siyuan.fu@intel.com>
> =B7=A2=CB=CD=CA=B1=BC=E4: 2020=C4=EA10=D4=C228=C8=D5 10:57
> =CA=D5=BC=FE=C8=CB: devel@edk2.groups.io
> =B3=AD=CB=CD: Dandan Bi <dandan.bi@intel.com>; Liming Gao
> <gaoliming@byosoft.com.cn>
> =D6=F7=CC=E2: [Patch] MdeModulePkg/Core: Support standalone MM in FV2 =
protocol
> GetNextFile().
>=20
> REF:https://bugzilla.tianocore.org/show_bug.cgi?id=3D3028
>=20
> The FirmwareVolume2Protocol->GetNextFile() produced by DXE Core can be
> used
> to search for a file based on the value of *FileType input. However, =
this
> service will always return EFI_NOT_FOUND if the input FileType is set =
to
> EFI_FV_FILETYPE_MM_STANDALONE or
> EFI_FV_FILETYPE_MM_CORE_STANDALONE, Which
> means user can't use this service to search any standalone MM image in
that
> FV.
> This patch update the FirmwareVolume2Protocol->GetNextFile() service =
to
> support searching standalone MM module.
>=20
> Signed-off-by: Siyuan Fu <siyuan.fu@intel.com>
> Cc: Dandan Bi <dandan.bi@intel.com>
> Cc: Liming Gao <gaoliming@byosoft.com.cn>
> ---
>  MdeModulePkg/Core/Dxe/FwVol/FwVolRead.c | 6 +++---
>  1 file changed, 3 insertions(+), 3 deletions(-)
>=20
> diff --git a/MdeModulePkg/Core/Dxe/FwVol/FwVolRead.c
> b/MdeModulePkg/Core/Dxe/FwVol/FwVolRead.c
> index 2861cbf676..8dcbbeb5ee 100644
> --- a/MdeModulePkg/Core/Dxe/FwVol/FwVolRead.c
> +++ b/MdeModulePkg/Core/Dxe/FwVol/FwVolRead.c
> @@ -1,7 +1,7 @@
>  /** @file
>    Implements functions to read firmware file
>=20
> -Copyright (c) 2006 - 2017, Intel Corporation. All rights =
reserved.<BR>
> +Copyright (c) 2006 - 2020, Intel Corporation. All rights =
reserved.<BR>
>  SPDX-License-Identifier: BSD-2-Clause-Patent
>=20
>  **/
> @@ -143,9 +143,9 @@ FvGetNextFile (
>      return EFI_ACCESS_DENIED;
>    }
>=20
> -  if (*FileType > EFI_FV_FILETYPE_SMM_CORE) {
> +  if (*FileType > EFI_FV_FILETYPE_MM_CORE_STANDALONE) {
>      //
> -    // File type needs to be in 0 - 0x0D
> +    // File type needs to be in 0 - 0x0F
>      //
>      return EFI_NOT_FOUND;
>    }
> --
> 2.19.1.windows.1