]> git.proxmox.com Git - mirror_edk2.git/blobdiff - EdkModulePkg/Universal/StatusCode/Dxe/DataHubStatusCodeWorker.c
Fix EDKT497. In the patch:
[mirror_edk2.git] / EdkModulePkg / Universal / StatusCode / Dxe / DataHubStatusCodeWorker.c
index 03095c38198378abf82bcb8ec235300036b21345..28714365a67b11a36a997bc719378fdf5fd97c2c 100644 (file)
 //\r
 // Initialize FIFO to cache records.\r
 //\r
-EFI_LOCK                  mFifoLock        = EFI_INITIALIZE_LOCK_VARIABLE  (EFI_TPL_HIGH_LEVEL);\r
-LIST_ENTRY                mRecordsFifo     = INITIALIZE_LIST_HEAD_VARIABLE (mRecordsFifo);\r
-UINTN                     mNumberOfRecords = 0;\r
-\r
+STATIC\r
+LIST_ENTRY                mRecordsFifo          = INITIALIZE_LIST_HEAD_VARIABLE (mRecordsFifo);\r
+STATIC\r
+LIST_ENTRY                mRecordsBuffer        = INITIALIZE_LIST_HEAD_VARIABLE (mRecordsBuffer);\r
+STATIC\r
 EFI_EVENT                 mLogDataHubEvent;\r
 //\r
 // Cache data hub protocol.\r
 //\r
+STATIC\r
 EFI_DATA_HUB_PROTOCOL     *mDataHubProtocol;\r
 \r
 \r
 /**\r
-  Return buffer of length DATAHUB_STATUSCODE_RECORD\r
+  Return one DATAHUB_STATUSCODE_RECORD space.\r
+  The size of free record pool would be extend, if the pool is empty. \r
+\r
  \r
   @retval  NULL   Can not allocate free memeory for record.\r
   @retval  !NULL  Point to buffer of record.\r
@@ -42,17 +46,39 @@ AcquireRecordBuffer (
   )\r
 {\r
   DATAHUB_STATUSCODE_RECORD *Record;\r
+  EFI_TPL                   CurrentTpl;\r
+  LIST_ENTRY                *Node;\r
+  UINT32                    Index;\r
+\r
+  CurrentTpl = gBS->RaiseTPL (EFI_TPL_HIGH_LEVEL);\r
+\r
+  if (!IsListEmpty (&mRecordsBuffer)) {\r
+    Node = GetFirstNode (&mRecordsBuffer);\r
+    RemoveEntryList (Node);\r
+\r
+    Record = CR (Node, DATAHUB_STATUSCODE_RECORD, Node, DATAHUB_STATUS_CODE_SIGNATURE);\r
+  } else {\r
+    if (CurrentTpl > EFI_TPL_NOTIFY) {\r
+      gBS->RestoreTPL (CurrentTpl);\r
+      return NULL;\r
+    }\r
 \r
-  Record   = (DATAHUB_STATUSCODE_RECORD *) AllocateZeroPool (sizeof (DATAHUB_STATUSCODE_RECORD));\r
-  if (NULL == Record) {\r
-    return NULL;\r
+    gBS->RestoreTPL (CurrentTpl);\r
+    Record   = (DATAHUB_STATUSCODE_RECORD *) AllocateZeroPool (sizeof (DATAHUB_STATUSCODE_RECORD) * 16);\r
+    if (NULL == Record) {\r
+      return NULL;\r
+    }\r
+\r
+    CurrentTpl = gBS->RaiseTPL (EFI_TPL_HIGH_LEVEL);\r
+    for (Index = 1; Index < 16; Index++) {\r
+      InsertTailList (&mRecordsBuffer, &Record[Index].Node);\r
+    }\r
   }\r
-  Record->Signature = DATAHUB_STATUS_CODE_SIGNATURE;\r
 \r
-  EfiAcquireLock (&mFifoLock);\r
+  Record->Signature = DATAHUB_STATUS_CODE_SIGNATURE;\r
   InsertTailList (&mRecordsFifo, &Record->Node);\r
-  mNumberOfRecords++;\r
-  EfiReleaseLock (&mFifoLock);\r
+\r
+  gBS->RestoreTPL (CurrentTpl);\r
 \r
   return Record;\r
 }\r
@@ -63,53 +89,35 @@ AcquireRecordBuffer (
 \r
   @param   Record        Point to record buffer which is acquired by AcquirRecordBuffer()\r
  \r
-  @retval  EFI_SUCCESS   If DataRecord is valid.\r
-  @retval  !EFI_SUCCESS  The record list has empty.\r
-\r
 **/\r
 VOID\r
 FreeRecordBuffer (\r
   IN  DATAHUB_STATUSCODE_RECORD  *Record\r
   )\r
