On Tue, 20 Jun 2023 at 13:46, Richard Henderson <richard.hender...@linaro.org> wrote: > > Changes for v5: > * Enabled only by cpu property x-rme=on, not -cpu max. > * Rebase vs SecEL2 fixes, now in master. > > This doesn't have the magic RMM memory, which previous patch sets > included for booting Huawei's forked TF-A. Upstream TF-A does not > have sufficient code to build either PLAT={qemu,qemu_sbsa} with > the RMM enabled, so that can't be tested either at the moment. > > All I can say is that this doesn't appear to break anything else > with x-rme=on. Which is less than satisfying.
I'm going to apply this to target-arm.next, since it's all reviewed. I've added this to the patch 20 commit message: # We're not currently completely sure whether FEAT_RME will # be OK to enable purely as a CPU-level property, or if it will # need board co-operation, so we're making these experimental # x- properties, so that the people developing the system # level software for RME can try to start using this and let # us know how it goes. The command line syntax for enabling # this will change in future, without backwards-compatibility. If you can send an extra patch that does the documentation changes that would be great; I figured we didn't need to hold this up or apply only 1-19 and not 20 just for the docs update. thanks -- PMM