Reviewed-by: Liming Gao <[email protected]> for Patch 6~8.

> -----Original Message-----
> From: Yao, Jiewen
> Sent: Friday, April 22, 2016 4:04 PM
> To: [email protected]
> Cc: Gao, Liming; Tian, Feng; Laszlo Ersek
> Subject: [PATCH 06/12] MdeModulePkg-FPDT(1): Add
> SMM_FPDT_FUNCTION_GET_BOOT_RECORD_DATA_BY_OFFSET definition.
> 
> This patch enhance performance data SMM communication by using fixed
> SMM communication buffer.
> 
> A new command
> SMM_FPDT_FUNCTION_GET_BOOT_RECORD_DATA_BY_OFFSET is added,
> because we need to support get partial PerformanceData to fixed SMM
> communication
> buffer. If performance data is bigger than fixed SMM communication buffer,
> the DXE agent need to call
> SMM_FPDT_FUNCTION_GET_BOOT_RECORD_DATA_BY_OFFSET
> multiple times to get all data out.
> 
> This is designed to meet Microsoft WSMT table definition on
> FIXED_COMM_BUFFERS
> requirement.
> 
> Cc: "Gao, Liming" <[email protected]>
> Cc: "Tian, Feng" <[email protected]>
> Cc: "Laszlo Ersek" <[email protected]>
> Contributed-under: TianoCore Contribution Agreement 1.0
> Signed-off-by: "Gao, Liming" <[email protected]>
> Reviewed-by: "Tian, Feng" <[email protected]>
> ---
>  MdeModulePkg/Include/Guid/FirmwarePerformance.h | 8 +++++---
>  1 file changed, 5 insertions(+), 3 deletions(-)
> 
> diff --git a/MdeModulePkg/Include/Guid/FirmwarePerformance.h
> b/MdeModulePkg/Include/Guid/FirmwarePerformance.h
> index a4aeda2..4697a2c 100644
> --- a/MdeModulePkg/Include/Guid/FirmwarePerformance.h
> +++ b/MdeModulePkg/Include/Guid/FirmwarePerformance.h
> @@ -1,7 +1,7 @@
>  /** @file
>    ACPI Firmware Performance Data Table (FPDT) implementation specific
> definitions.
> 
> -  Copyright (c) 2011 - 2013, Intel Corporation. All rights reserved.<BR>
> +  Copyright (c) 2011 - 2016, Intel Corporation. All rights reserved.<BR>
>    This program and the accompanying materials
>    are licensed and made available under the terms and conditions of the BSD
> License
>    which accompanies this distribution.  The full text of the license may be
> found at
> @@ -116,14 +116,16 @@ typedef struct {
>  //
>  // Log BOOT RECORD from SMM driver on boot time.
>  //
> -#define SMM_FPDT_FUNCTION_GET_BOOT_RECORD_SIZE          1
> -#define SMM_FPDT_FUNCTION_GET_BOOT_RECORD_DATA          2
> +#define SMM_FPDT_FUNCTION_GET_BOOT_RECORD_SIZE           1
> +#define SMM_FPDT_FUNCTION_GET_BOOT_RECORD_DATA           2
> +#define SMM_FPDT_FUNCTION_GET_BOOT_RECORD_DATA_BY_OFFSET 3
> 
>  typedef struct {
>    UINTN             Function;
>    EFI_STATUS        ReturnStatus;
>    UINTN             BootRecordSize;
>    VOID              *BootRecordData;
> +  UINTN             BootRecordOffset;
>  } SMM_BOOT_RECORD_COMMUNICATE;
> 
>  extern EFI_GUID gEfiFirmwarePerformanceGuid;
> --
> 2.7.4.windows.1

_______________________________________________
edk2-devel mailing list
[email protected]
https://lists.01.org/mailman/listinfo/edk2-devel

Reply via email to