[edk2-devel] [PATCH] MdeModulePkg/SmbiosDxe: Fix BcdRevision is not match with SMBIOS version

Michael D Kinney michael.d.kinney at intel.com
Sat Sep 23 03:06:16 UTC 2023


The PR looks correct

I only see the original version of the patch on the mailing list.  Can you send the V2 patch for review?

Mike

From: Guo, Gua <gua.guo at intel.com>
Sent: Friday, September 22, 2023 5:22 PM
To: Kinney, Michael D <michael.d.kinney at intel.com>; devel at edk2.groups.io; Lien, HoraceX <horacex.lien at intel.com>; Gao, Liming <gaoliming at byosoft.com.cn>
Cc: Liu, Zhiguang <zhiguang.liu at intel.com>; Bi, Dandan <dandan.bi at intel.com>; Zeng, Star <star.zeng at intel.com>; Gao, Zhichao <zhichao.gao at intel.com>
Subject: RE: [edk2-devel] [PATCH] MdeModulePkg/SmbiosDxe: Fix BcdRevision is not match with SMBIOS version

I think currently MdeModulePkg/SmbiosDxe: Fix BcdRevision is not match with SMBIOS version by liencx * Pull Request #4771 * tianocore/edk2 (github.com)<https://github.com/tianocore/edk2/pull/4771/> logic is correctly, so I want to give the patch Reviewed-by.

Currently,
  EntryPointStructureData.MinorVersion allow HEX (mPrivateData.Smbios.MajorVersion) (0 - 0xFF)
  EntryPointStructureData.MajorVersion allow HEX (mPrivateData.Smbios.MinorVersion) (0 - 0xFF)
  EntryPointStructureData.SmbiosBcdRevision allow BCD only value (0x[0-9][0-9]), otherwise, 0

These items is assigned here
[cid:image004.png at 01D9ED90.3BAA2460]

Thanks,
Gua
From: Kinney, Michael D <michael.d.kinney at intel.com<mailto:michael.d.kinney at intel.com>>
Sent: Saturday, September 23, 2023 12:20 AM
To: Guo, Gua <gua.guo at intel.com<mailto:gua.guo at intel.com>>; devel at edk2.groups.io<mailto:devel at edk2.groups.io>; Lien, HoraceX <horacex.lien at intel.com<mailto:horacex.lien at intel.com>>; Gao, Liming <gaoliming at byosoft.com.cn<mailto:gaoliming at byosoft.com.cn>>
Cc: Liu, Zhiguang <zhiguang.liu at intel.com<mailto:zhiguang.liu at intel.com>>; Bi, Dandan <dandan.bi at intel.com<mailto:dandan.bi at intel.com>>; Zeng, Star <star.zeng at intel.com<mailto:star.zeng at intel.com>>; Gao, Zhichao <zhichao.gao at intel.com<mailto:zhichao.gao at intel.com>>; Kinney, Michael D <michael.d.kinney at intel.com<mailto:michael.d.kinney at intel.com>>
Subject: RE: [edk2-devel] [PATCH] MdeModulePkg/SmbiosDxe: Fix BcdRevision is not match with SMBIOS version

I agree that if either the Major or Minor value of the version can not be represented in BCD format that setting then entire value to zero is correct.  Please update with that logic.

Mike

From: Guo, Gua <gua.guo at intel.com<mailto:gua.guo at intel.com>>
Sent: Thursday, September 21, 2023 10:10 PM
To: devel at edk2.groups.io<mailto:devel at edk2.groups.io>; Guo, Gua <gua.guo at intel.com<mailto:gua.guo at intel.com>>; Kinney, Michael D <michael.d.kinney at intel.com<mailto:michael.d.kinney at intel.com>>; Lien, HoraceX <horacex.lien at intel.com<mailto:horacex.lien at intel.com>>; Gao, Liming <gaoliming at byosoft.com.cn<mailto:gaoliming at byosoft.com.cn>>
Cc: Liu, Zhiguang <zhiguang.liu at intel.com<mailto:zhiguang.liu at intel.com>>; Bi, Dandan <dandan.bi at intel.com<mailto:dandan.bi at intel.com>>; Zeng, Star <star.zeng at intel.com<mailto:star.zeng at intel.com>>; Gao, Zhichao <zhichao.gao at intel.com<mailto:zhichao.gao at intel.com>>
Subject: RE: [edk2-devel] [PATCH] MdeModulePkg/SmbiosDxe: Fix BcdRevision is not match with SMBIOS version

