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-gaos...@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