public inbox for devel@edk2.groups.io
 help / color / mirror / Atom feed
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 --]

           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