I agree with this fix. 

Reviewed-by: Jeff Fan <[email protected]>

-----Original Message-----
From: edk2-devel [mailto:[email protected]] On Behalf Of Michael 
Kinney
Sent: Thursday, October 29, 2015 11:19 AM
To: [email protected]
Subject: [edk2] [PATCH 0/2] Add API to manage Local APIC SoftwareEnable

The LocalApicLib does not provide any APIs to manage the Local APIC 
SoftwareEnable state without side effects on other registers.  This feature is 
required by the SourceLevelDebugAgent to make sure the Local APIC is in the 
correct state to use the Local APIC Timer.

Contributed-under: TianoCore Contribution Agreement 1.0
Signed-off-by: Michael Kinney <[email protected]>

Michael Kinney (2):
  UefiCpuPkg: LocalApicLib: Add API to set SoftwareEnable bit
  SourceLevelDebugPkg: DebugAgent: Set Local APIC SoftwareEnable

 .../DebugAgent/DebugAgentCommon/DebugTimer.c       |  1 +
 .../DebugAgent/DxeDebugAgent/DxeDebugAgentLib.c    |  1 +
 .../DebugAgent/SmmDebugAgent/SmmDebugAgentLib.c    |  1 +
 UefiCpuPkg/Include/Library/LocalApicLib.h          | 16 ++++++++-
 UefiCpuPkg/Library/BaseXApicLib/BaseXApicLib.c     | 38 +++++++++++++++++++---
 .../BaseXApicX2ApicLib/BaseXApicX2ApicLib.c        | 38 +++++++++++++++++++---
 6 files changed, 86 insertions(+), 9 deletions(-)

--
1.9.5.msysgit.1

_______________________________________________
edk2-devel mailing list
[email protected]
https://lists.01.org/mailman/listinfo/edk2-devel
_______________________________________________
edk2-devel mailing list
[email protected]
https://lists.01.org/mailman/listinfo/edk2-devel

Reply via email to