public inbox for devel@edk2.groups.io
 help / color / mirror / Atom feed
From: "Laszlo Ersek" <lersek@redhat.com>
To: devel@edk2.groups.io, ray.ni@intel.com, "Dong,
	Eric" <eric.dong@intel.com>
Cc: "Kinney, Michael D" <michael.d.kinney@intel.com>,
	"Gao, Liming" <liming.gao@intel.com>
Subject: Re: [edk2-devel] [Patch v5 1/2] MdePkg: Add new MM MP Protocol definition.
Date: Thu, 11 Jul 2019 18:13:09 +0200	[thread overview]
Message-ID: <385ee804-aa92-886d-0a65-f49636266495@redhat.com> (raw)
In-Reply-To: <734D49CCEBEEF84792F5B80ED585239D5C223BE7@SHSMSX104.ccr.corp.intel.com>

Hi Ray,

On 07/11/19 08:42, Ni, Ray wrote:
> + MdePkg maintainers
> 
> We need to follow the open source community process to include MdePkg
> maintainers for this change review.
> 
> Laszlo,
> You are in the Cc list and had given Regression-Tested-By for V3 patch series.
> It would be great if you could give patch owner a hint to avoid breaking the process
> though I understand it's not responsibility of the people in the CC list. 😊

I'm unaware of any changes to the process.

In the commit message of the patch, Cc: tags should be included for all
"M" folks under XxxPkg, and also for all "R" folks who are listed with
interest in the given area (possibly the entirety of XxxPkg). The
contributor's git config should not include a
"sendemail.suppresscc=bodycc" setting; otherwise those Cc: tags will not
actually copy the intended reviewers.

For a given XxxPkg patch, at least one Acked-by or Required-by is needed
from an "M" person listed under XxxPkg in "Maintainers.txt", on the
mailing list, before the patch can be pushed.

Reviews and comments from others are welcome for the patch. The patch
should not be pushed as long as reasonable questions are still open for
it (regardless of origin). A patch should generally spend at least 24
hours on the list before it is pushed, even if an "M" approval arrives
earlier than that (so that others have a chance at noticing the patch
and at commenting). Once no questions seem to remain open, the patch has
spent the minimum time on the list, and there is public "M" approval,
the patch can be pushed.

