qemu-devel
[Top][All Lists]
Advanced

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

Re: [RFC PATCH v5 00/30] Add LoongArch softmmu support


From: Mark Cave-Ayland
Subject: Re: [RFC PATCH v5 00/30] Add LoongArch softmmu support
Date: Sat, 5 Feb 2022 13:32:39 +0000
User-agent: Mozilla/5.0 (X11; Linux x86_64; rv:91.0) Gecko/20100101 Thunderbird/91.5.0

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.


ATB,

Mark.



reply via email to

[Prev in Thread] Current Thread [Next in Thread]