From: matthewfcarlson@gmail.com
To: devel@edk2.groups.io
Cc: Matthew Carlson <matthewfcarlson@gmail.com>
Subject: [PATCH v1 1/3] UnitTestFrameworkPkg : BaseTimerLib: Adds a host-based timer Lib
Date: Tue, 7 Jul 2020 18:31:20 -0700 [thread overview]
Message-ID: <20200708013122.1301-2-matthewfcarlson@gmail.com> (raw)
In-Reply-To: <20200708013122.1301-1-matthewfcarlson@gmail.com>
From: Matthew Carlson <macarl@microsoft.com>
This adds a host based BaseTimerLib that won't assert.
Signed-off-by: Matthew Carlson <matthewfcarlson@gmail.com>
---
UnitTestFrameworkPkg/Library/Host/BaseTimerLibHost/TimerLibHost.c | 137 ++++++++++++++++++++
UnitTestFrameworkPkg/Library/Host/BaseTimerLibHost/BaseTimerLibHost.inf | 40 ++++++
UnitTestFrameworkPkg/Library/Host/BaseTimerLibHost/BaseTimerLibHost.uni | 19 +++
UnitTestFrameworkPkg/UnitTestFrameworkPkgHost.dsc.inc | 1 +
4 files changed, 197 insertions(+)
diff --git a/UnitTestFrameworkPkg/Library/Host/BaseTimerLibHost/TimerLibHost.c b/UnitTestFrameworkPkg/Library/Host/BaseTimerLibHost/TimerLibHost.c
new file mode 100644
index 000000000000..61304e66f54f
--- /dev/null
+++ b/UnitTestFrameworkPkg/Library/Host/BaseTimerLibHost/TimerLibHost.c
@@ -0,0 +1,137 @@
+/** @file
+ A non-functional instance of the Timer Library.
+
+ Copyright (c) 2007 - 2011, Intel Corporation. All rights reserved.<BR>
+ SPDX-License-Identifier: BSD-2-Clause-Patent
+
+**/
+
+#include <Base.h>
+#include <Library/TimerLib.h>
+#include <Library/DebugLib.h>
+#include <stdio.h>
+#include <stdlib.h>
+#include <time.h>
+
+/**
+ Stalls the CPU for at least the given number of microseconds.
+
+ Stalls the CPU for the number of microseconds specified by MicroSeconds.
+
+ @param MicroSeconds The minimum number of microseconds to delay.
+
+ @return The value of MicroSeconds inputted.
+
+**/
+UINTN
+EFIAPI
+MicroSecondDelay (
+ IN UINTN MicroSeconds
+ )
+{
+ return NanoSecondDelay(MicroSeconds * 1000000) / 1000000;
+}
+
+/**
+ Stalls the CPU for at least the given number of nanoseconds.
+
+ Stalls the CPU for the number of nanoseconds specified by NanoSeconds.
+
+ @param NanoSeconds The minimum number of nanoseconds to delay.
+
+ @return The value of NanoSeconds inputted.
+
+**/
+UINTN
+EFIAPI
+NanoSecondDelay (
+ IN UINTN NanoSeconds
+ )
+{
+ /*UINT64 start = GetPerformanceCounter();
+ UINT64 goal = start + NanoSeconds;
+ DEBUG(( DEBUG_ERROR, "Delaying for %d ns. Start: %d to %d\n", NanoSeconds, start, goal));
+ // check if we need to wait for the counter to roll over
+ if (start > goal) {
+ DEBUG(( DEBUG_ERROR, "Start is more than goal\n"));
+ while (GetPerformanceCounter() > goal);
+ }
+ while (GetPerformanceCounter() < goal);
+ return NanoSeconds;*/
+ return 0;
+}
+
+/**
+ Retrieves the current value of a 64-bit free running performance counter.
+
+ The counter can either count up by 1 or count down by 1. If the physical
+ performance counter counts by a larger increment, then the counter values
+ must be translated. The properties of the counter can be retrieved from
+ GetPerformanceCounterProperties().
+
+ @return The current value of the free running performance counter.
+
+**/
+UINT64
+EFIAPI
+GetPerformanceCounter (
+ VOID
+ )
+{
+ struct timespec ts;
+ timespec_get(&ts, TIME_UTC);
+ return ts.tv_nsec;
+}
+
+/**
+ Retrieves the 64-bit frequency in Hz and the range of performance counter
+ values.
+
+ If StartValue is not NULL, then the value that the performance counter starts
+ with immediately after is it rolls over is returned in StartValue. If
+ EndValue is not NULL, then the value that the performance counter end with
+ immediately before it rolls over is returned in EndValue. The 64-bit
+ frequency of the performance counter in Hz is always returned. If StartValue
+ is less than EndValue, then the performance counter counts up. If StartValue
+ is greater than EndValue, then the performance counter counts down. For
+ example, a 64-bit free running counter that counts up would have a StartValue
+ of 0 and an EndValue of 0xFFFFFFFFFFFFFFFF. A 24-bit free running counter
+ that counts down would have a StartValue of 0xFFFFFF and an EndValue of 0.
+
+ @param StartValue The value the performance counter starts with when it
+ rolls over.
+ @param EndValue The value that the performance counter ends with before
+ it rolls over.
+
+ @return The frequency in Hz.
+
+**/
+UINT64
+EFIAPI
+GetPerformanceCounterProperties (
+ OUT UINT64 *StartValue, OPTIONAL
+ OUT UINT64 *EndValue OPTIONAL
+ )
+{
+ return (UINT64)(-1);
+}
+
+/**
+ Converts elapsed ticks of performance counter to time in nanoseconds.
+
+ This function converts the elapsed ticks of running performance counter to
+ time value in unit of nanoseconds.
+
+ @param Ticks The number of elapsed ticks of running performance counter.
+
+ @return The elapsed time in nanoseconds.
+
+**/
+UINT64
+EFIAPI
+GetTimeInNanoSecond (
+ IN UINT64 Ticks
+ )
+{
+ return Ticks;
+}
diff --git a/UnitTestFrameworkPkg/Library/Host/BaseTimerLibHost/BaseTimerLibHost.inf b/UnitTestFrameworkPkg/Library/Host/BaseTimerLibHost/BaseTimerLibHost.inf
new file mode 100644
index 000000000000..ac091cd10af1
--- /dev/null
+++ b/UnitTestFrameworkPkg/Library/Host/BaseTimerLibHost/BaseTimerLibHost.inf
@@ -0,0 +1,40 @@
+## @file
+# NULL instance of Timer Library as a template.
+#
+# A non-functional instance of the Timer Library that can be used as a template
+# for the implementation of a functional timer library instance. This library instance can
+# also be used to test build DXE, Runtime, DXE SAL, and DXE SMM modules that require timer
+# services as well as EBC modules that require timer services.
+#
+# Copyright (c) 2007 - 2018, Intel Corporation. All rights reserved.<BR>
+#
+# SPDX-License-Identifier: BSD-2-Clause-Patent
+#
+#
+##
+
+[Defines]
+ INF_VERSION = 0x00010005
+ BASE_NAME = BaseTimerLibHost
+ MODULE_UNI_FILE = BaseTimerLibHost.uni
+ FILE_GUID = a2a413b8-43c3-40cc-a298-be7a214d3564
+ MODULE_TYPE = BASE
+ VERSION_STRING = 1.0
+ LIBRARY_CLASS = TimerLib
+
+
+#
+# VALID_ARCHITECTURES = IA32 X64 EBC AARCH
+#
+
+[Sources]
+ TimerLibHost.c
+
+
+[Packages]
+ MdePkg/MdePkg.dec
+
+
+[LibraryClasses]
+ DebugLib
+
diff --git a/UnitTestFrameworkPkg/Library/Host/BaseTimerLibHost/BaseTimerLibHost.uni b/UnitTestFrameworkPkg/Library/Host/BaseTimerLibHost/BaseTimerLibHost.uni
new file mode 100644
index 000000000000..6e100bd383d2
--- /dev/null
+++ b/UnitTestFrameworkPkg/Library/Host/BaseTimerLibHost/BaseTimerLibHost.uni
@@ -0,0 +1,19 @@
+// /** @file
+// NULL instance of Timer Library as a template.
+//
+// A non-functional instance of the Timer Library that can be used as a template
+// for the implementation of a functional timer library instance. This library instance can
+// also be used to test build DXE, Runtime, DXE SAL, and DXE SMM modules that require timer
+// services as well as EBC modules that require timer services.
+//
+// Copyright (c) 2007 - 2014, Intel Corporation. All rights reserved.<BR>
+//
+// SPDX-License-Identifier: BSD-2-Clause-Patent
+//
+// **/
+
+
+#string STR_MODULE_ABSTRACT #language en-US "Host instance of Timer Library"
+
+#string STR_MODULE_DESCRIPTION #language en-US "An instance of the Timer Library that can be used for host based unit tests."
+
diff --git a/UnitTestFrameworkPkg/UnitTestFrameworkPkgHost.dsc.inc b/UnitTestFrameworkPkg/UnitTestFrameworkPkgHost.dsc.inc
index e954968efc52..bc7b7afa1729 100644
--- a/UnitTestFrameworkPkg/UnitTestFrameworkPkgHost.dsc.inc
+++ b/UnitTestFrameworkPkg/UnitTestFrameworkPkgHost.dsc.inc
@@ -13,6 +13,7 @@
UnitTestLib|UnitTestFrameworkPkg/Library/UnitTestLib/UnitTestLibCmocka.inf
DebugLib|UnitTestFrameworkPkg/Library/Posix/DebugLibPosix/DebugLibPosix.inf
MemoryAllocationLib|UnitTestFrameworkPkg/Library/Posix/MemoryAllocationLibPosix/MemoryAllocationLibPosix.inf
+ TimerLib|UnitTestFrameworkPkg/Library/Host/BaseTimerLibHost/BaseTimerLibHost.inf
[BuildOptions]
GCC:*_*_*_CC_FLAGS = -fno-pie
--
2.25.1.vfs.1.1
next prev parent reply other threads:[~2020-07-08 1:31 UTC|newest]
Thread overview: 9+ messages / expand[flat|nested] mbox.gz Atom feed top
2020-07-08 1:31 [PATCH v1 0/3] *** SUBJECT HERE *** matthewfcarlson
2020-07-08 1:31 ` matthewfcarlson [this message]
2020-07-08 17:03 ` [edk2-devel] [PATCH v1 1/3] UnitTestFrameworkPkg : BaseTimerLib: Adds a host-based timer Lib Michael D Kinney
2020-07-08 17:10 ` Michael D Kinney
2020-07-08 17:30 ` Michael D Kinney
2020-07-08 17:42 ` Matthew Carlson
2020-07-09 18:59 ` Matthew Carlson
2020-07-08 1:31 ` [PATCH v1 2/3] CryptoPkg: BaseCryptLib: Add unit tests (Host and Shell based) matthewfcarlson
2020-07-08 1:31 ` [PATCH v1 3/3] AzurePipelines : Pr Gate: Turn on HBUT for CryptoPkg matthewfcarlson
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-list from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=20200708013122.1301-2-matthewfcarlson@gmail.com \
--to=devel@edk2.groups.io \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox