[edk2-devel] [PATCH 1/1] OvmfPkg: fix BuildResourceDescriptorHob call in PlatformAddHobCB()

[Ard Biesheuvel]

On Thu, 26 Jan 2023 at 02:48, Yao, Jiewen <jiewen.yao at intel.com> wrote:
>
> Reviewed-by: Jiewen Yao <Jiewen.yao at intel.com>
>

Merged as #3947, thanks

> > -----Original Message-----
> > From: devel at edk2.groups.io <devel at edk2.groups.io> On Behalf Of Gerd
> > Hoffmann
> > Sent: Thursday, January 26, 2023 1:11 AM
> > To: devel at edk2.groups.io
> > Cc: Pawel Polawski <ppolawsk at redhat.com>; Tom Lendacky
> > <thomas.lendacky at amd.com>; Yao, Jiewen <jiewen.yao at intel.com>; Oliver
> > Steffen <osteffen at redhat.com>; Gerd Hoffmann <kraxel at redhat.com>; Ard
> > Biesheuvel <ardb+tianocore at kernel.org>; Justen, Jordan L
> > <jordan.l.justen at intel.com>
> > Subject: [edk2-devel] [PATCH 1/1] OvmfPkg: fix BuildResourceDescriptorHob call
> > in PlatformAddHobCB()
> >
> > BuildResourceDescriptorHob() expects the third parameter be the Length,
> > not the End address.
> >
> > Fixes: 328076cfdf45 ("OvmfPkg/PlatformInitLib: Add PlatformAddHobCB")
> > Reported-by: Tom Lendacky <thomas.lendacky at amd.com>
> > Signed-off-by: Gerd Hoffmann <kraxel at redhat.com>
> > ---
> >  OvmfPkg/Library/PlatformInitLib/MemDetect.c | 2 +-
> >  1 file changed, 1 insertion(+), 1 deletion(-)
> >
> > diff --git a/OvmfPkg/Library/PlatformInitLib/MemDetect.c
> > b/OvmfPkg/Library/PlatformInitLib/MemDetect.c
> > index 5aeeeff89f57..38cece9173e8 100644
> > --- a/OvmfPkg/Library/PlatformInitLib/MemDetect.c
> > +++ b/OvmfPkg/Library/PlatformInitLib/MemDetect.c
> > @@ -200,7 +200,7 @@ PlatformAddHobCB (
> >
> >        break;
> >      case EfiAcpiAddressRangeReserved:
> > -      BuildResourceDescriptorHob (EFI_RESOURCE_MEMORY_RESERVED, 0, Base,
> > End);
> > +      BuildResourceDescriptorHob (EFI_RESOURCE_MEMORY_RESERVED, 0,
> > Base, End - Base);
> >        DEBUG ((DEBUG_INFO, "%a: Reserved [0x%Lx, 0x%Lx)\n", __FUNCTION__,
> > Base, End));
> >        break;
> >      default:
> > --
> > 2.39.1
> >
> >
> >
> >
> >
>
>
>
> 
>
>


-=-=-=-=-=-=-=-=-=-=-=-
Groups.io Links: You receive all messages sent to this group.
View/Reply Online (#99065): https://edk2.groups.io/g/devel/message/99065
Mute This Topic: https://groups.io/mt/96524909/1813853
Group Owner: devel+owner at edk2.groups.io
Unsubscribe: https://edk2.groups.io/g/devel/unsub [edk2-devel-archive at redhat.com]
-=-=-=-=-=-=-=-=-=-=-=-