[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
[Qemu-commits] [qemu/qemu] 35c702: target/riscv: Fix PMP propagation for
From: |
Peter Maydell |
Subject: |
[Qemu-commits] [qemu/qemu] 35c702: target/riscv: Fix PMP propagation for tlb |
Date: |
Tue, 20 Dec 2022 05:39:24 -0800 |
Branch: refs/heads/staging
Home: https://github.com/qemu/qemu
Commit: 35c70283c4517f0c8a559229bed124c4c77b9ae9
https://github.com/qemu/qemu/commit/35c70283c4517f0c8a559229bed124c4c77b9ae9
Author: LIU Zhiwei <zhiwei_liu@linux.alibaba.com>
Date: 2022-12-19 (Mon, 19 Dec 2022)
Changed paths:
M target/riscv/cpu_helper.c
M target/riscv/pmp.c
M target/riscv/pmp.h
Log Message:
-----------
target/riscv: Fix PMP propagation for tlb
Only the pmp index that be checked by pmp_hart_has_privs can be used
by pmp_get_tlb_size to avoid an error pmp index.
Before modification, we may use an error pmp index. For example,
we check address 0x4fc, and the size 0x4 in pmp_hart_has_privs. If there
is an pmp rule, valid range is [0x4fc, 0x500), then pmp_hart_has_privs
will return true;
However, this checked pmp index is discarded as pmp_hart_has_privs
return bool value. In pmp_is_range_in_tlb, it will traverse all pmp
rules. The tlb_sa will be 0x0, and tlb_ea will be 0xfff. If there is
a pmp rule [0x10, 0x14), it will be misused as it is legal in
pmp_get_tlb_size.
As we have already known the correct pmp index, just remove the
remove the pmp_is_range_in_tlb and get tlb size directly from
pmp_get_tlb_size.
Signed-off-by: LIU Zhiwei <zhiwei_liu@linux.alibaba.com>
Reviewed-by: Alistair Francis <alistair.francis@wdc.com>
Message-Id: <20221012060016.30856-1-zhiwei_liu@linux.alibaba.com>
Signed-off-by: Alistair Francis <alistair.francis@wdc.com>
Commit: 84bae5acff89134ea52df20cd98da2511235c782
https://github.com/qemu/qemu/commit/84bae5acff89134ea52df20cd98da2511235c782
Author: Wilfred Mallawa <wilfred.mallawa@wdc.com>
Date: 2022-12-19 (Mon, 19 Dec 2022)
Changed paths:
M include/hw/registerfields.h
Log Message:
-----------
hw/registerfields: add `FIELDx_1CLEAR()` macro
Adds a helper macro that implements the register `w1c`
functionality.
Ex:
uint32_t data = FIELD32_1CLEAR(val, REG, FIELD);
If ANY bits of the specified `FIELD` is set
then the respective field is cleared and returned to `data`.
If the field is cleared (0), then no change and
val is returned.
Signed-off-by: Wilfred Mallawa <wilfred.mallawa@wdc.com>
Reviewed-by: Alistair Francis <alistair.francis@wdc.com>
Message-Id: <20221017054950.317584-2-wilfred.mallawa@opensource.wdc.com>
Signed-off-by: Alistair Francis <alistair.francis@wdc.com>
Commit: aa2014b9fe28cfa38990f29b4cb6708e3b3ce3f5
https://github.com/qemu/qemu/commit/aa2014b9fe28cfa38990f29b4cb6708e3b3ce3f5
Author: Wilfred Mallawa <wilfred.mallawa@wdc.com>
Date: 2022-12-19 (Mon, 19 Dec 2022)
Changed paths:
M hw/ssi/ibex_spi_host.c
Log Message:
-----------
hw/ssi/ibex_spi: implement `FIELD32_1CLEAR` macro
use the `FIELD32_1CLEAR` macro to implement register
`rw1c` functionality to `ibex_spi`.
This change was tested by running the `SPI_HOST` from TockOS.
Signed-off-by: Wilfred Mallawa <wilfred.mallawa@wdc.com>
Reviewed-by: Alistair Francis <alistair.francis@wdc.com>
Message-Id: <20221017054950.317584-3-wilfred.mallawa@opensource.wdc.com>
Signed-off-by: Alistair Francis <alistair.francis@wdc.com>
Commit: 1e4c859fb8f450ca96e4ddcc20abe9bbc92b95a9
https://github.com/qemu/qemu/commit/1e4c859fb8f450ca96e4ddcc20abe9bbc92b95a9
Author: Richard Henderson <richard.henderson@linaro.org>
Date: 2022-12-19 (Mon, 19 Dec 2022)
Changed paths:
M tcg/riscv/tcg-target.c.inc
Log Message:
-----------
tcg/riscv: Fix range matched by TCG_CT_CONST_M12
We were matching a signed 13-bit range, not a 12-bit range.
Expand the commentary within the function and be explicit
about all of the ranges.
Reported-by: LIU Zhiwei <zhiwei_liu@linux.alibaba.com>
Signed-off-by: Richard Henderson <richard.henderson@linaro.org>
Reviewed-by: LIU Zhiwei <zhiwei_liu@linux.alibaba.com>
Reviewed-by: Alistair Francis <alistair.francis@wdc.com>
Message-Id: <20221022095821.2441874-1-richard.henderson@linaro.org>
Signed-off-by: Alistair Francis <alistair.francis@wdc.com>
Commit: 5183fe57a7c9181339c36e5c2e754cf89d6cd8ec
https://github.com/qemu/qemu/commit/5183fe57a7c9181339c36e5c2e754cf89d6cd8ec
Author: Richard Henderson <richard.henderson@linaro.org>
Date: 2022-12-19 (Mon, 19 Dec 2022)
Changed paths:
M tcg/riscv/tcg-target.c.inc
Log Message:
-----------
tcg/riscv: Fix reg overlap case in tcg_out_addsub2
There was a typo using opc_addi instead of opc_add with the
two registers. While we're at it, simplify the gating test
to al == bl to improve dynamic scheduling even when the
output register does not overlap the inputs.
Reported-by: LIU Zhiwei <zhiwei_liu@linux.alibaba.com>
Signed-off-by: Richard Henderson <richard.henderson@linaro.org>
Reviewed-by: Alistair Francis <alistair.francis@wdc.com>
Message-Id: <20221020233836.2341671-1-richard.henderson@linaro.org>
Signed-off-by: Alistair Francis <alistair.francis@wdc.com>
Commit: f9fb5aa543064b6d0531a23e15f5d652a49e90a7
https://github.com/qemu/qemu/commit/f9fb5aa543064b6d0531a23e15f5d652a49e90a7
Author: Richard Henderson <richard.henderson@linaro.org>
Date: 2022-12-19 (Mon, 19 Dec 2022)
Changed paths:
M tcg/riscv/tcg-target.c.inc
Log Message:
-----------
tcg/riscv: Fix base register for user-only qemu_ld/st
When guest_base != 0, we were not coordinating the usage of
TCG_REG_TMP0 as base properly, leading to a previous zero-extend
of the input address being discarded.
Shuffle the alignment check to the front, because that does not
depend on the zero-extend, and it keeps the register usage clear.
Set base after each step of the address arithmetic instead of before.
Return the base register used from tcg_out_tlb_load, so as to
keep that register choice localized to that function.
Reported-by: LIU Zhiwei <zhiwei_liu@linux.alibaba.com>
Signed-off-by: Richard Henderson <richard.henderson@linaro.org>
Reviewed-by: LIU Zhiwei <zhiwei_liu@linux.alibaba.com>
Reviewed-by: Alistair Francis <alistair.francis@wdc.com>
Message-Id: <20221023233337.2846860-1-richard.henderson@linaro.org>
Signed-off-by: Alistair Francis <alistair.francis@wdc.com>
Commit: a46dc52fa788ad7240148fd8beade86a5ba05811
https://github.com/qemu/qemu/commit/a46dc52fa788ad7240148fd8beade86a5ba05811
Author: Wilfred Mallawa <wilfred.mallawa@wdc.com>
Date: 2022-12-19 (Mon, 19 Dec 2022)
Changed paths:
M hw/riscv/opentitan.c
M include/hw/riscv/opentitan.h
Log Message:
-----------
hw/riscv/opentitan: bump opentitan
This patch updates the OpenTitan model to match
the specified register layout as per [1]. Which is also the latest
commit of OpenTitan supported by TockOS.
Note: Pinmux and Padctrl has been merged into Pinmux [2][3], this patch removes
any references to Padctrl. Note: OpenTitan doc [2] has not yet specified
much detail regarding this, except for a note that states `TODO: this
section needs to be updated to reflect the pinmux/padctrl merger`
[1]
https://github.com/lowRISC/opentitan/blob/d072ac505f82152678d6e04be95c72b728a347b8/hw/top_earlgrey/sw/autogen/top_earlgrey_memory.h
[2] https://docs.opentitan.org/hw/top_earlgrey/doc/design/
[3] https://docs.opentitan.org/hw/ip/pinmux/doc/#overview
Signed-off-by: Wilfred Mallawa <wilfred.mallawa@wdc.com>
Reviewed-by: Alistair Francis <alistair.francis@wdc.com>
Reviewed-by: Bin Meng <bmeng.cn@gmail.com>
Message-Id: <20221025043335.339815-2-wilfred.mallawa@opensource.wdc.com>
Signed-off-by: Alistair Francis <alistair.francis@wdc.com>
Commit: 5cb1257c2381e059789ba36f7662de38d591e8b2
https://github.com/qemu/qemu/commit/5cb1257c2381e059789ba36f7662de38d591e8b2
Author: Wilfred Mallawa <wilfred.mallawa@wdc.com>
Date: 2022-12-19 (Mon, 19 Dec 2022)
Changed paths:
M hw/riscv/opentitan.c
M include/hw/riscv/opentitan.h
Log Message:
-----------
hw/riscv/opentitan: add aon_timer base unimpl
Adds the updated `aon_timer` base as an unimplemented device. This is
used by TockOS, patch ensures the guest doesn't hit load faults.
Signed-off-by: Wilfred Mallawa <wilfred.mallawa@wdc.com>
Reviewed-by: Bin Meng <bmeng.cn@gmail.com>
Reviewed-by: Alistair Francis <alistair.francis@wdc.com>
Message-Id: <20221025043335.339815-3-wilfred.mallawa@opensource.wdc.com>
Signed-off-by: Alistair Francis <alistair.francis@wdc.com>
Commit: 8ccb1fcaf8f4ebc3c61cd2caec331527f8702b08
https://github.com/qemu/qemu/commit/8ccb1fcaf8f4ebc3c61cd2caec331527f8702b08
Author: Mayuresh Chitale <mchitale@ventanamicro.com>
Date: 2022-12-19 (Mon, 19 Dec 2022)
Changed paths:
M target/riscv/cpu.h
M target/riscv/cpu_bits.h
M target/riscv/csr.c
M target/riscv/machine.c
Log Message:
-----------
target/riscv: Add smstateen support
Smstateen extension specifies a mechanism to close
the potential covert channels that could cause security issues.
This patch adds the CSRs defined in the specification and
the corresponding predicates and read/write functions.
Signed-off-by: Mayuresh Chitale <mchitale@ventanamicro.com>
Reviewed-by: Weiwei Li <liweiwei@iscas.ac.cn>
Reviewed-by: Alistair Francis <alistair.francis@wdc.com>
Message-Id: <20221016124726.102129-2-mchitale@ventanamicro.com>
Signed-off-by: Alistair Francis <alistair.francis@wdc.com>
Commit: e09de58f9f2e2c3fd03aa88ef5a8b0ccdc9652d4
https://github.com/qemu/qemu/commit/e09de58f9f2e2c3fd03aa88ef5a8b0ccdc9652d4
Author: Mayuresh Chitale <mchitale@ventanamicro.com>
Date: 2022-12-19 (Mon, 19 Dec 2022)
Changed paths:
M target/riscv/csr.c
Log Message:
-----------
target/riscv: smstateen check for h/s/envcfg
Accesses to henvcfg, henvcfgh and senvcfg are allowed only if the corresponding
bit in mstateen0/hstateen0 is enabled. Otherwise an illegal instruction trap is
generated.
Signed-off-by: Mayuresh Chitale <mchitale@ventanamicro.com>
Reviewed-by: Weiwei Li<liweiwei@iscas.ac.cn>
Reviewed-by: Alistair Francis <alistair.francis@wdc.com>
Message-Id: <20221016124726.102129-3-mchitale@ventanamicro.com>
Signed-off-by: Alistair Francis <alistair.francis@wdc.com>
Commit: cf237a1f7e887141a5f152e133ef591bbafa4e1a
https://github.com/qemu/qemu/commit/cf237a1f7e887141a5f152e133ef591bbafa4e1a
Author: Mayuresh Chitale <mchitale@ventanamicro.com>
Date: 2022-12-19 (Mon, 19 Dec 2022)
Changed paths:
M target/riscv/translate.c
Log Message:
-----------
target/riscv: generate virtual instruction exception
This patch adds a mechanism to generate a virtual instruction
instruction exception instead of an illegal instruction exception
during instruction decode when virt is enabled.
Signed-off-by: Mayuresh Chitale <mchitale@ventanamicro.com>
Reviewed-by: Weiwei Li <liweiwei@iscas.ac.cn>
Reviewed-by: Alistair Francis <alistair.francis@wdc.com>
Message-Id: <20221016124726.102129-4-mchitale@ventanamicro.com>
Signed-off-by: Alistair Francis <alistair.francis@wdc.com>
Commit: 360860c1655e3a8df13d82f8705e48e069a4fb6a
https://github.com/qemu/qemu/commit/360860c1655e3a8df13d82f8705e48e069a4fb6a
Author: LIU Zhiwei <zhiwei_liu@linux.alibaba.com>
Date: 2022-12-19 (Mon, 19 Dec 2022)
Changed paths:
M target/riscv/cpu.h
M target/riscv/cpu_helper.c
M target/riscv/debug.c
M target/riscv/debug.h
M target/riscv/helper.h
M target/riscv/insn_trans/trans_privileged.c.inc
M target/riscv/insn_trans/trans_rvi.c.inc
M target/riscv/insn_trans/trans_rvv.c.inc
M target/riscv/translate.c
Log Message:
-----------
target/riscv: Add itrigger support when icount is not enabled
When icount is not enabled, there is no API in QEMU that can get the
guest instruction number.
Translate the guest code in a way that each TB only has one instruction.
After executing the instruction, decrease the count by 1 until it reaches 0
where the itrigger fires.
Note that only when priviledge matches the itrigger configuration,
the count will decrease.
Signed-off-by: LIU Zhiwei <zhiwei_liu@linux.alibaba.com>
Reviewed-by: Alistair Francis <alistair.francis@wdc.com>
Message-Id: <20221013062946.7530-2-zhiwei_liu@linux.alibaba.com>
Signed-off-by: Alistair Francis <alistair.francis@wdc.com>
Commit: d7ede82aef2fb158cfbc4f6647ff2bd60552abeb
https://github.com/qemu/qemu/commit/d7ede82aef2fb158cfbc4f6647ff2bd60552abeb
Author: LIU Zhiwei <zhiwei_liu@linux.alibaba.com>
Date: 2022-12-19 (Mon, 19 Dec 2022)
Changed paths:
M target/riscv/cpu.h
M target/riscv/cpu_helper.c
M target/riscv/debug.c
M target/riscv/debug.h
Log Message:
-----------
target/riscv: Add itrigger support when icount is enabled
The max count in itrigger can be 0x3FFF, which will cause a no trivial
translation and execution overload.
When icount is enabled, QEMU provides API that can fetch guest
instruction number. Thus, we can set an timer for itrigger with
the count as deadline.
Only when timer expires or priviledge mode changes, do lazy update
to count.
Signed-off-by: LIU Zhiwei <zhiwei_liu@linux.alibaba.com>
Reviewed-by: Alistair Francis <alistair.francis@wdc.com>
Message-Id: <20221013062946.7530-3-zhiwei_liu@linux.alibaba.com>
Signed-off-by: Alistair Francis <alistair.francis@wdc.com>
Commit: 60f68c36b4892b04fdbb7a5fd012d84f0e485bac
https://github.com/qemu/qemu/commit/60f68c36b4892b04fdbb7a5fd012d84f0e485bac
Author: LIU Zhiwei <zhiwei_liu@linux.alibaba.com>
Date: 2022-12-19 (Mon, 19 Dec 2022)
Changed paths:
M target/riscv/debug.c
Log Message:
-----------
target/riscv: Enable native debug itrigger
When QEMU is not in icount mode, execute instruction one by one. The
tdata1 can be read directly.
When QEMU is in icount mode, use a timer to simulate the itrigger. The
tdata1 may be not right because of lazy update of count in tdata1. Thus,
We should pack the adjusted count into tdata1 before read it back.
Signed-off-by: LIU Zhiwei <zhiwei_liu@linux.alibaba.com>
Reviewed-by: Alistair Francis <alistair.francis@wdc.com>
Message-Id: <20221013062946.7530-4-zhiwei_liu@linux.alibaba.com>
Signed-off-by: Alistair Francis <alistair.francis@wdc.com>
Commit: edc94d71aaf4b646081dd2b90a368df767d87b3e
https://github.com/qemu/qemu/commit/edc94d71aaf4b646081dd2b90a368df767d87b3e
Author: LIU Zhiwei <zhiwei_liu@linux.alibaba.com>
Date: 2022-12-19 (Mon, 19 Dec 2022)
Changed paths:
M target/riscv/cpu.h
M target/riscv/cpu_helper.c
M target/riscv/debug.c
M target/riscv/machine.c
Log Message:
-----------
target/riscv: Add itrigger_enabled field to CPURISCVState
Avoid calling riscv_itrigger_enabled() when calculate the tbflags.
As the itrigger enable status can only be changed when write
tdata1, migration load or itrigger fire, update env->itrigger_enabled
at these places.
Signed-off-by: LIU Zhiwei <zhiwei_liu@linux.alibaba.com>
Reviewed-by: Alistair Francis <alistair.francis@wdc.com>
Message-Id: <20221013062946.7530-5-zhiwei_liu@linux.alibaba.com>
Signed-off-by: Alistair Francis <alistair.francis@wdc.com>
Commit: 77b140735f406a88a943d04a5e23c6920f59839a
https://github.com/qemu/qemu/commit/77b140735f406a88a943d04a5e23c6920f59839a
Author: Frédéric Pétrot <frederic.petrot@univ-grenoble-alpes.fr>
Date: 2022-12-19 (Mon, 19 Dec 2022)
Changed paths:
M hw/intc/sifive_plic.c
Log Message:
-----------
hw/intc: sifive_plic: Renumber the S irqs for numa support
Commit 40244040a7a changed the way the S irqs are numbered. This breaks when
using numa configuration, e.g.:
./qemu-system-riscv64 -nographic -machine virt,dumpdtb=numa-tree.dtb \
-m 2G -smp cpus=16 \
-object memory-backend-ram,id=mem0,size=512M \
-object memory-backend-ram,id=mem1,size=512M \
-object memory-backend-ram,id=mem2,size=512M \
-object memory-backend-ram,id=mem3,size=512M \
-numa node,cpus=0-3,memdev=mem0,nodeid=0 \
-numa node,cpus=4-7,memdev=mem1,nodeid=1 \
-numa node,cpus=8-11,memdev=mem2,nodeid=2 \
-numa node,cpus=12-15,memdev=mem3,nodeid=3
leads to:
Unexpected error in object_property_find_err() at ../qom/object.c:1304:
qemu-system-riscv64: Property 'riscv.sifive.plic.unnamed-gpio-out[8]' not
found
This patch makes the nubering of the S irqs identical to what it was before.
Reviewed-by: Alistair Francis <alistair.francis@wdc.com>
Reviewed-by: Philippe Mathieu-Daudé <philmd@linaro.org>
Signed-off-by: Frédéric Pétrot <frederic.petrot@univ-grenoble-alpes.fr>
Message-Id: <20221114135122.1668703-1-frederic.petrot@univ-grenoble-alpes.fr>
Signed-off-by: Alistair Francis <alistair.francis@wdc.com>
Commit: 4d1f758b3eebb47784cbb94f12a7a2df0bd34264
https://github.com/qemu/qemu/commit/4d1f758b3eebb47784cbb94f12a7a2df0bd34264
Author: Anup Patel <apatel@ventanamicro.com>
Date: 2022-12-19 (Mon, 19 Dec 2022)
Changed paths:
M target/riscv/csr.c
Log Message:
-----------
target/riscv: Typo fix in sstc() predicate
We should use "&&" instead of "&" when checking hcounteren.TM and
henvcfg.STCE bits.
Fixes: 3ec0fe18a31f ("target/riscv: Add vstimecmp suppor")
Signed-off-by: Anup Patel <apatel@ventanamicro.com>
Reviewed-by: Alistair Francis <alistair.francis@wdc.com>
Message-Id: <20221108125703.1463577-2-apatel@ventanamicro.com>
Signed-off-by: Alistair Francis <alistair.francis@wdc.com>
Commit: 99439020b7f8e57fcc5e1f80a2f3adc62fbc1ff4
https://github.com/qemu/qemu/commit/99439020b7f8e57fcc5e1f80a2f3adc62fbc1ff4
Author: Atish Patra <atishp@rivosinc.com>
Date: 2022-12-19 (Mon, 19 Dec 2022)
Changed paths:
M hw/riscv/virt.c
M include/hw/riscv/virt.h
Log Message:
-----------
hw/riscv: virt: Remove the redundant ipi-id property
The imsic DT binding[1] has changed and no longer require an ipi-id.
The latest IMSIC driver dynamically allocates ipi id if slow-ipi
is not defined.
Get rid of the unused dt property which may lead to confusion.
[1]
https://lore.kernel.org/lkml/20221111044207.1478350-5-apatel@ventanamicro.com/
Signed-off-by: Atish Patra <atishp@rivosinc.com>
Reviewed-by: Alistair Francis <alistair.francis@wdc.com>
Message-Id: <20221122080529.1692533-1-atishp@rivosinc.com>
Signed-off-by: Alistair Francis <alistair.francis@wdc.com>
Commit: f57686d37eedde66b060cf275f668cc201bf5980
https://github.com/qemu/qemu/commit/f57686d37eedde66b060cf275f668cc201bf5980
Author: Jim Shu <jim.shu@sifive.com>
Date: 2022-12-19 (Mon, 19 Dec 2022)
Changed paths:
M target/riscv/cpu_helper.c
Log Message:
-----------
target/riscv: support cache-related PMU events in virtual mode
let tlb_fill() function also increments PMU counter when it is from
two-stage translation, so QEMU could also monitor these PMU events when
CPU runs in VS/VU mode (like running guest OS).
Signed-off-by: Jim Shu <jim.shu@sifive.com>
Reviewed-by: Alistair Francis <alistair.francis@wdc.com>
Message-Id: <20221123090635.6574-1-jim.shu@sifive.com>
Signed-off-by: Alistair Francis <alistair.francis@wdc.com>
Commit: 752e2599e4eb32a24f6e998e6210f045f56f7f97
https://github.com/qemu/qemu/commit/752e2599e4eb32a24f6e998e6210f045f56f7f97
Author: Bin Meng <bmeng@tinylab.org>
Date: 2022-12-19 (Mon, 19 Dec 2022)
Changed paths:
M target/riscv/cpu.c
Log Message:
-----------
target/riscv: Add some comments for sstatus CSR in riscv_cpu_dump_state()
sstatus register dump is currently missing in riscv_cpu_dump_state().
As sstatus is a copy of mstatus, which is described in the priv spec,
it seems redundant to print the same information twice.
Add some comments for this to let people know this is intentional.
Signed-off-by: Bin Meng <bmeng@tinylab.org>
Reviewed-by: Alistair Francis <alistair.francis@wdc.com>
Message-Id: <20221125050354.3166023-1-bmeng@tinylab.org>
Signed-off-by: Alistair Francis <alistair.francis@wdc.com>
Commit: 5da4d80248a9ab55578b0102a8236c43d9a03be1
https://github.com/qemu/qemu/commit/5da4d80248a9ab55578b0102a8236c43d9a03be1
Author: Conor Dooley <conor.dooley@microchip.com>
Date: 2022-12-19 (Mon, 19 Dec 2022)
Changed paths:
M hw/misc/mchp_pfsoc_ioscb.c
M include/hw/misc/mchp_pfsoc_ioscb.h
Log Message:
-----------
hw/misc: pfsoc: add fabric clocks to ioscb
On PolarFire SoC, some peripherals (eg the PCI root port) are clocked by
"Clock Conditioning Circuitry" in the FPGA. The specific clock depends
on the FPGA bitstream & can be locked to one particular {D,P}LL - in the
Icicle Kit Reference Design v2022.09 or later this is/will be the case.
Linux v6.1+ will have a driver for this peripheral and devicetrees that
previously relied on "fixed-frequency" clock nodes have been switched
over to clock-controller nodes. The IOSCB region is represented in QEMU,
but the specific region of it that the CCCs occupy has not so v6.1-rcN
kernels fail to boot in QEMU.
Add the regions as unimplemented so that the status-quo in terms of boot
is maintained.
Acked-by: Alistair Francis <alistair.francis@wdc.com>
Signed-off-by: Conor Dooley <conor.dooley@microchip.com>
Message-Id: <20221117225518.4102575-2-conor@kernel.org>
Signed-off-by: Alistair Francis <alistair.francis@wdc.com>
Commit: c1def1bc4bf91641c88d10649348a3866b0ce412
https://github.com/qemu/qemu/commit/c1def1bc4bf91641c88d10649348a3866b0ce412
Author: Conor Dooley <conor.dooley@microchip.com>
Date: 2022-12-19 (Mon, 19 Dec 2022)
Changed paths:
M hw/riscv/microchip_pfsoc.c
M include/hw/riscv/microchip_pfsoc.h
Log Message:
-----------
hw/riscv: pfsoc: add missing FICs as unimplemented
The Fabric Interconnect Controllers provide interfaces between the FPGA
fabric and the core complex. There are 5 FICs on PolarFire SoC, numbered
0 through 4. FIC2 is an AXI4 slave interface from the FPGA fabric and
does not show up on the MSS memory map. FIC4 is dedicated to the User
Crypto Processor and does not show up on the MSS memory map either.
FIC 0, 1 & 3 do show up in the MSS memory map and neither FICs 0 or 1
are represented in QEMU, leading to load access violations while booting
Linux for Icicle if PCIe is enabled as the root port is connected via
either FIC 0 or 1.
Acked-by: Alistair Francis <alistair.francis@wdc.com>
Signed-off-by: Conor Dooley <conor.dooley@microchip.com>
Message-Id: <20221117225518.4102575-3-conor@kernel.org>
Signed-off-by: Alistair Francis <alistair.francis@wdc.com>
Commit: 3969fc2dcfd977694005a15706249f6ea448f128
https://github.com/qemu/qemu/commit/3969fc2dcfd977694005a15706249f6ea448f128
Author: Conor Dooley <conor.dooley@microchip.com>
Date: 2022-12-19 (Mon, 19 Dec 2022)
Changed paths:
M hw/misc/mchp_pfsoc_ioscb.c
M hw/misc/mchp_pfsoc_sysreg.c
M hw/riscv/microchip_pfsoc.c
M include/hw/misc/mchp_pfsoc_ioscb.h
M include/hw/misc/mchp_pfsoc_sysreg.h
M include/hw/riscv/microchip_pfsoc.h
Log Message:
-----------
hw/{misc, riscv}: pfsoc: add system controller as unimplemented
The system controller on PolarFire SoC is access via a mailbox. The
control registers for this mailbox lie in the "IOSCB" region & the
interrupt is cleared via write to the "SYSREG" region. It also has a
QSPI controller, usually connected to a flash chip, that is used for
storing FPGA bitstreams and used for In-Application Programming (IAP).
Linux has an implementation of the system controller, through which the
hwrng is accessed, leading to load/store access faults.
Add the QSPI as unimplemented and a very basic (effectively
unimplemented) version of the system controller's mailbox. Rather than
purely marking the regions as unimplemented, service the mailbox
requests by reporting failures and raising the interrupt so a guest can
better handle the lack of support.
Signed-off-by: Conor Dooley <conor.dooley@microchip.com>
Acked-by: Alistair Francis <alistair.francis@wdc.com>
Message-Id: <20221117225518.4102575-4-conor@kernel.org>
Signed-off-by: Alistair Francis <alistair.francis@wdc.com>
Commit: 7a455c883e06588c48350bce9fea16cda05c4cb9
https://github.com/qemu/qemu/commit/7a455c883e06588c48350bce9fea16cda05c4cb9
Author: Jim Shu <jim.shu@sifive.com>
Date: 2022-12-19 (Mon, 19 Dec 2022)
Changed paths:
M hw/intc/sifive_plic.c
Log Message:
-----------
hw/intc: sifive_plic: fix out-of-bound access of source_priority array
If the number of interrupt is not multiple of 32, PLIC will have
out-of-bound access to source_priority array. Compute the number of
interrupt in the last word to avoid this out-of-bound access of array.
Signed-off-by: Jim Shu <jim.shu@sifive.com>
Reviewed-by: Bin Meng <bmeng@tinylab.org>
Message-Id: <20221127165753.30533-1-jim.shu@sifive.com>
Signed-off-by: Alistair Francis <alistair.francis@wdc.com>
Commit: c46d48721334c34f0f275cd09ad36bdb71b66e88
https://github.com/qemu/qemu/commit/c46d48721334c34f0f275cd09ad36bdb71b66e88
Author: Bin Meng <bmeng@tinylab.org>
Date: 2022-12-19 (Mon, 19 Dec 2022)
Changed paths:
M target/riscv/op_helper.c
Log Message:
-----------
target/riscv: Fix mret exception cause when no pmp rule is configured
The priv spec v1.12 says:
If no PMP entry matches an M-mode access, the access succeeds. If
no PMP entry matches an S-mode or U-mode access, but at least one
PMP entry is implemented, the access fails. Failed accesses generate
an instruction, load, or store access-fault exception.
At present the exception cause is set to 'illegal instruction' but
should have been 'instruction access fault'.
Fixes: d102f19a2085 ("target/riscv/pmp: Raise exception if no PMP entry is
configured")
Signed-off-by: Bin Meng <bmeng@tinylab.org>
Reviewed-by: Wilfred Mallawa <wilfred.mallawa@wdc.com>
Reviewed-by: Alistair Francis <alistair.francis@wdc.com>
Message-Id: <20221205065303.204095-1-bmeng@tinylab.org>
Signed-off-by: Alistair Francis <alistair.francis@wdc.com>
Commit: f1608b76626be63e3bad1d20ed78e6725d1ca1b1
https://github.com/qemu/qemu/commit/f1608b76626be63e3bad1d20ed78e6725d1ca1b1
Author: Richard Henderson <richard.henderson@linaro.org>
Date: 2022-12-19 (Mon, 19 Dec 2022)
Changed paths:
M target/riscv/translate.c
M tests/tcg/Makefile.target
M tests/tcg/riscv64/Makefile.target
A tests/tcg/riscv64/test-noc.S
Log Message:
-----------
target/riscv: Set pc_succ_insn for !rvc illegal insn
Failure to set pc_succ_insn may result in a TB covering zero bytes,
which triggers an assert within the code generator.
Cc: qemu-stable@nongnu.org
Resolves: https://gitlab.com/qemu-project/qemu/-/issues/1224
Signed-off-by: Richard Henderson <richard.henderson@linaro.org>
Reviewed-by: Alistair Francis <alistair.francis@wdc.com>
Reviewed-by: Philippe Mathieu-Daudé <philmd@linaro.org>
Message-Id: <20221203175744.151365-1-richard.henderson@linaro.org>
Signed-off-by: Alistair Francis <alistair.francis@wdc.com>
Commit: 0f81f8f81e0c15a9a0e89c9448f99d68a3c66819
https://github.com/qemu/qemu/commit/0f81f8f81e0c15a9a0e89c9448f99d68a3c66819
Author: Bin Meng <bmeng@tinylab.org>
Date: 2022-12-19 (Mon, 19 Dec 2022)
Changed paths:
M target/riscv/op_helper.c
Log Message:
-----------
target/riscv: Simplify helper_sret() a little bit
There are 2 paths in helper_sret() and the same mstatus update codes
are replicated. Extract the common parts to simplify it a little bit.
Signed-off-by: Bin Meng <bmeng@tinylab.org>
Reviewed-by: Alistair Francis <alistair.francis@wdc.com>
Message-Id: <20221207090037.281452-1-bmeng@tinylab.org>
Signed-off-by: Alistair Francis <alistair.francis@wdc.com>
Commit: 7fe1db4b09d7d5b1c71a0bc9901eaf9367ef733b
https://github.com/qemu/qemu/commit/7fe1db4b09d7d5b1c71a0bc9901eaf9367ef733b
Author: Bin Meng <bmeng@tinylab.org>
Date: 2022-12-19 (Mon, 19 Dec 2022)
Changed paths:
M target/riscv/op_helper.c
Log Message:
-----------
target/riscv: Clear mstatus.MPRV when leaving M-mode for priv spec 1.12+
Since priv spec v1.12, MRET and SRET now clear mstatus.MPRV when
leaving M-mode.
Signed-off-by: Bin Meng <bmeng@tinylab.org>
Reviewed-by: Alistair Francis <alistair.francis@wdc.com>
Message-Id: <20221207090037.281452-2-bmeng@tinylab.org>
Signed-off-by: Alistair Francis <alistair.francis@wdc.com>
Commit: f4b6a28b571209e29e47bebc797ef55b08a41d69
https://github.com/qemu/qemu/commit/f4b6a28b571209e29e47bebc797ef55b08a41d69
Author: Christoph Muellner <christoph.muellner@vrull.eu>
Date: 2022-12-19 (Mon, 19 Dec 2022)
Changed paths:
M target/riscv/cpu.c
M target/riscv/cpu.h
M target/riscv/insn32.decode
A target/riscv/insn_trans/trans_rvzawrs.c.inc
M target/riscv/translate.c
Log Message:
-----------
RISC-V: Add Zawrs ISA extension support
This patch adds support for the Zawrs ISA extension.
Given the current (incomplete) implementation of reservation sets
there seems to be no way to provide a full emulation of the WRS
instruction (wake on reservation set invalidation or timeout or
interrupt). Therefore, we just exit the TB and return to the main loop.
The specification can be found here:
https://github.com/riscv/riscv-zawrs/blob/main/zawrs.adoc
Note, that the Zawrs extension is frozen, but not ratified yet.
Changes since v3:
* Remove "RFC" since the extension is frozen
* Rebase on master and fix integration issues
* Fix entry ordering in extension list
Changes since v2:
* Rebase on master and resolve conflicts
* Adjustments according to a specification change
* Inline REQUIRE_ZAWRS() since it has only one user
Changes since v1:
* Adding zawrs to the ISA string that is passed to the kernel
Signed-off-by: Christoph Müllner <christoph.muellner@vrull.eu>
Reviewed-by: Alistair Francis <alistair.francis@wdc.com>
Message-Id: <20221005144948.3421504-1-christoph.muellner@vrull.eu>
Signed-off-by: Alistair Francis <alistair.francis@wdc.com>
Commit: 911eb12e44b9b3b516203ae4f2a9dd832c69bc2b
https://github.com/qemu/qemu/commit/911eb12e44b9b3b516203ae4f2a9dd832c69bc2b
Author: Bin Meng <bmeng@tinylab.org>
Date: 2022-12-19 (Mon, 19 Dec 2022)
Changed paths:
M hw/intc/Kconfig
M hw/riscv/Kconfig
Log Message:
-----------
hw/riscv: Select MSI_NONBROKEN in SIFIVE_PLIC
hw/pci/Kconfig says MSI_NONBROKEN should be selected by interrupt
controllers regardless of how MSI is implemented. msi_nonbroken is
initialized to true in sifive_plic_realize().
Let SIFIVE_PLIC select MSI_NONBROKEN and drop the selection from
RISC-V machines.
Signed-off-by: Bin Meng <bmeng@tinylab.org>
Reviewed-by: Alistair Francis <alistair.francis@wdc.com>
Reviewed-by: Philippe Mathieu-Daudé <philmd@linaro.org>
Reviewed-by: Wilfred Mallawa <wilfred.mallawa@wdc.com>
Message-Id: <20221211030829.802437-1-bmeng@tinylab.org>
Signed-off-by: Alistair Francis <alistair.francis@wdc.com>
Commit: dea41a1582acde7cdea253a35c913a52b55dddcc
https://github.com/qemu/qemu/commit/dea41a1582acde7cdea253a35c913a52b55dddcc
Author: Bin Meng <bmeng@tinylab.org>
Date: 2022-12-19 (Mon, 19 Dec 2022)
Changed paths:
M hw/intc/Kconfig
Log Message:
-----------
hw/intc: Select MSI_NONBROKEN in RISC-V AIA interrupt controllers
hw/pci/Kconfig says MSI_NONBROKEN should be selected by interrupt
controllers regardless of how MSI is implemented. msi_nonbroken is
initialized to true in both riscv_aplic_realize() and
riscv_imsic_realize().
Select MSI_NONBROKEN in RISCV_APLIC and RISCV_IMSIC.
Signed-off-by: Bin Meng <bmeng@tinylab.org>
Reviewed-by: Alistair Francis <alistair.francis@wdc.com>
Reviewed-by: Philippe Mathieu-Daudé <philmd@linaro.org>
Message-Id: <20221211030829.802437-2-bmeng@tinylab.org>
Signed-off-by: Alistair Francis <alistair.francis@wdc.com>
Commit: 75fc2ecbf339aa42f9bf70a89d34ace5ac9380bc
https://github.com/qemu/qemu/commit/75fc2ecbf339aa42f9bf70a89d34ace5ac9380bc
Author: Bin Meng <bmeng@tinylab.org>
Date: 2022-12-19 (Mon, 19 Dec 2022)
Changed paths:
M hw/riscv/Kconfig
Log Message:
-----------
hw/riscv: Fix opentitan dependency to SIFIVE_PLIC
Since commit ef6310064820 ("hw/riscv: opentitan: Update to the latest build")
the IBEX PLIC model was replaced with the SiFive PLIC model in the
'opentitan' machine but we forgot the add the dependency there.
Signed-off-by: Bin Meng <bmeng@tinylab.org>
Reviewed-by: Wilfred Mallawa <wilfred.mallawa@wdc.com>
Reviewed-by: Alistair Francis <alistair.francis@wdc.com>
Message-Id: <20221211030829.802437-3-bmeng@tinylab.org>
Signed-off-by: Alistair Francis <alistair.francis@wdc.com>
Commit: 5ef11917fa1eda1f46b320bdcd7f42f591b2cebf
https://github.com/qemu/qemu/commit/5ef11917fa1eda1f46b320bdcd7f42f591b2cebf
Author: Bin Meng <bmeng@tinylab.org>
Date: 2022-12-19 (Mon, 19 Dec 2022)
Changed paths:
M hw/riscv/Kconfig
Log Message:
-----------
hw/riscv: Sort machines Kconfig options in alphabetical order
SHAKTI_C machine Kconfig option was inserted in disorder. Fix it.
Signed-off-by: Bin Meng <bmeng@tinylab.org>
Reviewed-by: Alistair Francis <alistair.francis@wdc.com>
Reviewed-by: Philippe Mathieu-Daudé <philmd@linaro.org>
Reviewed-by: Wilfred Mallawa <wilfred.mallawa@wdc.com>
Message-Id: <20221211030829.802437-4-bmeng@tinylab.org>
Signed-off-by: Alistair Francis <alistair.francis@wdc.com>
Commit: 0b407d698c75729e53f78ed5b2b6d2b8cfdc4263
https://github.com/qemu/qemu/commit/0b407d698c75729e53f78ed5b2b6d2b8cfdc4263
Author: Bin Meng <bmeng@tinylab.org>
Date: 2022-12-19 (Mon, 19 Dec 2022)
Changed paths:
M hw/riscv/spike.c
Log Message:
-----------
hw/riscv: spike: Remove misleading comments
PLIC is not included in the 'spike' machine.
Signed-off-by: Bin Meng <bmeng@tinylab.org>
Reviewed-by: Wilfred Mallawa <wilfred.mallawa@wdc.com>
Reviewed-by: Alistair Francis <alistair.francis@wdc.com>
Message-Id: <20221211030829.802437-5-bmeng@tinylab.org>
Signed-off-by: Alistair Francis <alistair.francis@wdc.com>
Commit: 45072e4706f2888579f91f95b54ce0a7dc70c8b6
https://github.com/qemu/qemu/commit/45072e4706f2888579f91f95b54ce0a7dc70c8b6
Author: Bin Meng <bmeng@tinylab.org>
Date: 2022-12-19 (Mon, 19 Dec 2022)
Changed paths:
M hw/intc/sifive_plic.c
M include/hw/intc/sifive_plic.h
Log Message:
-----------
hw/intc: sifive_plic: Drop PLICMode_H
H-mode has been removed since priv spec 1.10. Drop it.
Signed-off-by: Bin Meng <bmeng@tinylab.org>
Reviewed-by: Wilfred Mallawa <wilfred.mallawa@wdc.com>
Reviewed-by: Alistair Francis <alistair.francis@wdc.com>
Message-Id: <20221211030829.802437-6-bmeng@tinylab.org>
Signed-off-by: Alistair Francis <alistair.francis@wdc.com>
Commit: 94877809fecdf25eb954fca5d74e0c4f4506496f
https://github.com/qemu/qemu/commit/94877809fecdf25eb954fca5d74e0c4f4506496f
Author: Bin Meng <bmeng@tinylab.org>
Date: 2022-12-19 (Mon, 19 Dec 2022)
Changed paths:
M hw/intc/sifive_plic.c
Log Message:
-----------
hw/intc: sifive_plic: Improve robustness of the PLIC config parser
At present the PLIC config parser can only handle legal config string
like "MS,MS". However if a config string like ",MS,MS,,MS,MS,," is
given the parser won't get the correct configuration.
This commit improves the config parser to make it more robust.
Signed-off-by: Bin Meng <bmeng@tinylab.org>
Acked-by: Alistair Francis <alistair.francis@wdc.com>
Message-Id: <20221211030829.802437-7-bmeng@tinylab.org>
Signed-off-by: Alistair Francis <alistair.francis@wdc.com>
Commit: 824451bca4b7601731e8893e13042e2099ae618e
https://github.com/qemu/qemu/commit/824451bca4b7601731e8893e13042e2099ae618e
Author: Bin Meng <bmeng@tinylab.org>
Date: 2022-12-19 (Mon, 19 Dec 2022)
Changed paths:
M hw/intc/sifive_plic.c
Log Message:
-----------
hw/intc: sifive_plic: Use error_setg() to propagate the error up via errp in
sifive_plic_realize()
The realize() callback has an errp for us to propagate the error up.
While we are here, correct the wrong multi-line comment format.
Signed-off-by: Bin Meng <bmeng@tinylab.org>
Reviewed-by: Alistair Francis <alistair.francis@wdc.com>
Reviewed-by: Philippe Mathieu-Daudé <philmd@linaro.org>
Message-Id: <20221211030829.802437-8-bmeng@tinylab.org>
Signed-off-by: Alistair Francis <alistair.francis@wdc.com>
Commit: 5451fbe416b366176a01b1d55e02b70817d6331c
https://github.com/qemu/qemu/commit/5451fbe416b366176a01b1d55e02b70817d6331c
Author: Bin Meng <bmeng@tinylab.org>
Date: 2022-12-19 (Mon, 19 Dec 2022)
Changed paths:
M hw/intc/sifive_plic.c
Log Message:
-----------
hw/intc: sifive_plic: Update "num-sources" property default value
At present the default value of "num-sources" property is zero,
which does not make a lot of sense, as in sifive_plic_realize()
we see s->bitfield_words is calculated by:
s->bitfield_words = (s->num_sources + 31) >> 5;
if the we don't configure "num-sources" property its default value
zero makes s->bitfield_words zero too, which isn't true because
interrupt source 0 still occupies one word.
Let's change the default value to 1 meaning that only interrupt
source 0 is supported by default and a sanity check in realize().
While we are here, add a comment to describe the exact meaning of
this property that the number should include interrupt source 0.
Signed-off-by: Bin Meng <bmeng@tinylab.org>
Reviewed-by: Alistair Francis <alistair.francis@wdc.com>
Message-Id: <20221211030829.802437-9-bmeng@tinylab.org>
Signed-off-by: Alistair Francis <alistair.francis@wdc.com>
Commit: e143cee00d8f353558b83b649887f4b18d38ebc9
https://github.com/qemu/qemu/commit/e143cee00d8f353558b83b649887f4b18d38ebc9
Author: Bin Meng <bmeng@tinylab.org>
Date: 2022-12-19 (Mon, 19 Dec 2022)
Changed paths:
M include/hw/riscv/microchip_pfsoc.h
Log Message:
-----------
hw/riscv: microchip_pfsoc: Fix the number of interrupt sources of PLIC
Per chapter 6.5.2 in [1], the number of interupt sources including
interrupt source 0 should be 187.
[1] PolarFire SoC MSS TRM:
https://ww1.microchip.com/downloads/aemDocuments/documents/FPGA/ProductDocuments/ReferenceManuals/PolarFire_SoC_FPGA_MSS_Technical_Reference_Manual_VC.pdf
Fixes: 56f6e31e7b7e ("hw/riscv: Initial support for Microchip PolarFire SoC
Icicle Kit board")
Signed-off-by: Bin Meng <bmeng@tinylab.org>
Reviewed-by: Wilfred Mallawa <wilfred.mallawa@wdc.com>
Reviewed-by: Alistair Francis <alistair.francis@wdc.com>
Reviewed-by: Conor Dooley <conor.dooley@microchip.com>
Message-Id: <20221211030829.802437-10-bmeng@tinylab.org>
Signed-off-by: Alistair Francis <alistair.francis@wdc.com>
Commit: 76920cd62ce530a763d4eb61c33e9b456ee6b599
https://github.com/qemu/qemu/commit/76920cd62ce530a763d4eb61c33e9b456ee6b599
Author: Bin Meng <bmeng@tinylab.org>
Date: 2022-12-19 (Mon, 19 Dec 2022)
Changed paths:
M include/hw/riscv/sifive_e.h
Log Message:
-----------
hw/riscv: sifive_e: Fix the number of interrupt sources of PLIC
Per chapter 10 in Freedom E310 manuals [1][2][3], E310 G002 and G003
supports 52 interrupt sources while G000 supports 51 interrupt sources.
We use the value of G002 and G003, so it is 53 (including source 0).
[1] G000 manual:
https://sifive.cdn.prismic.io/sifive/4faf3e34-4a42-4c2f-be9e-c77baa4928c7_fe310-g000-manual-v3p2.pdf
[2] G002 manual:
https://sifive.cdn.prismic.io/sifive/034760b5-ac6a-4b1c-911c-f4148bb2c4a5_fe310-g002-v1p5.pdf
[3] G003 manual:
https://sifive.cdn.prismic.io/sifive/3af39c59-6498-471e-9dab-5355a0d539eb_fe310-g003-manual.pdf
Fixes: eb637edb1241 ("SiFive Freedom E Series RISC-V Machine")
Signed-off-by: Bin Meng <bmeng@tinylab.org>
Reviewed-by: Wilfred Mallawa <wilfred.mallawa@wdc.com>
Reviewed-by: Alistair Francis <alistair.francis@wdc.com>
Message-Id: <20221211030829.802437-11-bmeng@tinylab.org>
Signed-off-by: Alistair Francis <alistair.francis@wdc.com>
Commit: ba26bc14018ff3e997a455f6e8ef683c2d57250b
https://github.com/qemu/qemu/commit/ba26bc14018ff3e997a455f6e8ef683c2d57250b
Author: Bin Meng <bmeng@tinylab.org>
Date: 2022-12-19 (Mon, 19 Dec 2022)
Changed paths:
M hw/riscv/sifive_u.c
Log Message:
-----------
hw/riscv: sifive_u: Avoid using magic number for "riscv, ndev"
At present magic number is used to create "riscv,ndev" property
in the dtb. Let's use the macro SIFIVE_U_PLIC_NUM_SOURCES that
is used to instantiate the PLIC model instead.
Signed-off-by: Bin Meng <bmeng@tinylab.org>
Reviewed-by: Wilfred Mallawa <wilfred.mallawa@wdc.com>
Reviewed-by: Alistair Francis <alistair.francis@wdc.com>
Message-Id: <20221211030829.802437-12-bmeng@tinylab.org>
Signed-off-by: Alistair Francis <alistair.francis@wdc.com>
Commit: 831b90e348562320cb0804856c6cda9d4d12f631
https://github.com/qemu/qemu/commit/831b90e348562320cb0804856c6cda9d4d12f631
Author: Bin Meng <bmeng@tinylab.org>
Date: 2022-12-19 (Mon, 19 Dec 2022)
Changed paths:
M hw/riscv/virt.c
M include/hw/riscv/virt.h
Log Message:
-----------
hw/riscv: virt: Fix the value of "riscv, ndev" in the dtb
Commit 28d8c281200f ("hw/riscv: virt: Add optional AIA IMSIC support to virt
machine")
changed the value of VIRT_IRQCHIP_NUM_SOURCES from 127 to 53, which
is VIRTIO_NDEV and also used as the value of "riscv,ndev" property
in the dtb. Unfortunately this is wrong as VIRT_IRQCHIP_NUM_SOURCES
should include interrupt source 0 but "riscv,ndev" does not.
While we are here, we also fix the comments of platform bus irq range
which is now "64 to 96", but should be "64 to 95", introduced since
commit 1832b7cb3f64 ("hw/riscv: virt: Create a platform bus").
Fixes: 28d8c281200f ("hw/riscv: virt: Add optional AIA IMSIC support to virt
machine")
Signed-off-by: Bin Meng <bmeng@tinylab.org>
Reviewed-by: Alistair Francis <alistair.francis@wdc.com>
Message-Id: <20221211030829.802437-13-bmeng@tinylab.org>
Signed-off-by: Alistair Francis <alistair.francis@wdc.com>
Commit: 41a66488577885bf2685c6c09e5d4da105ff7219
https://github.com/qemu/qemu/commit/41a66488577885bf2685c6c09e5d4da105ff7219
Author: Bin Meng <bmeng@tinylab.org>
Date: 2022-12-19 (Mon, 19 Dec 2022)
Changed paths:
M hw/intc/sifive_plic.c
M include/hw/riscv/microchip_pfsoc.h
M include/hw/riscv/shakti_c.h
M include/hw/riscv/sifive_e.h
M include/hw/riscv/sifive_u.h
M include/hw/riscv/virt.h
Log Message:
-----------
hw/intc: sifive_plic: Change "priority-base" to start from interrupt source 0
At present the SiFive PLIC model "priority-base" expects interrupt
priority register base starting from source 1 instead source 0,
that's why on most platforms "priority-base" is set to 0x04 except
'opentitan' machine. 'opentitan' should have set "priority-base"
to 0x04 too.
Note the irq number calculation in sifive_plic_{read,write} is
correct as the codes make up for the irq number by adding 1.
Let's simply update "priority-base" to start from interrupt source
0 and add a comment to make it crystal clear.
Signed-off-by: Bin Meng <bmeng@tinylab.org>
Reviewed-by: Alistair Francis <alistair.francis@wdc.com>
Reviewed-by: Wilfred Mallawa <wilfred.mallawa@wdc.com>
Message-Id: <20221211030829.802437-14-bmeng@tinylab.org>
Signed-off-by: Alistair Francis <alistair.francis@wdc.com>
Commit: f64fbfe567dc981ffb89ba99365532ee819e9d70
https://github.com/qemu/qemu/commit/f64fbfe567dc981ffb89ba99365532ee819e9d70
Author: Bin Meng <bmeng@tinylab.org>
Date: 2022-12-19 (Mon, 19 Dec 2022)
Changed paths:
M hw/riscv/opentitan.c
Log Message:
-----------
hw/riscv: opentitan: Drop "hartid-base" and "priority-base" initialization
"hartid-base" and "priority-base" are zero by default. There is no
need to initialize them to zero again.
Signed-off-by: Bin Meng <bmeng@tinylab.org>
Reviewed-by: Wilfred Mallawa <wilfred.mallawa@wdc.com>
Reviewed-by: Alistair Francis <alistair.francis@wdc.com>
Message-Id: <20221211030829.802437-15-bmeng@tinylab.org>
Signed-off-by: Alistair Francis <alistair.francis@wdc.com>
Commit: e59b3c6ece6a1351aeca6b916cd9674e23d15e89
https://github.com/qemu/qemu/commit/e59b3c6ece6a1351aeca6b916cd9674e23d15e89
Author: Bin Meng <bmeng@tinylab.org>
Date: 2022-12-19 (Mon, 19 Dec 2022)
Changed paths:
M hw/intc/sifive_plic.c
Log Message:
-----------
hw/intc: sifive_plic: Fix the pending register range check
The pending register upper limit is currently set to
plic->num_sources >> 3, which is wrong, e.g.: considering
plic->num_sources is 7, the upper limit becomes 0 which fails
the range check if reading the pending register at pending_base.
Fixes: 1e24429e40df ("SiFive RISC-V PLIC Block")
Signed-off-by: Bin Meng <bmeng@tinylab.org>
Reviewed-by: Alistair Francis <alistair.francis@wdc.com>
Message-Id: <20221211030829.802437-16-bmeng@tinylab.org>
Signed-off-by: Alistair Francis <alistair.francis@wdc.com>
Commit: a064d584103214bec4478aa60466c43d491bfcdd
https://github.com/qemu/qemu/commit/a064d584103214bec4478aa60466c43d491bfcdd
Author: Peter Maydell <peter.maydell@linaro.org>
Date: 2022-12-20 (Tue, 20 Dec 2022)
Changed paths:
M hw/intc/Kconfig
M hw/intc/sifive_plic.c
M hw/misc/mchp_pfsoc_ioscb.c
M hw/misc/mchp_pfsoc_sysreg.c
M hw/riscv/Kconfig
M hw/riscv/microchip_pfsoc.c
M hw/riscv/opentitan.c
M hw/riscv/sifive_u.c
M hw/riscv/spike.c
M hw/riscv/virt.c
M hw/ssi/ibex_spi_host.c
M include/hw/intc/sifive_plic.h
M include/hw/misc/mchp_pfsoc_ioscb.h
M include/hw/misc/mchp_pfsoc_sysreg.h
M include/hw/registerfields.h
M include/hw/riscv/microchip_pfsoc.h
M include/hw/riscv/opentitan.h
M include/hw/riscv/shakti_c.h
M include/hw/riscv/sifive_e.h
M include/hw/riscv/sifive_u.h
M include/hw/riscv/virt.h
M target/riscv/cpu.c
M target/riscv/cpu.h
M target/riscv/cpu_bits.h
M target/riscv/cpu_helper.c
M target/riscv/csr.c
M target/riscv/debug.c
M target/riscv/debug.h
M target/riscv/helper.h
M target/riscv/insn32.decode
M target/riscv/insn_trans/trans_privileged.c.inc
M target/riscv/insn_trans/trans_rvi.c.inc
M target/riscv/insn_trans/trans_rvv.c.inc
A target/riscv/insn_trans/trans_rvzawrs.c.inc
M target/riscv/machine.c
M target/riscv/op_helper.c
M target/riscv/pmp.c
M target/riscv/pmp.h
M target/riscv/translate.c
M tcg/riscv/tcg-target.c.inc
M tests/tcg/Makefile.target
M tests/tcg/riscv64/Makefile.target
A tests/tcg/riscv64/test-noc.S
Log Message:
-----------
Merge tag 'pull-riscv-to-apply-20221219-3' of
https://github.com/alistair23/qemu into staging
First RISC-V PR for QEMU 8.0
* Fix PMP propagation for tlb
* Collection of bug fixes
* Add the `FIELDx_1CLEAR()` macro
* Bump the OpenTitan supported version
* Add smstateen support
* Support native debug icount trigger
* Remove the redundant ipi-id property in the virt machine
* Support cache-related PMU events in virtual mode
* Add some missing PolarFire SoC io regions
* Fix mret exception cause when no pmp rule is configured
* Fix bug where disabling compressed instructions would crash QEMU
* Add Zawrs ISA extension support
* A range of code refactoring and cleanups
# gpg: Signature made Mon 19 Dec 2022 02:16:00 GMT
# gpg: using RSA key F6C4AC46D4934868D3B8CE8F21E10D29DF977054
# gpg: Good signature from "Alistair Francis <alistair@alistair23.me>" [unknown]
# gpg: WARNING: This key is not certified with a trusted signature!
# gpg: There is no indication that the signature belongs to the owner.
# Primary key fingerprint: F6C4 AC46 D493 4868 D3B8 CE8F 21E1 0D29 DF97 7054
* tag 'pull-riscv-to-apply-20221219-3' of https://github.com/alistair23/qemu:
(45 commits)
hw/intc: sifive_plic: Fix the pending register range check
hw/riscv: opentitan: Drop "hartid-base" and "priority-base" initialization
hw/intc: sifive_plic: Change "priority-base" to start from interrupt source 0
hw/riscv: virt: Fix the value of "riscv, ndev" in the dtb
hw/riscv: sifive_u: Avoid using magic number for "riscv, ndev"
hw/riscv: sifive_e: Fix the number of interrupt sources of PLIC
hw/riscv: microchip_pfsoc: Fix the number of interrupt sources of PLIC
hw/intc: sifive_plic: Update "num-sources" property default value
hw/intc: sifive_plic: Use error_setg() to propagate the error up via errp in
sifive_plic_realize()
hw/intc: sifive_plic: Improve robustness of the PLIC config parser
hw/intc: sifive_plic: Drop PLICMode_H
hw/riscv: spike: Remove misleading comments
hw/riscv: Sort machines Kconfig options in alphabetical order
hw/riscv: Fix opentitan dependency to SIFIVE_PLIC
hw/intc: Select MSI_NONBROKEN in RISC-V AIA interrupt controllers
hw/riscv: Select MSI_NONBROKEN in SIFIVE_PLIC
RISC-V: Add Zawrs ISA extension support
target/riscv: Clear mstatus.MPRV when leaving M-mode for priv spec 1.12+
target/riscv: Simplify helper_sret() a little bit
target/riscv: Set pc_succ_insn for !rvc illegal insn
...
Signed-off-by: Peter Maydell <peter.maydell@linaro.org>
Compare: https://github.com/qemu/qemu/compare/33698d3abf8c...a064d5841032
[Prev in Thread] |
Current Thread |
[Next in Thread] |
- [Qemu-commits] [qemu/qemu] 35c702: target/riscv: Fix PMP propagation for tlb,
Peter Maydell <=