summaryrefslogtreecommitdiffstats
path: root/UefiCpuPkg/PiSmmCpuDxeSmm
diff options
context:
space:
mode:
authorNeal Gompa <ngompa@fedoraproject.org>2021-07-05 05:36:03 -0400
committermergify[bot] <37929162+mergify[bot]@users.noreply.github.com>2021-07-06 07:33:42 +0000
commitb3db0cb1f8d163f22b769c205c6347376a315dcd (patch)
tree44cf4b294b7978e1632b33918ac4e931fecb8607 /UefiCpuPkg/PiSmmCpuDxeSmm
parent44ced03798d3af47577af3e7a865be33383766d0 (diff)
downloadedk2-b3db0cb1f8d163f22b769c205c6347376a315dcd.tar.gz
edk2-b3db0cb1f8d163f22b769c205c6347376a315dcd.tar.bz2
edk2-b3db0cb1f8d163f22b769c205c6347376a315dcd.zip
MdeModulePkg/PartitionDxe: Ignore PMBR BootIndicator per UEFI spec
Per UEFI Spec 2.8 (UEFI_Spec_2_8_final.pdf, page 114) 5.2.3 Protective MBR Table 20. Protective MBR Partition Record protecting the entire disk The description for BootIndicator states the following: > Set to 0x00 to indicate a non-bootable partition. If set to any > value other than 0x00 the behavior of this flag on non-UEFI > systems is undefined. Must be ignored by UEFI implementations. Unfortunately, we have been incorrectly assuming that the BootIndicator value must be 0x00, which leads to problems when the 'pmbr_boot' flag is set on a disk containing a GPT (such as with GNU parted). When the flag is set, the value changes to 0x01, causing this check to fail and the system is rendered unbootable despite it being valid from the perspective of the UEFI spec. To resolve this, we drop the check for the BootIndicator so that we stop caring about the value set there, which restores the capability to boot such disks. Ref: https://bugzilla.tianocore.org/show_bug.cgi?id=3474 Cc: Chris Murphy <chrismurphy@fedoraproject.org> Cc: David Duncan <davdunc@amazon.com> Cc: Lazlo Ersek <lersek@redhat.com> Cc: Hao A Wu <hao.a.wu@intel.com> Cc: Ray Ni <ray.ni@intel.com> Cc: Zhichao Gao <zhichao.gao@intel.com> Signed-off-by: Neal Gompa <ngompa@fedoraproject.org> Message-Id: <20210705093603.575707-1-ngompa@fedoraproject.org> Reviewed-by: Laszlo Ersek <lersek@redhat.com> Reviewed-by: Hao A Wu <hao.a.wu@intel.com>
Diffstat (limited to 'UefiCpuPkg/PiSmmCpuDxeSmm')
0 files changed, 0 insertions, 0 deletions