[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH v3 0/6] Add Microblaze configuration options
From: |
Alistair Francis |
Subject: |
Re: [Qemu-devel] [PATCH v3 0/6] Add Microblaze configuration options |
Date: |
Fri, 29 May 2015 17:08:36 +1000 |
On Fri, May 29, 2015 at 4:58 PM, Edgar E. Iglesias
<address@hidden> wrote:
> On Fri, May 29, 2015 at 04:28:51PM +1000, Alistair Francis wrote:
>> Firstly this patch series tidies up some code and removes
>> a "xlnx." prefix.
>>
>> Then it moves the Microblaze PVR registers to the end
>> of the CPUMBState to preserve them during reset. This
>> allows most of the operations on them to be moved from
>> the reset to the realise. Except for the machine specific
>> ones which will be moved when the other properties are
>> converted across to standard QEMU props, after this patch
>> series is accepted (either merged or the method I'm using
>> is approved). See the individual commit for more details.
>>
>> Next it adds the "use-stack-protection" property
>> to the Microblaze CPU, which allows stack protection to be
>> disabled.
>>
>> It also converts the previously hardcoded method of enabling
>> the FPU to use standard QEMU properties. This simplifies the
>> logic in the target-microblaze translate.c.
>>
>> V3:
>> - Split disabling stack protection into a seperate patch
>> - Add comments detailing the use-fpu values
>> V2:
>> - Small fixes
>> - Disable stack protection by default
>> - Remove the memset and cpu_reset functions from the MB realise
>> Changes since RFC:
>> - Preserve the PVR registers during resets
>> - Move most of the logic into realise functions
>> - Small name and function changes
>
> Thanks Alistair,
>
> I've added this into my MicroBlaze queue.
Thanks Edgar,
I'll start working on the next set next week
Thanks,
Alistair
>
> Cheers,
> Edgar
>
>
>
>>
>> Alistair Francis (6):
>> target-microblaze: Fix up indentation
>> target-microblaze: Preserve the pvr registers during reset
>> target-microblaze: Allow the stack protection to be disabled
>> target-microblaze: Tidy up the base-vectors property
>> target-microblaze: Convert use-fpu to a CPU property
>> target-microblaze: Disable stack protection by default
>>
>> hw/microblaze/petalogix_ml605_mmu.c | 7 +++-
>> target-microblaze/cpu-qom.h | 9 +++++-
>> target-microblaze/cpu.c | 60
>> ++++++++++++++++++++++-------------
>> target-microblaze/cpu.h | 11 ++++--
>> target-microblaze/helper.c | 8 ++--
>> target-microblaze/op_helper.c | 10 +++---
>> target-microblaze/translate.c | 14 +++-----
>> 7 files changed, 72 insertions(+), 47 deletions(-)
>>
>
- [Qemu-devel] [PATCH v3 1/6] target-microblaze: Fix up indentation, (continued)
- [Qemu-devel] [PATCH v3 1/6] target-microblaze: Fix up indentation, Alistair Francis, 2015/05/29
- [Qemu-devel] [PATCH v3 2/6] target-microblaze: Preserve the pvr registers during reset, Alistair Francis, 2015/05/29
- [Qemu-devel] [PATCH v3 3/6] target-microblaze: Allow the stack protection to be disabled, Alistair Francis, 2015/05/29
- [Qemu-devel] [PATCH v3 5/6] target-microblaze: Convert use-fpu to a CPU property, Alistair Francis, 2015/05/29
- [Qemu-devel] [PATCH v3 6/6] target-microblaze: Disable stack protection by default, Alistair Francis, 2015/05/29
- [Qemu-devel] [PATCH v3 4/6] target-microblaze: Tidy up the base-vectors property, Alistair Francis, 2015/05/29
- Re: [Qemu-devel] [PATCH v3 0/6] Add Microblaze configuration options, Edgar E. Iglesias, 2015/05/29
- Re: [Qemu-devel] [PATCH v3 0/6] Add Microblaze configuration options,
Alistair Francis <=