Hi Mike

Is it still have concern ? I think the patch just code enhance, it shouldn't have side effects, but also make the Edk2 code stronger for error handle, it should be good.

Thanks,
Gua

From: devel at edk2.groups.io<mailto:devel at edk2.groups.io> <devel at edk2.groups.io<mailto:devel at edk2.groups.io>> On Behalf Of Guo, Gua
Sent: Wednesday, September 20, 2023 9:52 AM
To: Kinney, Michael D <michael.d.kinney at intel.com<mailto:michael.d.kinney at intel.com>>; devel at edk2.groups.io<mailto:devel at edk2.groups.io>; Lien, HoraceX <horacex.lien at intel.com<mailto:horacex.lien at intel.com>>; Gao, Liming <gaoliming at byosoft.com.cn<mailto:gaoliming at byosoft.com.cn>>
Cc: Liu, Zhiguang <zhiguang.liu at intel.com<mailto:zhiguang.liu at intel.com>>; Bi, Dandan <dandan.bi at intel.com<mailto:dandan.bi at intel.com>>; Zeng, Star <star.zeng at intel.com<mailto:star.zeng at intel.com>>; Gao, Zhichao <zhichao.gao at intel.com<mailto:zhichao.gao at intel.com>>
Subject: Re: [edk2-devel] [PATCH] MdeModulePkg/SmbiosDxe: Fix BcdRevision is not match with SMBIOS version

In fact, spec say MajorVersion (Offset 06h) and MinorVersion (Offset 07h) are duplicate item with BCD Revision (Offset 1Eh), if it duplicate item, I think the value should be the same.

BCD Revision only exist on 32-bit Entry Point (_SM_), on 64-bit Entry Point (_SM3_) is obsolete, so I guess SMBIOS organization may think it make someone confusion, so it obsolete on 64-bit Entry Point, but for 32-bit Entry Point, maybe we need to let MajorVersion (Offset 06h) and MinorVersion (Offset 07h) as same as BCD Revision (Offset 1Eh). And also have better error handle for out of BCD format.

[cid:image005.png at 01D9ED90.3BAA2460]

Thanks,
Gua

From: Kinney, Michael D <michael.d.kinney at intel.com<mailto:michael.d.kinney at intel.com>>
Sent: Wednesday, September 20, 2023 9:35 AM
To: Guo, Gua <gua.guo at intel.com<mailto:gua.guo at intel.com>>; devel at edk2.groups.io<mailto:devel at edk2.groups.io>; Lien, HoraceX <horacex.lien at intel.com<mailto:horacex.lien at intel.com>>; Gao, Liming <gaoliming at byosoft.com.cn<mailto:gaoliming at byosoft.com.cn>>
Cc: Liu, Zhiguang <zhiguang.liu at intel.com<mailto:zhiguang.liu at intel.com>>; Bi, Dandan <dandan.bi at intel.com<mailto:dandan.bi at intel.com>>; Zeng, Star <star.zeng at intel.com<mailto:star.zeng at intel.com>>; Gao, Zhichao <zhichao.gao at intel.com<mailto:zhichao.gao at intel.com>>; Kinney, Michael D <michael.d.kinney at intel.com<mailto:michael.d.kinney at intel.com>>
Subject: RE: [edk2-devel] [PATCH] MdeModulePkg/SmbiosDxe: Fix BcdRevision is not match with SMBIOS version

What is the difference between "version of the specification implemented in table structures" and "compliance with a revision of this specification"?

Is it really a spec violation for them to be difference values?

Mike

From: Guo, Gua <gua.guo at intel.com<mailto:gua.guo at intel.com>>
Sent: Tuesday, September 19, 2023 6:03 PM
To: devel at edk2.groups.io<mailto:devel at edk2.groups.io>; Kinney, Michael D <michael.d.kinney at intel.com<mailto:michael.d.kinney at intel.com>>; Lien, HoraceX <horacex.lien at intel.com<mailto:horacex.lien at intel.com>>; Gao, Liming <gaoliming at byosoft.com.cn<mailto:gaoliming at byosoft.com.cn>>
Cc: Liu, Zhiguang <zhiguang.liu at intel.com<mailto:zhiguang.liu at intel.com>>; Bi, Dandan <dandan.bi at intel.com<mailto:dandan.bi at intel.com>>; Zeng, Star <star.zeng at intel.com<mailto:star.zeng at intel.com>>; Gao, Zhichao <zhichao.gao at intel.com<mailto:zhichao.gao at intel.com>>
Subject: RE: [edk2-devel] [PATCH] MdeModulePkg/SmbiosDxe: Fix BcdRevision is not match with SMBIOS version


Hi Mike



May I get your comment and hope it can eliminate your concern ?



Question1: Let's back why this change need to do the enhance.

From

>    EntryPointStructureData.MajorVersion      = mPrivateData.Smbios.MajorVersion;

>    EntryPointStructureData.MinorVersion      = mPrivateData.Smbios.MinorVersion;

>    EntryPointStructureData.SmbiosBcdRevision = (UINT8)((PcdGet16 (PcdSmbiosVersion) >> 4) & 0xf0) | (UINT8)(PcdGet16 (PcdSmbiosVersion) & 0x0f);

To

>    EntryPointStructureData.MajorVersion      = mPrivateData.Smbios.MajorVersion;

>    EntryPointStructureData.MinorVersion      = mPrivateData.Smbios.MinorVersion;

>    EntryPointStructureData.SmbiosBcdRevision = ((mPrivateData.Smbios.MajorVersion & 0x0f) << 4) | (mPrivateData.Smbios.MinorVersion & 0x0f);



Answer1:

If mPrivateData.Smbios.MajorVersion equal to 3h and mPrivateData.Smbios.MajorVersion equal to 4h means (Spec Version 3.4), but  PcdSmbiosVersion = 0305h will cause SmbiosBcdRevision become 35h (Spec Version 3.5).

3.4 != 3.5 violate SMBIOS spec, so I think the change is reasonable to prevent this case happening.



Question2: Forcing to 0 does not sound right.



Answer2: I think it maybe reasonable based on below description.



Ref to SMBIOS Spec: https://www.dmtf.org/sites/default/files/standards/documents/DSP0134_3.7.0.pdf -> Section 5.2.1

SMBIOS Spec mention on MajorVersion (Offset 06h) = 0Ah and MinorVersion (Offset 07h) = 11 range have chance 10.22, this example is out of BCD format

[cid:image006.png at 01D9ED90.3BAA2460]



For this case, Spec have allow to set to 00h to ignore the SMBIOS BCD Revision (Offset 1Eh), when it happen, only consider SMBIOS Major Version (Offset 06h) and SMBIOS Minor Version (Offset 07h).

[cid:image007.png at 01D9ED90.3BAA2460]







Thanks,

Gua



