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.