Attention is currently required from: Julius Werner.
Hung-Te Lin has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/28017 )
Change subject: arm64: mmu: Spot check security state for TTB mapping
......................................................................
Patch Set 2:
(1 comment)
File src/arch/arm64/armv8/mmu.c:
https://review.coreboot.org/c/coreboot/+/28017/comment/b170db55_606c3b32
PS2, Line 324: assert_correct_ttb_mapping(_ettb - 1);
that gives me using gcc-12.2: […]
Well, this should be a false alert. Although I'm not sure how what's the best way to fix it - changing to `_ttb + REGION_SIZE(ttb) - 1` ?
--
To view, visit
https://review.coreboot.org/c/coreboot/+/28017
To unsubscribe, or for help writing mail filters, visit
https://review.coreboot.org/settings
Gerrit-Project: coreboot
Gerrit-Branch: master
Gerrit-Change-Id: I70cda4f76f201b03d69a9ece063a3830b15ac04b
Gerrit-Change-Number: 28017
Gerrit-PatchSet: 2
Gerrit-Owner: Julius Werner
jwerner@chromium.org
Gerrit-Reviewer: Hung-Te Lin
hungte@chromium.org
Gerrit-Reviewer: Patrick Georgi
patrick@coreboot.org
Gerrit-Reviewer: Paul Menzel
paulepanter@mailbox.org
Gerrit-Reviewer: build bot (Jenkins)
no-reply@coreboot.org
Gerrit-CC: Elyes Haouas
ehaouas@noos.fr
Gerrit-Attention: Julius Werner
jwerner@chromium.org
Gerrit-Comment-Date: Mon, 05 Dec 2022 09:25:14 +0000
Gerrit-HasComments: Yes
Gerrit-Has-Labels: No
Comment-In-Reply-To: Elyes Haouas
ehaouas@noos.fr
Gerrit-MessageType: comment