Re: [PATCH v8 0/4] target/arm: Add FEAT_MEC to max cpu

[Gustavo Romero <gustavo.romero@linaro.org>]

Hi folks,

Richard, thanks for v8. Pierrick, thanks for testing it. :)

On 7/14/25 14:09, Pierrick Bouvier wrote:
> On 7/14/25 8:58 AM, Richard Henderson wrote:
>> Changes for v8:
>>    - Re-order SCTLR2 and TCR2 so that they are independent of MEC.
>>    - Enable the SCTLR2 and TCR2 enable bits.
>>    - Squash 3 smaller MEC patches together.
>>
>> This still fails the RME tests, because we still need TF-A rebuilt
>> with ENABLE_FEAT_SCTLR2 and ENABLE_FEAT_TCR2.  Pierrick, since you
>> have just done such a build, could you re-test with this series?
>>
> 
> I tested that on my local Realm enabled setup and I can confirm this solved the issue and current series works.
> Both flags are needed in TF-A. ENABLE_FEAT_TCR2 is needed to boot host, and ENABLE_FEAT_SCTLR2 is needed to boot nested guest.

I'm a bit confused because the QEMU RME tests, afaics, uses OP-TEE, not TF-A. I've built TF-A
using the scripts in [0], enabling ENABLE_FEAT_TCR2 and ENABLE_FEAT_SCTLR2, but no way to get
it booting. I understand we can embed a OP-TEE into the TF_A via BL32=<optee_image> when
building TF-A. Is that what you're using?

Thanks.


Cheers,
Gustavo

[0] https://github.com/pbo-linaro/qemu-linux-stack.git

> As I'm off today, I'll update that properly tomorrow when I have time, and not rush things. I'll update RME images for sbsa and virt tests + device passthrough test, and post associated patches.
> 
> Thanks,
> Pierrick