On 07/19/16 02:57, Jeff Fan wrote:
> Except for MtrrSetAllMtrrs(), others services are not suggested to be invoked 
> by
> APs because they are not necessary and may invoke DEBUG() to dump MTRR 
> setting.
> 
> Cc: Feng Tian <[email protected]>
> Cc: Michael Kinney <[email protected]>
> Cc: Laszlo Ersek <[email protected]>
> Contributed-under: TianoCore Contribution Agreement 1.0
> Signed-off-by: Jeff Fan <[email protected]>
> ---
>  UefiCpuPkg/Library/MtrrLib/MtrrLib.c | 4 ++++
>  1 file changed, 4 insertions(+)
> 
> diff --git a/UefiCpuPkg/Library/MtrrLib/MtrrLib.c 
> b/UefiCpuPkg/Library/MtrrLib/MtrrLib.c
> index f667a8f..5aef772 100644
> --- a/UefiCpuPkg/Library/MtrrLib/MtrrLib.c
> +++ b/UefiCpuPkg/Library/MtrrLib/MtrrLib.c
> @@ -1,6 +1,10 @@
>  /** @file
>    MTRR setting library
>  
> +  @par Note: 
> +    Most of services in this library instance are suggested to be invoked by 
> BSP only,
> +    except for MtrrSetAllMtrrs() which is used to sync BSP's MTRR setting to 
> APs.
> +
>    Copyright (c) 2008 - 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
> 

Reviewed-by: Laszlo Ersek <[email protected]>
_______________________________________________
edk2-devel mailing list
[email protected]
https://lists.01.org/mailman/listinfo/edk2-devel

Reply via email to