Re: [PATCH] UefiCpuPkg/CpuDxe: change level of DEBUG message

["Dong, Eric" <eric.dong@intel.com>]

Reviewed-by: Eric Dong <eric.dong@intel.com>

> -----Original Message-----
> From: Wang, Jian J
> Sent: Wednesday, August 29, 2018 11:22 AM
> To: edk2-devel@lists.01.org
> Cc: Dong, Eric <eric.dong@intel.com>; Laszlo Ersek <lersek@redhat.com>
> Subject: [PATCH] UefiCpuPkg/CpuDxe: change level of DEBUG message
> 
> It's reported the debug message in CpuDxe driver is quite annoying in boot
> and shell, and slow down the boot process. To solve this issue, this patch
> changes the DEBUG_INFO to DEBUG_VERBOSE. On a typical Intel real
> platform, at least 16s boot time can be saved.
> 
> Cc: Eric Dong <eric.dong@intel.com>
> Cc: Laszlo Ersek <lersek@redhat.com>
> Contributed-under: TianoCore Contribution Agreement 1.1
> Signed-off-by: Jian J Wang <jian.j.wang@intel.com>
> ---
>  UefiCpuPkg/CpuDxe/CpuDxe.c       | 2 +-
>  UefiCpuPkg/CpuDxe/CpuPageTable.c | 4 ++--
>  2 files changed, 3 insertions(+), 3 deletions(-)
> 
> diff --git a/UefiCpuPkg/CpuDxe/CpuDxe.c b/UefiCpuPkg/CpuDxe/CpuDxe.c
> index cfd4c415ae..76661cbcd9 100644
> --- a/UefiCpuPkg/CpuDxe/CpuDxe.c
> +++ b/UefiCpuPkg/CpuDxe/CpuDxe.c
> @@ -404,7 +404,7 @@ CpuSetMemoryAttributes (
>    // to avoid unnecessary computing.
>    //
>    if (mIsFlushingGCD) {
> -    DEBUG((DEBUG_INFO, "  Flushing GCD\n"));
> +    DEBUG((DEBUG_VERBOSE, "  Flushing GCD\n"));
>      return EFI_SUCCESS;
>    }
> 
> diff --git a/UefiCpuPkg/CpuDxe/CpuPageTable.c
> b/UefiCpuPkg/CpuDxe/CpuPageTable.c
> index df021798c0..609df58e3a 100644
> --- a/UefiCpuPkg/CpuDxe/CpuPageTable.c
> +++ b/UefiCpuPkg/CpuDxe/CpuPageTable.c
> @@ -528,7 +528,7 @@ SplitPage (
>      ASSERT (SplitAttribute == Page4K);
>      if (SplitAttribute == Page4K) {
>        NewPageEntry = AllocatePagesFunc (1);
> -      DEBUG ((DEBUG_INFO, "Split - 0x%x\n", NewPageEntry));
> +      DEBUG ((DEBUG_VERBOSE, "Split - 0x%x\n", NewPageEntry));
>        if (NewPageEntry == NULL) {
>          return RETURN_OUT_OF_RESOURCES;
>        }
> @@ -549,7 +549,7 @@ SplitPage (
>      ASSERT (SplitAttribute == Page2M || SplitAttribute == Page4K);
>      if ((SplitAttribute == Page2M || SplitAttribute == Page4K)) {
>        NewPageEntry = AllocatePagesFunc (1);
> -      DEBUG ((DEBUG_INFO, "Split - 0x%x\n", NewPageEntry));
> +      DEBUG ((DEBUG_VERBOSE, "Split - 0x%x\n", NewPageEntry));
>        if (NewPageEntry == NULL) {
>          return RETURN_OUT_OF_RESOURCES;
>        }
> --
> 2.16.2.windows.1