From: "Khor, Swee Aun via groups.io" <swee.aun.khor=intel.com@groups.io>
To: "Yao, Jiewen" <jiewen.yao@intel.com>,
"kraxel@redhat.com" <kraxel@redhat.com>,
"ardb+tianocore@kernel.org" <ardb+tianocore@kernel.org>
Cc: "Wu, Jiaxin" <jiaxin.wu@intel.com>,
"devel@edk2.groups.io" <devel@edk2.groups.io>
Subject: Re: [edk2-devel] Add SmmCpuPlatformHookLib IsLmceStatusCheckNeeded interface PR review request
Date: Tue, 31 Dec 2024 01:05:40 +0000 [thread overview]
Message-ID: <DS7PR11MB61750B478519B3683ADD7BA6AF0A2@DS7PR11MB6175.namprd11.prod.outlook.com> (raw)
In-Reply-To: <DS7PR11MB61751B053D534C017D92F8A5AF022@DS7PR11MB6175.namprd11.prod.outlook.com>
[-- Attachment #1: Type: text/plain, Size: 1739 bytes --]
Hi Ard, JieWen and Greg,
Please help to review my PR.Thanks.
Regards,
Swee Aun
From: Khor, Swee Aun
Sent: Monday, December 23, 2024 2:28 PM
To: Yao, Jiewen <jiewen.yao@intel.com>; kraxel@redhat.com
Cc: Wu, Jiaxin <jiaxin.wu@intel.com>
Subject: Add SmmCpuPlatformHookLib IsLmceStatusCheckNeeded interface PR review request
Hi JieWen and Greg,
Could you please help to review PR https://github.com/tianocore/edk2/pull/6557?
This PR is to add a new interface for SmmCpuPlatformHookLib which can used to check whether the LMCE status check is required depending on platform requirements.
By default, 2 edk2 implementations (UefiCpuPkg/SmmCpuPlatformHookLibNull and OvmfPkg/SmmCpuPlatformHookLibQemu ) returns TRUE indicating LMCE status check should be performed. Platform specific implementation could return FALSE if would like to skip the LMCE check due to any reason.
This new interface will be consumed by UefiCpuPkg/PiSmmCpuDxeSmm/MpService.c SmmWaitForApArrival(). However, the consumer part PR will be submitted later to avoid immediate downstream build error. https://github.com/tianocore/edk2/pull/6557 PR allow downstream to implement platform specific implementation, thus the transition will be smooth when downstream received the updated SmmWaitForApArrival().
Please let me know if you have any questions.
Regards,
Swee Aun
-=-=-=-=-=-=-=-=-=-=-=-
Groups.io Links: You receive all messages sent to this group.
View/Reply Online (#120953): https://edk2.groups.io/g/devel/message/120953
Mute This Topic: https://groups.io/mt/110420494/7686176
Group Owner: devel+owner@edk2.groups.io
Unsubscribe: https://edk2.groups.io/g/devel/unsub [rebecca@openfw.io]
-=-=-=-=-=-=-=-=-=-=-=-
[-- Attachment #2: Type: text/html, Size: 5987 bytes --]
parent reply other threads:[~2025-01-04 9:11 UTC|newest]
Thread overview: expand[flat|nested] mbox.gz Atom feed
[parent not found: <DS7PR11MB61751B053D534C017D92F8A5AF022@DS7PR11MB6175.namprd11.prod.outlook.com>]
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=DS7PR11MB61750B478519B3683ADD7BA6AF0A2@DS7PR11MB6175.namprd11.prod.outlook.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