|
Bugzilla – Full Text Bug Listing |
| Summary: | measured boot is broken in VMs in Tumbleweed | ||
|---|---|---|---|
| Product: | [openSUSE] openSUSE Tumbleweed | Reporter: | Alberto Planas Dominguez <aplanas> |
| Component: | Security | Assignee: | Joey Lee <jlee> |
| Status: | RESOLVED FIXED | QA Contact: | E-mail List <qa-bugs> |
| Severity: | Critical | ||
| Priority: | P5 - None | CC: | fvogt, guillaume.gardet, kukuk, richard.fan |
| Version: | Current | ||
| Target Milestone: | --- | ||
| Hardware: | Other | ||
| OS: | Other | ||
| Whiteboard: | |||
| Found By: | --- | Services Priority: | |
| Business Priority: | Blocker: | --- | |
| Marketing QA Status: | --- | IT Deployment: | --- |
| Bug Depends on: | |||
| Bug Blocks: | 1197267, 1197324 | ||
|
Description
Alberto Planas Dominguez
2022-03-14 17:06:33 UTC
@jlee, seems that can be OVMF. Fabian realizes that in TW we are not building Tcg2ConfigPei, but we do in older versions. Seems that TPM_ENABLE / CONFIG is now separated in two: TPM1_ENABLE and TPM2_ENABLE, to be aligned with the Arm configuration. Seems that the CONFIG option is still in use in some arm code, but that can be a bug, as there is a patch to remove the CONFIG option. Fabian just send: https://build.opensuse.org/request/show/961877 After some more digging, TPM1 is by default TRUE, and TPM2 is FALSE. So this PR should do. (In reply to Alberto Planas Dominguez from comment #2) > Seems that TPM_ENABLE / CONFIG is now separated in two: TPM1_ENABLE and > TPM2_ENABLE, to be aligned with the Arm configuration. Seems that the CONFIG > option is still in use in some arm code, but that can be a bug, as there is > a patch to remove the CONFIG option. Yep. Would be nice if that could be clarified with upstream. TPM_CONFIG_ENABLE was removed with https://github.com/tianocore/edk2/commit/5711ff4d0b56ff4c58dc7a780e706bc58aed2253, but Arm kept TPM2_CONFIG_ENABLE: https://github.com/tianocore/edk2/commit/e521b3c54edf6f6251c71da44086a1022ad90c94 (In reply to Alberto Planas Dominguez from comment #3) > Fabian just send: https://build.opensuse.org/request/show/961877 > > After some more digging, TPM1 is by default TRUE, and TPM2 is FALSE. So this > PR should do. I have pushed Fabian's fix to Virtualization project and also pushing to Factory. It works to me to create /sys/kernel/security/tpm0 in guest. (In reply to Fabian Vogt from comment #4) > (In reply to Alberto Planas Dominguez from comment #2) > > Seems that TPM_ENABLE / CONFIG is now separated in two: TPM1_ENABLE and > > TPM2_ENABLE, to be aligned with the Arm configuration. Seems that the CONFIG > > option is still in use in some arm code, but that can be a bug, as there is > > a patch to remove the CONFIG option. > > Yep. Would be nice if that could be clarified with upstream. > TPM_CONFIG_ENABLE was removed with > https://github.com/tianocore/edk2/commit/ > 5711ff4d0b56ff4c58dc7a780e706bc58aed2253, but Arm kept TPM2_CONFIG_ENABLE: > https://github.com/tianocore/edk2/commit/ > e521b3c54edf6f6251c71da44086a1022ad90c94 Yes, looks that the TPM2_CONFIG_ENABLE can also be removed from ArmVirtPkg. *** Bug 1197324 has been marked as a duplicate of this bug. *** *** Bug 1197267 has been marked as a duplicate of this bug. *** Fixes got released some time ago. |