-----Original Message-----
From: devel at edk2.groups.io<mailto:devel at edk2.groups.io> <devel at edk2.groups.io<mailto:devel at edk2.groups.io>> On Behalf Of Michael D Kinney
Sent: Friday, September 15, 2023 1:48 AM
To: Lien, HoraceX <horacex.lien at intel.com<mailto:horacex.lien at intel.com>>; devel at edk2.groups.io<mailto:devel at edk2.groups.io>; Gao, Liming <gaoliming at byosoft.com.cn<mailto:gaoliming at byosoft.com.cn>>
Cc: Liu, Zhiguang <zhiguang.liu at intel.com<mailto:zhiguang.liu at intel.com>>; Bi, Dandan <dandan.bi at intel.com<mailto:dandan.bi at intel.com>>; Zeng, Star <star.zeng at intel.com<mailto:star.zeng at intel.com>>; Gao, Zhichao <zhichao.gao at intel.com<mailto:zhichao.gao at intel.com>>; Kinney, Michael D <michael.d.kinney at intel.com<mailto:michael.d.kinney at intel.com>>
Subject: Re: [edk2-devel] [PATCH] MdeModulePkg/SmbiosDxe: Fix BcdRevision is not match with SMBIOS version



Forcing to 0 does not sound right.



You did not answer my question about the property of the global variables.



Without knowing the format of the information in the global variables you cannot safely use them.  If they are in BCD then no need to check for out of range.  If they are hex values, then you have to use conversion functions.



Mike



> -----Original Message-----

> From: Lien, HoraceX <horacex.lien at intel.com<mailto:horacex.lien at intel.com>>

> Sent: Thursday, September 14, 2023 2:33 AM

> To: Kinney, Michael D <michael.d.kinney at intel.com<mailto:michael.d.kinney at intel.com>>;

> devel at edk2.groups.io<mailto:devel at edk2.groups.io>; Gao, Liming <gaoliming at byosoft.com.cn<mailto:gaoliming at byosoft.com.cn>>

> Cc: Liu, Zhiguang <zhiguang.liu at intel.com<mailto:zhiguang.liu at intel.com>>; Bi, Dandan

> <dandan.bi at intel.com<mailto:dandan.bi at intel.com>>; Zeng, Star <star.zeng at intel.com<mailto:star.zeng at intel.com>>; Gao, Zhichao

> <zhichao.gao at intel.com<mailto:zhichao.gao at intel.com>>

> Subject: RE: [edk2-devel] [PATCH] MdeModulePkg/SmbiosDxe: Fix

> BcdRevision is not match with SMBIOS version

>

> Hi Mike,

>

> https://github.com/tianocore/edk2/pull/4771

> I have changed code following rule: It is only accept range 0-9 for

> Major and Minor version to fill in SmbiosBcdRevision, if one of Major

> or Minor is greater than 9 then fill in 00h.

>

> Please help to review this, thanks :)

>

> Thanks,

> Horace Lien

>

> -----Original Message-----

> From: Lien, HoraceX

> Sent: Friday, September 8, 2023 5:35 PM

> To: Kinney, Michael D <michael.d.kinney at intel.com<mailto:michael.d.kinney at intel.com>>;

> devel at edk2.groups.io<mailto:devel at edk2.groups.io>; Gao, Liming <gaoliming at byosoft.com.cn<mailto:gaoliming at byosoft.com.cn>>

> Cc: Liu, Zhiguang <Zhiguang.Liu at intel.com<mailto:Zhiguang.Liu at intel.com>>; Bi, Dandan

> <dandan.bi at intel.com<mailto:dandan.bi at intel.com>>; Zeng, Star <star.zeng at intel.com<mailto:star.zeng at intel.com>>; Gao, Zhichao

> <zhichao.gao at intel.com<mailto:zhichao.gao at intel.com>>

> Subject: RE: [edk2-devel] [PATCH] MdeModulePkg/SmbiosDxe: Fix

> BcdRevision is not match with SMBIOS version

>

> Hi Mike,

>

> No, we didn't guarantee this before. Add comments to descript BCD

> field is good point.

> I have reviewed SMBIOS spec for SmbiosBcdRevision field, it mentions

> "If the value is 00h, only the Major and Minor Versions in offsets 6

> and 7 of the Entry Point Structure provide the version information. ".

> So, I have new idea to implement this, I will filter range 0-9 for

> Major/Minor version to fill in SmbiosBcdRevision, if one of Major or

> Minor is greater than 9 then fill in 00h.

> Do you think it is ok?

>

> Thanks for your reply.

>

> Thanks,

> Horace Lien

