Message ID | 20220108091419.2027710-1-yangxiaojuan@loongson.cn (mailing list archive) |
---|---|
Headers | show |
Series | Add LoongArch softmmu support. | expand |
Hi Xiaojuan, I've just finished reviewing the first part (target modifications) as I'm not familiar with QEMU device emulation. You may have to revise the target part carefully, and re-organize at the series level to accelerate upstreaming though, as Richard pointed out in the other patch series (Song Gao's LoongArch linux-user support series) that the series as a whole is blocked. On 1/8/22 17:13, Xiaojuan Yang wrote: > This series patch add softmmu support for LoongArch. > Base on the linux-user emulation support V14 patch. > *https://patchew.org/QEMU/20220106094200.1801206-1-gaosong@loongson.cn/ There's a recognized syntax for marking patch series dependency [1], so that your series could be auto-applied by Patchew for ease of consumption. You can look at Song Gao's v14 LoongArch linux-user series for example usage. [1]: https://www.qemu.org/docs/master/devel/submitting-a-patch.html#base-patches-against-current-git-master > 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 > > > 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 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: Update README > target/loongarch: Add CSR registers definition > target/loongarch: Add basic vmstate description of CPU. There are serious issues with your commit message... First of all, some of your commit message titles end with a period, while some don't; the QEMU convention is to NOT use one. So please fix all commits like this to remove the trailing period. > target/loongarch: Implement qmp_query_cpu_definitions() > target/loongarch: Add constant timer support "Implement the constant timer" would be more concise and idiomatic English. > target/loongarch: Add MMU support for LoongArch CPU. > target/loongarch: Add LoongArch CSR instruction > target/loongarch: Add LoongArch IOCSR instruction You don't need to emphasize "LoongArch" because the component prefix "target/loongarch" says it all. Also all of these commits add support for multiple instructions at once, so you would say "instructions". You may need to check all places for simple plural form mistakes like these. > target/loongarch: Add TLB instruction support > target/loongarch: Add other core instructions support > target/loongarch: Add LoongArch interrupt and exception handle "handlers"? > target/loongarch: Add timer related instructions support. > target/loongarch: Add gdb support. > hw/pci-host: Add ls7a1000 PCIe Host bridge support for Loongson3 > Platform "Add the LS7A1000 PCIe host bridge" would be enough; although currently the LS7A chip is only paired with Loongson 3 CPUs, there's no intrinsic reasons to only support this combination ever. > hw/loongarch: Add support loongson3-ls7a machine type. "Support the loongson3-ls7a machine type" > hw/loongarch: Add LoongArch cpu interrupt support(CPUINTC) You may just say "Implement the LoongArch CPUINTC"; people naturally look in the diff to get what CPUINTC means. Same for other following commits with similar commit messages. > 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 This patch is missing component prefix in its title. > hw/loongarch: Add some devices support for 3A5000. What's "some"? You may want to clarify a bit, or to split patches if you cannot make your title short and concise. > hw/loongarch: Add LoongArch ls7a rtc device support The LS7A RTC is usable for MIPS-based Loongson systems too, like the 3A4000/LS7A1000 combination, so is this really LoongArch-specific? > 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 "for" loongarch64 systems > .../devices/loongarch64-softmmu/default.mak | 3 + > configs/targets/loongarch64-softmmu.mak | 4 + > 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 | 376 +++++++++ > hw/intc/loongarch_ipi.c | 164 ++++ > hw/intc/loongarch_pch_msi.c | 75 ++ > hw/intc/loongarch_pch_pic.c | 428 ++++++++++ > hw/intc/meson.build | 4 + > hw/intc/trace-events | 25 + > 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 | 685 +++++++++++++++ > hw/loongarch/meson.build | 6 + > hw/meson.build | 1 + > hw/pci-host/Kconfig | 4 + > hw/pci-host/ls7a.c | 218 +++++ > 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 | 69 ++ > include/hw/intc/loongarch_ipi.h | 48 ++ > include/hw/intc/loongarch_pch_msi.h | 21 + > include/hw/intc/loongarch_pch_pic.h | 74 ++ > include/hw/loongarch/loongarch.h | 75 ++ > 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 | 25 + > target/loongarch/constant_timer.c | 63 ++ > target/loongarch/cpu-csr.h | 236 ++++++ > target/loongarch/cpu-param.h | 2 +- > target/loongarch/cpu.c | 377 ++++++++- > target/loongarch/cpu.h | 220 ++++- > 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_core.c.inc | 412 ++++++++++ > target/loongarch/insn_trans/trans_extra.c.inc | 36 +- > target/loongarch/insns.decode | 44 + > target/loongarch/internals.h | 29 + > target/loongarch/iocsr_helper.c | 120 +++ > target/loongarch/machine.c | 101 +++ > target/loongarch/meson.build | 11 + > target/loongarch/op_helper.c | 57 ++ > target/loongarch/tlb_helper.c | 777 ++++++++++++++++++ > 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 ++ > 69 files changed, 6958 insertions(+), 30 deletions(-) > create mode 100644 configs/devices/loongarch64-softmmu/default.mak > create mode 100644 configs/targets/loongarch64-softmmu.mak > 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_core.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 >
Hi, Xuerui Thank you for all you advice, I will modify the target part carefully. Xiaojuan On 01/09/2022 05:26 PM, WANG Xuerui wrote: > Hi Xiaojuan, > > I've just finished reviewing the first part (target modifications) as I'm not familiar with QEMU device emulation. You may have to revise the target part carefully, and re-organize at the series level to accelerate upstreaming though, as Richard pointed out in the other patch series (Song Gao's LoongArch linux-user support series) that the series as a whole is blocked. > > > On 1/8/22 17:13, Xiaojuan Yang wrote: >> This series patch add softmmu support for LoongArch. >> Base on the linux-user emulation support V14 patch. >> *https://patchew.org/QEMU/20220106094200.1801206-1-gaosong@loongson.cn/ > > There's a recognized syntax for marking patch series dependency [1], so that your series could be auto-applied by Patchew for ease of consumption. You can look at Song Gao's v14 LoongArch linux-user series for example usage. > > [1]: https://www.qemu.org/docs/master/devel/submitting-a-patch.html#base-patches-against-current-git-master > >> 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 >> >> >> 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 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: Update README >> target/loongarch: Add CSR registers definition >> target/loongarch: Add basic vmstate description of CPU. > > There are serious issues with your commit message... > > First of all, some of your commit message titles end with a period, while some don't; the QEMU convention is to NOT use one. So please fix all commits like this to remove the trailing period. > >> target/loongarch: Implement qmp_query_cpu_definitions() >> target/loongarch: Add constant timer support > "Implement the constant timer" would be more concise and idiomatic English. >> target/loongarch: Add MMU support for LoongArch CPU. >> target/loongarch: Add LoongArch CSR instruction >> target/loongarch: Add LoongArch IOCSR instruction > You don't need to emphasize "LoongArch" because the component prefix "target/loongarch" says it all. Also all of these commits add support for multiple instructions at once, so you would say "instructions". You may need to check all places for simple plural form mistakes like these. >> target/loongarch: Add TLB instruction support >> target/loongarch: Add other core instructions support >> target/loongarch: Add LoongArch interrupt and exception handle > "handlers"? >> target/loongarch: Add timer related instructions support. >> target/loongarch: Add gdb support. >> hw/pci-host: Add ls7a1000 PCIe Host bridge support for Loongson3 >> Platform > "Add the LS7A1000 PCIe host bridge" would be enough; although currently the LS7A chip is only paired with Loongson 3 CPUs, there's no intrinsic reasons to only support this combination ever. >> hw/loongarch: Add support loongson3-ls7a machine type. > "Support the loongson3-ls7a machine type" >> hw/loongarch: Add LoongArch cpu interrupt support(CPUINTC) > You may just say "Implement the LoongArch CPUINTC"; people naturally look in the diff to get what CPUINTC means. Same for other following commits with similar commit messages. >> 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 > This patch is missing component prefix in its title. >> hw/loongarch: Add some devices support for 3A5000. > What's "some"? You may want to clarify a bit, or to split patches if you cannot make your title short and concise. >> hw/loongarch: Add LoongArch ls7a rtc device support > The LS7A RTC is usable for MIPS-based Loongson systems too, like the 3A4000/LS7A1000 combination, so is this really LoongArch-specific? >> 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 > "for" loongarch64 systems >> .../devices/loongarch64-softmmu/default.mak | 3 + >> configs/targets/loongarch64-softmmu.mak | 4 + >> 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 | 376 +++++++++ >> hw/intc/loongarch_ipi.c | 164 ++++ >> hw/intc/loongarch_pch_msi.c | 75 ++ >> hw/intc/loongarch_pch_pic.c | 428 ++++++++++ >> hw/intc/meson.build | 4 + >> hw/intc/trace-events | 25 + >> 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 | 685 +++++++++++++++ >> hw/loongarch/meson.build | 6 + >> hw/meson.build | 1 + >> hw/pci-host/Kconfig | 4 + >> hw/pci-host/ls7a.c | 218 +++++ >> 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 | 69 ++ >> include/hw/intc/loongarch_ipi.h | 48 ++ >> include/hw/intc/loongarch_pch_msi.h | 21 + >> include/hw/intc/loongarch_pch_pic.h | 74 ++ >> include/hw/loongarch/loongarch.h | 75 ++ >> 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 | 25 + >> target/loongarch/constant_timer.c | 63 ++ >> target/loongarch/cpu-csr.h | 236 ++++++ >> target/loongarch/cpu-param.h | 2 +- >> target/loongarch/cpu.c | 377 ++++++++- >> target/loongarch/cpu.h | 220 ++++- >> 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_core.c.inc | 412 ++++++++++ >> target/loongarch/insn_trans/trans_extra.c.inc | 36 +- >> target/loongarch/insns.decode | 44 + >> target/loongarch/internals.h | 29 + >> target/loongarch/iocsr_helper.c | 120 +++ >> target/loongarch/machine.c | 101 +++ >> target/loongarch/meson.build | 11 + >> target/loongarch/op_helper.c | 57 ++ >> target/loongarch/tlb_helper.c | 777 ++++++++++++++++++ >> 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 ++ >> 69 files changed, 6958 insertions(+), 30 deletions(-) >> create mode 100644 configs/devices/loongarch64-softmmu/default.mak >> create mode 100644 configs/targets/loongarch64-softmmu.mak >> 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_core.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 >>
On 08/01/2022 09:13, Xiaojuan Yang wrote: > This series patch add softmmu support for LoongArch. > Base on the linux-user emulation support V14 patch. > * https://patchew.org/QEMU/20220106094200.1801206-1-gaosong@loongson.cn/ > 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 > > > 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 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: Update README > target/loongarch: Add CSR registers 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 + > 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 | 376 +++++++++ > hw/intc/loongarch_ipi.c | 164 ++++ > hw/intc/loongarch_pch_msi.c | 75 ++ > hw/intc/loongarch_pch_pic.c | 428 ++++++++++ > hw/intc/meson.build | 4 + > hw/intc/trace-events | 25 + > 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 | 685 +++++++++++++++ > hw/loongarch/meson.build | 6 + > hw/meson.build | 1 + > hw/pci-host/Kconfig | 4 + > hw/pci-host/ls7a.c | 218 +++++ > 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 | 69 ++ > include/hw/intc/loongarch_ipi.h | 48 ++ > include/hw/intc/loongarch_pch_msi.h | 21 + > include/hw/intc/loongarch_pch_pic.h | 74 ++ > include/hw/loongarch/loongarch.h | 75 ++ > 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 | 25 + > target/loongarch/constant_timer.c | 63 ++ > target/loongarch/cpu-csr.h | 236 ++++++ > target/loongarch/cpu-param.h | 2 +- > target/loongarch/cpu.c | 377 ++++++++- > target/loongarch/cpu.h | 220 ++++- > 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_core.c.inc | 412 ++++++++++ > target/loongarch/insn_trans/trans_extra.c.inc | 36 +- > target/loongarch/insns.decode | 44 + > target/loongarch/internals.h | 29 + > target/loongarch/iocsr_helper.c | 120 +++ > target/loongarch/machine.c | 101 +++ > target/loongarch/meson.build | 11 + > target/loongarch/op_helper.c | 57 ++ > target/loongarch/tlb_helper.c | 777 ++++++++++++++++++ > 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 ++ > 69 files changed, 6958 insertions(+), 30 deletions(-) > create mode 100644 configs/devices/loongarch64-softmmu/default.mak > create mode 100644 configs/targets/loongarch64-softmmu.mak > 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_core.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 review for v4: this is looking much better to me, since without knowing much about Loongarch I can now at least get a feel for the IRQs/memory layouts by following the standard QEMU APIs. There are some queries/questions about some MMIO regions (and initialisation of the CPU in sysemu mode) but I don't think they will be too hard to fix once we understand the memory layout. Finally a couple of slightly more routine things that this series is missing: an update to the MAINTAINERS file for the target/loongarch directory, and also some .rst documentation describing the basic "virt" machine and its devices e.g. something like https://qemu.readthedocs.io/en/latest/system/target-sparc64.html. ATB, Mark.
Hi, Mark On 01/15/2022 10:11 PM, Mark Cave-Ayland wrote: > On 08/01/2022 09:13, Xiaojuan Yang wrote: > >> This series patch add softmmu support for LoongArch. >> Base on the linux-user emulation support V14 patch. >> * https://patchew.org/QEMU/20220106094200.1801206-1-gaosong@loongson.cn/ >> 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 >> >> >> 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 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: Update README >> target/loongarch: Add CSR registers 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 + >> 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 | 376 +++++++++ >> hw/intc/loongarch_ipi.c | 164 ++++ >> hw/intc/loongarch_pch_msi.c | 75 ++ >> hw/intc/loongarch_pch_pic.c | 428 ++++++++++ >> hw/intc/meson.build | 4 + >> hw/intc/trace-events | 25 + >> 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 | 685 +++++++++++++++ >> hw/loongarch/meson.build | 6 + >> hw/meson.build | 1 + >> hw/pci-host/Kconfig | 4 + >> hw/pci-host/ls7a.c | 218 +++++ >> 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 | 69 ++ >> include/hw/intc/loongarch_ipi.h | 48 ++ >> include/hw/intc/loongarch_pch_msi.h | 21 + >> include/hw/intc/loongarch_pch_pic.h | 74 ++ >> include/hw/loongarch/loongarch.h | 75 ++ >> 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 | 25 + >> target/loongarch/constant_timer.c | 63 ++ >> target/loongarch/cpu-csr.h | 236 ++++++ >> target/loongarch/cpu-param.h | 2 +- >> target/loongarch/cpu.c | 377 ++++++++- >> target/loongarch/cpu.h | 220 ++++- >> 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_core.c.inc | 412 ++++++++++ >> target/loongarch/insn_trans/trans_extra.c.inc | 36 +- >> target/loongarch/insns.decode | 44 + >> target/loongarch/internals.h | 29 + >> target/loongarch/iocsr_helper.c | 120 +++ >> target/loongarch/machine.c | 101 +++ >> target/loongarch/meson.build | 11 + >> target/loongarch/op_helper.c | 57 ++ >> target/loongarch/tlb_helper.c | 777 ++++++++++++++++++ >> 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 ++ >> 69 files changed, 6958 insertions(+), 30 deletions(-) >> create mode 100644 configs/devices/loongarch64-softmmu/default.mak >> create mode 100644 configs/targets/loongarch64-softmmu.mak >> 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_core.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 review for v4: this is looking much better to me, since without knowing much about Loongarch I can now at least get a feel for the IRQs/memory layouts by following the standard QEMU APIs. > > There are some queries/questions about some MMIO regions (and initialisation of the CPU in sysemu mode) but I don't think they will be too hard to fix once we understand the memory layout. > > Finally a couple of slightly more routine things that this series is missing: an update to the MAINTAINERS file for the target/loongarch directory, and also some .rst documentation describing the basic "virt" machine and its devices e.g. something like https://qemu.readthedocs.io/en/latest/system/target-sparc64.html. > Thank you for your advice, I will learn more about the memory layout and modify the MMIO regions and fixup other things in the next version. Thanks, Xiaojuan > > ATB, > > Mark.