Sean Rhodes has submitted this change. ( https://review.coreboot.org/c/coreboot/+/86427?usp=email )
(
3 is the latest approved patch-set. No files were changed between the latest approved patch-set and the submitted one. )Change subject: src/acpi/acpigen: Increase LENSTACK_SIZE from 10 to 15 ......................................................................
src/acpi/acpigen: Increase LENSTACK_SIZE from 10 to 15
Some upcoming patches run up against the existing limit, which was added 16 years ago without any justification as to the size. Bump the size from 10 to 15, to prevent tripping the runtime assertion.
TEST=Tested with rest of patch train
Change-Id: I8362b3a63a23bea0ce47920e5d41cd2535dbc084 Signed-off-by: Matt DeVillier matt.devillier@gmail.com Reviewed-on: https://review.coreboot.org/c/coreboot/+/86427 Tested-by: build bot (Jenkins) no-reply@coreboot.org Reviewed-by: Sean Rhodes sean@starlabs.systems Reviewed-by: Patrick Rudolph patrick.rudolph@9elements.com --- M src/acpi/acpigen.c 1 file changed, 1 insertion(+), 1 deletion(-)
Approvals: Sean Rhodes: Looks good to me, but someone else must approve build bot (Jenkins): Verified Patrick Rudolph: Looks good to me, approved
diff --git a/src/acpi/acpigen.c b/src/acpi/acpigen.c index f682522..7d24b9e 100644 --- a/src/acpi/acpigen.c +++ b/src/acpi/acpigen.c @@ -1,7 +1,7 @@ /* SPDX-License-Identifier: GPL-2.0-only */
/* How much nesting do we support? */ -#define ACPIGEN_LENSTACK_SIZE 10 +#define ACPIGEN_LENSTACK_SIZE 15
/* If you need to change this, change acpigen_pop_len too */ #define ACPIGEN_RSVD_PKGLEN_BYTES 3