An "M" person can defer to another "M" or "R" person, if he/she chooses
so. This is best done formally, i.e., wait until the deferred-to person
approves the patch, and then the first "M" person can give their own
Acked-by. (Standing for "I haven't reviewed in detail, but given the
circumstances / other reviews, I'm OK with this patch".)

When the patch is pushed, all feedback tags, given on the list for the
patch (regardless of origin), should be picked up, by whoever is pushing
the patch.

Let's consider the current state of this series, as an example.

- The first patch (for MdePkg) has an "M" review, from Liming.
- The second patch (for UefiCpuPkg) has an "M" review, from you.
- The series has an open question (regarding regression testing), from
myself.
- The series was posted more than 24 hours ago (as far as I can see).

So once I (hopefully) report back with an R-t-b for the series, the
series becomes pushable (assuming noone opens another question
meanwhile). In that case, any "M" person is welcome to push the patch
series -- in the current case, it will likely be Eric. When Eric
prepares for pushing the series, he's expected to pick up
- Liming's R-b for the first patch,
- my (to be posted) R-t-b for the first patch,
- your R-b for the second patch,
- my (to be posted) R-t-b for the second patch.

(I will likely send my R-t-b in response to the blurb (the "v5 0/2"
email), which means that the feedback tag applies to every patch in the
series.)

This "tag pickup" procedure is difficult to get right when using a MUA
that does not support a "threaded" view, and it is easy to get right
with a MUA that does. When I'm about to push a series, I tend to perform
one full scan over the thread:

- Whenever I reach patch-level feedback, I run "git rebase -i" for the
full series, just to pick up that one tag. (A single "reword" action
among the "pick"s.)

- Whenever I reach series-level feedback (grouped under the blurb), I
run "git rebase -i" for the full series again, and I apply the tag to
every patch in the series (all actions are set to "reword").

Thanks
Laszlo

>> -----Original Message-----
>> From: Dong, Eric
>> Sent: Wednesday, July 10, 2019 3:56 PM
>> To: devel@edk2.groups.io
>> Cc: Ni, Ray <ray.ni@intel.com>; Laszlo Ersek <lersek@redhat.com>
>> Subject: [Patch v5 1/2] MdePkg: Add new MM MP Protocol definition.
>>
>> REF: https://bugzilla.tianocore.org/show_bug.cgi?id=1937
>>
>> EFI MM MP Protocol is defined in the PI 1.5 specification.
>>
>> The MM MP protocol provides a set of functions to allow execution of
>> procedures on processors that have entered MM. This protocol has the
>> following properties:
>> 1. The caller can invoke execution of a procedure on a processor, other than
>> the caller, that has also entered MM. Supports blocking and non-blocking
>> modes of operation.
>> 2. The caller can invoke a procedure on multiple processors. Supports
>> blocking and non-blocking modes of operation.
>>
>> Cc: Ray Ni <ray.ni@intel.com>
>> Cc: Laszlo Ersek <lersek@redhat.com>
>> Signed-off-by: Eric Dong <eric.dong@intel.com>
>> Reviewed-by: Ray Ni <ray.ni@intel.com>
>> ---
>>  MdePkg/Include/Pi/PiMultiPhase.h |  16 ++
>>  MdePkg/Include/Protocol/MmMp.h   | 333
>> +++++++++++++++++++++++++++++++
>>  MdePkg/MdePkg.dec                |   3 +
>>  3 files changed, 352 insertions(+)
>>  create mode 100644 MdePkg/Include/Protocol/MmMp.h
>>
>> diff --git a/MdePkg/Include/Pi/PiMultiPhase.h
>> b/MdePkg/Include/Pi/PiMultiPhase.h
>> index eb12527767..a5056799e1 100644
>> --- a/MdePkg/Include/Pi/PiMultiPhase.h
>> +++ b/MdePkg/Include/Pi/PiMultiPhase.h
>> @@ -176,4 +176,20 @@ VOID
>>    IN OUT VOID  *Buffer
>>    );
>>
>> +/**
>> +  The function prototype for invoking a function on an Application Processor.
>> +
>> +  This definition is used by the UEFI MM MP Serices Protocol.
>> +
>> +  @param[in] ProcedureArgument    The pointer to private data buffer.
>> +
>> +  @retval EFI_SUCCESS             Excutive the procedure successfully
>> +
>> +**/
>> +typedef
>> +EFI_STATUS
>> +(EFIAPI *EFI_AP_PROCEDURE2)(
>> +  IN VOID  *ProcedureArgument
>> +);
>> +
>>  #endif
>> diff --git a/MdePkg/Include/Protocol/MmMp.h
>> b/MdePkg/Include/Protocol/MmMp.h new file mode 100644 index
>> 0000000000..beace1386c
>> --- /dev/null
>> +++ b/MdePkg/Include/Protocol/MmMp.h
>> @@ -0,0 +1,333 @@
>> +/** @file
>> +  EFI MM MP Protocol is defined in the PI 1.5 specification.
>> +
>> +  The MM MP protocol provides a set of functions to allow execution of
>> + procedures on processors that  have entered MM. This protocol has the
>> following properties:
>> +  1. The caller can only invoke execution of a procedure on a processor,
>> other than the caller, that
>> +     has also entered MM.
>> +  2. It is possible to invoke a procedure on multiple processors. Supports
>> blocking and non-blocking
>> +     modes of operation.
>> +
>> +  Copyright (c) 2019, Intel Corporation. All rights reserved.<BR>
>> +  SPDX-License-Identifier: BSD-2-Clause-Patent
>> +
>> +**/
>> +
>> +#ifndef _MM_MP_H_
>> +#define _MM_MP_H_
>> +
>> +#include <Pi/PiMmCis.h>
>> +
>> +#define EFI_MM_MP_PROTOCOL_GUID \
>> +  { \
>> +    0x5d5450d7, 0x990c, 0x4180, {0xa8, 0x3, 0x8e, 0x63, 0xf0, 0x60,
>> +0x83, 0x7  }  \
>> +  }
>> +
>> +//
>> +// Revision definition.
>> +//
>> +#define EFI_MM_MP_PROTOCOL_REVISION    0x00
>> +
>> +//
>> +// Attribute flags
>> +//
>> +#define EFI_MM_MP_TIMEOUT_SUPPORTED    0x01
>> +
>> +//
>> +// Completion token
>> +//
>> +typedef VOID* MM_COMPLETION;
>> +
>> +typedef struct {
>> +  MM_COMPLETION  Completion;
>> +  EFI_STATUS     Status;
>> +} MM_DISPATCH_COMPLETION_TOKEN;
>> +
>> +typedef struct _EFI_MM_MP_PROTOCOL  EFI_MM_MP_PROTOCOL;
>> +
>> +/**
>> +  Service to retrieves the number of logical processor in the platform.
>> +
>> +  @param[in]  This                The EFI_MM_MP_PROTOCOL instance.
>> +  @param[out] NumberOfProcessors  Pointer to the total number of logical
>> processors in the system,
>> +                                  including the BSP and all APs.
>> +
>> +  @retval EFI_SUCCESS             The number of processors was retrieved
>> successfully
>> +  @retval EFI_INVALID_PARAMETER   NumberOfProcessors is NULL
>> +**/
>> +typedef
>> +EFI_STATUS
>> +(EFIAPI *EFI_MM_GET_NUMBER_OF_PROCESSORS) (
>> +  IN CONST EFI_MM_MP_PROTOCOL  *This,
>> +  OUT      UINTN               *NumberOfProcessors
>> +);
>> +
>> +
>> +/**
>> +  This service allows the caller to invoke a procedure one of the
>> +application processors (AP). This
>> +  function uses an optional token parameter to support blocking and
>> +non-blocking modes. If the token
>> +  is passed into the call, the function will operate in a non-blocking
>> +fashion and the caller can
>> +  check for completion with CheckOnProcedure or WaitForProcedure.
>> +
>> +  @param[in]     This                   The EFI_MM_MP_PROTOCOL instance.
>> +  @param[in]     Procedure              A pointer to the procedure to be run on
>> the designated target
>> +                                        AP of the system. Type EFI_AP_PROCEDURE2 is defined
>> below in
>> +                                        related definitions.
>> +  @param[in]     CpuNumber              The zero-based index of the processor
>> number of the target
>> +                                        AP, on which the code stream is supposed to run. If the
>> number
>> +                                        points to the calling processor then it will not run the
>> +                                        supplied code.
>> +  @param[in]     TimeoutInMicroseconds  Indicates the time limit in
>> microseconds for this AP to
>> +                                        finish execution of Procedure, either for blocking or
>> +                                        non-blocking mode. Zero means infinity. If the timeout
>> +                                        expires before this AP returns from Procedure, then
>> Procedure
>> +                                        on the AP is terminated. If the timeout expires in
>> blocking
>> +                                        mode, the call returns EFI_TIMEOUT. If the timeout
>> expires
>> +                                        in non-blocking mode, the timeout determined can be
>> through
>> +                                        CheckOnProcedure or WaitForProcedure.
>> +                                        Note that timeout support is optional. Whether an
>> +                                        implementation supports this feature, can be
>> determined via
>> +                                        the Attributes data member.
>> +  @param[in,out] ProcedureArguments     Allows the caller to pass a list of
>> parameters to the code
>> +                                        that is run by the AP. It is an optional common mailbox
>> +                                        between APs and the caller to share information.
>> +  @param[in,out] Token                  This is parameter is broken into two
>> components:
>> +                                        1.Token->Completion is an optional parameter that
>> allows the
>> +                                        caller to execute the procedure in a blocking or non-
>> blocking
>> +                                        fashion. If it is NULL the call is blocking, and the call will
>> +                                        not return until the AP has completed the procedure. If
>> the
>> +                                        token is not NULL, the call will return immediately. The
>> caller
>> +                                        can check whether the procedure has completed with
>> +                                        CheckOnProcedure or WaitForProcedure.
>> +                                        2.Token->Status The implementation updates the
>> address pointed
>> +                                        at by this variable with the status code returned by
>> Procedure
>> +                                        when it completes execution on the target AP, or with
>> EFI_TIMEOUT
>> +                                        if the Procedure fails to complete within the optional
>> timeout.
>> +                                        The implementation will update this variable with
>> EFI_NOT_READY
>> +                                        prior to starting Procedure on the target AP.
>> +  @param[in,out] CPUStatus              This optional pointer may be used to get
>> the status code returned
>> +                                        by Procedure when it completes execution on the
>> target AP, or with
>> +                                        EFI_TIMEOUT if the Procedure fails to complete within
>> the optional
>> +                                        timeout. The implementation will update this variable
>> with
>> +                                        EFI_NOT_READY prior to starting Procedure on the
>> target AP.
>> +
>> +  @retval EFI_SUCCESS                   In the blocking case, this indicates that
>> Procedure has completed
>> +                                        execution on the target AP.
>> +                                        In the non-blocking case this indicates that the
>> procedure has
>> +                                        been successfully scheduled for execution on the target
>> AP.
>> +  @retval EFI_INVALID_PARAMETER         The input arguments are out of
>> range. Either the target AP is the
>> +                                        caller of the function, or the Procedure or Token is NULL
>> +  @retval EFI_NOT_READY                 If the target AP is busy executing another
>> procedure
>> +  @retval EFI_ALREADY_STARTED           Token is already in use for another
>> procedure
>> +  @retval EFI_TIMEOUT                   In blocking mode, the timeout expired
>> before the specified AP
>> +                                        has finished **/ typedef
>> +EFI_STATUS (EFIAPI *EFI_MM_DISPATCH_PROCEDURE) (
>> +  IN CONST EFI_MM_MP_PROTOCOL            *This,
>> +  IN       EFI_AP_PROCEDURE2             Procedure,
>> +  IN       UINTN                         CpuNumber,
>> +  IN       UINTN                         TimeoutInMicroseconds,
>> +  IN OUT   VOID                          *ProcedureArguments OPTIONAL,
>> +  IN OUT   MM_COMPLETION                 *Token,
>> +  IN OUT   EFI_STATUS                    *CPUStatus
>> +);
>> +
>> +/**
>> +  This service allows the caller to invoke a procedure on all running
>> +application processors (AP)
>> +  except the caller. This function uses an optional token parameter to
>> +support blocking and
>> +  nonblocking modes. If the token is passed into the call, the function
>> +will operate in a non-blocking
>> +  fashion and the caller can check for completion with CheckOnProcedure or
>> WaitForProcedure.
>> +
>> +  It is not necessary for the implementation to run the procedure on every
>> processor on the platform.
>> +  Processors that are powered down in such a way that they cannot
>> + respond to interrupts, may be  excluded from the broadcast.
>> +
>> +
>> +  @param[in]     This                   The EFI_MM_MP_PROTOCOL instance.
>> +  @param[in]     Procedure              A pointer to the code stream to be run on
>> the APs that have
>> +                                        entered MM. Type EFI_AP_PROCEDURE is defined
>> below in related
>> +                                        definitions.
>> +  @param[in]     TimeoutInMicroseconds  Indicates the time limit in
>> microseconds for the APs to finish
>> +                                        execution of Procedure, either for blocking or non-
>> blocking mode.
>> +                                        Zero means infinity. If the timeout expires before all
>> APs return
>> +                                        from Procedure, then Procedure on the failed APs is
>> terminated. If
>> +                                        the timeout expires in blocking mode, the call returns
>> EFI_TIMEOUT.
>> +                                        If the timeout expires in non-blocking mode, the
>> timeout determined
>> +                                        can be through CheckOnProcedure or
>> WaitForProcedure.
>> +                                        Note that timeout support is optional. Whether an
>> implementation
>> +                                        supports this feature can be determined via the
>> Attributes data
>> +                                        member.
>> +  @param[in,out] ProcedureArguments     Allows the caller to pass a list of
>> parameters to the code
>> +                                        that is run by the AP. It is an optional common mailbox
>> +                                        between APs and the caller to share information.
>> +  @param[in,out] Token                  This is parameter is broken into two
>> components:
>> +                                        1.Token->Completion is an optional parameter that
>> allows the
>> +                                        caller to execute the procedure in a blocking or non-
>> blocking
>> +                                        fashion. If it is NULL the call is blocking, and the call will
>> +                                        not return until the AP has completed the procedure. If
>> the
>> +                                        token is not NULL, the call will return immediately. The
>> caller
>> +                                        can check whether the procedure has completed with
>> +                                        CheckOnProcedure or WaitForProcedure.
>> +                                        2.Token->Status The implementation updates the
>> address pointed
>> +                                        at by this variable with the status code returned by
>> Procedure
>> +                                        when it completes execution on the target AP, or with
>> EFI_TIMEOUT
>> +                                        if the Procedure fails to complete within the optional
>> timeout.
>> +                                        The implementation will update this variable with
>> EFI_NOT_READY
>> +                                        prior to starting Procedure on the target AP
>> +  @param[in,out] CPUStatus              This optional pointer may be used to get
>> the individual status
>> +                                        returned by every AP that participated in the broadcast.
>> This
>> +                                        parameter if used provides the base address of an array
>> to hold
>> +                                        the EFI_STATUS value of each AP in the system. The size
>> of the
>> +                                        array can be ascertained by the
>> GetNumberOfProcessors function.
>> +                                        As mentioned above, the broadcast may not include
>> every processor
>> +                                        in the system. Some implementations may exclude
>> processors that
>> +                                        have been powered down in such a way that they are
>> not responsive
>> +                                        to interrupts. Additionally the broadcast excludes the
>> processor
>> +                                        which is making the BroadcastProcedure call. For every
>> excluded
>> +                                        processor, the array entry must
>> + contain a value of EFI_NOT_STARTED
>> +
>> +  @retval EFI_SUCCESS                   In the blocking case, this indicates that
>> Procedure has completed
>> +                                        execution on the APs. In the non-blocking case this
>> indicates that
>> +                                        the procedure has been successfully scheduled for
>> execution on the
>> +                                        APs.
>> +  @retval EFI_INVALID_PARAMETER         Procedure or Token is NULL.
>> +  @retval EFI_NOT_READY                 If a target AP is busy executing another
>> procedure.
>> +  @retval EFI_TIMEOUT                   In blocking mode, the timeout expired
>> before all enabled APs have
>> +                                        finished.
>> +  @retval EFI_ALREADY_STARTED           Before the AP procedure associated
>> with the Token is finished, the
>> +                                        same Token cannot be used to dispatch or broadcast
>> another procedure.
>> +
>> +**/
>> +typedef
>> +EFI_STATUS
>> +(EFIAPI *EFI_MM_BROADCAST_PROCEDURE) (
>> +  IN CONST EFI_MM_MP_PROTOCOL            *This,
>> +  IN       EFI_AP_PROCEDURE2             Procedure,
>> +  IN       UINTN                         TimeoutInMicroseconds,
>> +  IN OUT   VOID                          *ProcedureArguments OPTIONAL,
>> +  IN OUT   MM_COMPLETION                 *Token,
>> +  IN OUT   EFI_STATUS                    *CPUStatus
>> +);
>> +
>> +
>> +/**
>> +  This service allows the caller to set a startup procedure that will
>> +be executed when an AP powers
>> +  up from a state where core configuration and context is lost. The
>> +procedure is execution has the
>> +  following properties:
>> +  1. The procedure executes before the processor is handed over to the
>> operating system.
>> +  2. All processors execute the same startup procedure.
>> +  3. The procedure may run in parallel with other procedures invoked
>> +through the functions in this
>> +  protocol, or with processors that are executing an MM handler or running
>> in the operating system.
>> +
>> +
>> +  @param[in]      This                 The EFI_MM_MP_PROTOCOL instance.
>> +  @param[in]      Procedure            A pointer to the code stream to be run on
>> the designated target AP
>> +                                       of the system. Type EFI_AP_PROCEDURE is defined
>> below in Volume 2
>> +                                       with the related definitions of
>> +                                       EFI_MP_SERVICES_PROTOCOL.StartupAllAPs.
>> +                                       If caller may pass a value of NULL to deregister any
>> existing
>> +                                       startup procedure.
>> +  @param[in,out]  ProcedureArguments   Allows the caller to pass a list of
>> parameters to the code that is
>> +                                       run by the AP. It is an optional common mailbox
>> between APs and
>> +                                       the caller to share information
>> +
>> +  @retval EFI_SUCCESS                  The Procedure has been set successfully.
>> +  @retval EFI_INVALID_PARAMETER        The Procedure is NULL.
>> +**/
>> +typedef
>> +EFI_STATUS
>> +(EFIAPI *EFI_MM_SET_STARTUP_PROCEDURE) (
>> +  IN CONST EFI_MM_MP_PROTOCOL *This,
>> +  IN       EFI_AP_PROCEDURE   Procedure,
>> +  IN OUT   VOID               *ProcedureArguments OPTIONAL
>> +);
>> +
>> +/**
>> +  When non-blocking execution of a procedure on an AP is invoked with
>> +DispatchProcedure,
>> +  via the use of a token, this function can be used to check for completion of
>> the procedure on the AP.
>> +  The function takes the token that was passed into the
>> +DispatchProcedure call. If the procedure
>> +  is complete, and therefore it is now possible to run another
>> +procedure on the same AP, this function
>> +  returns EFI_SUCESS. In this case the status returned by the procedure
>> +that executed on the AP is
>> +  returned in the token's Status field. If the procedure has not yet
>> +completed, then this function
>> +  returns EFI_NOT_READY.
>> +
>> +  When a non-blocking execution of a procedure is invoked with
>> + BroadcastProcedure, via the  use of a token, this function can be used
>> + to check for completion of the procedure on all the  broadcast APs.
>> + The function takes the token that was passed into the
>> + BroadcastProcedure  call. If the procedure is complete on all
>> + broadcast APs this function returns EFI_SUCESS. In this  case the Status
>> field in the token passed into the function reflects the overall result of the
>> invocation, which may be EFI_SUCCESS, if all executions succeeded, or the
>> first observed failure.
>> +  If the procedure has not yet completed on the broadcast APs, the
>> + function returns  EFI_NOT_READY.
>> +
>> +  @param[in]      This                 The EFI_MM_MP_PROTOCOL instance.
>> +  @param[in]      Token                This parameter describes the token that was
>> passed into
>> +                                       DispatchProcedure or BroadcastProcedure.
>> +
>> +  @retval EFI_SUCCESS                  Procedure has completed.
>> +  @retval EFI_NOT_READY                The Procedure has not completed.
>> +  @retval EFI_INVALID_PARAMETER        Token or Token->Completion is
>> NULL
>> +  @retval EFI_NOT_FOUND                Token is not currently in use for a non-
>> blocking call
>> +
>> +**/
>> +typedef
>> +EFI_STATUS
>> +(EFIAPI *EFI_CHECK_FOR_PROCEDURE) (
>> +  IN CONST EFI_MM_MP_PROTOCOL            *This,
>> +  IN       MM_COMPLETION                 Token
>> +);
>> +
>> +/**
>> +  When a non-blocking execution of a procedure on an AP is invoked via
>> +DispatchProcedure,
>> +  this function will block the caller until the remote procedure has completed
>> on the designated AP.
>> +  The non-blocking procedure invocation is identified by the Token
>> +parameter, which must match the
>> +  token that used when DispatchProcedure was called. Upon completion
>> +the status returned by
>> +  the procedure that executed on the AP is used to update the token's
>> Status field.
>> +
>> +  When a non-blocking execution of a procedure on an AP is invoked via
>> + BroadcastProcedure  this function will block the caller until the
>> + remote procedure has completed on all of the APs that  entered MM. The
>> + non-blocking procedure invocation is identified by the Token
>> + parameter, which  must match the token that used when
>> + BroadcastProcedure was called. Upon completion the  overall status
>> + returned by the procedures that executed on the broadcast AP is used to
>> update the  token's Status field. The overall status may be EFI_SUCCESS, if all
>> executions succeeded, or the  first observed failure.
>> +
>> +
>> +  @param[in]      This                 The EFI_MM_MP_PROTOCOL instance.
>> +  @param[in]      Token                This parameter describes the token that was
>> passed into
>> +                                       DispatchProcedure or BroadcastProcedure.
>> +
>> +  @retval EFI_SUCCESS                  Procedure has completed.
>> +  @retval EFI_INVALID_PARAMETER        Token or Token->Completion is
>> NULL
>> +  @retval EFI_NOT_FOUND                Token is not currently in use for a non-
>> blocking call
>> +
>> +**/
>> +typedef
>> +EFI_STATUS
>> +(EFIAPI *EFI_WAIT_FOR_PROCEDURE) (
>> +  IN CONST EFI_MM_MP_PROTOCOL            *This,
>> +  IN       MM_COMPLETION                 Token
>> +);
>> +
>> +
>> +
>> +///
>> +/// The MM MP protocol provides a set of functions to allow execution
>> +of procedures on processors that /// have entered MM.
>> +///
>> +struct _EFI_MM_MP_PROTOCOL {
>> +  UINT32                            Revision;
>> +  UINT32                            Attributes;
>> +  EFI_MM_GET_NUMBER_OF_PROCESSORS   GetNumberOfProcessors;
>> +  EFI_MM_DISPATCH_PROCEDURE         DispatchProcedure;
>> +  EFI_MM_BROADCAST_PROCEDURE        BroadcastProcedure;
>> +  EFI_MM_SET_STARTUP_PROCEDURE      SetStartupProcedure;
>> +  EFI_CHECK_FOR_PROCEDURE           CheckForProcedure;
>> +  EFI_WAIT_FOR_PROCEDURE            WaitForProcedure;
>> +};
>> +
>> +extern EFI_GUID gEfiMmMpProtocolGuid;
>> +
>> +#endif
>> diff --git a/MdePkg/MdePkg.dec b/MdePkg/MdePkg.dec index
>> 6c563375ee..b382efd578 100644
>> --- a/MdePkg/MdePkg.dec
>> +++ b/MdePkg/MdePkg.dec
>> @@ -1167,6 +1167,9 @@
>>    # Protocols defined in PI 1.5.
>>    #
>>
>> +  ## Include/Protocol/MmMp.h
>> +  gEfiMmMpProtocolGuid = { 0x5d5450d7, 0x990c, 0x4180, { 0xa8, 0x3,
>> + 0x8e, 0x63, 0xf0, 0x60, 0x83, 0x7 }}
>> +
>>    ## Include/Protocol/MmEndOfDxe.h
>>    gEfiMmEndOfDxeProtocolGuid = { 0x24e70042, 0xd5c5, 0x4260, { 0x8c, 0x39,
>> 0xa, 0xd3, 0xaa, 0x32, 0xe9, 0x3d }}
>>
>> --
>> 2.21.0.windows.1
> 
> 
> 
> 


  reply	other threads:[~2019-07-11 16:13 UTC|newest]

Thread overview: 11+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2019-07-10  7:56 [Patch v5 0/2] Enable new MM MP protocol Dong, Eric
2019-07-10  7:56 ` [Patch v5 1/2] MdePkg: Add new MM MP Protocol definition Dong, Eric
2019-07-11  6:42   ` Ni, Ray
2019-07-11 16:13     ` Laszlo Ersek [this message]
2019-07-10  7:56 ` [Patch v5 2/2] UefiCpuPkg/PiSmmCpuDxeSmm: Enable MM MP Protocol Dong, Eric
2019-07-10  8:56   ` Ni, Ray
2019-07-11 17:53   ` [edk2-devel] " Laszlo Ersek
2019-07-12  0:56     ` Dong, Eric
2019-07-10 16:10 ` [edk2-devel] [Patch v5 0/2] Enable new MM MP protocol Laszlo Ersek
     [not found] ` <15AFFCA66AC7A422.21469@groups.io>
2019-07-11  6:39   ` [edk2-devel] [Patch v5 1/2] MdePkg: Add new MM MP Protocol definition Dong, Eric
2019-07-11  6:50     ` Liming Gao

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=385ee804-aa92-886d-0a65-f49636266495@redhat.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