On 31 May 2018 at 21:39, Aaron Lindsay <alind...@codeaurora.org> wrote:
> On May 31 15:18, Peter Maydell wrote:
>> if (arm_feature(env, ARM_FEATURE_V7VE) {
>> /* v7 Virtualization Extensions. In real hardware this implies
>> * EL2 and also the presence of the Security Extensions.
>> * For QEMU, for backwards-compatibility we implement some
>> * CPUs or CPU configs which have no actual EL2 or EL3 but do
>> * include the various other features that V7VE implies.
>> * Presence of EL2 itself is ARM_FEATURE_EL2, and of the
>> * Security Extensions is ARM_FEATURE_EL3.
>> */
>> set_feature(env, ARM_FEATURE_ARM_DIV);
>
> Is it safe to assume from your comment above regarding keeping ARM_DIV
> separate from V7VE that the inclusion of it here is an oversight and
> that only LPAE and V7 should be set if V7VE is? (and that V8 should
> now directly imply both V7VE and ARM_DIV?)
No; V7VE always implies ARM_DIV. (ARM_DIV doesn't imply V7VE,
though, which is why it is a separate feature bit.)
thanks
-- PMM
