tyler.durden/audk
1
0
Fork 0
mirror of https://github.com/acidanthera/audk.git synced 2025-07-23 13:44:33 +02:00
Code Issues Packages Projects Releases Wiki Activity

OvmfPkg: QemuFlashFvbServicesRuntimeDxe: no dual addressing needed

Browse Source 
Currently the EFI_FW_VOL_INSTANCE and ESAL_FWB_GLOBAL structures declare
the following entries as arrays, with two entries each:

- EFI_FW_VOL_INSTANCE.FvBase[2]
- ESAL_FWB_GLOBAL.FvInstance[2]

In every case, the entry at subscript zero is meant as "physical address",
while the entry at subscript one is meant as "virtual address" -- a
pointer to the same object. The virtual address entry is originally
initialized to the physical address, and then it is converted to the
virtual mapping in FvbVirtualddressChangeEvent().

Functions that (a) read the listed fields and (b) run both before and
after the virtual address change event -- since this is a runtime DXE
driver -- derive the correct array subscript by calling the
EfiGoneVirtual() function from UefiRuntimeLib.

The problem with the above infrastructure is that it's entirely
superfluous.

EfiGoneVirtual() "knows" whether EFI has gone virtual only because the
UefiRuntimeLib constructor registers the exact same kind of virtual
address change callback, and the callback flips a static variabe to TRUE,
and EfiGoneVirtual() queries that static variable.

In effect this means for QemuFlashFvbServicesRuntimeDxe: "when there is a
virtual address change, convert the entries with subscript one from
physical to virtual, and from then on use the entries with subscript one".

This would only make sense if QemuFlashFvbServicesRuntimeDxe ever needed
the original (physical) addresses (ie. the entries with subscript zero)
after the virtual address change, but that is not the case.

Replace the arrays with single elements. The subscript zero elements
simply disappear, and the single elements take the role of the prior
subscript one elements.

Contributed-under: TianoCore Contribution Agreement 1.0
Signed-off-by: Laszlo Ersek <lersek@redhat.com>
Reviewed-by: Jordan Justen <jordan.l.justen@intel.com>

git-svn-id: https://svn.code.sf.net/p/edk2/code/trunk/edk2@18670 6f19259b-4bc3-4df7-8a09-765794883524
This commit is contained in:
Laszlo Ersek 2015-10-26 14:58:33 +00:00 committed by lersek
parent f97a5b5e4c
commit 109301e5a1
2 changed files with 30 additions and 69 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

77
OvmfPkg/QemuFlashFvbServicesRuntimeDxe/FwBlockService.c
View File