-/*++\r
-\r
-Routine Description:\r
-\r
-  Release a mRecordBuffer entry allocated by AquireEmptyRecordBuffer ().\r
-\r
-Arguments:\r
-\r
-  RecordBuffer          - Data to free\r
-\r
-Returns:\r
-\r
-  EFI_SUCCESS           - If DataRecord is valid\r
-  EFI_UNSUPPORTED       - The record list has empty\r
-\r
---*/\r
 {\r
+  EFI_TPL  CurrentTpl;\r
+\r
   ASSERT (Record != NULL);\r
-  ASSERT (mNumberOfRecords != 0);\r
 \r
-  EfiAcquireLock (&mFifoLock);\r
+  CurrentTpl = gBS->RaiseTPL (EFI_TPL_HIGH_LEVEL);\r
+\r
   RemoveEntryList (&Record->Node);\r
-  mNumberOfRecords--;\r
-  EfiReleaseLock (&mFifoLock);\r
+  InsertTailList (&mRecordsBuffer, &Record->Node);\r
 \r
-  FreePool (Record);\r
+  gBS->RestoreTPL (CurrentTpl);\r
 }\r
 \r
 \r
 /**\r
   Report status code into DataHub.\r
  \r
-  @param  CodeType      Indicates the type of status code being reported.  Type EFI_STATUS_CODE_TYPE is defined in "Related Definitions¡± below.\r
+  @param  CodeType      Indicates the type of status code being reported.  Type EFI_STATUS_CODE_TYPE is defined in "Related Definitions" below.\r
  \r
   @param  Value         Describes the current status of a hardware or software entity.  \r
                         This included information about the class and subclass that is used to classify the entity \r
                         as well as an operation.  For progress codes, the operation is the current activity. \r
                         For error codes, it is the exception.  For debug codes, it is not defined at this time. \r
-                        Type EFI_STATUS_CODE_VALUE is defined in ¡°Related Definitions¡± below.  \r
+                        Type EFI_STATUS_CODE_VALUE is defined in "Related Definitions" below.  \r
                         Specific values are discussed in the Intel? Platform Innovation Framework for EFI Status Code Specification.\r
  \r
   @param  Instance      The enumeration of a hardware or software entity within the system.  \r
@@ -126,6 +134,7 @@ Returns:
   @param  Data          This optional parameter may be used to pass additional data\r
  \r
   @retval EFI_OUT_OF_RESOURCES   Can not acquire record buffer.\r
+  @retval EFI_DEVICE_ERROR       EFI serial device can not work after ExitBootService() is called .\r
   @retval EFI_SUCCESS            Success to cache status code and signal log data event.\r
 \r
 **/\r
@@ -148,10 +157,10 @@ DataHubStatusCodeReportWorker (
   // See whether in runtime phase or not.\r
   //\r
   if (EfiAtRuntime ()) {\r
-    return EFI_SUCCESS;\r
+    return EFI_DEVICE_ERROR;\r
   }\r
 \r
-  Record = (DATAHUB_STATUSCODE_RECORD *) AcquireRecordBuffer ();\r
+  Record = AcquireRecordBuffer ();\r
   if (Record == NULL) {\r
     //\r
     // There are no empty record buffer in private buffers\r
@@ -218,15 +227,18 @@ LogDataHubEventCallBack (
   )\r
 {\r
   DATAHUB_STATUSCODE_RECORD         *Record;\r
-  UINT                            Size;\r
+  UINT32                            Size;\r
   UINT64                            DataRecordClass;\r
   LIST_ENTRY                        *Node;\r
+  EFI_TPL                           CurrentTpl;\r
 \r
   //\r
   // Log DataRecord in Data Hub.\r
   // Journal records fifo to find all record entry.\r
   //\r
   //\r
+  CurrentTpl = gBS->RaiseTPL (EFI_TPL_HIGH_LEVEL);\r
+\r
   for (Node = mRecordsFifo.ForwardLink; Node != &mRecordsFifo;) {\r
     Record = CR (Node, DATAHUB_STATUSCODE_RECORD, Node, DATAHUB_STATUS_CODE_SIGNATURE);\r
     Node   = Node->ForwardLink;\r
@@ -234,7 +246,7 @@ LogDataHubEventCallBack (
     //\r
     // Add in the size of the header we added.\r
     //\r
-    Size = sizeof (DATAHUB_STATUSCODE_RECORD) + Record->Data.Size;\r
+    Size = sizeof (DATAHUB_STATUSCODE_RECORD) + (UINT32) Record->Data.Size;\r
 \r
     if ((Record->CodeType & EFI_STATUS_CODE_TYPE_MASK) == EFI_PROGRESS_CODE) {\r
       DataRecordClass = EFI_DATA_RECORD_CLASS_PROGRESS_CODE;\r
@@ -262,11 +274,15 @@ LogDataHubEventCallBack (
                         &gEfiStatusCodeRuntimeProtocolGuid,\r
                         DataRecordClass,\r
                         Record,\r
-                        (UINT32) Size\r
+                        Size\r
                         );\r
 \r
+\r
+\r
     FreeRecordBuffer (Record);\r
   }\r
+\r
+  gBS->RestoreTPL (CurrentTpl);\r
 }\r
 \r
 \r