diff options
author | Cinnamon Shia <cinnamon.shia@hpe.com> | 2016-05-02 12:34:09 +0800 |
---|---|---|
committer | Star Zeng <star.zeng@intel.com> | 2016-05-03 12:37:57 +0800 |
commit | 0cd1e699e5f76b3323b5881932af8a62bc5edf2a (patch) | |
tree | e557e02b145cfd5a7d5f402b36d227a2b616801e | |
parent | 69af847603ca1f6439031c4e1fee844fdb06ca6f (diff) | |
download | edk2-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.c | 5 |
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.
|