Re: [edk2-devel] 回复: [edk2-devel] UEFI spec version in system table, shall we update it to 2.8?

["Poosapalli, Karunakar via groups.io" <KarunakarPoosapalli=Dell.com@groups.io>]

[-- Attachment #1: Type: text/plain, Size: 7472 bytes --]

Mike,

Agree with your points. Thanks for sharing the detailed info. Beautifully conveyed!

-Karunakar



Internal Use - Confidential
From: Kinney, Michael D <michael.d.kinney@intel.com>
Sent: Thursday, September 21, 2023 12:30 AM
To: Poosapalli, Karunakar; devel@edk2.groups.io; POLUDOV, FELIX; Yao, Jiewen
Cc: Kinney, Michael D
Subject: RE: [edk2-devel] 回复: [edk2-devel] UEFI spec version in system table, shall we update it to 2.8?


[EXTERNAL EMAIL]
Compliance of TianoCore is different than Compliance of a platform firmware image.


  *   TianoCore should only update the version when all of the TianoCore contents conform to the spec version.
  *   If the subset of TainoCore features integrated into a platform firmware image meets all of a newer spec version requirements and the rest of the platform components also meet a newer spec version, then the platform firmware image can change the version value to match.

Without the full analysis the TianoCore version can not be updated.

The main purpose of the version values in the UEFI tables is for UEFI Option ROMs, UEFI Applications, and UEFI OS Loaders to know what UEFI API support is present and only call services that are known to be present with the expected behavior of those APIs as documented in that version of the specification.  The use case we do not want to see is platform firmware publishing a higher version value without a guarantee that all the APIs and behavior match. That could cause unexpected behavior from UEFI Drivers, UEFI Applications, or UEFI OS Loaders.

Mike

From: Poosapalli, Karunakar <KarunakarPoosapalli@Dell.com<mailto:KarunakarPoosapalli@Dell.com>>
Sent: Tuesday, September 19, 2023 9:41 PM
To: Kinney, Michael D <michael.d.kinney@intel.com<mailto:michael.d.kinney@intel.com>>; devel@edk2.groups.io<mailto:devel@edk2.groups.io>; POLUDOV, FELIX <felixp@ami.com<mailto:felixp@ami.com>>; Yao, Jiewen <jiewen.yao@intel.com<mailto:jiewen.yao@intel.com>>
Subject: RE: [edk2-devel] 回复: [edk2-devel] UEFI spec version in system table, shall we update it to 2.8?

Hi @Kinney, Michael D<mailto:michael.d.kinney@intel.com>,

Thanks for sharing your inputs. Agree with your point.

As latest tianocore edk2 already added some of the interfaces or functionalities implemented from v2.8 by different communities,
Can we say it’s compliant with v2.8? but as there is no analysis done to see whether all updates from 2.8 is implemented or not we can’t say it’s fully 2.8 Supported ?

Karunakar



Internal Use - Confidential
From: Kinney, Michael D <michael.d.kinney@intel.com<mailto:michael.d.kinney@intel.com>>
Sent: Wednesday, September 20, 2023 2:34 AM
To: devel@edk2.groups.io<mailto:devel@edk2.groups.io>; Poosapalli, Karunakar; POLUDOV, FELIX; Yao, Jiewen
Cc: Kinney, Michael D
Subject: RE: [edk2-devel] 回复: [edk2-devel] UEFI spec version in system table, shall we update it to 2.8?


[EXTERNAL EMAIL]
I do not think anyone has done the complete analysis since UEFI 2.7.  Some updates from newer specs
have been added as needed by different community members.

Unless there are dedicated tasks to review and verify that every spec change has been added to edk2, the version can not be updated.

Mike

From: devel@edk2.groups.io<mailto:devel@edk2.groups.io> <devel@edk2.groups.io<mailto:devel@edk2.groups.io>> On Behalf Of Poosapalli, Karunakar via groups.io
Sent: Tuesday, September 19, 2023 11:18 AM
To: Poosapalli, Karunakar <KarunakarPoosapalli@Dell.com<mailto:KarunakarPoosapalli@Dell.com>>; devel@edk2.groups.io<mailto:devel@edk2.groups.io>; POLUDOV, FELIX <felixp@ami.com<mailto:felixp@ami.com>>; Yao, Jiewen <jiewen.yao@intel.com<mailto:jiewen.yao@intel.com>>
Subject: Re: [edk2-devel] 回复: [edk2-devel] UEFI spec version in system table, shall we update it to 2.8?

Hi All,

Can you please share your thoughts if any idea in EDK2 compliance with UEFI 2.8 Support ?

From UEFI sources and latest edk2 master branch is still at 2.7. Same for latest stable tag edk2-stable202308.
https://github.com/tianocore/edk2.git [github.com]<https://urldefense.com/v3/__https:/github.com/tianocore/edk2.git__;!!LpKI!nIohUbt0OghPJnCKOpCtgOh1X26Ds1hTOXDuhgvA7ameCb6qMli7Zw8P-z0ruFYDjM8sum-VJAV7sB8_8FzbvKH80JAhH-G8$>
#define EFI_SYSTEM_TABLE_REVISION       EFI_2_70_SYSTEM_TABLE_REVISION
#define EFI_SPECIFICATION_VERSION       EFI_SYSTEM_TABLE_REVISION


But we could see some of the features from 2.8 is already available and UEFI 2.9 spec also released but system table in edk2 is still pointing to 2.7.

Curious to know, Whether latest edk2 is not fully compliance with 2.8 or only system table update is missing for 2.8


From: karunakar_poosapalli via groups.io <karunakar_poosapalli=dell.com@groups.io<mailto:karunakar_poosapalli=dell.com@groups.io>>
Sent: Tuesday, September 19, 2023 10:41 PM
To: Poosapalli, Karunakar; devel@edk2.groups.io<mailto:devel@edk2.groups.io>
Subject: Re: [edk2-devel] 回复: [edk2-devel] UEFI spec version in system table, shall we update it to 2.8?


[EXTERNAL EMAIL]

In latest tianocore edk2 (https://github.com/tianocore/edk2 [github.com]<https://urldefense.com/v3/__https:/github.com/tianocore/edk2__;!!LpKI!mQ_BG3pFqn4l_WXu4-SMapIb2kzSZmeFEwZ9PNGLNx6j_YdYF6DUWQBuLZwoJhMzMQIiKG1dMBmDJH6eqgKzHbK-JfJWjjqXU1bQ$> ), we can see System table is still pointing to 2.70 Version(EFI_2_70_SYSTEM_TABLE_REVISION)

https://github.com/tianocore/edk2/blob/master/MdePkg/Include/Uefi/UefiSpec.h [github.com]<https://urldefense.com/v3/__https:/github.com/tianocore/edk2/blob/master/MdePkg/Include/Uefi/UefiSpec.h__;!!LpKI!mQ_BG3pFqn4l_WXu4-SMapIb2kzSZmeFEwZ9PNGLNx6j_YdYF6DUWQBuLZwoJhMzMQIiKG1dMBmDJH6eqgKzHbK-JfJWjvHlQENb$>

//

// EFI Runtime Services Table

//

#define EFI_SYSTEM_TABLE_SIGNATURE      SIGNATURE_64 ('I','B','I',' ','S','Y','S','T')

#define EFI_2_80_SYSTEM_TABLE_REVISION  ((2 << 16) | (80))

#define EFI_2_70_SYSTEM_TABLE_REVISION  ((2 << 16) | (70))

#define EFI_2_60_SYSTEM_TABLE_REVISION  ((2 << 16) | (60))

#define EFI_2_50_SYSTEM_TABLE_REVISION  ((2 << 16) | (50))

#define EFI_2_40_SYSTEM_TABLE_REVISION  ((2 << 16) | (40))

#define EFI_2_31_SYSTEM_TABLE_REVISION  ((2 << 16) | (31))

#define EFI_2_30_SYSTEM_TABLE_REVISION  ((2 << 16) | (30))

#define EFI_2_20_SYSTEM_TABLE_REVISION  ((2 << 16) | (20))

#define EFI_2_10_SYSTEM_TABLE_REVISION  ((2 << 16) | (10))

#define EFI_2_00_SYSTEM_TABLE_REVISION  ((2 << 16) | (00))

#define EFI_1_10_SYSTEM_TABLE_REVISION  ((1 << 16) | (10))

#define EFI_1_02_SYSTEM_TABLE_REVISION  ((1 << 16) | (02))

#define EFI_SYSTEM_TABLE_REVISION       EFI_2_70_SYSTEM_TABLE_REVISION

#define EFI_SPECIFICATION_VERSION       EFI_SYSTEM_TABLE_REVISION



#define EFI_RUNTIME_SERVICES_SIGNATURE  SIGNATURE_64 ('R','U','N','T','S','E','R','V')

#define EFI_RUNTIME_SERVICES_REVISION   EFI_SPECIFICATION_VERSION



Does it mean latest tianocore is still not UEFI 2.8 version support or it supports 2.8 functionalities but the table version number is not yet updated?


Internal Use - Confidential



-=-=-=-=-=-=-=-=-=-=-=-
Groups.io Links: You receive all messages sent to this group.
View/Reply Online (#108928): https://edk2.groups.io/g/devel/message/108928
Mute This Topic: https://groups.io/mt/81906348/7686176
Group Owner: devel+owner@edk2.groups.io
Unsubscribe: https://edk2.groups.io/g/devel/unsub [rebecca@openfw.io]
-=-=-=-=-=-=-=-=-=-=-=-



[-- Attachment #2: Type: text/html, Size: 25874 bytes --]