summaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorCinnamon Shia <cinnamon.shia@hpe.com>2016-05-02 12:34:09 +0800
committerStar Zeng <star.zeng@intel.com>2016-05-03 12:37:57 +0800
commit0cd1e699e5f76b3323b5881932af8a62bc5edf2a (patch)
treee557e02b145cfd5a7d5f402b36d227a2b616801e
parent69af847603ca1f6439031c4e1fee844fdb06ca6f (diff)
downloadedk2-0cd1e699e5f76b3323b5881932af8a62bc5edf2a.tar.gz
edk2-0cd1e699e5f76b3323b5881932af8a62bc5edf2a.tar.bz2
edk2-0cd1e699e5f76b3323b5881932af8a62bc5edf2a.zip
PerformancePkg/Dp_App: Fix the error message "Timer library instance error!"
When executing shell dp command, there is an error message "Timer library instance error!" The error message "Timer library instance error!" should be for the case about duration > EndTimeStamp if CountUp or duration > StartTimeStamp if CountDown. But if the EndTimeStamp of an entry is not added, it should not the case to catch. This change fixes the error message "Timer library instance error!" from the "BdsAttempt" entry which is logged when trying to boot a boot option. Contributed-under: TianoCore Contribution Agreement 1.0 Signed-off-by: Cinnamon Shia <cinnamon.shia@hpe.com> Reviewed-by: Star Zeng <star.zeng@intel.com> Reviewed-by: Jaben Carsey <jaben.carsey@intel.com>
-rw-r--r--PerformancePkg/Dp_App/DpUtilities.c5
1 files changed, 5 insertions, 0 deletions
diff --git a/PerformancePkg/Dp_App/DpUtilities.c b/PerformancePkg/Dp_App/DpUtilities.c
index b49844a058..13d38979d3 100644
--- a/PerformancePkg/Dp_App/DpUtilities.c
+++ b/PerformancePkg/Dp_App/DpUtilities.c
@@ -2,6 +2,7 @@
Utility functions used by the Dp application.
Copyright (c) 2009 - 2016, Intel Corporation. All rights reserved.<BR>
+ (C) Copyright 2016 Hewlett Packard Enterprise Development LP<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
@@ -82,6 +83,10 @@ GetDuration (
UINT64 Duration;
BOOLEAN Error;
+ if (Measurement->EndTimeStamp == 0) {
+ return 0;
+ }
+
// PERF_START macros are called with a value of 1 to indicate
// the beginning of time. So, adjust the start ticker value
// to the real beginning of time.