On 5/23/25 11:31, Steven Lee wrote:
Clang's sanitizer reports a runtime error when booting with
'-net nic -net user', due to a null pointer being passed
to memory_region_find(), which subsequently triggers a crash in
flatview_lookup().
Root cause:
- Missing NIC configuration in the CA35 initialization.
Fix:
- Reduce ca35 ram size from 2GiB to 1GiB to align with ast2700a1-evb,
where the ram-container is defined as 1GiB in its class.
- Add nic configuration in ast2700fc's ca35 init function.
v2:
- Split the CA35 memory mapping into a separate patch.
- Added a new patch to fix BMC memory mapping in the fby35 machine,
which had a similar issue (unmapped system_memory).
- Removed Change-Id tag from commit messages
v3:
- Fixed issue with incorrectly inheriting TYPE_ASPEED_MACHINE.
- Rewrote the commit message for "Map BMC memory into system memory".
- Added more details to the commit message for "Fix unimplemented region
overlap with VBootROM".
v4:
- Split the first patch "Fix null pointer dereference" into two patches.
- Remove the patch "hw/arm/aspeed_ast27x0: Fix unimplemented
region overlap with vbootrom".
v5:
- Rewrote the commit message for the first patch to:
"hw/arm/aspeed_ast2700-fc: Add network support".
Steven Lee (6):
hw/arm/aspeed_ast2700-fc: Add network support
hw/arm/aspeed_ast2700-fc: Reduce ca35 ram size to align with ast2700a1
hw/arm/aspeed_ast27x0: Fix unimplemented region overlap with vbootrom
hw/arm/aspeed_ast27x0-fc: Map ca35 memory into system memory
hw/arm/fby35: Map BMC memory into system memory
docs: Remove ast2700fc from Aspeed family boards
docs/system/arm/aspeed.rst | 2 +-
hw/arm/aspeed_ast27x0-fc.c | 10 +++++++++-
hw/arm/aspeed_ast27x0.c | 4 ++--
hw/arm/fby35.c | 1 +
4 files changed, 13 insertions(+), 4 deletions(-)
Applied to aspeed-next.
Thanks,
C.