Re: [Xen-devel] [PATCH v2 02/29] ArmPkg: allow patchable PCDs for memory, FD and FV addresses

["Olivier Martin" <olivier.martin@xxxxxxx>]

Reviewed-By: Olivier Martin <Olivier.martin@xxxxxxx>

> -----Original Message-----
> From: Ard Biesheuvel [mailto:ard.biesheuvel@xxxxxxxxxx]
> Sent: 26 January 2015 19:03
> To: edk2-devel@xxxxxxxxxxxxxxxxxxxxx; lersek@xxxxxxxxxx; Olivier
> Martin; roy.franz@xxxxxxxxxx; leif.lindholm@xxxxxxxxxx;
> stefano.stabellini@xxxxxxxxxxxxx; Ian.Campbell@xxxxxxxxxx;
> anthony.perard@xxxxxxxxxx; christoffer.dall@xxxxxxxxxx; xen-
> devel@xxxxxxxxxxxxx; ilias.biris@xxxxxxxxxx
> Cc: Ard Biesheuvel
> Subject: [PATCH v2 02/29] ArmPkg: allow patchable PCDs for memory, FD
> and FV addresses
> 
> In order to allow a runtime self relocating PrePi instance, change the
> allowable PCD types for the following PCDs:
> 
>   gArmTokenSpaceGuid.PcdSystemMemoryBase
>   gArmTokenSpaceGuid.PcdSystemMemorySize
>   gArmTokenSpaceGuid.PcdFdBaseAddress
>   gArmTokenSpaceGuid.PcdFvBaseAddress
> 
> to include PcdsPatchableInModule. This makes the build system correctly
> distinguish fixed PCDs from PCDs whose value may be different from the
> assigned value at compile time.
> 
> Note that this only affects platforms that explicitly mark these PCDs
> as
> PatchableInModule in the DSC. All existing platforms that use FixedPcd
> will not be affected by this change.
> 
> Contributed-under: TianoCore Contribution Agreement 1.0
> Signed-off-by: Ard Biesheuvel <ard.biesheuvel@xxxxxxxxxx>
> ---
>  ArmPkg/ArmPkg.dec | 25 ++++++++++++++-----------
>  1 file changed, 14 insertions(+), 11 deletions(-)
> 
> diff --git a/ArmPkg/ArmPkg.dec b/ArmPkg/ArmPkg.dec
> index d7a4826d931a..b01de13e5f78 100644
> --- a/ArmPkg/ArmPkg.dec
> +++ b/ArmPkg/ArmPkg.dec
> @@ -93,14 +93,6 @@
>    gArmTokenSpaceGuid.PcdSecureFvSize|0x0|UINT32|0x00000030
> 
>    #
> -  # ARM Normal (or Non Secure) Firmware PCDs
> -  #
> -  gArmTokenSpaceGuid.PcdFdBaseAddress|0|UINT64|0x0000002B
> -  gArmTokenSpaceGuid.PcdFdSize|0|UINT32|0x0000002C
> -  gArmTokenSpaceGuid.PcdFvBaseAddress|0|UINT64|0x0000002D
> -  gArmTokenSpaceGuid.PcdFvSize|0|UINT32|0x0000002E
> -
> -  #
>    # ARM Hypervisor Firmware PCDs
>    #
>    gArmTokenSpaceGuid.PcdHypFdBaseAddress|0|UINT32|0x0000003A
> @@ -127,6 +119,15 @@
>    # Maximum file size for TFTP servers that do not support 'tsize'
> extension
>    gArmTokenSpaceGuid.PcdMaxTftpFileSize|0x01000000|UINT32|0x00000000
> 
> +  #
> +  # ARM Normal (or Non Secure) Firmware PCDs
> +  #
> +  gArmTokenSpaceGuid.PcdFdSize|0|UINT32|0x0000002C
> +  gArmTokenSpaceGuid.PcdFvSize|0|UINT32|0x0000002E
> +
> +[PcdsFixedAtBuild.common, PcdsPatchableInModule.common]
> +  gArmTokenSpaceGuid.PcdFdBaseAddress|0|UINT64|0x0000002B
> +  gArmTokenSpaceGuid.PcdFvBaseAddress|0|UINT64|0x0000002D
> 
>  [PcdsFixedAtBuild.ARM]
>    #
> @@ -207,16 +208,18 @@
> 
> 
>  #
> -# These PCDs are also defined as 'PcdsDynamic' to be redefined when
> using UEFI in a
> -# context of virtual machine.
> +# These PCDs are also defined as 'PcdsDynamic' or
> 'PcdsPatchableInModule' to be
> +# redefined when using UEFI in a context of virtual machine.
>  #
> -[PcdsFixedAtBuild.common, PcdsDynamic.common]
> +[PcdsFixedAtBuild.common, PcdsDynamic.common,
> PcdsPatchableInModule.common]
> +
>    # System Memory (DRAM): These PCDs define the region of in-built
> system memory
>    # Some platforms can get DRAM extensions, these additional regions
> will be declared
>    # to UEFI by ArmPlatformLib
>    gArmTokenSpaceGuid.PcdSystemMemoryBase|0|UINT64|0x00000029
>    gArmTokenSpaceGuid.PcdSystemMemorySize|0|UINT64|0x0000002A
> 
> +[PcdsFixedAtBuild.common, PcdsDynamic.common]
>    #
>    # ARM Architectural Timer
>    #
> --
> 1.8.3.2
> 





_______________________________________________
Xen-devel mailing list
Xen-devel@xxxxxxxxxxxxx
http://lists.xen.org/xen-devel