Skip to content

fix(fw_cfg): clear ACPI table checksums before recalculation#417

Merged
Lencerf merged 1 commit intogoogle:mainfrom
Lencerf:fix/acpi/clea
Mar 6, 2026
Merged

fix(fw_cfg): clear ACPI table checksums before recalculation#417
Lencerf merged 1 commit intogoogle:mainfrom
Lencerf:fix/acpi/clea

Conversation

@Lencerf
Copy link
Collaborator

@Lencerf Lencerf commented Mar 6, 2026

The fw_cfg ADD_CHECKSUM command includes the checksum field itself when calculating the new checksum for an ACPI table. If the checksum is not cleared beforehand, this results in an incorrect final checksum.

The fw_cfg `ADD_CHECKSUM` command includes the checksum field itself
when calculating the new checksum for an ACPI table. If the checksum is
not cleared beforehand, this results in an incorrect final checksum.

Signed-off-by: Changyuan Lyu <changyuanl@google.com>
@coveralls
Copy link

Pull Request Test Coverage Report for Build 22752951058

Details

  • 0 of 16 (0.0%) changed or added relevant lines in 3 files are covered.
  • No unchanged relevant lines lost coverage.
  • Overall coverage decreased (-0.02%) to 26.784%

Changes Missing Coverage Covered Lines Changed/Added Lines %
alioth/src/board/board_x86_64/board_x86_64.rs 0 1 0.0%
alioth/src/device/fw_cfg/acpi.rs 0 1 0.0%
alioth/src/firmware/acpi/acpi.rs 0 14 0.0%
Totals Coverage Status
Change from base Build 22752936433: -0.02%
Covered Lines: 3814
Relevant Lines: 14728

💛 - Coveralls

@Lencerf Lencerf merged commit eb9728b into google:main Mar 6, 2026
5 checks passed
@Lencerf Lencerf deleted the fix/acpi/clea branch March 6, 2026 07:11
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants