Hi Andre, > > Revert "sunxi: enable MMU_PGPROT proper page table protection" > > > > This reverts commit 16cfccda4dbf0b53473b8212897f5c63bfbeb6e1. > > > > This causes the follow issue on the A64 boards I have including > > Pine64+, Pine64-LTS and the Pinephone. Revert until the issue > > can be properly investigated. > > > > CPU: Allwinner A64 (SUN50I) > > Model: Pine64+ > > DRAM: 2 GiB > > Insufficient RAM for page table: 0x5000 > 0x4000. Please increase the size > > in get_page_table_size() > > Mmh, how did you get this? I just ran mainline on a Pine64-LTS, and > didn't see any issues. Are you running on some specific setup? I am on > GCC 11.2.0, TF-A is some v2.12.0-724-gf745e004a, and that's with > pine64-lts_defconfig.
Fedora 42/43, gcc 15.2, binutils 2.44, TF-A 2.13. I've seen it across Pine64+, Pine64-LTS and Pinephone. No special setup and I reproduced in on the vanilla upstream checkout from a couple of days ago when I had the time to do the bisect. > > resetting ... > > System reset not supported on this platform > > > > Signed-off-by: Peter Robinson <pbrobin...@gmail.com> > > --- > > arch/arm/Kconfig | 1 - > > 1 file changed, 1 deletion(-) > > > > diff --git a/arch/arm/Kconfig b/arch/arm/Kconfig > > index 40368abc297..b8be270e4d7 100644 > > --- a/arch/arm/Kconfig > > +++ b/arch/arm/Kconfig > > @@ -1207,7 +1207,6 @@ config ARCH_SUNXI > > select DM_SPI_FLASH if SPI && MTD > > select DM_KEYBOARD > > select DM_SERIAL > > - select MMU_PGPROT if ARM64 > > Regardless that's quite a sledgehammer. If that's a real issue, we > could surely provide our own get_page_table_size() implementation which > is more generous? You commit that reverts says "This might trigger runtime exceptions in misbehaving drivers, which should then be fixed." which shows that it might cause a problem, and I agree that any drivers should be fixed, but the above output tells me absolutely nothing about even where to begin to look so to use your sledgehammer analogy, if I knew whether I needed a ball-peen hammer or a claw hammer I could but I am basically blind and with my testing the sledge hammer solves the problem. Peter
