Hi, Mark
On 02/05/2022 09:32 PM, Mark Cave-Ayland wrote:
> On 28/01/2022 03:40, Xiaojuan Yang wrote:
>
>> This series patch add softmmu support for LoongArch.
>> The latest kernel:
>> * https://github.com/loongson/linux/tree/loongarch-next
>> The latest uefi:
>> * https://github.com/loongson/edk2
>> * https://github.com/loongson/edk2-platforms
>> The manual:
>> * https://github.com/loongson/LoongArch-Documentation/releases/tag/2021.10.11
>>
>> You can get LoongArch qemu series like this:
>> git clone https://github.com/loongson/qemu.git
>> git checkout tcg-dev
>>
>> Changes for v5:
>>
>> 1. Fix host bridge map irq function.
>> 2. Move cpu timer init function into machine init.
>> 3. Adjust memory region layout.
>> 4. Add the documentation at docs/system/loongarch/loongson3.rst.
>> - Introduction to 3a5000 virt.
>> - Output of "info mtree".
>>
>> Changes for v4:
>> 1. Uefi code is open and add some fdt interface to pass info between qemu and uefi.
>> 2. Use a per cpu address space for iocsr.
>> 3. Modify the tlb emulation.
>> 4. Machine and board code mainly follow Mark's advice.
>> 5. Adjust pci host space map.
>> 6. Use more memregion to simplify the interrupt controller's emulate.
>>
>>
>> Changes for v3:
>> 1.Target code mainly follow Richard's code review comments.
>> 2.Put the csr and iocsr read/write instruction emulate into 2 different patch.
>> 3.Simply the tlb emulation.
>> 4.Delete some unused csr registers defintion.
>> 5.Machine and board code mainly follow Mark's advice, discard the obsolete interface.
>> 6.NUMA function is removed for it is not completed.
>> 7.Adjust some format problem and the Naming problem
>>
>>
>> Changes for v3:
>> 1.Target code mainly follow Richard's code review comments.
>> 2.Put the csr and iocsr read/write instruction emulate into 2 different patch.
>> 3.Simply the tlb emulation.
>> 4.Delete some unused csr registers defintion.
>> 5.Machine and board code mainly follow Mark's advice, discard the obsolete interface.
>> 6.NUMA function is removed for it is not completed.
>> 7.Adjust some format problem and the Naming problem
>>
>>
>> Changes for v2:
>> 1.Combine patch 2 and 3 into one.
>> 2.Adjust the order of the patch.
>> 3.Put all the binaries on the github.
>> 4.Modify some emulate errors when use the kernel from the github.
>> 5.Adjust some format problem and the Naming problem
>> 6.Others mainly follow Richard's code review comments.
>>
>> Please help review!
>>
>> Thanks
>>
>>
>> Xiaojuan Yang (30):
>> target/loongarch: Add system emulation introduction
>> target/loongarch: Add CSRs definition
>> target/loongarch: Add basic vmstate description of CPU.
>> target/loongarch: Implement qmp_query_cpu_definitions()
>> target/loongarch: Add constant timer support
>> target/loongarch: Add MMU support for LoongArch CPU.
>> target/loongarch: Add LoongArch CSR instruction
>> target/loongarch: Add LoongArch IOCSR instruction
>> target/loongarch: Add TLB instruction support
>> target/loongarch: Add other core instructions support
>> target/loongarch: Add LoongArch interrupt and exception handle
>> target/loongarch: Add timer related instructions support.
>> target/loongarch: Add gdb support.
>> hw/pci-host: Add ls7a1000 PCIe Host bridge support for Loongson3
>> Platform
>> hw/loongarch: Add support loongson3-ls7a machine type.
>> hw/loongarch: Add LoongArch cpu interrupt support(CPUINTC)
>> hw/loongarch: Add LoongArch ipi interrupt support(IPI)
>> hw/intc: Add LoongArch ls7a interrupt controller support(PCH-PIC)
>> hw/intc: Add LoongArch ls7a msi interrupt controller support(PCH-MSI)
>> hw/intc: Add LoongArch extioi interrupt controller(EIOINTC)
>> hw/loongarch: Add irq hierarchy for the system
>> Enable common virtio pci support for LoongArch
>> hw/loongarch: Add some devices support for 3A5000.
>> hw/loongarch: Add LoongArch ls7a rtc device support
>> hw/loongarch: Add default bios startup support.
>> hw/loongarch: Add -kernel and -initrd options support
>> hw/loongarch: Add LoongArch smbios support
>> hw/loongarch: Add LoongArch acpi support
>> hw/loongarch: Add fdt support.
>> tests/tcg/loongarch64: Add hello/memory test in loongarch64 system
>>
>> .../devices/loongarch64-softmmu/default.mak | 3 +
>> configs/targets/loongarch64-softmmu.mak | 4 +
>> docs/system/loongarch/loongson3.rst | 78 ++
>> gdb-xml/loongarch-base64.xml | 43 +
>> gdb-xml/loongarch-fpu64.xml | 57 ++
>> hw/Kconfig | 1 +
>> hw/acpi/Kconfig | 4 +
>> hw/acpi/ls7a.c | 374 +++++++++
>> hw/acpi/meson.build | 1 +
>> hw/intc/Kconfig | 15 +
>> hw/intc/loongarch_extioi.c | 409 ++++++++++
>> hw/intc/loongarch_ipi.c | 164 ++++
>> hw/intc/loongarch_pch_msi.c | 75 ++
>> hw/intc/loongarch_pch_pic.c | 488 +++++++++++
>> hw/intc/meson.build | 4 +
>> hw/intc/trace-events | 27 +
>> hw/loongarch/Kconfig | 22 +
>> hw/loongarch/acpi-build.c | 636 +++++++++++++++
>> hw/loongarch/fw_cfg.c | 33 +
>> hw/loongarch/fw_cfg.h | 15 +
>> hw/loongarch/loongson3.c | 692 ++++++++++++++++
>> hw/loongarch/meson.build | 6 +
>> hw/meson.build | 1 +
>> hw/pci-host/Kconfig | 4 +
>> hw/pci-host/ls7a.c | 211 +++++
>> hw/pci-host/meson.build | 1 +
>> hw/rtc/Kconfig | 3 +
>> hw/rtc/ls7a_rtc.c | 322 ++++++++
>> hw/rtc/meson.build | 1 +
>> include/exec/poison.h | 2 +
>> include/hw/acpi/ls7a.h | 53 ++
>> include/hw/intc/loongarch_extioi.h | 77 ++
>> include/hw/intc/loongarch_ipi.h | 48 ++
>> include/hw/intc/loongarch_pch_msi.h | 21 +
>> include/hw/intc/loongarch_pch_pic.h | 81 ++
>> include/hw/loongarch/loongarch.h | 77 ++
>> include/hw/pci-host/ls7a.h | 79 ++
>> include/hw/pci/pci_ids.h | 3 +
>> include/sysemu/arch_init.h | 1 +
>> linux-user/loongarch64/cpu_loop.c | 8 +-
>> qapi/machine-target.json | 6 +-
>> qapi/machine.json | 2 +-
>> softmmu/qdev-monitor.c | 3 +-
>> target/Kconfig | 1 +
>> target/loongarch/Kconfig | 2 +
>> target/loongarch/README | 28 +
>> target/loongarch/constant_timer.c | 62 ++
>> target/loongarch/cpu-csr.h | 236 ++++++
>> target/loongarch/cpu-param.h | 2 +-
>> target/loongarch/cpu.c | 362 ++++++++-
>> target/loongarch/cpu.h | 217 ++++-
>> target/loongarch/csr_helper.c | 112 +++
>> target/loongarch/disas.c | 57 ++
>> target/loongarch/fpu_helper.c | 2 +-
>> target/loongarch/gdbstub.c | 97 +++
>> target/loongarch/helper.h | 26 +
>> target/loongarch/insn_trans/trans_extra.c.inc | 36 +-
>> .../insn_trans/trans_privileged.c.inc | 410 ++++++++++
>> target/loongarch/insns.decode | 44 +
>> target/loongarch/internals.h | 28 +
>> target/loongarch/iocsr_helper.c | 139 ++++
>> target/loongarch/machine.c | 102 +++
>> target/loongarch/meson.build | 11 +
>> target/loongarch/op_helper.c | 51 ++
>> target/loongarch/tlb_helper.c | 761 ++++++++++++++++++
>> target/loongarch/translate.c | 9 +-
>> tests/tcg/loongarch64/Makefile.softmmu-target | 33 +
>> tests/tcg/loongarch64/system/boot.S | 58 ++
>> tests/tcg/loongarch64/system/kernel.ld | 30 +
>> tests/tcg/loongarch64/system/regdef.h | 86 ++
>> 70 files changed, 7126 insertions(+), 31 deletions(-)
>> create mode 100644 configs/devices/loongarch64-softmmu/default.mak
>> create mode 100644 configs/targets/loongarch64-softmmu.mak
>> create mode 100644 docs/system/loongarch/loongson3.rst
>> create mode 100644 gdb-xml/loongarch-base64.xml
>> create mode 100644 gdb-xml/loongarch-fpu64.xml
>> create mode 100644 hw/acpi/ls7a.c
>> create mode 100644 hw/intc/loongarch_extioi.c
>> create mode 100644 hw/intc/loongarch_ipi.c
>> create mode 100644 hw/intc/loongarch_pch_msi.c
>> create mode 100644 hw/intc/loongarch_pch_pic.c
>> create mode 100644 hw/loongarch/Kconfig
>> create mode 100644 hw/loongarch/acpi-build.c
>> create mode 100644 hw/loongarch/fw_cfg.c
>> create mode 100644 hw/loongarch/fw_cfg.h
>> create mode 100644 hw/loongarch/loongson3.c
>> create mode 100644 hw/loongarch/meson.build
>> create mode 100644 hw/pci-host/ls7a.c
>> create mode 100644 hw/rtc/ls7a_rtc.c
>> create mode 100644 include/hw/acpi/ls7a.h
>> create mode 100644 include/hw/intc/loongarch_extioi.h
>> create mode 100644 include/hw/intc/loongarch_ipi.h
>> create mode 100644 include/hw/intc/loongarch_pch_msi.h
>> create mode 100644 include/hw/intc/loongarch_pch_pic.h
>> create mode 100644 include/hw/loongarch/loongarch.h
>> create mode 100644 include/hw/pci-host/ls7a.h
>> create mode 100644 target/loongarch/Kconfig
>> create mode 100644 target/loongarch/constant_timer.c
>> create mode 100644 target/loongarch/cpu-csr.h
>> create mode 100644 target/loongarch/csr_helper.c
>> create mode 100644 target/loongarch/gdbstub.c
>> create mode 100644 target/loongarch/insn_trans/trans_privileged.c.inc
>> create mode 100644 target/loongarch/iocsr_helper.c
>> create mode 100644 target/loongarch/machine.c
>> create mode 100644 target/loongarch/tlb_helper.c
>> create mode 100644 tests/tcg/loongarch64/Makefile.softmmu-target
>> create mode 100644 tests/tcg/loongarch64/system/boot.S
>> create mode 100644 tests/tcg/loongarch64/system/kernel.ld
>> create mode 100644 tests/tcg/loongarch64/system/regdef.h
>
> I've just done another set of reviews on v5 which is looking quite promising now. Please see the individual patch replies for further information.
>
> The main thing you are still missing is a patch adding entries to the MAINTAINERS files for all of your CPU and device files so that users and developers are able to contact someone if they have any issues or need patches reviewed. I would be tempted to include this in patch 1 and move it to the end of the series.
>
> Finally have you set up an account on GitLab and pushed your branch there to make sure that it passes QEMU CI? Whilst I have done a visual review of the patches, there are many extra checks performed by the CI to help catch runtime bugs.
>
Thank you for your advice, I have take a carefully look at all your advice and will modify them at the next version and the MAINTAINERS files will be added too.
We put The QEMU CI test at https://gitlab.com/gaosong/qemu, the branch is tcg-ci. The user emulation test passed but the system emulation test is unstable, we will fix it as soon as possible.
Thanks,
Xiaojuan
>
> ATB,
>
> Mark.