>

> -----Original Message-----

> From: Kinney, Michael D <michael.d.kinney at intel.com<mailto:michael.d.kinney at intel.com>>

> Sent: Friday, September 8, 2023 6:05 AM

> To: Lien, HoraceX <horacex.lien at intel.com<mailto:horacex.lien at intel.com>>; devel at edk2.groups.io<mailto:devel at edk2.groups.io>; Gao,

> Liming <gaoliming at byosoft.com.cn<mailto:gaoliming at byosoft.com.cn>>

> Cc: Liu, Zhiguang <zhiguang.liu at intel.com<mailto:zhiguang.liu at intel.com>>; Bi, Dandan

> <dandan.bi at intel.com<mailto:dandan.bi at intel.com>>; Zeng, Star <star.zeng at intel.com<mailto:star.zeng at intel.com>>; Gao, Zhichao

> <zhichao.gao at intel.com<mailto:zhichao.gao at intel.com>>; Kinney, Michael D

> <michael.d.kinney at intel.com<mailto:michael.d.kinney at intel.com>>

> Subject: RE: [edk2-devel] [PATCH] MdeModulePkg/SmbiosDxe: Fix

> BcdRevision is not match with SMBIOS version

>

> I was asking about the property of the global variables being used in

> this patch.  Are they already guaranteed to be in BSD format and in

> range 0..9.  If so, then no additional code changes would be required.

> However, it would be good to add comments about the properties of

> those global variables and why they can be used to directly assign to

> fields that are required to be in BSD format.

>

> Mike

>

> > -----Original Message-----

> > From: Lien, HoraceX <horacex.lien at intel.com<mailto:horacex.lien at intel.com>>

> > Sent: Thursday, September 7, 2023 2:41 AM

> > To: devel at edk2.groups.io<mailto:devel at edk2.groups.io>; Kinney, Michael D

> > <michael.d.kinney at intel.com<mailto:michael.d.kinney at intel.com>>; Gao, Liming <gaoliming at byosoft.com.cn<mailto:gaoliming at byosoft.com.cn>>

> > Cc: Liu, Zhiguang <zhiguang.liu at intel.com<mailto:zhiguang.liu at intel.com>>; Bi, Dandan

> > <dandan.bi at intel.com<mailto:dandan.bi at intel.com>>; Zeng, Star <star.zeng at intel.com<mailto:star.zeng at intel.com>>; Gao,

> > Zhichao <zhichao.gao at intel.com<mailto:zhichao.gao at intel.com>>; Lien, HoraceX

> > <horacex.lien at intel.com<mailto:horacex.lien at intel.com>>

> > Subject: RE: [edk2-devel] [PATCH] MdeModulePkg/SmbiosDxe: Fix

> > BcdRevision is not match with SMBIOS version

> >

> > Hi Mike,

> >

> > Could you please reply for me?

> > If you want to filter range 0-9, then I will send PR again.

> >

> > Thanks,

> > Horace Lien

> >

> > -----Original Message-----

> > From: devel at edk2.groups.io<mailto:devel at edk2.groups.io> <devel at edk2.groups.io<mailto:devel at edk2.groups.io>> On Behalf Of Lien,

> > HoraceX

> > Sent: Friday, September 1, 2023 3:06 PM

> > To: Kinney, Michael D <michael.d.kinney at intel.com<mailto:michael.d.kinney at intel.com>>;

> > devel at edk2.groups.io<mailto:devel at edk2.groups.io>

> > Cc: Liu, Zhiguang <zhiguang.liu at intel.com<mailto:zhiguang.liu at intel.com>>; Bi, Dandan

> > <dandan.bi at intel.com<mailto:dandan.bi at intel.com>>; Zeng, Star <star.zeng at intel.com<mailto:star.zeng at intel.com>>; Gao,

> > Zhichao <zhichao.gao at intel.com<mailto:zhichao.gao at intel.com>>

> > Subject: Re: [edk2-devel] [PATCH] MdeModulePkg/SmbiosDxe: Fix

