fdt: bump FDT_MEM_REGIONS to 64
ClosedPublic
Actions

Authored by adrian on Sun, Apr 6, 8:42 PM.

Details

Reviewers

andrew
manu
phk

Commits

rG489489b4bc55: fdt: bump FDT_MEM_REGIONS to 64

Summary

The Snapdragon X1E / Lenovo T14s device tree has 47 reserved memory
regions. The reserved region parsing doesn't boundary check and will
just end up array bounds smashing, which leads to a reboot before
any console IO is written.

Diff Detail

Repository

rG FreeBSD src repository

Lint

Lint Not Applicable

Unit

Tests Not Applicable

Event Timeline

adrian created this revision.Sun, Apr 6, 8:42 PM

Herald added a subscriber: imp. · View Herald TranscriptSun, Apr 6, 8:42 PM

adrian requested review of this revision.Sun, Apr 6, 8:42 PM

Harbormaster completed remote builds in B63356: Diff 153215.Sun, Apr 6, 8:43 PM

adrian added a parent revision: D49686: qcom_gcc: migrate to use sleep_clk directly.Sun, Apr 6, 8:44 PM

adrian added reviewers: andrew, manu, phk.

adrian added a child revision: D49690: arm64: add the lenovo t14s snapdragon X1E device tree to the build list.

phk accepted this revision.Mon, Apr 7, 4:55 AM

This revision is now accepted and ready to land.Mon, Apr 7, 4:55 AM

manu accepted this revision.Mon, Apr 7, 5:52 AM

oh added a subscriber: oh.Mon, Apr 7, 9:09 AM

Closed by commit rG489489b4bc55: fdt: bump FDT_MEM_REGIONS to 64 (authored by adrian). · Explain WhyTue, Apr 8, 3:48 AM

This revision was automatically updated to reflect the committed changes.

adrian added a commit: rG489489b4bc55: fdt: bump FDT_MEM_REGIONS to 64.

Revision Contents
Changeset List

Path

Size

sys/

dev/

fdt/

fdt_common.h

2 lines

Diff 153291

View Options

fdt: bump FDT_MEM_REGIONS to 64ClosedPublicActions