Add Microblaze configuration options
Get series with b4
b4 am [email protected]Versions newest first · linked by author and title
- v3 Latest6 patches 5 repliesIn review
- v25 patches 9 repliesSuperseded
- v15 patches 11 repliesSuperseded
Changes in v3 from the cover letter
- Split disabling stack protection into a seperate patch
- Add comments detailing the use-fpu values
Patches in v3 No applies recorded yet
| # | Subject and files | Size | Replies | Applied |
|---|---|---|---|---|
| 1/6 | target-microblaze: Fix up indentationtarget-microblaze/ | +5 | 0 replies | |
| 2/6 | target-microblaze: Preserve the pvr registers during resettarget-microblaze/ target-microblaze/ | +28 | 0 replies | |
| 3/6 | target-microblaze: Allow the stack protection to be disabledtarget-microblaze/ target-microblaze/ target-microblaze/ +1 more | +13 | 1 reply | |
| 4/6 | target-microblaze: Tidy up the base-vectors propertytarget-microblaze/ target-microblaze/ target-microblaze/ | +8 | 0 replies | |
| 5/6 | target-microblaze: Convert use-fpu to a CPU propertyhw/ target-microblaze/ target-microblaze/ +1 more | +19 | 1 reply | |
| 6/6 | target-microblaze: Disable stack protection by defaulttarget-microblaze/ | +1 | 1 reply |
Reviews trailers per patch
All 6 patches have a Reviewed-by or Acked-by.
| Patch | Peter Crosthwaite | Edgar E. Iglesias |
|---|---|---|
| 1/6 | R-b | R-b |
| 2/6 | R-b | R-b |
| 3/6 | R-b | |
| 4/6 | R-b | |
| 5/6 | R-b | R-b |
| 6/6 | R-b |
Trailers given in replies are filled; those carried in the patches from earlier versions are outlined with dashes.
Cover letter [Qemu-devel] [PATCH v3 0/6] Add Microblaze configuration options
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
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(-)