> > BcdRevision is not match with SMBIOS version

> >

> > Hi Mike,

> >

> > I have change code to

> > EntryPointStructureData.SmbiosBcdRevision =

> > ((mPrivateData.Smbios.MajorVersion & 0x0f) << 4) |

> > (mPrivateData.Smbios.MinorVersion & 0x0f); Add &0x0F to mask upper

> > nibble bit, do we still need to guarantee that range is between 0-9?

> > Because the old code only filtered 4 bits, instead of accurately

> > filtering the number range 0-9.

> >

> > Thanks,

> > Horace Lien

> >

> > -----Original Message-----

> > From: Kinney, Michael D <michael.d.kinney at intel.com<mailto:michael.d.kinney at intel.com>>

> > Sent: Thursday, August 31, 2023 11:56 PM

> > To: devel at edk2.groups.io<mailto:devel at edk2.groups.io>; Lien, HoraceX <horacex.lien at intel.com<mailto:horacex.lien at intel.com>>

> > Cc: Liu, Zhiguang <zhiguang.liu at intel.com<mailto:zhiguang.liu at intel.com>>; Bi, Dandan

> > <dandan.bi at intel.com<mailto:dandan.bi at intel.com>>; Zeng, Star <star.zeng at intel.com<mailto:star.zeng at intel.com>>; Gao,

> > Zhichao <zhichao.gao at intel.com<mailto:zhichao.gao at intel.com>>; Kinney, Michael D

> > <michael.d.kinney at intel.com<mailto:michael.d.kinney at intel.com>>

> > Subject: RE: [edk2-devel] [PATCH] MdeModulePkg/SmbiosDxe: Fix

> > BcdRevision is not match with SMBIOS version

> >

> > Are mPrivateData.Smbios.MajorVersion and

> > mPrivateData.Smbios.MinorVersion guaranteed to be in range 0..9?

> >

> > Mike

> >

> > > -----Original Message-----

> > > From: devel at edk2.groups.io<mailto:devel at edk2.groups.io> <devel at edk2.groups.io<mailto:devel at edk2.groups.io>> On Behalf Of

> > > horacex.lien at intel.com<mailto:horacex.lien at intel.com>

> > > Sent: Wednesday, August 30, 2023 2:13 AM

> > > To: devel at edk2.groups.io<mailto:devel at edk2.groups.io>

> > > Cc: Lien, HoraceX <horacex.lien at intel.com<mailto:horacex.lien at intel.com>>; Liu, Zhiguang

> > > <zhiguang.liu at intel.com<mailto:zhiguang.liu at intel.com>>; Bi, Dandan <dandan.bi at intel.com<mailto:dandan.bi at intel.com>>; Zeng,

> > > Star <star.zeng at intel.com<mailto:star.zeng at intel.com>>; Gao, Zhichao <zhichao.gao at intel.com<mailto:zhichao.gao at intel.com>>

> > > Subject: [edk2-devel] [PATCH] MdeModulePkg/SmbiosDxe: Fix

> > > BcdRevision is not match with SMBIOS version

> > >

> > > From: HoraceX Lien <horacex.lien at intel.com<mailto:horacex.lien at intel.com>>

> > >

> > > REF: https://bugzilla.tianocore.org/show_bug.cgi?id=4544

> > >

> > > These value of Major/Minor version are updated from SMBIOS memory

> > > data, but BCD Revision is updated from PCD PcdSmbiosVersion.

> > > We should also update PCD PcdSmbiosVersion from SMBIOS memory

> > > data, to ensure that get consistent version value.

> > >

> > > Cc: Zhiguang Liu <zhiguang.liu at intel.com<mailto:zhiguang.liu at intel.com>>

> > > Cc: Dandan Bi <dandan.bi at intel.com<mailto:dandan.bi at intel.com>>

> > > Cc: Star Zeng <star.zeng at intel.com<mailto:star.zeng at intel.com>>

> > > Cc: Zhichao Gao <zhichao.gao at intel.com<mailto:zhichao.gao at intel.com>>

