On 13 May 2014 17:15, Fabian Aggeler <aggel...@ethz.ch> wrote: > --- a/target-arm/cpu.h > +++ b/target-arm/cpu.h > @@ -631,6 +631,7 @@ enum arm_features { > ARM_FEATURE_CBAR, /* has cp15 CBAR */ > ARM_FEATURE_CRC, /* ARMv8 CRC instructions */ > ARM_FEATURE_CBAR_RO, /* has cp15 CBAR and it is read-only */ > + ARM_FEATURE_SECURITY_EXTENSIONS, /* has Security Extensions */ > };
Also this feature name is pretty long and unwieldy; something shorter would be nice. I think we should probably go with ARM_FEATURE_EL2/ARM_FEATURE_EL3 as per Edgar's patchset. thanks -- PMM