Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

SCT: add test for EFI_CONFORMANCE_PROFILE_TABLE (Bugzilla Bug 4042) #230

Open
tianocore-issues opened this issue Sep 1, 2022 · 5 comments

Comments

@tianocore-issues
Copy link

This issue was created automatically with bugzilla2github

Bugzilla Bug 4042

Date: 2022-09-01T10:56:55+00:00
From: Heinrich Schuchardt <<xypron.glpk>>
To: Heinrich Schuchardt <<xypron.glpk>>
CC: Carolyn.Gjertsen, edhaya.chandran, gaojie, xypron.glpk

Last updated: 2024-11-07T10:49:50+00:00

@tianocore-issues
Copy link
Author

Comment 19462

Date: 2022-09-01 10:56:55 +0000
From: Heinrich Schuchardt <<xypron.glpk>>

  • Industry Specification: ---
  • Release Observed: EDK II Master
  • Releases to Fix: EDK II Master
  • Target OS: ---
  • Bugzilla Assignee(s): Heinrich Schuchardt <<xypron.glpk>>

The UEFI 2.10 version introduced the EFI_CONFORMANCE_PROFILE_TABLE.

Let's add a test that it only contains known GUIDs.

I will try to create a patch.

@tianocore-issues
Copy link
Author

Comment 19465

Date: 2022-09-01 17:06:41 +0000
From: Heinrich Schuchardt <<xypron.glpk>>

We actually can only test the following:

Version == EFI_CONFORMANCE_PROFILES_TABLE_VERSION (0x01)

NumberOfProfiles >= 1
(an empty table would mean that the implementation does not conform to anything)

Currently EFI_CONFORMANCE_PROFILES_UEFI_SPEC_GUID is the only known GUID. But any GUID is allowable in the table according to the specification ("Additional profiles can be defined and published in other specifications"). So testing the actual GUID values is not reasonable.

Best regards

Heinrich

@tianocore-issues
Copy link
Author

Comment 19975

Date: 2022-11-09 03:03:44 +0000
From: Edhay <<edhaya.chandran>>

Created attachment 1049
Please ignore this patch

Attachment: BZ-1049-updated_patch.patch

@tianocore-issues
Copy link
Author

Comment 19977

Date: 2022-11-09 03:10:50 +0000
From: Edhay <<edhaya.chandran>>

Hi Heinrich,
The patch above was uploaded by mistake.
I am not able to remove the same. Please ignore it.

@tianocore-issues
Copy link
Author

Comment 23520

Date: 2024-11-07 10:49:50 +0000
From: Edhay <<edhaya.chandran>>

Hi Heinrich,
Could you please provide a patch for this feature.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

1 participant