[Patch] UefiCpuPkg: Update SecCore to get BFV size based on BFV header FvLength

public inbox for devel@edk2.groups.io
 help / color / mirror / Atom feed

* [Patch] UefiCpuPkg: Update SecCore to get BFV size based on BFV header FvLength
@ 2017-12-06  8:02 Liming Gao
  2017-12-06  8:24 ` Zeng, Star
                   ` (2 more replies)
  0 siblings, 3 replies; 4+ messages in thread
From: Liming Gao @ 2017-12-06  8:02 UTC (permalink / raw)
  To: edk2-devel; +Cc: Eric Dong, Star Zeng

Contributed-under: TianoCore Contribution Agreement 1.1
Signed-off-by: Liming Gao <liming.gao@intel.com>
Cc: Eric Dong <eric.dong@intel.com>
Cc: Star Zeng <star.zeng@intel.com>
---
 UefiCpuPkg/SecCore/SecMain.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/UefiCpuPkg/SecCore/SecMain.c b/UefiCpuPkg/SecCore/SecMain.c
index 173bbfcfcb..c241d3704a 100644
--- a/UefiCpuPkg/SecCore/SecMain.c
+++ b/UefiCpuPkg/SecCore/SecMain.c
@@ -189,7 +189,7 @@ SecStartup (
   //
   SecCoreData.DataSize               = (UINT16) sizeof (EFI_SEC_PEI_HAND_OFF);
   SecCoreData.BootFirmwareVolumeBase = BootFirmwareVolume;
-  SecCoreData.BootFirmwareVolumeSize = (UINTN)(0x100000000ULL - (UINTN) BootFirmwareVolume);
+  SecCoreData.BootFirmwareVolumeSize = (UINTN)((EFI_FIRMWARE_VOLUME_HEADER *) BootFirmwareVolume)->FvLength;
   SecCoreData.TemporaryRamBase       = (VOID*)(UINTN) TempRamBase;
   SecCoreData.TemporaryRamSize       = SizeOfRam;
   SecCoreData.PeiTemporaryRamBase    = SecCoreData.TemporaryRamBase;
-- 
2.11.0.windows.1



^ permalink raw reply related	[flat|nested] 4+ messages in thread

* Re: [Patch] UefiCpuPkg: Update SecCore to get BFV size based on BFV header FvLength
  2017-12-06  8:02 [Patch] UefiCpuPkg: Update SecCore to get BFV size based on BFV header FvLength Liming Gao
@ 2017-12-06  8:24 ` Zeng, Star
  2017-12-07  2:49 ` Yao, Jiewen
  2017-12-07  2:54 ` Dong, Eric
  2 siblings, 0 replies; 4+ messages in thread
From: Zeng, Star @ 2017-12-06  8:24 UTC (permalink / raw)
  To: Gao, Liming, edk2-devel@lists.01.org; +Cc: Dong, Eric, Zeng, Star

Reviewed-by: Star Zeng <star.zeng@intel.com>

-----Original Message-----
From: Gao, Liming 
Sent: Wednesday, December 6, 2017 4:02 PM
To: edk2-devel@lists.01.org
Cc: Dong, Eric <eric.dong@intel.com>; Zeng, Star <star.zeng@intel.com>
Subject: [Patch] UefiCpuPkg: Update SecCore to get BFV size based on BFV header FvLength

Contributed-under: TianoCore Contribution Agreement 1.1
Signed-off-by: Liming Gao <liming.gao@intel.com>
Cc: Eric Dong <eric.dong@intel.com>
Cc: Star Zeng <star.zeng@intel.com>
---
 UefiCpuPkg/SecCore/SecMain.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/UefiCpuPkg/SecCore/SecMain.c b/UefiCpuPkg/SecCore/SecMain.c index 173bbfcfcb..c241d3704a 100644
--- a/UefiCpuPkg/SecCore/SecMain.c
+++ b/UefiCpuPkg/SecCore/SecMain.c
@@ -189,7 +189,7 @@ SecStartup (
   //
   SecCoreData.DataSize               = (UINT16) sizeof (EFI_SEC_PEI_HAND_OFF);
   SecCoreData.BootFirmwareVolumeBase = BootFirmwareVolume;
-  SecCoreData.BootFirmwareVolumeSize = (UINTN)(0x100000000ULL - (UINTN) BootFirmwareVolume);
+  SecCoreData.BootFirmwareVolumeSize = 
+ (UINTN)((EFI_FIRMWARE_VOLUME_HEADER *) BootFirmwareVolume)->FvLength;
   SecCoreData.TemporaryRamBase       = (VOID*)(UINTN) TempRamBase;
   SecCoreData.TemporaryRamSize       = SizeOfRam;
   SecCoreData.PeiTemporaryRamBase    = SecCoreData.TemporaryRamBase;
--
2.11.0.windows.1



^ permalink raw reply	[flat|nested] 4+ messages in thread

* Re: [Patch] UefiCpuPkg: Update SecCore to get BFV size based on BFV header FvLength
  2017-12-06  8:02 [Patch] UefiCpuPkg: Update SecCore to get BFV size based on BFV header FvLength Liming Gao
  2017-12-06  8:24 ` Zeng, Star
@ 2017-12-07  2:49 ` Yao, Jiewen
  2017-12-07  2:54 ` Dong, Eric
  2 siblings, 0 replies; 4+ messages in thread
From: Yao, Jiewen @ 2017-12-07  2:49 UTC (permalink / raw)
  To: Gao, Liming, edk2-devel@lists.01.org; +Cc: Dong, Eric, Zeng, Star

Reviewed-by: Jiewen.yao@intel.com

> -----Original Message-----
> From: edk2-devel [mailto:edk2-devel-bounces@lists.01.org] On Behalf Of Liming
> Gao
> Sent: Wednesday, December 6, 2017 4:02 PM
> To: edk2-devel@lists.01.org
> Cc: Dong, Eric <eric.dong@intel.com>; Zeng, Star <star.zeng@intel.com>
> Subject: [edk2] [Patch] UefiCpuPkg: Update SecCore to get BFV size based on
> BFV header FvLength
> 
> Contributed-under: TianoCore Contribution Agreement 1.1
> Signed-off-by: Liming Gao <liming.gao@intel.com>
> Cc: Eric Dong <eric.dong@intel.com>
> Cc: Star Zeng <star.zeng@intel.com>
> ---
>  UefiCpuPkg/SecCore/SecMain.c | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
> 
> diff --git a/UefiCpuPkg/SecCore/SecMain.c b/UefiCpuPkg/SecCore/SecMain.c
> index 173bbfcfcb..c241d3704a 100644
> --- a/UefiCpuPkg/SecCore/SecMain.c
> +++ b/UefiCpuPkg/SecCore/SecMain.c
> @@ -189,7 +189,7 @@ SecStartup (
>    //
>    SecCoreData.DataSize               = (UINT16) sizeof
> (EFI_SEC_PEI_HAND_OFF);
>    SecCoreData.BootFirmwareVolumeBase = BootFirmwareVolume;
> -  SecCoreData.BootFirmwareVolumeSize = (UINTN)(0x100000000ULL - (UINTN)
> BootFirmwareVolume);
> +  SecCoreData.BootFirmwareVolumeSize =
> (UINTN)((EFI_FIRMWARE_VOLUME_HEADER *)
> BootFirmwareVolume)->FvLength;
>    SecCoreData.TemporaryRamBase       = (VOID*)(UINTN) TempRamBase;
>    SecCoreData.TemporaryRamSize       = SizeOfRam;
>    SecCoreData.PeiTemporaryRamBase    = SecCoreData.TemporaryRamBase;
> --
> 2.11.0.windows.1
> 
> _______________________________________________
> edk2-devel mailing list
> edk2-devel@lists.01.org
> https://lists.01.org/mailman/listinfo/edk2-devel


^ permalink raw reply	[flat|nested] 4+ messages in thread

* Re: [Patch] UefiCpuPkg: Update SecCore to get BFV size based on BFV header FvLength
  2017-12-06  8:02 [Patch] UefiCpuPkg: Update SecCore to get BFV size based on BFV header FvLength Liming Gao
  2017-12-06  8:24 ` Zeng, Star
  2017-12-07  2:49 ` Yao, Jiewen
@ 2017-12-07  2:54 ` Dong, Eric
  2 siblings, 0 replies; 4+ messages in thread
From: Dong, Eric @ 2017-12-07  2:54 UTC (permalink / raw)
  To: Gao, Liming, edk2-devel@lists.01.org; +Cc: Zeng, Star

Reviewed-by: Eric Dong <eric.dong@intel.com>

-----Original Message-----
From: Gao, Liming 
Sent: Wednesday, December 6, 2017 4:02 PM
To: edk2-devel@lists.01.org
Cc: Dong, Eric; Zeng, Star
Subject: [Patch] UefiCpuPkg: Update SecCore to get BFV size based on BFV header FvLength

Contributed-under: TianoCore Contribution Agreement 1.1
Signed-off-by: Liming Gao <liming.gao@intel.com>
Cc: Eric Dong <eric.dong@intel.com>
Cc: Star Zeng <star.zeng@intel.com>
---
 UefiCpuPkg/SecCore/SecMain.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/UefiCpuPkg/SecCore/SecMain.c b/UefiCpuPkg/SecCore/SecMain.c index 173bbfcfcb..c241d3704a 100644
--- a/UefiCpuPkg/SecCore/SecMain.c
+++ b/UefiCpuPkg/SecCore/SecMain.c
@@ -189,7 +189,7 @@ SecStartup (
   //
   SecCoreData.DataSize               = (UINT16) sizeof (EFI_SEC_PEI_HAND_OFF);
   SecCoreData.BootFirmwareVolumeBase = BootFirmwareVolume;
-  SecCoreData.BootFirmwareVolumeSize = (UINTN)(0x100000000ULL - (UINTN) BootFirmwareVolume);
+  SecCoreData.BootFirmwareVolumeSize = 
+ (UINTN)((EFI_FIRMWARE_VOLUME_HEADER *) BootFirmwareVolume)->FvLength;
   SecCoreData.TemporaryRamBase       = (VOID*)(UINTN) TempRamBase;
   SecCoreData.TemporaryRamSize       = SizeOfRam;
   SecCoreData.PeiTemporaryRamBase    = SecCoreData.TemporaryRamBase;
--
2.11.0.windows.1



^ permalink raw reply	[flat|nested] 4+ messages in thread

end of thread, other threads:[~2017-12-07  2:50 UTC | newest]

Thread overview: 4+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2017-12-06  8:02 [Patch] UefiCpuPkg: Update SecCore to get BFV size based on BFV header FvLength Liming Gao
2017-12-06  8:24 ` Zeng, Star
2017-12-07  2:49 ` Yao, Jiewen
2017-12-07  2:54 ` Dong, Eric

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox