PcAtChipsetPkg AcpiTimerLib: Get more accurate TSC Frequency

Minimize the code overhead between the two TSC reads by adding
new internal API to calculate TSC Frequency instead of reusing
MicroSecondDelay ().

Cc: Michael D Kinney <michael.d.kinney@intel.com>
Cc: Liming Gao <liming.gao@intel.com>
Cc: Paul A Lohr <paul.a.lohr@intel.com>
Contributed-under: TianoCore Contribution Agreement 1.0
Signed-off-by: Star Zeng <star.zeng@intel.com>
Reviewed-by: Liming Gao <liming.gao@intel.com>
Reviewed-by: Michael D Kinney <michael.d.kinney@intel.com>

diff --git a/PcAtChipsetPkg/Library/AcpiTimerLib/AcpiTimerLib.c b/PcAtChipsetPkg/Library/AcpiTimerLib/AcpiTimerLib.c
index 806a4f7ce24c7431b3173328948795ef09df9753..e6fea231123d9740964fa24b08ba9bc8b92aec19 100644 (file)
--- a/PcAtChipsetPkg/Library/AcpiTimerLib/AcpiTimerLib.c
+++ b/PcAtChipsetPkg/Library/AcpiTimerLib/AcpiTimerLib.c
@@ -1,7 +1,7 @@
 /** @file\r
   ACPI Timer implements one instance of Timer Library.\r
 \r
-  Copyright (c) 2013 - 2015, Intel Corporation. All rights reserved.<BR>\r
+  Copyright (c) 2013 - 2016, Intel Corporation. All rights reserved.<BR>\r
   This program and the accompanying materials\r
   are licensed and made available under the terms and conditions of the BSD License\r
   which accompanies this distribution.  The full text of the license may be found at\r
@@ -335,3 +335,57 @@ GetTimeInNanoSecond (
 \r
   return NanoSeconds;\r
 }\r
+\r
+/**\r
+  Calculate TSC frequency.\r
+\r
+  The TSC counting frequency is determined by comparing how far it counts\r
+  during a 100us period as determined by the ACPI timer. The ACPI timer is\r
+  used because it counts at a known frequency.\r
+  The TSC is sampled, followed by waiting for ACPI_TIMER_FREQUENCY / 10000\r
+  clocks of the ACPI timer, or 100us. The TSC is then sampled again. The\r
+  difference multiplied by 10000 is the TSC frequency. There will be a small\r
+  error because of the overhead of reading the ACPI timer. An attempt is\r
+  made to determine and compensate for this error.\r
+\r
+  @return The number of TSC counts per second.\r
+\r
+**/\r
+UINT64\r
+InternalCalculateTscFrequency (\r
+  VOID\r
+  )\r
+{\r
+  UINT64      StartTSC;\r
+  UINT64      EndTSC;\r
+  UINT16      TimerAddr;\r
+  UINT32      Ticks;\r
+  UINT64      TscFrequency;\r
+  BOOLEAN     InterruptState;\r
+\r
+  InterruptState = SaveAndDisableInterrupts ();\r
+\r
+  TimerAddr = InternalAcpiGetAcpiTimerIoPort ();\r
+  Ticks = IoRead32 (TimerAddr) + (ACPI_TIMER_FREQUENCY / 10000);    // Set Ticks to 100us in the future\r
+\r
+  StartTSC = AsmReadTsc ();                                         // Get base value for the TSC\r
+  //\r
+  // Wait until the ACPI timer has counted 100us.\r
+  // Timer wrap-arounds are handled correctly by this function.\r
+  // When the current ACPI timer value is greater than 'Ticks', the while loop will exit.\r
+  //\r
+  while (((Ticks - IoRead32 (TimerAddr)) & BIT23) == 0) {\r
+    CpuPause();\r
+  }\r
+  EndTSC = AsmReadTsc ();                                           // TSC value 100us later\r
+\r
+  TscFrequency = MultU64x32 (\r
+                   (EndTSC - StartTSC),                             // Number of TSC counts in 100us\r
+                   10000                                            // Number of 100us in a second\r
+                   );\r
+\r
+  SetInterruptState (InterruptState);\r
+\r
+  return TscFrequency;\r
+}\r
+\r

