Add TPM Physical Presence >=128 operation value support.

The Tcg/TrEE PhysicalPresence library will depend on Tcg/TrEE PpVendor library.
The default NULL library instance is provided in this package.
OEM can create OemPpVendorLib as override to handle >=128 operation value.

Contributed-under: TianoCore Contribution Agreement 1.0
Signed-off-by: "Yao, Jiewen" <jiewen.yao@intel.com>
Reviewed-by: "Dong, Guo" <guo.dong@intel.com>




git-svn-id: https://svn.code.sf.net/p/edk2/code/trunk/edk2@16597 6f19259b-4bc3-4df7-8a09-765794883524

6 files changed, 124 insertions, 79 deletions

diff --git a/SecurityPkg/Tcg/TcgSmm/TcgSmm.c b/SecurityPkg/Tcg/TcgSmm/TcgSmm.c
index 6080eff3b7..96fb456ccd 100644
--- a/SecurityPkg/Tcg/TcgSmm/TcgSmm.c
+++ b/SecurityPkg/Tcg/TcgSmm/TcgSmm.c
@@ -8,7 +8,7 @@
 

   PhysicalPresenceCallback() and MemoryClearCallback() will receive untrusted input and do some check.

 

-Copyright (c) 2011 - 2014, Intel Corporation. All rights reserved.<BR>

+Copyright (c) 2011 - 2015, Intel Corporation. All rights reserved.<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 

