From: "Ard Biesheuvel" <ardb@kernel.org>
To: "Bi, Dandan" <dandan.bi@intel.com>
Cc: "Wang, Yin" <yin.wang@intel.com>,
"devel@edk2.groups.io" <devel@edk2.groups.io>,
"Yao, Jiewen" <jiewen.yao@intel.com>,
"Justen, Jordan L" <jordan.l.justen@intel.com>,
Gerd Hoffmann <kraxel@redhat.com>,
Sunil V L <sunilvl@ventanamicro.com>,
"Warkentin, Andrei" <andrei.warkentin@intel.com>,
"Dong, Eric" <eric.dong@intel.com>
Subject: Re: [edk2-devel] [PATCH v5 1/1] OvmfPkg: Fix Hii form name mismatch with EFI variable
Date: Wed, 6 Sep 2023 09:02:00 +0200 [thread overview]
Message-ID: <CAMj1kXHTpnxbQ23Xe5pw=BJD0WR5gyuqC1HUSh28MA5mBO9-3g@mail.gmail.com> (raw)
In-Reply-To: <MN6PR11MB82421ECFA7EED17261429C1EEAEFA@MN6PR11MB8242.namprd11.prod.outlook.com>
On Wed, 6 Sept 2023 at 05:03, Bi, Dandan <dandan.bi@intel.com> wrote:
>
> Reviewed-by: Dandan Bi <dandan.bi@intel.com>
>
Thanks for the review
Merging as #4789
Thanks,
>
>
> Thanks,
> Dandan
> -----Original Message-----
> From: Wang, Yin <yin.wang@intel.com>
> Sent: Friday, September 1, 2023 4:21 PM
> To: devel@edk2.groups.io
> Cc: Wang, Yin <yin.wang@intel.com>; Ard Biesheuvel <ardb+tianocore@kernel.org>; Yao, Jiewen <jiewen.yao@intel.com>; Justen, Jordan L <jordan.l.justen@intel.com>; Gerd Hoffmann <kraxel@redhat.com>; Sunil V L <sunilvl@ventanamicro.com>; Warkentin, Andrei <andrei.warkentin@intel.com>; Bi, Dandan <dandan.bi@intel.com>; Dong, Eric <eric.dong@intel.com>
> Subject: [PATCH v5 1/1] OvmfPkg: Fix Hii form name mismatch with EFI variable
>
> The Hii form is named "MainFormState" while the EFI variable is named "PlatformConfig".
> This discrepancy in names causes the following SCT cases to fail on
> RiscVVirtQemu:
> ExtractConfigConformance
> ExtractConfigFunction
> ExtractConfig_Func
>
> Cc: Ard Biesheuvel <ardb+tianocore@kernel.org>
> Cc: Jiewen Yao <jiewen.yao@intel.com>
> Cc: Jordan Justen <jordan.l.justen@intel.com>
> Cc: Gerd Hoffmann <kraxel@redhat.com>
> Cc: Sunil V L <sunilvl@ventanamicro.com>
> Cc: Andrei Warkentin <andrei.warkentin@intel.com>
> Cc: Dandan Bi <dandan.bi@intel.com>
> Cc: Eric Dong <eric.dong@intel.com>
> Signed-off-by: Yin Wang <yin.wang@intel.com>
> Acked-by: Gerd Hoffmann <kraxel@redhat.com>
> Acked-by: Sunil V L <sunilvl@ventanamicro.com>
> ---
> v5: Re-organize commit message
> v4: Update commit message about version history
>
> - aefcc918: Enhance ExtractConfig() to match UEFI specification,
> adding check of whether "<ConfigHdr> matches the GUID and name"
> - 16acacf2: Fix Hii form name mismatch in ExtractConfig()
> - This patch: Add one more fix for Hii form name mismatch
>
> v3: Gerd comments on v2 that caused issues may related to RISC-V specific config. To address this separately, the patch was restored to v1, and details were added to the commit message.
>
> v2: Add two more changes which caused other two SCT Hii cases to fail.
>
> v1: Initial version
> ---
> OvmfPkg/PlatformDxe/Platform.c | 2 +-
> 1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/OvmfPkg/PlatformDxe/Platform.c b/OvmfPkg/PlatformDxe/Platform.c index cd3bfd554c..c32439683d 100644
> --- a/OvmfPkg/PlatformDxe/Platform.c
> +++ b/OvmfPkg/PlatformDxe/Platform.c
> @@ -279,7 +279,7 @@ ExtractConfig (
> // ConfigRequestHdr = HiiConstructConfigHdr ( &gOvmfPlatformConfigGuid,- mVariableName,+ mHiiFormName, mImageHandle ); if (ConfigRequestHdr == NULL) {--
> 2.34.1
>
-=-=-=-=-=-=-=-=-=-=-=-
Groups.io Links: You receive all messages sent to this group.
View/Reply Online (#108309): https://edk2.groups.io/g/devel/message/108309
Mute This Topic: https://groups.io/mt/101090614/7686176
Group Owner: devel+owner@edk2.groups.io
Unsubscribe: https://edk2.groups.io/g/devel/unsub [rebecca@openfw.io]
-=-=-=-=-=-=-=-=-=-=-=-
prev parent reply other threads:[~2023-09-06 7:02 UTC|newest]
Thread overview: 4+ messages / expand[flat|nested] mbox.gz Atom feed top
2023-09-01 8:21 [edk2-devel] [PATCH v5 1/1] OvmfPkg: Fix Hii form name mismatch with EFI variable Wang, Yin
2023-09-06 2:36 ` Wang, Yin
2023-09-06 3:02 ` Dandan Bi
2023-09-06 7:02 ` Ard Biesheuvel [this message]
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-list from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to='CAMj1kXHTpnxbQ23Xe5pw=BJD0WR5gyuqC1HUSh28MA5mBO9-3g@mail.gmail.com' \
--to=devel@edk2.groups.io \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox