[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-arm] [PATCH v1 1/5] xlnx-zynqmp: Add a secure prop to en/disab
From: |
Peter Maydell |
Subject: |
Re: [Qemu-arm] [PATCH v1 1/5] xlnx-zynqmp: Add a secure prop to en/disable ARM Security Extensions |
Date: |
Tue, 24 May 2016 18:29:49 +0100 |
On 24 May 2016 at 17:57, Edgar E. Iglesias <address@hidden> wrote:
> On Tue, May 24, 2016 at 05:30:54PM +0100, Peter Maydell wrote:
>> On 19 May 2016 at 23:54, Edgar E. Iglesias <address@hidden> wrote:
>> > From: "Edgar E. Iglesias" <address@hidden>
>> >
>> > Add a secure prop to en/disable ARM Security Extensions.
>> > This is particulary useful for KVM runs.
>>
>> "particularly"
>>
>> > Default to disabled to match the behavior of KVM.
>>
>> This is a change in behaviour, though, right? Is that OK?
>
> IMO it's OK. But I don't have a very strong opinion. We
> didn't have EL3 in this machine to start with, it came in when
> we enabled it on the a53. We don't have a big user-base that
> really depends on either default I'd say.
I'll defer to your preference, but we should mention in the
commit message that we changed the behaviour (and perhaps
also that it was accidental that EL3 got enabled).
thanks
-- PMM
- [Qemu-arm] [PATCH v1 0/5] xlnx-zynqmp: Fix various issues with KVM runs, Edgar E. Iglesias, 2016/05/19
- [Qemu-arm] [PATCH v1 4/5] xlnx-zynqmp: Delay realization of GIC until post CPU realization, Edgar E. Iglesias, 2016/05/19
- [Qemu-arm] [PATCH v1 3/5] xlnx-zynqmp: Make the RPU subsystem optional, Edgar E. Iglesias, 2016/05/19
- [Qemu-arm] [PATCH v1 1/5] xlnx-zynqmp: Add a secure prop to en/disable ARM Security Extensions, Edgar E. Iglesias, 2016/05/19
- [Qemu-arm] [PATCH v1 2/5] xlnx-zynqmp: Break out RPU setup into a separate function, Edgar E. Iglesias, 2016/05/19
- [Qemu-arm] [PATCH v1 5/5] xlnx-zynqmp: Use the in kernel GIC model for KVM runs, Edgar E. Iglesias, 2016/05/19
- [Qemu-arm] [PATCH v1 2/5] xlnx-zynqmp: Break out RPU setup into a separate function, Edgar E. Iglesias, 2016/05/19
- [Qemu-arm] [PATCH v1 4/5] xlnx-zynqmp: Delay realization of GIC until post CPU realization, Edgar E. Iglesias, 2016/05/19
- [Qemu-arm] [PATCH v1 5/5] xlnx-zynqmp: Use the in kernel GIC model for KVM runs, Edgar E. Iglesias, 2016/05/19
- [Qemu-arm] [PATCH v1 3/5] xlnx-zynqmp: Make the RPU subsystem optional, Edgar E. Iglesias, 2016/05/19