public inbox for devel@edk2.groups.io
 help / color / mirror / Atom feed
From: "Tiger Liu(BJ-RD)" <tigerliu@zhaoxin.com>
To: "Kinney, Michael D" <michael.d.kinney@intel.com>,
	"devel@edk2.groups.io" <devel@edk2.groups.io>
Subject: Re: [edk2-devel] [edk2] AP core : call GetPeiServicesTablePointer function hang
Date: Tue, 19 Nov 2019 02:37:16 +0000	[thread overview]
Message-ID: <774de03803f24829be2be0c34b3b5aa7@zhaoxin.com> (raw)

Hi, Mike:
Got it!

Thanks

Best wishes,
-----邮件原件-----
发件人: Kinney, Michael D <michael.d.kinney@intel.com>
发送时间: 2019年11月19日 9:51
收件人: Tiger Liu(BJ-RD) <TigerLiu@zhaoxin.com>; devel@edk2.groups.io; Kinney, Michael D <michael.d.kinney@intel.com>
主题: RE: [edk2-devel] [edk2] AP core : call GetPeiServicesTablePointer function hang

That is the only reason the IDTs are different.

Earlier in this project, the same IDT was used by BSP and APs.  This meant APs could call PEI Services and we did not know until there was a side effect.  By using different IDTs, we catch APs calling PEI Services immediately.

Mike

> -----Original Message-----
> From: Tiger Liu(BJ-RD) <TigerLiu@zhaoxin.com>
> Sent: Monday, November 18, 2019 5:21 PM
> To: Kinney, Michael D <michael.d.kinney@intel.com>;
> devel@edk2.groups.io
> Subject: Re: [edk2-devel] [edk2] AP core : call
> GetPeiServicesTablePointer function hang
>
> Hi, Mike:
> Got it!
> Thank you very much!
>
> I have another question:
> Why does AP use an IDT table which is different with BSP?
> Is there other concern?
>
> Thanks
> -----邮件原件-----
> 发件人: Kinney, Michael D <michael.d.kinney@intel.com>
> 发送时间: 2019年11月19日 0:27
> 收件人: devel@edk2.groups.io; Tiger Liu(BJ-RD) <TigerLiu@zhaoxin.com>;
> Kinney, Michael D <michael.d.kinney@intel.com>
> 主题: RE: [edk2-devel] [edk2] AP core : call GetPeiServicesTablePointer
> function hang
>
> PEI Phase execution environment is focused on BSP execution.
> The PEI Services and most of the PPI services are not designed to be
> MP safe.  Since it is not safe for an AP to call PEI Services, the IDT
> entry with the pointer to the PEI Service Table is not available to an
> AP to prevent code from incorrectly calling a PEI Service from an AP.
>
> It is legal to run code on an AP in the PEI Phase, but the code
> executed must be carefully designed to not use PEI Services and is
> usually designed to execute using only stack/memory and system
> resources that have assigned for exclusive access by an AP.
> This is usually communicated from BSP to an AP through a mailbox in
> memory.  If an AP needs something like a PEI Service (e.g.
> memory allocation or logging), then the AP should make that request to
> the BSP to perform that operation for the AP.
>
> Mike
>
> > -----Original Message-----
> > From: devel@edk2.groups.io <devel@edk2.groups.io> On
> Behalf Of Tiger
> > Liu(BJ-RD)
> > Sent: Monday, November 18, 2019 1:08 AM
> > To: devel@edk2.groups.io
> > Subject: [edk2-devel] [edk2] AP core : call
> GetPeiServicesTablePointer
> > function hang
> >
> > Dear All:
> > I met a question during debug multi-processor code.
> >
> > I found if AP's procedure call
> > GetPeiServicesTablePointer function , then it would
> hang.
> > GetPeiServicesTablePointer (
> >   VOID
> >   )
> > {
> >   CONST EFI_PEI_SERVICES  **PeiServices;
> >   IA32_DESCRIPTOR   Idtr;
> >
> >   AsmReadIdtr (&Idtr);
> >   PeiServices = (CONST EFI_PEI_SERVICES **)
> (*(UINTN*)(Idtr.Base -
> > sizeof (UINTN)));
> >   ASSERT (PeiServices != NULL);
> >   return PeiServices;
> > }
> >
> > It seems BSP and AP use different IDT table? Why?
> >
> > Thanks
> >
> >
> > 保密声明:
> > 本邮件含有保密或专有信息,仅供指定收件人使用。严禁对本邮件或其内
> 容
> > 做任何未经授权的查阅、使用、复制或转发。
> > CONFIDENTIAL NOTE:
> > This email contains confidential or legally
> privileged information and
> > is for the sole use of its intended recipient. Any
> unauthorized
> > review, use, copying or forwarding of this email or
> the content of
> > this email is strictly prohibited.
> >
> > 
>
>
>
> 保密声明:
> 本邮件含有保密或专有信息,仅供指定收件人使用。严禁对本邮件或其内容
> 做任何未经授权的查阅、使用、复制或转发。
> CONFIDENTIAL NOTE:
> This email contains confidential or legally privileged information and
> is for the sole use of its intended recipient. Any unauthorized
> review, use, copying or forwarding of this email or the content of
> this email is strictly prohibited.


保密声明:
本邮件含有保密或专有信息,仅供指定收件人使用。严禁对本邮件或其内容做任何未经授权的查阅、使用、复制或转发。
CONFIDENTIAL NOTE:
This email contains confidential or legally privileged information and is for the sole use of its intended recipient. Any unauthorized review, use, copying or forwarding of this email or the content of this email is strictly prohibited.

             reply	other threads:[~2019-11-19  2:37 UTC|newest]

Thread overview: 5+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2019-11-19  2:37 Tiger Liu(BJ-RD) [this message]
  -- strict thread matches above, loose matches on Subject: below --
2019-11-19  1:20 [edk2-devel] [edk2] AP core : call GetPeiServicesTablePointer function hang Tiger Liu(BJ-RD)
2019-11-19  1:51 ` Michael D Kinney
2019-11-18  9:08 Tiger Liu(BJ-RD)
2019-11-18 16:27 ` Michael D Kinney

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=774de03803f24829be2be0c34b3b5aa7@zhaoxin.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