> > > Signed-off-by: HoraceX Lien <horacex.lien at intel.com<mailto:horacex.lien at intel.com>>

> > > ---

> > >  MdeModulePkg/Universal/SmbiosDxe/SmbiosDxe.c | 2 +-

> > >  1 file changed, 1 insertion(+), 1 deletion(-)

> > >

> > > diff --git a/MdeModulePkg/Universal/SmbiosDxe/SmbiosDxe.c

> > > b/MdeModulePkg/Universal/SmbiosDxe/SmbiosDxe.c

> > > index 1a86e69d3c..e3f6215033 100644

> > > --- a/MdeModulePkg/Universal/SmbiosDxe/SmbiosDxe.c

> > > +++ b/MdeModulePkg/Universal/SmbiosDxe/SmbiosDxe.c

> > > @@ -1072,7 +1072,7 @@ SmbiosCreateTable (

> > >      DEBUG ((DEBUG_INFO, "SmbiosCreateTable: Initialize 32-bit

> > > entry point structure\n"));

> > >

> > >      EntryPointStructureData.MajorVersion      =

> > > mPrivateData.Smbios.MajorVersion;

> > >

> > >      EntryPointStructureData.MinorVersion      =

> > > mPrivateData.Smbios.MinorVersion;

> > >

> > > -    EntryPointStructureData.SmbiosBcdRevision = (UINT8)((PcdGet16

> > > (PcdSmbiosVersion) >> 4) & 0xf0) | (UINT8)(PcdGet16

> > > (PcdSmbiosVersion) & 0x0f);

> > >

> > > +    EntryPointStructureData.SmbiosBcdRevision =

> > > (mPrivateData.Smbios.MajorVersion << 4) |

> > > mPrivateData.Smbios.MinorVersion;

> > >

> > >      PhysicalAddress                           = 0xffffffff;

> > >

> > >      Status                                    = gBS->AllocatePages (

> > >

> > >

> > > AllocateMaxAddress,

> > >

> > > --

> > > 2.31.1.windows.1

> > >

> > >

> > >

> > > -=-=-=-=-=-=

> > > Groups.io Links: You receive all messages sent to this group.

> > > View/Reply Online (#108150):

> > > https://edk2.groups.io/g/devel/message/108150

> > > Mute This Topic: https://groups.io/mt/101057293/1643496

> > > Group Owner: devel+owner at edk2.groups.io<mailto:devel+owner at edk2.groups.io>

> > > Unsubscribe: https://edk2.groups.io/g/devel/unsub

> > > [michael.d.kinney at intel.com]

> > > -=-=-=-=-=-=

> > >

> >

> >

> >

> >

> >















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


-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://listman.redhat.com/archives/edk2-devel-archive/attachments/20230923/91842c66/attachment-0001.htm>
-------------- next part --------------
A non-text attachment was scrubbed...
Name: image004.png
Type: image/png
Size: 57454 bytes
Desc: image004.png
URL: <http://listman.redhat.com/archives/edk2-devel-archive/attachments/20230923/91842c66/attachment-0004.png>
-------------- next part --------------
A non-text attachment was scrubbed...
Name: image005.png
Type: image/png
Size: 232151 bytes
Desc: image005.png
URL: <http://listman.redhat.com/archives/edk2-devel-archive/attachments/20230923/91842c66/attachment-0005.png>
-------------- next part --------------
A non-text attachment was scrubbed...
Name: image006.png
Type: image/png
Size: 71822 bytes
Desc: image006.png
URL: <http://listman.redhat.com/archives/edk2-devel-archive/attachments/20230923/91842c66/attachment-0006.png>
-------------- next part --------------
A non-text attachment was scrubbed...
Name: image007.png
Type: image/png
Size: 64751 bytes
Desc: image007.png
URL: <http://listman.redhat.com/archives/edk2-devel-archive/attachments/20230923/91842c66/attachment-0007.png>


More information about the edk2-devel-archive mailing list