riscv: 'max' CPU, detect user choice in TCG
Status from this message: Applied to riscv-to-apply.next
Get series with b4
b4 am [email protected]Versions newest first · linked by author and title
- v10 Latest20 patches 2 repliesApplied
- v920 patches 16 repliesApplied
- v8 posting 2/220 patches 23 repliesSuperseded
Earlier versions · 8
| Version | Posted | Patches | Replies | Status |
|---|---|---|---|---|
| v8 posting 1/2 | 12 patches | 0 replies | Superseded | |
| v7 | 12 patches | 3 replies | Superseded | |
| v6 | 12 patches | 10 replies | Superseded | |
| v5 | 11 patches | 9 replies | Superseded | |
| v4 | 11 patches | 0 replies | Superseded | |
| v3 | 8 patches | 3 replies | Superseded | |
| v2 | 7 patches | 8 replies | Superseded | |
| v1 | 7 patches | 28 replies | Superseded |
Changes in v10 from the cover letter
- patch 3:
- use #ifdef CONFIG_KVM instead of #ifndef CONFIG_USER_ONLY
- helper renamed to kvm_riscv_cpu_add_kvm_properties
- helper is declared in kvm_riscv.h
- v9 link: https://lore.kernel.org/qemu-riscv/[email protected]/
[1] https://lists.gnu.org/archive/html/qemu-devel/2023-09/msg01729.html
[2] https://lore.kernel.org/qemu-riscv/[email protected]/
Patches in v10 Applied
Reviews trailers per patch
All 20 patches have a Reviewed-by or Acked-by.
| Patch | Andrew Jones | Alistair Francis | Philippe Mathieu-Daudé | Weiwei Li |
|---|---|---|---|---|
| 1/20 | R-b | R-b | R-b | |
| 2/20 | R-b | R-b | ||
| 3/20 | R-b | |||
| 4/20 | R-b | R-b | R-b | |
| 5/20 | R-b | R-b | ||
| 6/20 | R-b | R-b | ||
| 7/20 | R-b | R-b | R-b | |
| 8/20 | R-b | R-b | ||
| 9/20 | R-b | R-b | ||
| 10/20 | R-b | R-b | R-b | |
| 11/20 | R-b | R-b T-b | ||
| 12/20 | R-b | R-b | R-b T-b | |
| 13/20 | R-b | R-b | ||
| 14/20 | R-b | R-b | R-b | |
| 15/20 | R-b | R-b | ||
| 16/20 | R-b | R-b | ||
| 17/20 | R-b | R-b | ||
| 18/20 | R-b | R-b | ||
| 19/20 | R-b | R-b | ||
| 20/20 | R-b | R-b |
Trailers given in replies are filled; those carried in the patches from earlier versions are outlined with dashes. Scroll the table sideways for all 4 reviewers.
Cover letter [PATCH v10 00/20] riscv: 'max' CPU, detect user choice in TCG
Hi,
In this version we have a change in patch 3. It was causing a build break with --enable-debug builds that weren't being detected by regular gitlab CI but was detected when running the merge CI [1]. The problem is reproducible by any risc-v --enable-debug build.
I ended up leaving kvm_riscv_cpu_add_kvm_properties implementation inside cpu.c, since it can't be easily moved to kvm.c at this point, while adding its function declaration in kvm_riscv.h since it's a KVM only function. This is temporary, not a pattern we want to promote. We'll move the function to kvm.c when splitting accelerators logic in "riscv: split TCG/KVM accelerators from cpu.c", patch 08 in the current version of that series [2].
Series rebased on top of master. Only patch 3 is missing acks.
Changes from v9:
- patch 3:
- use #ifdef CONFIG_KVM instead of #ifndef CONFIG_USER_ONLY
- helper renamed to kvm_riscv_cpu_add_kvm_properties
- helper is declared in kvm_riscv.h
- v9 link: https://lore.kernel.org/qemu-riscv/[email protected]/
[1] https://lists.gnu.org/archive/html/qemu-devel/2023-09/msg01729.html
[2] https://lore.kernel.org/qemu-riscv/[email protected]/
Daniel Henrique Barboza (20):
target/riscv/cpu.c: split CPU options from riscv_cpu_extensions[] target/riscv/cpu.c: skip 'bool' check when filtering KVM props target/riscv/cpu.c: split kvm prop handling to its own helper target/riscv: add DEFINE_PROP_END_OF_LIST() to riscv_cpu_options[] target/riscv/cpu.c: split non-ratified exts from
riscv_cpu_extensions[]
target/riscv/cpu.c: split vendor exts from riscv_cpu_extensions[] target/riscv/cpu.c: add riscv_cpu_add_qdev_prop_array() target/riscv/cpu.c: add riscv_cpu_add_kvm_unavail_prop_array() target/riscv/cpu.c: limit cfg->vext_spec log message target/riscv: add 'max' CPU type
avocado, risc-v: add tuxboot tests for 'max' CPU target/riscv: deprecate the 'any' CPU type target/riscv/cpu.c: use offset in isa_ext_is_enabled/update_enabled target/riscv: make CPUCFG() macro public target/riscv/cpu.c: introduce cpu_cfg_ext_auto_update() target/riscv/cpu.c: use cpu_cfg_ext_auto_update() during realize() target/riscv/cpu.c: introduce RISCVCPUMultiExtConfig target/riscv: use isa_ext_update_enabled() in
init_max_cpu_extensions()
target/riscv/cpu.c: honor user choice in cpu_cfg_ext_auto_update() target/riscv/cpu.c: consider user option with RVG
docs/about/deprecated.rst | 12 + target/riscv/cpu-qom.h | 1 + target/riscv/cpu.c | 564 +++++++++++++++++++++--------- target/riscv/cpu.h | 2 + target/riscv/kvm.c | 8 +- target/riscv/kvm_riscv.h | 3 + tests/avocado/tuxrun_baselines.py | 32 ++ 7 files changed, 452 insertions(+), 170 deletions(-)
Signature · 1 line
2.41.0