@ -132,11 +132,6 @@ FvbVirtualddressChangeEvent (
Call the passed in Child Notify event and convert the mFvbModuleGlobal Call the passed in Child Notify event and convert the mFvbModuleGlobal
date items to there virtual address. date items to there virtual address.
mFvbModuleGlobal->FvInstance[FVB_PHYSICAL] - Physical copy of instance
data
mFvbModuleGlobal->FvInstance[FVB_VIRTUAL] - Virtual pointer to common
instance data.
Arguments: Arguments:
(Standard EFI notify event - EFI_EVENT_NOTIFY) (Standard EFI notify event - EFI_EVENT_NOTIFY)
@ -150,16 +145,15 @@ FvbVirtualddressChangeEvent (
EFI_FW_VOL_INSTANCE *FwhInstance; EFI_FW_VOL_INSTANCE *FwhInstance;
UINTN Index; UINTN Index;
EfiConvertPointer (0x0, FwhInstance = mFvbModuleGlobal->FvInstance;
(VOID **) &mFvbModuleGlobal->FvInstance[FVB_VIRTUAL]); EfiConvertPointer (0x0, (VOID **) &mFvbModuleGlobal->FvInstance);
// //
// Convert the base address of all the instances // Convert the base address of all the instances
// //
Index = 0; Index = 0;
FwhInstance = mFvbModuleGlobal->FvInstance[FVB_PHYSICAL];
while (Index < mFvbModuleGlobal->NumFv) { while (Index < mFvbModuleGlobal->NumFv) {
EfiConvertPointer (0x0, (VOID **) &FwhInstance->FvBase[FVB_VIRTUAL]); EfiConvertPointer (0x0, (VOID **) &FwhInstance->FvBase);
FwhInstance = (EFI_FW_VOL_INSTANCE *) FwhInstance = (EFI_FW_VOL_INSTANCE *)
( (
(UINTN) ((UINT8 *) FwhInstance) + (UINTN) ((UINT8 *) FwhInstance) +
@ -177,8 +171,7 @@ EFI_STATUS
GetFvbInstance ( GetFvbInstance (
IN UINTN Instance, IN UINTN Instance,
IN ESAL_FWB_GLOBAL *Global, IN ESAL_FWB_GLOBAL *Global,
OUT EFI_FW_VOL_INSTANCE **FwhInstance, OUT EFI_FW_VOL_INSTANCE **FwhInstance
IN BOOLEAN Virtual
) )
/*++ /*++
@ -191,7 +184,6 @@ GetFvbInstance (
Global - Pointer to ESAL_FWB_GLOBAL that contains all Global - Pointer to ESAL_FWB_GLOBAL that contains all
instance data instance data
FwhInstance - The EFI_FW_VOL_INSTANCE fimrware instance structure FwhInstance - The EFI_FW_VOL_INSTANCE fimrware instance structure
Virtual - Whether CPU is in virtual or physical mode
Returns: Returns:
EFI_SUCCESS - Successfully returns EFI_SUCCESS - Successfully returns
@ -208,7 +200,7 @@ GetFvbInstance (
// //
// Find the right instance of the FVB private data // Find the right instance of the FVB private data
// //
FwhRecord = Global->FvInstance[Virtual]; FwhRecord = Global->FvInstance;
while (Instance > 0) { while (Instance > 0) {
FwhRecord = (EFI_FW_VOL_INSTANCE *) FwhRecord = (EFI_FW_VOL_INSTANCE *)
( (
@ -227,8 +219,7 @@ EFI_STATUS
FvbGetPhysicalAddress ( FvbGetPhysicalAddress (
IN UINTN Instance, IN UINTN Instance,
OUT EFI_PHYSICAL_ADDRESS *Address, OUT EFI_PHYSICAL_ADDRESS *Address,
IN ESAL_FWB_GLOBAL *Global, IN ESAL_FWB_GLOBAL *Global
IN BOOLEAN Virtual
) )
/*++ /*++
@ -243,7 +234,6 @@ FvbGetPhysicalAddress (
address of the firmware volume. address of the firmware volume.
Global - Pointer to ESAL_FWB_GLOBAL that contains all Global - Pointer to ESAL_FWB_GLOBAL that contains all
instance data instance data
Virtual - Whether CPU is in virtual or physical mode
Returns: Returns:
EFI_SUCCESS - Successfully returns EFI_SUCCESS - Successfully returns
@ -257,9 +247,9 @@ FvbGetPhysicalAddress (
// //
// Find the right instance of the FVB private data // Find the right instance of the FVB private data
// //
Status = GetFvbInstance (Instance, Global, &FwhInstance, Virtual); Status = GetFvbInstance (Instance, Global, &FwhInstance);
ASSERT_EFI_ERROR (Status); ASSERT_EFI_ERROR (Status);
*Address = FwhInstance->FvBase[Virtual]; *Address = FwhInstance->FvBase;
return EFI_SUCCESS; return EFI_SUCCESS;
} }
@ -268,8 +258,7 @@ EFI_STATUS
FvbGetVolumeAttributes ( FvbGetVolumeAttributes (
IN UINTN Instance, IN UINTN Instance,
OUT EFI_FVB_ATTRIBUTES_2 *Attributes, OUT EFI_FVB_ATTRIBUTES_2 *Attributes,
IN ESAL_FWB_GLOBAL *Global, IN ESAL_FWB_GLOBAL *Global
IN BOOLEAN Virtual
) )
/*++ /*++
@ -283,7 +272,6 @@ FvbGetVolumeAttributes (
Attributes - Output buffer which contains attributes Attributes - Output buffer which contains attributes
Global - Pointer to ESAL_FWB_GLOBAL that contains all Global - Pointer to ESAL_FWB_GLOBAL that contains all
instance data instance data
Virtual - Whether CPU is in virtual or physical mode
Returns: Returns:
EFI_SUCCESS - Successfully returns EFI_SUCCESS - Successfully returns
@ -297,7 +285,7 @@ FvbGetVolumeAttributes (
// //
// Find the right instance of the FVB private data // Find the right instance of the FVB private data
// //
Status = GetFvbInstance (Instance, Global, &FwhInstance, Virtual); Status = GetFvbInstance (Instance, Global, &FwhInstance);
ASSERT_EFI_ERROR (Status); ASSERT_EFI_ERROR (Status);
*Attributes = FwhInstance->VolumeHeader.Attributes; *Attributes = FwhInstance->VolumeHeader.Attributes;
@ -311,8 +299,7 @@ FvbGetLbaAddress (
OUT UINTN *LbaAddress, OUT UINTN *LbaAddress,
OUT UINTN *LbaLength, OUT UINTN *LbaLength,
OUT UINTN *NumOfBlocks, OUT UINTN *NumOfBlocks,
IN ESAL_FWB_GLOBAL *Global, IN ESAL_FWB_GLOBAL *Global
IN BOOLEAN Virtual
) )
/*++ /*++
@ -331,7 +318,6 @@ FvbGetLbaAddress (
BlockSize BlockSize
Global - Pointer to ESAL_FWB_GLOBAL that contains all Global - Pointer to ESAL_FWB_GLOBAL that contains all
instance data instance data
Virtual - Whether CPU is in virtual or physical mode
Returns: Returns:
EFI_SUCCESS - Successfully returns EFI_SUCCESS - Successfully returns
@ -351,7 +337,7 @@ FvbGetLbaAddress (
// //
// Find the right instance of the FVB private data // Find the right instance of the FVB private data
// //
Status = GetFvbInstance (Instance, Global, &FwhInstance, Virtual); Status = GetFvbInstance (Instance, Global, &FwhInstance);
ASSERT_EFI_ERROR (Status); ASSERT_EFI_ERROR (Status);
StartLba = 0; StartLba = 0;
@ -377,7 +363,7 @@ FvbGetLbaAddress (
if (Lba >= StartLba && Lba < NextLba) { if (Lba >= StartLba && Lba < NextLba) {
Offset = Offset + (UINTN) MultU64x32 ((Lba - StartLba), BlockLength); Offset = Offset + (UINTN) MultU64x32 ((Lba - StartLba), BlockLength);
if (LbaAddress != NULL) { if (LbaAddress != NULL) {
*LbaAddress = FwhInstance->FvBase[Virtual] + Offset; *LbaAddress = FwhInstance->FvBase + Offset;
} }
if (LbaLength != NULL) { if (LbaLength != NULL) {
@ -401,8 +387,7 @@ EFI_STATUS
FvbSetVolumeAttributes ( FvbSetVolumeAttributes (
IN UINTN Instance, IN UINTN Instance,
IN OUT EFI_FVB_ATTRIBUTES_2 *Attributes, IN OUT EFI_FVB_ATTRIBUTES_2 *Attributes,
IN ESAL_FWB_GLOBAL *Global, IN ESAL_FWB_GLOBAL *Global
IN BOOLEAN Virtual
) )
/*++ /*++
@ -419,7 +404,6 @@ FvbSetVolumeAttributes (
of the firmware volume of the firmware volume
Global - Pointer to ESAL_FWB_GLOBAL that contains all Global - Pointer to ESAL_FWB_GLOBAL that contains all
instance data instance data
Virtual - Whether CPU is in virtual or physical mode
Returns: Returns:
EFI_SUCCESS - Successfully returns EFI_SUCCESS - Successfully returns
@ -442,7 +426,7 @@ FvbSetVolumeAttributes (
// //
// Find the right instance of the FVB private data // Find the right instance of the FVB private data
// //
Status = GetFvbInstance (Instance, Global, &FwhInstance, Virtual); Status = GetFvbInstance (Instance, Global, &FwhInstance);
ASSERT_EFI_ERROR (Status); ASSERT_EFI_ERROR (Status);
AttribPtr = AttribPtr =
@ -562,7 +546,7 @@ FvbProtocolGetPhysicalAddress (
FvbDevice = FVB_DEVICE_FROM_THIS (This); FvbDevice = FVB_DEVICE_FROM_THIS (This);
return FvbGetPhysicalAddress (FvbDevice->Instance, Address, return FvbGetPhysicalAddress (FvbDevice->Instance, Address,
mFvbModuleGlobal, EfiGoneVirtual ()); mFvbModuleGlobal);
} }
EFI_STATUS EFI_STATUS
@ -604,8 +588,7 @@ FvbProtocolGetBlockSize (
NULL, NULL,
BlockSize, BlockSize,
NumOfBlocks, NumOfBlocks,
mFvbModuleGlobal, mFvbModuleGlobal
EfiGoneVirtual ()
); );
} }
@ -634,7 +617,7 @@ FvbProtocolGetAttributes (
FvbDevice = FVB_DEVICE_FROM_THIS (This); FvbDevice = FVB_DEVICE_FROM_THIS (This);
return FvbGetVolumeAttributes (FvbDevice->Instance, Attributes, return FvbGetVolumeAttributes (FvbDevice->Instance, Attributes,
mFvbModuleGlobal, EfiGoneVirtual ()); mFvbModuleGlobal);
} }
EFI_STATUS EFI_STATUS
@ -662,7 +645,7 @@ FvbProtocolSetAttributes (
FvbDevice = FVB_DEVICE_FROM_THIS (This); FvbDevice = FVB_DEVICE_FROM_THIS (This);
return FvbSetVolumeAttributes (FvbDevice->Instance, Attributes, return FvbSetVolumeAttributes (FvbDevice->Instance, Attributes,
mFvbModuleGlobal, EfiGoneVirtual ()); mFvbModuleGlobal);
} }
EFI_STATUS EFI_STATUS
@ -708,7 +691,7 @@ FvbProtocolEraseBlocks (
FvbDevice = FVB_DEVICE_FROM_THIS (This); FvbDevice = FVB_DEVICE_FROM_THIS (This);
Status = GetFvbInstance (FvbDevice->Instance, mFvbModuleGlobal, Status = GetFvbInstance (FvbDevice->Instance, mFvbModuleGlobal,
&FwhInstance, EfiGoneVirtual ()); &FwhInstance);
ASSERT_EFI_ERROR (Status); ASSERT_EFI_ERROR (Status);
NumOfBlocks = FwhInstance->NumOfBlocks; NumOfBlocks = FwhInstance->NumOfBlocks;
@ -1088,21 +1071,10 @@ FvbInitialize (
FwVolHeader->HeaderLength - FwVolHeader->HeaderLength -
sizeof (EFI_FIRMWARE_VOLUME_HEADER) sizeof (EFI_FIRMWARE_VOLUME_HEADER)
); );
mFvbModuleGlobal->FvInstance = AllocateRuntimePool (BufferSize);
ASSERT (mFvbModuleGlobal->FvInstance != NULL);
// FwhInstance = mFvbModuleGlobal->FvInstance;
// Only need to allocate once. There is only one copy of physical memory for
// the private data of each FV instance. But in virtual mode or in physical
// mode, the address of the the physical memory may be different.
//
mFvbModuleGlobal->FvInstance[FVB_PHYSICAL] = AllocateRuntimePool (
BufferSize);
ASSERT (mFvbModuleGlobal->FvInstance[FVB_PHYSICAL] != NULL);
//
// Make a virtual copy of the FvInstance pointer.
//
FwhInstance = mFvbModuleGlobal->FvInstance[FVB_PHYSICAL];
mFvbModuleGlobal->FvInstance[FVB_VIRTUAL] = FwhInstance;
mFvbModuleGlobal->NumFv = 0; mFvbModuleGlobal->NumFv = 0;
MaxLbaSize = 0; MaxLbaSize = 0;
@ -1111,8 +1083,7 @@ FvbInitialize (
(EFI_FIRMWARE_VOLUME_HEADER *) (UINTN) (EFI_FIRMWARE_VOLUME_HEADER *) (UINTN)
PcdGet32 (PcdOvmfFlashNvStorageVariableBase); PcdGet32 (PcdOvmfFlashNvStorageVariableBase);
FwhInstance->FvBase[FVB_PHYSICAL] = (UINTN) BaseAddress; FwhInstance->FvBase = (UINTN) BaseAddress;
FwhInstance->FvBase[FVB_VIRTUAL] = (UINTN) BaseAddress;
CopyMem ((UINTN *) &(FwhInstance->VolumeHeader), (UINTN *) FwVolHeader, CopyMem ((UINTN *) &(FwhInstance->VolumeHeader), (UINTN *) FwVolHeader,
FwVolHeader->HeaderLength); FwVolHeader->HeaderLength);

22
OvmfPkg/QemuFlashFvbServicesRuntimeDxe/FwBlockService.h
View File

@ -23,21 +23,15 @@
#ifndef _FW_BLOCK_SERVICE_H #ifndef _FW_BLOCK_SERVICE_H
#define _FW_BLOCK_SERVICE_H #define _FW_BLOCK_SERVICE_H
//
// BugBug: Add documentation here for data structure!!!!
//
#define FVB_PHYSICAL 0
#define FVB_VIRTUAL 1
typedef struct { typedef struct {
UINTN FvBase[2]; UINTN FvBase;
UINTN NumOfBlocks; UINTN NumOfBlocks;
EFI_FIRMWARE_VOLUME_HEADER VolumeHeader; EFI_FIRMWARE_VOLUME_HEADER VolumeHeader;
} EFI_FW_VOL_INSTANCE; } EFI_FW_VOL_INSTANCE;
typedef struct { typedef struct {
UINT32 NumFv; UINT32 NumFv;
EFI_FW_VOL_INSTANCE *FvInstance[2]; EFI_FW_VOL_INSTANCE *FvInstance;
} ESAL_FWB_GLOBAL; } ESAL_FWB_GLOBAL;
// //
@ -78,24 +72,21 @@ EFI_STATUS
FvbSetVolumeAttributes ( FvbSetVolumeAttributes (
IN UINTN Instance, IN UINTN Instance,
IN OUT EFI_FVB_ATTRIBUTES_2 *Attributes, IN OUT EFI_FVB_ATTRIBUTES_2 *Attributes,
IN ESAL_FWB_GLOBAL *Global, IN ESAL_FWB_GLOBAL *Global
IN BOOLEAN Virtual
); );
EFI_STATUS EFI_STATUS
FvbGetVolumeAttributes ( FvbGetVolumeAttributes (
IN UINTN Instance, IN UINTN Instance,
OUT EFI_FVB_ATTRIBUTES_2 *Attributes, OUT EFI_FVB_ATTRIBUTES_2 *Attributes,
IN ESAL_FWB_GLOBAL *Global, IN ESAL_FWB_GLOBAL *Global
IN BOOLEAN Virtual
); );
EFI_STATUS EFI_STATUS
FvbGetPhysicalAddress ( FvbGetPhysicalAddress (
IN UINTN Instance, IN UINTN Instance,
OUT EFI_PHYSICAL_ADDRESS *Address, OUT EFI_PHYSICAL_ADDRESS *Address,
IN ESAL_FWB_GLOBAL *Global, IN ESAL_FWB_GLOBAL *Global
IN BOOLEAN Virtual
); );
EFI_STATUS EFI_STATUS
@ -120,8 +111,7 @@ FvbGetLbaAddress (
OUT UINTN *LbaAddress, OUT UINTN *LbaAddress,
OUT UINTN *LbaLength, OUT UINTN *LbaLength,
OUT UINTN *NumOfBlocks, OUT UINTN *NumOfBlocks,
IN ESAL_FWB_GLOBAL *Global, IN ESAL_FWB_GLOBAL *Global
IN BOOLEAN Virtual
); );
// //