diff --git a/PcAtChipsetPkg/Library/AcpiTimerLib/BaseAcpiTimerLib.c b/PcAtChipsetPkg/Library/AcpiTimerLib/BaseAcpiTimerLib.c
index 21fdb79908b89baedc235c283f6ea63bcf45e29f..8819ebcfccef7446c88254b5c189052a3badfd64 100644 (file)
--- a/PcAtChipsetPkg/Library/AcpiTimerLib/BaseAcpiTimerLib.c
+++ b/PcAtChipsetPkg/Library/AcpiTimerLib/BaseAcpiTimerLib.c
@@ -1,7 +1,7 @@
 /** @file\r
   ACPI Timer implements one instance of Timer Library.\r
 \r
-  Copyright (c) 2013 - 2014, Intel Corporation. All rights reserved.<BR>\r
+  Copyright (c) 2013 - 2016, Intel Corporation. All rights reserved.<BR>\r
   This program and the accompanying materials\r
   are licensed and made available under the terms and conditions of the BSD License\r
   which accompanies this distribution.  The full text of the license may be found at\r
@@ -16,6 +16,26 @@
 #include <Library/TimerLib.h>\r
 #include <Library/BaseLib.h>\r
 \r
+/**\r
+  Calculate TSC frequency.\r
+\r
+  The TSC counting frequency is determined by comparing how far it counts\r
+  during a 100us period as determined by the ACPI timer. The ACPI timer is\r
+  used because it counts at a known frequency.\r
+  The TSC is sampled, followed by waiting for ACPI_TIMER_FREQUENCY / 10000\r
+  clocks of the ACPI timer, or 100us. The TSC is then sampled again. The\r
+  difference multiplied by 10000 is the TSC frequency. There will be a small\r
+  error because of the overhead of reading the ACPI timer. An attempt is\r
+  made to determine and compensate for this error.\r
+\r
+  @return The number of TSC counts per second.\r
+\r
+**/\r
+UINT64\r
+InternalCalculateTscFrequency (\r
+  VOID\r
+  );\r
+\r
 /**\r
   Internal function to retrieves the 64-bit frequency in Hz.\r
 \r
@@ -29,14 +49,5 @@ InternalGetPerformanceCounterFrequency (
   VOID\r
   ) \r
 {\r
-  BOOLEAN  InterruptState;\r
-  UINT64   Count;\r
-  UINT64   Frequency;\r
-  \r
-  InterruptState = SaveAndDisableInterrupts ();\r
-  Count = GetPerformanceCounter ();\r
-  MicroSecondDelay (100);\r
-  Frequency = MultU64x32 (GetPerformanceCounter () - Count, 10000);\r
-  SetInterruptState (InterruptState);\r
-  return Frequency;\r
+  return InternalCalculateTscFrequency ();\r
 }\r

diff --git a/PcAtChipsetPkg/Library/AcpiTimerLib/DxeAcpiTimerLib.c b/PcAtChipsetPkg/Library/AcpiTimerLib/DxeAcpiTimerLib.c
index 6f5c07a4f0b4d6d9a156e885a5aa6e1d0a216ed3..7f7b0f8f62947712e3a3a3d66df51ddeece1c14a 100644 (file)
--- a/PcAtChipsetPkg/Library/AcpiTimerLib/DxeAcpiTimerLib.c
+++ b/PcAtChipsetPkg/Library/AcpiTimerLib/DxeAcpiTimerLib.c
@@ -1,7 +1,7 @@
 /** @file\r
   ACPI Timer implements one instance of Timer Library.\r
 \r
-  Copyright (c) 2013 - 2014, Intel Corporation. All rights reserved.<BR>\r
+  Copyright (c) 2013 - 2016, Intel Corporation. All rights reserved.<BR>\r
   This program and the accompanying materials\r
   are licensed and made available under the terms and conditions of the BSD License\r
   which accompanies this distribution.  The full text of the license may be found at\r
@@ -16,6 +16,26 @@
 #include <Library/TimerLib.h>\r
 #include <Library/BaseLib.h>\r
 \r
+/**\r
+  Calculate TSC frequency.\r
+\r
+  The TSC counting frequency is determined by comparing how far it counts\r
+  during a 100us period as determined by the ACPI timer. The ACPI timer is\r
+  used because it counts at a known frequency.\r
+  The TSC is sampled, followed by waiting for ACPI_TIMER_FREQUENCY / 10000\r
+  clocks of the ACPI timer, or 100us. The TSC is then sampled again. The\r
+  difference multiplied by 10000 is the TSC frequency. There will be a small\r
+  error because of the overhead of reading the ACPI timer. An attempt is\r
+  made to determine and compensate for this error.\r
+\r
+  @return The number of TSC counts per second.\r
+\r
+**/\r
+UINT64\r
+InternalCalculateTscFrequency (\r
+  VOID\r
+  );\r
+\r
 //\r
 // Cached performance counter frequency\r
 //\r
@@ -34,15 +54,8 @@ InternalGetPerformanceCounterFrequency (
   VOID\r
   ) \r
 {\r
-  BOOLEAN  InterruptState;\r
-  UINT64   Count;\r
-\r
   if (mPerformanceCounterFrequency == 0) {\r
-    InterruptState = SaveAndDisableInterrupts ();\r
-    Count = GetPerformanceCounter ();\r
-    MicroSecondDelay (100);\r
-    mPerformanceCounterFrequency = MultU64x32 (GetPerformanceCounter () - Count, 10000);\r
-    SetInterruptState (InterruptState);\r
+    mPerformanceCounterFrequency = InternalCalculateTscFrequency ();\r
   }\r
   return  mPerformanceCounterFrequency;\r
 }\r