@@ -53,7 +53,7 @@ PhysicalPresenceCallback (
   EFI_STATUS                     Status;

   UINTN                          DataSize;

   EFI_PHYSICAL_PRESENCE          PpData;

-  UINT8                          Flags;

+  EFI_PHYSICAL_PRESENCE_FLAGS    Flags;

   BOOLEAN                        RequestConfirmed;

 

   //

@@ -67,23 +67,31 @@ PhysicalPresenceCallback (
                            &DataSize,

                            &PpData

                            );

-  if (EFI_ERROR (Status)) {

-    mTcgNvs->PhysicalPresence.ReturnCode = PP_SUBMIT_REQUEST_GENERAL_FAILURE;

-    DEBUG ((EFI_D_ERROR, "[TPM] Get PP variable failure! Status = %r\n", Status));

-    return EFI_SUCCESS;

-  }

 

   DEBUG ((EFI_D_INFO, "[TPM] PP callback, Parameter = %x\n", mTcgNvs->PhysicalPresence.Parameter));

   if (mTcgNvs->PhysicalPresence.Parameter == ACPI_FUNCTION_RETURN_REQUEST_RESPONSE_TO_OS) {

+    if (EFI_ERROR (Status)) {

+      mTcgNvs->PhysicalPresence.ReturnCode  = PP_RETURN_TPM_OPERATION_RESPONSE_FAILURE;

+      mTcgNvs->PhysicalPresence.LastRequest = 0;

+      mTcgNvs->PhysicalPresence.Response    = 0;

+      DEBUG ((EFI_D_ERROR, "[TPM] Get PP variable failure! Status = %r\n", Status));

+      return EFI_SUCCESS;

+    }

+    mTcgNvs->PhysicalPresence.ReturnCode  = PP_RETURN_TPM_OPERATION_RESPONSE_SUCCESS;

     mTcgNvs->PhysicalPresence.LastRequest = PpData.LastPPRequest;

     mTcgNvs->PhysicalPresence.Response    = PpData.PPResponse;

   } else if ((mTcgNvs->PhysicalPresence.Parameter == ACPI_FUNCTION_SUBMIT_REQUEST_TO_BIOS) 

           || (mTcgNvs->PhysicalPresence.Parameter == ACPI_FUNCTION_SUBMIT_REQUEST_TO_BIOS_2)) {

+    if (EFI_ERROR (Status)) {

+      mTcgNvs->PhysicalPresence.ReturnCode = TCG_PP_SUBMIT_REQUEST_TO_PREOS_GENERAL_FAILURE;

+      DEBUG ((EFI_D_ERROR, "[TPM] Get PP variable failure! Status = %r\n", Status));

+      return EFI_SUCCESS;

+    }

     if (mTcgNvs->PhysicalPresence.Request == PHYSICAL_PRESENCE_SET_OPERATOR_AUTH) {

       //

       // This command requires UI to prompt user for Auth data.

       //

-      mTcgNvs->PhysicalPresence.ReturnCode = PP_SUBMIT_REQUEST_NOT_IMPLEMENTED;

+      mTcgNvs->PhysicalPresence.ReturnCode = TCG_PP_SUBMIT_REQUEST_TO_PREOS_NOT_IMPLEMENTED;

       return EFI_SUCCESS;

     }

 

@@ -100,15 +108,35 @@ PhysicalPresenceCallback (
     }

 

     if (EFI_ERROR (Status)) { 

-      mTcgNvs->PhysicalPresence.ReturnCode = PP_SUBMIT_REQUEST_GENERAL_FAILURE;

+      mTcgNvs->PhysicalPresence.ReturnCode = TCG_PP_SUBMIT_REQUEST_TO_PREOS_GENERAL_FAILURE;

       return EFI_SUCCESS;

     }

-    mTcgNvs->PhysicalPresence.ReturnCode = PP_SUBMIT_REQUEST_SUCCESS;

+    mTcgNvs->PhysicalPresence.ReturnCode = TCG_PP_SUBMIT_REQUEST_TO_PREOS_SUCCESS;

+

+    if (mTcgNvs->PhysicalPresence.Request >= TCG_PHYSICAL_PRESENCE_VENDOR_SPECIFIC_OPERATION) {

+      DataSize = sizeof (EFI_PHYSICAL_PRESENCE_FLAGS);

+      Status = mSmmVariable->SmmGetVariable (

+                               PHYSICAL_PRESENCE_FLAGS_VARIABLE,

+                               &gEfiPhysicalPresenceGuid,

+                               NULL,

+                               &DataSize,

+                               &Flags

+                               );

+      if (EFI_ERROR (Status)) {

+        Flags.PPFlags = TCG_BIOS_TPM_MANAGEMENT_FLAG_NO_PPI_PROVISION;

+      }

+      mTcgNvs->PhysicalPresence.ReturnCode = TcgPpVendorLibSubmitRequestToPreOSFunction (mTcgNvs->PhysicalPresence.Request, Flags.PPFlags);

+    }

   } else if (mTcgNvs->PhysicalPresence.Parameter == ACPI_FUNCTION_GET_USER_CONFIRMATION_STATUS_FOR_REQUEST) {

+    if (EFI_ERROR (Status)) {

+      mTcgNvs->PhysicalPresence.ReturnCode = TCG_PP_GET_USER_CONFIRMATION_BLOCKED_BY_BIOS_CONFIGURATION;

+      DEBUG ((EFI_D_ERROR, "[TPM] Get PP variable failure! Status = %r\n", Status));

+      return EFI_SUCCESS;

+    }

     //

     // Get the Physical Presence flags

     //

-    DataSize = sizeof (UINT8);

+    DataSize = sizeof (EFI_PHYSICAL_PRESENCE_FLAGS);

     Status = mSmmVariable->SmmGetVariable (

                              PHYSICAL_PRESENCE_FLAGS_VARIABLE,

                              &gEfiPhysicalPresenceGuid,

@@ -117,7 +145,7 @@ PhysicalPresenceCallback (
                              &Flags

                              );

     if (EFI_ERROR (Status)) {

-      mTcgNvs->PhysicalPresence.ReturnCode = PP_SUBMIT_REQUEST_GENERAL_FAILURE;

+      mTcgNvs->PhysicalPresence.ReturnCode = TCG_PP_GET_USER_CONFIRMATION_BLOCKED_BY_BIOS_CONFIGURATION;

       DEBUG ((EFI_D_ERROR, "[TPM] Get PP flags failure! Status = %r\n", Status));

       return EFI_SUCCESS;

     }

@@ -135,27 +163,27 @@ PhysicalPresenceCallback (
       case PHYSICAL_PRESENCE_SET_OWNER_INSTALL_FALSE:

       case PHYSICAL_PRESENCE_ENABLE_ACTIVATE_OWNER_TRUE:

       case PHYSICAL_PRESENCE_DEACTIVATE_DISABLE_OWNER_FALSE:

-        if ((Flags & FLAG_NO_PPI_PROVISION) != 0) {

+        if ((Flags.PPFlags & TCG_BIOS_TPM_MANAGEMENT_FLAG_NO_PPI_PROVISION) != 0) {

           RequestConfirmed = TRUE;

         }

         break;

 

       case PHYSICAL_PRESENCE_CLEAR:

       case PHYSICAL_PRESENCE_ENABLE_ACTIVATE_CLEAR:

-        if ((Flags & FLAG_NO_PPI_CLEAR) != 0) {

+        if ((Flags.PPFlags & TCG_BIOS_TPM_MANAGEMENT_FLAG_NO_PPI_CLEAR) != 0) {

           RequestConfirmed = TRUE;

         }

         break;

 

       case PHYSICAL_PRESENCE_DEFERRED_PP_UNOWNERED_FIELD_UPGRADE:

-        if ((Flags & FLAG_NO_PPI_MAINTENANCE) != 0) {

+        if ((Flags.PPFlags & TCG_BIOS_TPM_MANAGEMENT_FLAG_NO_PPI_MAINTENANCE) != 0) {

           RequestConfirmed = TRUE;

         }

         break;

 

       case PHYSICAL_PRESENCE_ENABLE_ACTIVATE_CLEAR_ENABLE_ACTIVATE:

       case PHYSICAL_PRESENCE_CLEAR_ENABLE_ACTIVATE:

-        if ((Flags & FLAG_NO_PPI_CLEAR) != 0 && (Flags & FLAG_NO_PPI_PROVISION) != 0) {

+        if ((Flags.PPFlags & TCG_BIOS_TPM_MANAGEMENT_FLAG_NO_PPI_CLEAR) != 0 && (Flags.PPFlags & TCG_BIOS_TPM_MANAGEMENT_FLAG_NO_PPI_PROVISION) != 0) {

           RequestConfirmed = TRUE;

         }

         break;  

@@ -171,15 +199,20 @@ PhysicalPresenceCallback (
         //

         // This command requires UI to prompt user for Auth data

         //

-        mTcgNvs->PhysicalPresence.ReturnCode = PP_REQUEST_NOT_IMPLEMENTED; 

+        mTcgNvs->PhysicalPresence.ReturnCode = TCG_PP_GET_USER_CONFIRMATION_NOT_IMPLEMENTED; 

         return EFI_SUCCESS;

+      default:

+        break;

     }

 

     if (RequestConfirmed) {

-      mTcgNvs->PhysicalPresence.ReturnCode = PP_REQUEST_ALLOWED_AND_PPUSER_NOT_REQUIRED;

+      mTcgNvs->PhysicalPresence.ReturnCode = TCG_PP_GET_USER_CONFIRMATION_ALLOWED_AND_PPUSER_NOT_REQUIRED;

     } else {

-      mTcgNvs->PhysicalPresence.ReturnCode = PP_REQUEST_ALLOWED_AND_PPUSER_REQUIRED;

+      mTcgNvs->PhysicalPresence.ReturnCode = TCG_PP_GET_USER_CONFIRMATION_ALLOWED_AND_PPUSER_REQUIRED;

     }    

+    if (mTcgNvs->PhysicalPresence.Request >= TCG_PHYSICAL_PRESENCE_VENDOR_SPECIFIC_OPERATION) {

+      mTcgNvs->PhysicalPresence.ReturnCode = TcgPpVendorLibGetUserConfirmationStatusFunction (mTcgNvs->PhysicalPresence.Request, Flags.PPFlags);

+    }

   } 

 

   return EFI_SUCCESS;

diff --git a/SecurityPkg/Tcg/TcgSmm/TcgSmm.h b/SecurityPkg/Tcg/TcgSmm/TcgSmm.h
index eee8bc3d71..d8eb82fac7 100644
--- a/SecurityPkg/Tcg/TcgSmm/TcgSmm.h
+++ b/SecurityPkg/Tcg/TcgSmm/TcgSmm.h
@@ -1,7 +1,7 @@
 /** @file

   The header file for TCG SMM driver.

   

-Copyright (c) 2012 - 2013, Intel Corporation. All rights reserved.<BR>

+Copyright (c) 2012 - 2015, Intel Corporation. All rights reserved.<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 

@@ -36,6 +36,7 @@ WITHOUT WARRANTIES OR REPRESENTATIONS OF ANY KIND, EITHER EXPRESS OR IMPLIED.
 #include <Library/DxeServicesLib.h>

 #include <Library/TpmMeasurementLib.h>

 #include <Library/PcdLib.h>

+#include <Library/TcgPpVendorLib.h>

 

 #pragma pack(1)

 typedef struct {

@@ -83,23 +84,10 @@ typedef struct {
 #define ACPI_FUNCTION_GET_USER_CONFIRMATION_STATUS_FOR_REQUEST     8

 

 //

-// The return code for Get User Confirmation Status for Operation

+// The return code for Return TPM Operation Response to OS Environment

 //

-#define PP_REQUEST_NOT_IMPLEMENTED                                 0

-#define PP_REQUEST_BIOS_ONLY                                       1

-#define PP_REQUEST_BLOCKED                                         2

-#define PP_REQUEST_ALLOWED_AND_PPUSER_REQUIRED                     3

-#define PP_REQUEST_ALLOWED_AND_PPUSER_NOT_REQUIRED                 4

-

-//

-// The return code for Sumbit TPM Request to Pre-OS Environment

-// and Sumbit TPM Request to Pre-OS Environment 2

-//

-#define PP_SUBMIT_REQUEST_SUCCESS                                  0

-#define PP_SUBMIT_REQUEST_NOT_IMPLEMENTED                          1

-#define PP_SUBMIT_REQUEST_GENERAL_FAILURE                          2

-#define PP_SUBMIT_REQUEST_BLOCKED_BY_BIOS_SETTINGS                 3

-

+#define PP_RETURN_TPM_OPERATION_RESPONSE_SUCCESS                   0

+#define PP_RETURN_TPM_OPERATION_RESPONSE_FAILURE                   1

 

 //

 // The definition for TCG MOR

diff --git a/SecurityPkg/Tcg/TcgSmm/TcgSmm.inf b/SecurityPkg/Tcg/TcgSmm/TcgSmm.inf
index f39e089bed..be7a96bc62 100644
--- a/SecurityPkg/Tcg/TcgSmm/TcgSmm.inf
+++ b/SecurityPkg/Tcg/TcgSmm/TcgSmm.inf
@@ -9,7 +9,7 @@
 #  This driver will have external input - variable and ACPINvs data in SMM mode.

 #  This external input must be validated carefully to avoid security issue.

 #

-# Copyright (c) 2011 - 2014, Intel Corporation. All rights reserved.<BR>

+# Copyright (c) 2011 - 2015, Intel Corporation. All rights reserved.<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

@@ -49,6 +49,7 @@
   DxeServicesLib

   TpmMeasurementLib

   PcdLib

+  TcgPpVendorLib

 

 [Guids]

   ## SOMETIMES_PRODUCES ## Variable:L"PhysicalPresence"

diff --git a/SecurityPkg/Tcg/TrEESmm/TrEESmm.c b/SecurityPkg/Tcg/TrEESmm/TrEESmm.c
index 14303d5458..a522cd97c6 100644
--- a/SecurityPkg/Tcg/TrEESmm/TrEESmm.c
+++ b/SecurityPkg/Tcg/TrEESmm/TrEESmm.c
@@ -9,7 +9,7 @@
 

   PhysicalPresenceCallback() and MemoryClearCallback() will receive untrusted input and do some check.

 

-Copyright (c) 2013 - 2014, Intel Corporation. All rights reserved.<BR>

+Copyright (c) 2013 - 2015, Intel Corporation. All rights reserved.<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 

@@ -66,11 +66,11 @@ PhysicalPresenceCallback (
   IN OUT UINTN                   *CommBufferSize

   )

 {

-  EFI_STATUS                     Status;

-  UINTN                          DataSize;

-  EFI_TREE_PHYSICAL_PRESENCE     PpData;

-  UINT8                          Flags;

-  BOOLEAN                        RequestConfirmed;

+  EFI_STATUS                        Status;

+  UINTN                             DataSize;

+  EFI_TREE_PHYSICAL_PRESENCE        PpData;

+  EFI_TREE_PHYSICAL_PRESENCE_FLAGS  Flags;

+  BOOLEAN                           RequestConfirmed;

 

   //

   // Get the Physical Presence variable

@@ -83,24 +83,33 @@ PhysicalPresenceCallback (
                            &DataSize,

                            &PpData

                            );

-  if (EFI_ERROR (Status)) {

-    mTcgNvs->PhysicalPresence.ReturnCode = PP_SUBMIT_REQUEST_GENERAL_FAILURE;

-    DEBUG ((EFI_D_ERROR, "[TPM] Get PP variable failure! Status = %r\n", Status));

-    return EFI_SUCCESS;

-  }

 

   DEBUG ((EFI_D_INFO, "[TPM2] PP callback, Parameter = %x, Request = %x\n", mTcgNvs->PhysicalPresence.Parameter, mTcgNvs->PhysicalPresence.Request));

 

   if (mTcgNvs->PhysicalPresence.Parameter == ACPI_FUNCTION_RETURN_REQUEST_RESPONSE_TO_OS) {

+    if (EFI_ERROR (Status)) {

+      mTcgNvs->PhysicalPresence.ReturnCode  = PP_RETURN_TPM_OPERATION_RESPONSE_FAILURE;

+      mTcgNvs->PhysicalPresence.LastRequest = 0;

+      mTcgNvs->PhysicalPresence.Response    = 0;

+      DEBUG ((EFI_D_ERROR, "[TPM2] Get PP variable failure! Status = %r\n", Status));

+      return EFI_SUCCESS;

+    }

+    mTcgNvs->PhysicalPresence.ReturnCode  = PP_RETURN_TPM_OPERATION_RESPONSE_SUCCESS;

     mTcgNvs->PhysicalPresence.LastRequest = PpData.LastPPRequest;

     mTcgNvs->PhysicalPresence.Response    = PpData.PPResponse;

   } else if ((mTcgNvs->PhysicalPresence.Parameter == ACPI_FUNCTION_SUBMIT_REQUEST_TO_BIOS) 

           || (mTcgNvs->PhysicalPresence.Parameter == ACPI_FUNCTION_SUBMIT_REQUEST_TO_BIOS_2)) {

-    if (mTcgNvs->PhysicalPresence.Request > TREE_PHYSICAL_PRESENCE_NO_ACTION_MAX) {

+    if (EFI_ERROR (Status)) {

+      mTcgNvs->PhysicalPresence.ReturnCode = TREE_PP_SUBMIT_REQUEST_TO_PREOS_GENERAL_FAILURE;

+      DEBUG ((EFI_D_ERROR, "[TPM2] Get PP variable failure! Status = %r\n", Status));

+      return EFI_SUCCESS;

+    }

+    if ((mTcgNvs->PhysicalPresence.Request > TREE_PHYSICAL_PRESENCE_NO_ACTION_MAX) &&

+        (mTcgNvs->PhysicalPresence.Request < TREE_PHYSICAL_PRESENCE_VENDOR_SPECIFIC_OPERATION) ) {

       //

       // This command requires UI to prompt user for Auth data.

       //

-      mTcgNvs->PhysicalPresence.ReturnCode = PP_SUBMIT_REQUEST_NOT_IMPLEMENTED;

+      mTcgNvs->PhysicalPresence.ReturnCode = TREE_PP_SUBMIT_REQUEST_TO_PREOS_NOT_IMPLEMENTED;

       return EFI_SUCCESS;

     }

 

@@ -117,16 +126,36 @@ PhysicalPresenceCallback (
     }

 

     if (EFI_ERROR (Status)) { 

-      mTcgNvs->PhysicalPresence.ReturnCode = PP_SUBMIT_REQUEST_GENERAL_FAILURE;

-      DEBUG ((EFI_D_ERROR, "[TPM] Set PP variable failure! Status = %r\n", Status));

+      mTcgNvs->PhysicalPresence.ReturnCode = TREE_PP_SUBMIT_REQUEST_TO_PREOS_GENERAL_FAILURE;

+      DEBUG ((EFI_D_ERROR, "[TPM2] Set PP variable failure! Status = %r\n", Status));

       return EFI_SUCCESS;

     }

-    mTcgNvs->PhysicalPresence.ReturnCode = PP_SUBMIT_REQUEST_SUCCESS;

+    mTcgNvs->PhysicalPresence.ReturnCode = TREE_PP_SUBMIT_REQUEST_TO_PREOS_SUCCESS;

+

+    if (mTcgNvs->PhysicalPresence.Request >= TREE_PHYSICAL_PRESENCE_VENDOR_SPECIFIC_OPERATION) {

+      DataSize = sizeof (EFI_TREE_PHYSICAL_PRESENCE_FLAGS);

+      Status = mSmmVariable->SmmGetVariable (

+                               TREE_PHYSICAL_PRESENCE_FLAGS_VARIABLE,

+                               &gEfiTrEEPhysicalPresenceGuid,

+                               NULL,

+                               &DataSize,

+                               &Flags

+                               );

+      if (EFI_ERROR (Status)) {

+        Flags.PPFlags = 0;

+      }

+      mTcgNvs->PhysicalPresence.ReturnCode = TrEEPpVendorLibSubmitRequestToPreOSFunction (mTcgNvs->PhysicalPresence.Request, Flags.PPFlags);

+    }

   } else if (mTcgNvs->PhysicalPresence.Parameter == ACPI_FUNCTION_GET_USER_CONFIRMATION_STATUS_FOR_REQUEST) {

+    if (EFI_ERROR (Status)) {

+      mTcgNvs->PhysicalPresence.ReturnCode = TREE_PP_GET_USER_CONFIRMATION_BLOCKED_BY_BIOS_CONFIGURATION;

+      DEBUG ((EFI_D_ERROR, "[TPM2] Get PP variable failure! Status = %r\n", Status));

+      return EFI_SUCCESS;

+    }

     //

     // Get the Physical Presence flags

     //

-    DataSize = sizeof (UINT8);

+    DataSize = sizeof (EFI_TREE_PHYSICAL_PRESENCE_FLAGS);

     Status = mSmmVariable->SmmGetVariable (

                              TREE_PHYSICAL_PRESENCE_FLAGS_VARIABLE,

                              &gEfiTrEEPhysicalPresenceGuid,

@@ -135,8 +164,8 @@ PhysicalPresenceCallback (
                              &Flags

                              );

     if (EFI_ERROR (Status)) {

-      mTcgNvs->PhysicalPresence.ReturnCode = PP_SUBMIT_REQUEST_GENERAL_FAILURE;

-      DEBUG ((EFI_D_ERROR, "[TPM] Get PP flags failure! Status = %r\n", Status));

+      mTcgNvs->PhysicalPresence.ReturnCode = TREE_PP_GET_USER_CONFIRMATION_BLOCKED_BY_BIOS_CONFIGURATION;

+      DEBUG ((EFI_D_ERROR, "[TPM2] Get PP flags failure! Status = %r\n", Status));

       return EFI_SUCCESS;

     }

 

@@ -148,7 +177,7 @@ PhysicalPresenceCallback (
       case TREE_PHYSICAL_PRESENCE_CLEAR_CONTROL_CLEAR_2:

       case TREE_PHYSICAL_PRESENCE_CLEAR_CONTROL_CLEAR_3:

       case TREE_PHYSICAL_PRESENCE_CLEAR_CONTROL_CLEAR_4:

-        if ((Flags & TREE_FLAG_NO_PPI_CLEAR) != 0) {

+        if ((Flags.PPFlags & TREE_BIOS_TPM_MANAGEMENT_FLAG_NO_PPI_CLEAR) != 0) {

           RequestConfirmed = TRUE;

         }

         break;

@@ -164,17 +193,22 @@ PhysicalPresenceCallback (
         if (mTcgNvs->PhysicalPresence.Request <= TREE_PHYSICAL_PRESENCE_NO_ACTION_MAX) {

           RequestConfirmed = TRUE;

         } else {

-          mTcgNvs->PhysicalPresence.ReturnCode = PP_REQUEST_NOT_IMPLEMENTED; 

-          return EFI_SUCCESS;

+          if (mTcgNvs->PhysicalPresence.Request < TREE_PHYSICAL_PRESENCE_VENDOR_SPECIFIC_OPERATION) {

+            mTcgNvs->PhysicalPresence.ReturnCode = TREE_PP_GET_USER_CONFIRMATION_NOT_IMPLEMENTED; 

+            return EFI_SUCCESS;

+          }

         }

         break;

     }

 

     if (RequestConfirmed) {

-      mTcgNvs->PhysicalPresence.ReturnCode = PP_REQUEST_ALLOWED_AND_PPUSER_NOT_REQUIRED;

+      mTcgNvs->PhysicalPresence.ReturnCode = TREE_PP_GET_USER_CONFIRMATION_ALLOWED_AND_PPUSER_NOT_REQUIRED;

     } else {

-      mTcgNvs->PhysicalPresence.ReturnCode = PP_REQUEST_ALLOWED_AND_PPUSER_REQUIRED;

+      mTcgNvs->PhysicalPresence.ReturnCode = TREE_PP_GET_USER_CONFIRMATION_ALLOWED_AND_PPUSER_REQUIRED;

     }    

+    if (mTcgNvs->PhysicalPresence.Request >= TREE_PHYSICAL_PRESENCE_VENDOR_SPECIFIC_OPERATION) {

+      mTcgNvs->PhysicalPresence.ReturnCode = TrEEPpVendorLibGetUserConfirmationStatusFunction (mTcgNvs->PhysicalPresence.Request, Flags.PPFlags);

+    }

   } 

 

   return EFI_SUCCESS;

diff --git a/SecurityPkg/Tcg/TrEESmm/TrEESmm.h b/SecurityPkg/Tcg/TrEESmm/TrEESmm.h
index 8f4117e1e4..a0e1182248 100644
--- a/SecurityPkg/Tcg/TrEESmm/TrEESmm.h
+++ b/SecurityPkg/Tcg/TrEESmm/TrEESmm.h
@@ -1,7 +1,7 @@
 /** @file

   The header file for TrEE SMM driver.

   

-Copyright (c) 2013, Intel Corporation. All rights reserved.<BR>

+Copyright (c) 2013 - 2015, Intel Corporation. All rights reserved.<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 

@@ -37,6 +37,7 @@ WITHOUT WARRANTIES OR REPRESENTATIONS OF ANY KIND, EITHER EXPRESS OR IMPLIED.
 #include <Library/DxeServicesLib.h>

 #include <Library/TpmMeasurementLib.h>

 #include <Library/Tpm2DeviceLib.h>

+#include <Library/TrEEPpVendorLib.h>

 

 #pragma pack(1)

 typedef struct {

@@ -84,23 +85,10 @@ typedef struct {
 #define ACPI_FUNCTION_GET_USER_CONFIRMATION_STATUS_FOR_REQUEST     8

 

 //

-// The return code for Get User Confirmation Status for Operation

+// The return code for Return TPM Operation Response to OS Environment

 //

-#define PP_REQUEST_NOT_IMPLEMENTED                                 0

-#define PP_REQUEST_BIOS_ONLY                                       1

-#define PP_REQUEST_BLOCKED                                         2

-#define PP_REQUEST_ALLOWED_AND_PPUSER_REQUIRED                     3

-#define PP_REQUEST_ALLOWED_AND_PPUSER_NOT_REQUIRED                 4

-

-//

-// The return code for Sumbit TPM Request to Pre-OS Environment

-// and Sumbit TPM Request to Pre-OS Environment 2

-//

-#define PP_SUBMIT_REQUEST_SUCCESS                                  0

-#define PP_SUBMIT_REQUEST_NOT_IMPLEMENTED                          1

-#define PP_SUBMIT_REQUEST_GENERAL_FAILURE                          2

-#define PP_SUBMIT_REQUEST_BLOCKED_BY_BIOS_SETTINGS                 3

-

+#define PP_RETURN_TPM_OPERATION_RESPONSE_SUCCESS                   0

+#define PP_RETURN_TPM_OPERATION_RESPONSE_FAILURE                   1

 

 //

 // The definition for TCG MOR

diff --git a/SecurityPkg/Tcg/TrEESmm/TrEESmm.inf b/SecurityPkg/Tcg/TrEESmm/TrEESmm.inf
index 0436ba5c51..de71ffdc1b 100644
--- a/SecurityPkg/Tcg/TrEESmm/TrEESmm.inf
+++ b/SecurityPkg/Tcg/TrEESmm/TrEESmm.inf
@@ -9,7 +9,7 @@
 #  This driver will have external input - variable and ACPINvs data in SMM mode.

 #  This external input must be validated carefully to avoid security issue.

 #

-# Copyright (c) 2013 - 2014, Intel Corporation. All rights reserved.<BR>

+# Copyright (c) 2013 - 2015, Intel Corporation. All rights reserved.<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

@@ -49,6 +49,7 @@
   DxeServicesLib

   TpmMeasurementLib

   Tpm2DeviceLib

+  TrEEPpVendorLib

 

 [Guids]

   ## SOMETIMES_PRODUCES ## Variable:L"PhysicalPresence"