[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH 04/13] target-arm: Add secure qemu machine optio
From: |
Peter Maydell |
Subject: |
Re: [Qemu-devel] [PATCH 04/13] target-arm: Add secure qemu machine option |
Date: |
Fri, 5 Dec 2014 15:39:46 +0000 |
On 5 December 2014 at 15:33, Greg Bellows <address@hidden> wrote:
>
>
> On 5 December 2014 at 09:18, Peter Maydell <address@hidden> wrote:
>>
>> On 3 December 2014 at 20:05, Greg Bellows <address@hidden> wrote:
>> > Added 'secure' qemu boolean option to qemu_machine_opts[].
>> >
>> > Signed-off-by: Greg Bellows <address@hidden>
>> > ---
>> > vl.c | 4 ++++
>> > 1 file changed, 4 insertions(+)
>> >
>> > diff --git a/vl.c b/vl.c
>> > index eb89d62..5d640f7 100644
>> > --- a/vl.c
>> > +++ b/vl.c
>> > @@ -387,6 +387,10 @@ static QemuOptsList qemu_machine_opts = {
>> > .name = "iommu",
>> > .type = QEMU_OPT_BOOL,
>> > .help = "Set on/off to enable/disable Intel IOMMU (VT-d)",
>> > + },{
>> > + .name = "secure",
>> > + .type = QEMU_OPT_BOOL,
>> > + .help = "Set on/off to enable/disable secure state",
>> > },
>>
>> If patch 5 adds 'secure' as a machine property for only those
>> boards where it makes sense, why do we need this global switch?
>>
>
> That is what I thought as well, but this is apparently needed as we get an
> invalid machine property otherwise. Below is the error, I'll look again,
> but it appeared all machine properties were defined here.
>
> qemu-system-aarch64: -machine type=vexpress-a15,secure=off: Invalid
> parameter 'secure'
That would seem to defeat the point of the machine opts design,
so it looks a bit strange. Marcel: how is this supposed to work
for board-specific -machine options?
thanks
-- PMM
- [Qemu-devel] [PATCH 01/13] target-arm: Add vexpress class and machine types, (continued)
- [Qemu-devel] [PATCH 05/13] target-arm: Add vexpress machine secure property, Greg Bellows, 2014/12/03
- [Qemu-devel] [PATCH 06/13] target-arm: Change vexpress daughterboard init arg, Greg Bellows, 2014/12/03
- [Qemu-devel] [PATCH 04/13] target-arm: Add secure qemu machine option, Greg Bellows, 2014/12/03
- Re: [Qemu-devel] [PATCH 04/13] target-arm: Add secure qemu machine option, Peter Maydell, 2014/12/05
- Re: [Qemu-devel] [PATCH 04/13] target-arm: Add secure qemu machine option, Greg Bellows, 2014/12/05
- Re: [Qemu-devel] [PATCH 04/13] target-arm: Add secure qemu machine option,
Peter Maydell <=
- Re: [Qemu-devel] [PATCH 04/13] target-arm: Add secure qemu machine option, Marcel Apfelbaum, 2014/12/05
- Re: [Qemu-devel] [PATCH 04/13] target-arm: Add secure qemu machine option, Greg Bellows, 2014/12/05
- Re: [Qemu-devel] [PATCH 04/13] target-arm: Add secure qemu machine option, Marcel Apfelbaum, 2014/12/05
- Re: [Qemu-devel] [PATCH 04/13] target-arm: Add secure qemu machine option, Greg Bellows, 2014/12/05
[Qemu-devel] [PATCH 07/13] target-arm: Add virt class and machine types, Greg Bellows, 2014/12/03
[Qemu-devel] [PATCH 09/13] target-arm: Add feature unset function, Greg Bellows, 2014/12/03
[Qemu-devel] [PATCH 08/13] target-arm: Add virt machine secure property, Greg Bellows, 2014/12/03
[Qemu-devel] [PATCH 10/13] target-arm: Add ARMCPU secure property, Greg Bellows, 2014/12/03