Re: [edk2-devel] [PATCH v2 2/6] OvmfPkg/AmdSev: add Grub Firmware Volume Package

["James Bottomley" <jejb@linux.ibm.com>]

On Tue, 2020-11-24 at 16:58 +0100, Laszlo Ersek wrote:
> On 11/24/20 15:54, Laszlo Ersek wrote:
[...]
> Perhaps I can make that recommendation / request more detailed too:
> 
> 
> * drop:
> 
> - UefiRuntimeServicesTableLib: no use of "gRT"
> 
> - ReportStatusCodeLib: commit 0a0566d5edad is not relevant, because
> we justifiedly removed TryRunningQemuKernel()
> 
> - XenPlatformLib: as discussed before; substitute FALSE for each
> XenDetected() call, and compress the resultant code

All dropped.

> * keep:
> 
> - BaseLib: for CpuDeadLoop()
> 
> - MemoryAllocationLib: for FreePool()
> 
> - UefiBootServicesTableLib: for gBS->xxx()
> 
> - BaseMemoryLib: for CompareMem()
> 
> - DebugLib: for DEBUG() and ASSERT()
> 
> - PcdLib: for PcdGet16 (PcdOvmfHostBridgePciDevId)
> 
> - UefiBootManagerLib: for APIs central to the functionality of
> PlatformBootManagerLibGrub
> 
> - BootLogoLib: for BootLogoEnableLogo()
> 
> - DevicePathLib: for a bunch of device path manipulation
> 
> - PciLib: mainly for the functions called in PciAcpiInitialization()
> 
> - UefiLib: for EfiEventGroupSignal() etc
> 
> - PlatformBmPrintScLib: for PlatformBmPrintScRegisterHandler() --
> this is responsible for printing the boot option processing steps to
> the UEFI console
> 
> - Tcg2PhysicalPresenceLib: for
> Tcg2PhysicalItPresenceLibProcessRequest() -- we preserve TPM support
> 
> 
> Then leaving the trimming of [Packages] to the end makes sense --
> after trimming everything else, try to remove each package DEC in
> isolation, and see if the lib instance continues to build.

I can't seem to remove any packages.  Even the source debug agent one
is used by the uart devices.

It seems I can get rid of PcdFSBClock

and in the protocols section (with some header file removal)

gEfiDecompressProtocolGuid
gEfiS3SaveStateProtocolGuid

James