OvmfPkg: move PciEncoding into AcpiPlatformLib

Bhyve supports providing ACPI tables by FwCfg. Therefore,
InstallQemuFwCfgTables should be moved to AcpiPlatformLib to reuse the
code. As first step, move PciEncoding into AcpiPlatformLib.

Signed-off-by: Corvin Köhne <corvink@FreeBSD.org>
Acked-by: Peter Grehan <grehan@freebsd.org>

19 files changed, 38 insertions, 274 deletions

diff --git a/ArmVirtPkg/ArmVirtQemu.dsc b/ArmVirtPkg/ArmVirtQemu.dsc
index d998f816b2..1e0225951a 100644
--- a/ArmVirtPkg/ArmVirtQemu.dsc
+++ b/ArmVirtPkg/ArmVirtQemu.dsc
@@ -111,6 +111,7 @@
   ArmMmuLib|ArmPkg/Library/ArmMmuLib/ArmMmuPeiLib.inf

 

 [LibraryClasses.common.DXE_DRIVER]

+  AcpiPlatformLib|OvmfPkg/Library/AcpiPlatformLib/DxeAcpiPlatformLib.inf

   ReportStatusCodeLib|MdeModulePkg/Library/DxeReportStatusCodeLib/DxeReportStatusCodeLib.inf

 

 !if $(TPM2_ENABLE) == TRUE

diff --git a/ArmVirtPkg/ArmVirtQemuKernel.dsc b/ArmVirtPkg/ArmVirtQemuKernel.dsc
index c9d2b18392..8ef5927b53 100644
--- a/ArmVirtPkg/ArmVirtQemuKernel.dsc
+++ b/ArmVirtPkg/ArmVirtQemuKernel.dsc
@@ -83,6 +83,7 @@
   TpmPlatformHierarchyLib|SecurityPkg/Library/PeiDxeTpmPlatformHierarchyLibNull/PeiDxeTpmPlatformHierarchyLib.inf

 

 [LibraryClasses.common.DXE_DRIVER]

+  AcpiPlatformLib|OvmfPkg/Library/AcpiPlatformLib/DxeAcpiPlatformLib.inf

   ReportStatusCodeLib|MdeModulePkg/Library/DxeReportStatusCodeLib/DxeReportStatusCodeLib.inf

 

 [LibraryClasses.common.UEFI_DRIVER]

diff --git a/OvmfPkg/AcpiPlatformDxe/AcpiPlatform.h b/OvmfPkg/AcpiPlatformDxe/AcpiPlatform.h
index 3ec5098658..1328f6d1cb 100644
--- a/OvmfPkg/AcpiPlatformDxe/AcpiPlatform.h
+++ b/OvmfPkg/AcpiPlatformDxe/AcpiPlatform.h
@@ -10,12 +10,6 @@
 #define ACPI_PLATFORM_H_

 

 #include <Protocol/AcpiTable.h> // EFI_ACPI_TABLE_PROTOCOL

-#include <Protocol/PciIo.h>     // EFI_PCI_IO_PROTOCOL

-

-typedef struct {

-  EFI_PCI_IO_PROTOCOL    *PciIo;

-  UINT64                 PciAttributes;

-} ORIGINAL_ATTRIBUTES;

 

 typedef struct S3_CONTEXT S3_CONTEXT;

 

@@ -43,18 +37,6 @@ InstallAcpiTables (
   IN   EFI_ACPI_TABLE_PROTOCOL  *AcpiTable

   );

 

-VOID

-EnablePciDecoding (

-  OUT ORIGINAL_ATTRIBUTES  **OriginalAttributes,

-  OUT UINTN                *Count

-  );

-

-VOID

-RestorePciDecoding (

-  IN ORIGINAL_ATTRIBUTES  *OriginalAttributes,

-  IN UINTN                Count

-  );

