OvmfPkg: Use TdHob instead of e820tables to get memory info in TDVF

[sunceping]

Currently, TDVF gets LowMemory and FistNonAddress from the e820tables
via fw_cfg, while TD-Hob can also provide the memory info of LowMemory
and FistNonAddress.

In current stage e820tables are not measured but TD-Hob is measured in
early phase by TDVF.

So, from the security perspective we'd better use the information from
TD-Hob instead of e820tables.

Cc: Erdem Aktas [email protected]
Cc: Jiewen Yao [email protected]
Cc: Min Xu [email protected]
Cc: Gerd Hoffmann [email protected]
Cc: Elena Reshetova [email protected]
Signed-off-by: Ceping Sun [email protected]

Description

<Include a description of the change and why this change was made.>

<For each item, place an "x" in between [ and ] if true. Example: [x] (you can also check items in GitHub UI)>

<Create the PR as a Draft PR if it is only created to run CI checks.>

<Delete lines in <> tags before creating the PR.>

• Breaking change?
  • Breaking change - Does this PR cause a break in build or boot behavior?
  • Examples: Does it add a new library class or move a module to a different repo.
• Impacts security?
  • Security - Does this PR have a direct security impact?
  • Examples: Crypto algorithm change or buffer overflow fix.
• Includes tests?
  • Tests - Does this PR include any explicit test code?
  • Examples: Unit tests or integration tests.

How This Was Tested

<Describe the test(s) that were run to verify the changes.>

Integration Instructions

<Describe how these changes should be integrated. Use N/A if nothing is required.>

[kraxel]

Yes, the PlatformReservationConflict should not be needed when running on TDX:

For memory detection I'd suggest to use the cloud hypervisor idea, see PlatformScanE820Pvh. Have a relatively small helper function which reads the tdx hob, translates the entries to e820, goes call the callback with that. Should make the patch much smaller.

[sunceping]

For memory detection I'd suggest to use the cloud hypervisor idea, see PlatformScanE820Pvh. Have a relatively small helper function which reads the tdx hob, translates the entries to e820, goes call the callback with that. Should make the patch much smaller.

Thanks, now updated.

[kraxel]

Patch #1 looks good to me.
Patch #2 should not be needed any more due to patch #1 switching PlatformScanE820 to NOT use fw_cfg on TDX.

[sunceping]

Patch #1 looks good to me. Patch #2 should not be needed any more due to patch #1 switching PlatformScanE820 to NOT use fw_cfg on TDX.

Thanks, patch 2 has now been removed.