-

 EFI_STATUS

 AllocateS3Context (

   OUT S3_CONTEXT  **S3Context,

diff --git a/OvmfPkg/AcpiPlatformDxe/AcpiPlatformDxe.inf b/OvmfPkg/AcpiPlatformDxe/AcpiPlatformDxe.inf
index 3fd0483b50..b22aad95e0 100644
--- a/OvmfPkg/AcpiPlatformDxe/AcpiPlatformDxe.inf
+++ b/OvmfPkg/AcpiPlatformDxe/AcpiPlatformDxe.inf
@@ -26,7 +26,6 @@
   BootScript.c

   CloudHvAcpi.c

   EntryPoint.c

-  PciDecoding.c

   QemuFwCfgAcpi.c

 

 [Packages]

@@ -35,15 +34,13 @@
   OvmfPkg/OvmfPkg.dec

 

 [LibraryClasses]

+  AcpiPlatformLib

   BaseLib

   BaseMemoryLib

-  DebugLib

   MemoryAllocationLib

   OrderedCollectionLib

-  PcdLib

   QemuFwCfgLib

   QemuFwCfgS3Lib

-  UefiBootServicesTableLib

   UefiDriverEntryPoint

   HobLib

   TpmMeasurementLib

diff --git a/OvmfPkg/AcpiPlatformDxe/QemuFwCfgAcpi.c b/OvmfPkg/AcpiPlatformDxe/QemuFwCfgAcpi.c
index a073b292b7..3de039d574 100644
--- a/OvmfPkg/AcpiPlatformDxe/QemuFwCfgAcpi.c
+++ b/OvmfPkg/AcpiPlatformDxe/QemuFwCfgAcpi.c
@@ -11,6 +11,7 @@
 #include <IndustryStandard/Acpi.h>            // EFI_ACPI_DESCRIPTION_HEADER

 #include <IndustryStandard/QemuLoader.h>      // QEMU_LOADER_FNAME_SIZE

 #include <IndustryStandard/UefiTcgPlatform.h>

+#include <Library/AcpiPlatformLib.h>

 #include <Library/BaseLib.h>                  // AsciiStrCmp()

 #include <Library/BaseMemoryLib.h>            // CopyMem()

 #include <Library/DebugLib.h>                 // DEBUG()

diff --git a/OvmfPkg/AmdSev/AmdSevX64.dsc b/OvmfPkg/AmdSev/AmdSevX64.dsc
index d24909e847..2c6ed7c974 100644
--- a/OvmfPkg/AmdSev/AmdSevX64.dsc
+++ b/OvmfPkg/AmdSev/AmdSevX64.dsc
@@ -329,6 +329,7 @@
   PciLib|OvmfPkg/Library/DxePciLibI440FxQ35/DxePciLibI440FxQ35.inf

 

 [LibraryClasses.common.DXE_DRIVER]

+  AcpiPlatformLib|OvmfPkg/Library/AcpiPlatformLib/DxeAcpiPlatformLib.inf

   PcdLib|MdePkg/Library/DxePcdLib/DxePcdLib.inf

   TimerLib|OvmfPkg/Library/AcpiTimerLib/DxeAcpiTimerLib.inf

   ResetSystemLib|OvmfPkg/Library/ResetSystemLib/DxeResetSystemLib.inf

diff --git a/OvmfPkg/Bhyve/AcpiPlatformDxe/AcpiPlatform.h b/OvmfPkg/Bhyve/AcpiPlatformDxe/AcpiPlatform.h
index 54d1af073e..b75292b735 100644
--- a/OvmfPkg/Bhyve/AcpiPlatformDxe/AcpiPlatform.h
+++ b/OvmfPkg/Bhyve/AcpiPlatformDxe/AcpiPlatform.h
@@ -21,11 +21,6 @@
 #include <Library/XenPlatformLib.h>

 #include <IndustryStandard/Acpi.h>

 

-typedef struct {

-  EFI_PCI_IO_PROTOCOL    *PciIo;

-  UINT64                 PciAttributes;

-} ORIGINAL_ATTRIBUTES;

-

 typedef struct S3_CONTEXT S3_CONTEXT;

 

 EFI_STATUS

@@ -58,16 +53,4 @@ InstallAcpiTables (
   IN   EFI_ACPI_TABLE_PROTOCOL  *AcpiTable

   );

 

-VOID

-EnablePciDecoding (

-  OUT ORIGINAL_ATTRIBUTES  **OriginalAttributes,

-  OUT UINTN                *Count

-  );

-

-VOID

-RestorePciDecoding (

-  IN ORIGINAL_ATTRIBUTES  *OriginalAttributes,

-  IN UINTN                Count

-  );

-

 #endif /* _ACPI_PLATFORM_H_INCLUDED_ */

diff --git a/OvmfPkg/Bhyve/AcpiPlatformDxe/AcpiPlatformDxe.inf b/OvmfPkg/Bhyve/AcpiPlatformDxe/AcpiPlatformDxe.inf
index 75ed8e4a7d..2e228d815b 100644
--- a/OvmfPkg/Bhyve/AcpiPlatformDxe/AcpiPlatformDxe.inf
+++ b/OvmfPkg/Bhyve/AcpiPlatformDxe/AcpiPlatformDxe.inf
@@ -26,7 +26,6 @@
   AcpiPlatform.h

   Bhyve.c

   EntryPoint.c

-  PciDecoding.c

 

 [Packages]

   MdePkg/MdePkg.dec

diff --git a/OvmfPkg/Bhyve/AcpiPlatformDxe/PciDecoding.c b/OvmfPkg/Bhyve/AcpiPlatformDxe/PciDecoding.c
deleted file mode 100644
index 0dcf3517f1..0000000000
--- a/OvmfPkg/Bhyve/AcpiPlatformDxe/PciDecoding.c
+++ /dev/null
@@ -1,232 +0,0 @@
-/** @file

-  Temporarily enable IO and MMIO decoding for all PCI devices while QEMU

-  regenerates the ACPI tables.

-

-  Copyright (C) 2016, Red Hat, Inc.

-

-  SPDX-License-Identifier: BSD-2-Clause-Patent

-**/

-

-#include <Library/MemoryAllocationLib.h>

-

-#include "AcpiPlatform.h"

-

-/**

-  Collect all PciIo protocol instances in the system. Save their original

-  attributes, and enable IO and MMIO decoding for each.

-

-  This is a best effort function; it doesn't return status codes. Its

-  caller is supposed to proceed even if this function fails.

-

-  @param[out] OriginalAttributes  On output, a dynamically allocated array of

-                                  ORIGINAL_ATTRIBUTES elements. The array lists

-                                  the PciIo protocol instances found in the

-                                  system at the time of the call, plus the

-                                  original PCI attributes for each.

-

-                                  Before returning, the function enables IO and

-                                  MMIO decoding for each PciIo instance it

-                                  finds.

-

-                                  On error, or when no such instances are

-                                  found, OriginalAttributes is set to NULL.

-

-  @param[out] Count               On output, the number of elements in

-                                  OriginalAttributes. On error it is set to

-                                  zero.

-**/

-VOID

-EnablePciDecoding (

-  OUT ORIGINAL_ATTRIBUTES  **OriginalAttributes,

-  OUT UINTN                *Count

-  )

-{

-  EFI_STATUS           Status;

-  UINTN                NoHandles;

-  EFI_HANDLE           *Handles;

-  ORIGINAL_ATTRIBUTES  *OrigAttrs;

-  UINTN                Idx;

-

-  *OriginalAttributes = NULL;

-  *Count              = 0;

-

-  if (PcdGetBool (PcdPciDisableBusEnumeration)) {

-    //

-    // The platform downloads ACPI tables from QEMU in general, but there are

-    // no root bridges in this execution. We're done.

-    //

-    return;

-  }

-

-  Status = gBS->LocateHandleBuffer (

-                  ByProtocol,

-                  &gEfiPciIoProtocolGuid,

-                  NULL /* SearchKey */,

-                  &NoHandles,

-                  &Handles

-                  );

-  if (Status == EFI_NOT_FOUND) {

-    //

-    // No PCI devices were found on either of the root bridges. We're done.

-    //

-    return;

-  }

-

-  if (EFI_ERROR (Status)) {

-    DEBUG ((

-      DEBUG_WARN,

-      "%a: LocateHandleBuffer(): %r\n",

-      __func__,

-      Status

-      ));

-    return;

-  }

-

-  OrigAttrs = AllocatePool (NoHandles * sizeof *OrigAttrs);

-  if (OrigAttrs == NULL) {

-    DEBUG ((

-      DEBUG_WARN,

-      "%a: AllocatePool(): out of resources\n",

-      __func__

-      ));

-    goto FreeHandles;

-  }

-

-  for (Idx = 0; Idx < NoHandles; ++Idx) {

-    EFI_PCI_IO_PROTOCOL  *PciIo;

-    UINT64               Attributes;

-

-    //

-    // Look up PciIo on the handle and stash it

-    //

-    Status = gBS->HandleProtocol (

-                    Handles[Idx],

-                    &gEfiPciIoProtocolGuid,

-                    (VOID **)&PciIo

-                    );

-    ASSERT_EFI_ERROR (Status);

-    OrigAttrs[Idx].PciIo = PciIo;

-

-    //

-    // Stash the current attributes

-    //

-    Status = PciIo->Attributes (

-                      PciIo,

-                      EfiPciIoAttributeOperationGet,

-                      0,

-                      &OrigAttrs[Idx].PciAttributes

-                      );

-    if (EFI_ERROR (Status)) {

-      DEBUG ((

-        DEBUG_WARN,

-        "%a: EfiPciIoAttributeOperationGet: %r\n",

-        __func__,

-        Status

-        ));

-      goto RestoreAttributes;

-    }

-

-    //

-    // Retrieve supported attributes

-    //

-    Status = PciIo->Attributes (

-                      PciIo,

-                      EfiPciIoAttributeOperationSupported,

-                      0,

-                      &Attributes

-                      );

-    if (EFI_ERROR (Status)) {

-      DEBUG ((

-        DEBUG_WARN,

-        "%a: EfiPciIoAttributeOperationSupported: %r\n",

-        __func__,

-        Status

-        ));

-      goto RestoreAttributes;

-    }

-

-    //

-    // Enable IO and MMIO decoding

-    //

-    Attributes &= EFI_PCI_IO_ATTRIBUTE_IO | EFI_PCI_IO_ATTRIBUTE_MEMORY;

-    Status      = PciIo->Attributes (

-                           PciIo,

-                           EfiPciIoAttributeOperationEnable,

-                           Attributes,

-                           NULL

-                           );

-    if (EFI_ERROR (Status)) {

-      DEBUG ((

-        DEBUG_WARN,

-        "%a: EfiPciIoAttributeOperationEnable: %r\n",

-        __func__,

-        Status

-        ));

-      goto RestoreAttributes;

-    }

-  }

-

-  //

-  // Success

-  //

-  FreePool (Handles);

-  *OriginalAttributes = OrigAttrs;

-  *Count              = NoHandles;

-  return;

-

-RestoreAttributes:

-  while (Idx > 0) {

-    --Idx;

-    OrigAttrs[Idx].PciIo->Attributes (

-                            OrigAttrs[Idx].PciIo,

-                            EfiPciIoAttributeOperationSet,

-                            OrigAttrs[Idx].PciAttributes,

-                            NULL

-                            );

-  }

-

-  FreePool (OrigAttrs);

-

-FreeHandles:

-  FreePool (Handles);

-}

-

-/**

-  Restore the original PCI attributes saved with EnablePciDecoding().

-

-  @param[in] OriginalAttributes  The array allocated and populated by

-                                 EnablePciDecoding(). This parameter may be

-                                 NULL. If OriginalAttributes is NULL, then the

-                                 function is a no-op; otherwise the PciIo

-                                 attributes will be restored, and the

-                                 OriginalAttributes array will be freed.

-

-  @param[in] Count               The Count value stored by EnablePciDecoding(),

-                                 the number of elements in OriginalAttributes.

-                                 Count may be zero if and only if

-                                 OriginalAttributes is NULL.

-**/

-VOID

-RestorePciDecoding (

-  IN ORIGINAL_ATTRIBUTES  *OriginalAttributes,

-  IN UINTN                Count

-  )

-{

-  UINTN  Idx;

-

-  ASSERT ((OriginalAttributes == NULL) == (Count == 0));

-  if (OriginalAttributes == NULL) {

-    return;

-  }

-

-  for (Idx = 0; Idx < Count; ++Idx) {

-    OriginalAttributes[Idx].PciIo->Attributes (

-                                     OriginalAttributes[Idx].PciIo,

-                                     EfiPciIoAttributeOperationSet,

-                                     OriginalAttributes[Idx].PciAttributes,

-                                     NULL

-                                     );

-  }

-

-  FreePool (OriginalAttributes);

-}

diff --git a/OvmfPkg/CloudHv/CloudHvX64.dsc b/OvmfPkg/CloudHv/CloudHvX64.dsc
index dd9aa403d6..e000deed9e 100644
--- a/OvmfPkg/CloudHv/CloudHvX64.dsc
+++ b/OvmfPkg/CloudHv/CloudHvX64.dsc
@@ -366,6 +366,7 @@
   PciLib|OvmfPkg/Library/DxePciLibI440FxQ35/DxePciLibI440FxQ35.inf

 

 [LibraryClasses.common.DXE_DRIVER]

+  AcpiPlatformLib|OvmfPkg/Library/AcpiPlatformLib/DxeAcpiPlatformLib.inf

   PcdLib|MdePkg/Library/DxePcdLib/DxePcdLib.inf

   TimerLib|OvmfPkg/Library/AcpiTimerLib/DxeAcpiTimerLib.inf

   ResetSystemLib|OvmfPkg/Library/ResetSystemLib/DxeResetSystemLib.inf

diff --git a/OvmfPkg/Include/Library/AcpiPlatformLib.h b/OvmfPkg/Include/Library/AcpiPlatformLib.h
index 6172650003..167d6c49d6 100644
--- a/OvmfPkg/Include/Library/AcpiPlatformLib.h
+++ b/OvmfPkg/Include/Library/AcpiPlatformLib.h
@@ -8,6 +8,12 @@
 #define ACPI_PLATFORM_LIB_H_

 

 #include <Protocol/AcpiTable.h>

+#include <Protocol/PciIo.h>

+

+typedef struct {

+  EFI_PCI_IO_PROTOCOL    *PciIo;

+  UINT64                 PciAttributes;

+} ORIGINAL_ATTRIBUTES;

 

 /**

   Searches and returns the address of the ACPI Root System Description Pointer (RSDP) in system memory.

@@ -51,4 +57,16 @@ InstallAcpiTablesFromRsdp (
   IN EFI_ACPI_2_0_ROOT_SYSTEM_DESCRIPTION_POINTER  *Rsdp

   );

 

+VOID

+EnablePciDecoding (

+  OUT ORIGINAL_ATTRIBUTES  **OriginalAttributes,

+  OUT UINTN                *Count

+  );

+

+VOID

+RestorePciDecoding (

+  IN ORIGINAL_ATTRIBUTES  *OriginalAttributes,

+  IN UINTN                Count

+  );

+

 #endif

diff --git a/OvmfPkg/IntelTdx/IntelTdxX64.dsc b/OvmfPkg/IntelTdx/IntelTdxX64.dsc
index db4953a82e..193657ff2d 100644
--- a/OvmfPkg/IntelTdx/IntelTdxX64.dsc
+++ b/OvmfPkg/IntelTdx/IntelTdxX64.dsc
@@ -289,6 +289,7 @@
   PciLib|OvmfPkg/Library/DxePciLibI440FxQ35/DxePciLibI440FxQ35.inf

 

 [LibraryClasses.common.DXE_DRIVER]

+  AcpiPlatformLib|OvmfPkg/Library/AcpiPlatformLib/DxeAcpiPlatformLib.inf

   PcdLib|MdePkg/Library/DxePcdLib/DxePcdLib.inf

   TimerLib|OvmfPkg/Library/AcpiTimerLib/DxeAcpiTimerLib.inf

   ResetSystemLib|OvmfPkg/Library/ResetSystemLib/DxeResetSystemLib.inf

diff --git a/OvmfPkg/Library/AcpiPlatformLib/DxeAcpiPlatformLib.inf b/OvmfPkg/Library/AcpiPlatformLib/DxeAcpiPlatformLib.inf
index dfe0e5623d..4be501bb2c 100644
--- a/OvmfPkg/Library/AcpiPlatformLib/DxeAcpiPlatformLib.inf
+++ b/OvmfPkg/Library/AcpiPlatformLib/DxeAcpiPlatformLib.inf
@@ -16,11 +16,18 @@
 

 [Sources]

   DxeAcpiPlatformLib.c

+  PciDecoding.c

 

 [Packages]

+  MdeModulePkg/MdeModulePkg.dec

   MdePkg/MdePkg.dec

   OvmfPkg/OvmfPkg.dec

 

 [LibraryClasses]

   BaseLib

   DebugLib

+  PcdLib

+  UefiBootServicesTableLib

+

+[Pcd]

+  gEfiMdeModulePkgTokenSpaceGuid.PcdPciDisableBusEnumeration

diff --git a/OvmfPkg/AcpiPlatformDxe/PciDecoding.c b/OvmfPkg/Library/AcpiPlatformLib/PciDecoding.c
index c7dbfb1baa..58fbfeefda 100644
--- a/OvmfPkg/AcpiPlatformDxe/PciDecoding.c
+++ b/OvmfPkg/Library/AcpiPlatformLib/PciDecoding.c
@@ -7,12 +7,11 @@
   SPDX-License-Identifier: BSD-2-Clause-Patent

 **/

 

+#include <Library/AcpiPlatformLib.h>

 #include <Library/DebugLib.h>                  // DEBUG()

 #include <Library/MemoryAllocationLib.h>       // AllocatePool()

 #include <Library/UefiBootServicesTableLib.h>  // gBS

 

-#include "AcpiPlatform.h"

-

 /**

   Collect all PciIo protocol instances in the system. Save their original

   attributes, and enable IO and MMIO decoding for each.

diff --git a/OvmfPkg/Microvm/MicrovmX64.dsc b/OvmfPkg/Microvm/MicrovmX64.dsc
index 82da3a7e7c..2f75856393 100644
--- a/OvmfPkg/Microvm/MicrovmX64.dsc
+++ b/OvmfPkg/Microvm/MicrovmX64.dsc
@@ -368,6 +368,7 @@
   PciExpressLib|OvmfPkg/Library/BaseCachingPciExpressLib/BaseCachingPciExpressLib.inf

 

 [LibraryClasses.common.DXE_DRIVER]

+  AcpiPlatformLib|OvmfPkg/Library/AcpiPlatformLib/DxeAcpiPlatformLib.inf

   PcdLib|MdePkg/Library/DxePcdLib/DxePcdLib.inf

   ResetSystemLib|OvmfPkg/Library/ResetSystemLib/DxeResetSystemLibMicrovm.inf

   HobLib|MdePkg/Library/DxeHobLib/DxeHobLib.inf

diff --git a/OvmfPkg/OvmfPkgIa32.dsc b/OvmfPkg/OvmfPkgIa32.dsc
index 08b04e65ca..8d1ef4e833 100644
--- a/OvmfPkg/OvmfPkgIa32.dsc
+++ b/OvmfPkg/OvmfPkgIa32.dsc
@@ -373,6 +373,7 @@
   PciLib|OvmfPkg/Library/DxePciLibI440FxQ35/DxePciLibI440FxQ35.inf

 

 [LibraryClasses.common.DXE_DRIVER]

+  AcpiPlatformLib|OvmfPkg/Library/AcpiPlatformLib/DxeAcpiPlatformLib.inf

   PcdLib|MdePkg/Library/DxePcdLib/DxePcdLib.inf

   TimerLib|OvmfPkg/Library/AcpiTimerLib/DxeAcpiTimerLib.inf

   ResetSystemLib|OvmfPkg/Library/ResetSystemLib/DxeResetSystemLib.inf

diff --git a/OvmfPkg/OvmfPkgIa32X64.dsc b/OvmfPkg/OvmfPkgIa32X64.dsc
index d456896a45..afee349910 100644
--- a/OvmfPkg/OvmfPkgIa32X64.dsc
+++ b/OvmfPkg/OvmfPkgIa32X64.dsc
@@ -378,6 +378,7 @@
   PciLib|OvmfPkg/Library/DxePciLibI440FxQ35/DxePciLibI440FxQ35.inf

 

 [LibraryClasses.common.DXE_DRIVER]

+  AcpiPlatformLib|OvmfPkg/Library/AcpiPlatformLib/DxeAcpiPlatformLib.inf

   PcdLib|MdePkg/Library/DxePcdLib/DxePcdLib.inf

   TimerLib|OvmfPkg/Library/AcpiTimerLib/DxeAcpiTimerLib.inf

   ResetSystemLib|OvmfPkg/Library/ResetSystemLib/DxeResetSystemLib.inf

diff --git a/OvmfPkg/OvmfPkgX64.dsc b/OvmfPkg/OvmfPkgX64.dsc
index e1c0a6fe9f..24ac834e06 100644
--- a/OvmfPkg/OvmfPkgX64.dsc
+++ b/OvmfPkg/OvmfPkgX64.dsc
@@ -399,6 +399,7 @@
   PciLib|OvmfPkg/Library/DxePciLibI440FxQ35/DxePciLibI440FxQ35.inf

 

 [LibraryClasses.common.DXE_DRIVER]

+  AcpiPlatformLib|OvmfPkg/Library/AcpiPlatformLib/DxeAcpiPlatformLib.inf

   PcdLib|MdePkg/Library/DxePcdLib/DxePcdLib.inf

   TimerLib|OvmfPkg/Library/AcpiTimerLib/DxeAcpiTimerLib.inf

   ResetSystemLib|OvmfPkg/Library/ResetSystemLib/DxeResetSystemLib.inf

diff --git a/OvmfPkg/RiscVVirt/RiscVVirtQemu.dsc b/OvmfPkg/RiscVVirt/RiscVVirtQemu.dsc
index 04573bc0f3..34b2037824 100644
--- a/OvmfPkg/RiscVVirt/RiscVVirtQemu.dsc
+++ b/OvmfPkg/RiscVVirt/RiscVVirtQemu.dsc
@@ -113,6 +113,7 @@
 !endif

 

 [LibraryClasses.common.DXE_DRIVER]

+  AcpiPlatformLib|OvmfPkg/Library/AcpiPlatformLib/DxeAcpiPlatformLib.inf

   ReportStatusCodeLib|MdeModulePkg/Library/DxeReportStatusCodeLib/DxeReportStatusCodeLib.inf

   PciExpressLib|OvmfPkg/Library/BaseCachingPciExpressLib/BaseCachingPciExpressLib.inf