On Tue, Sep 12, 2023 at 8:27 PM Michael Tokarev <m...@tls.msk.ru> wrote: > > 11.09.2023 09:42, Alistair Francis:> target/riscv: don't read CSR in > riscv_csrrw_do64 (2023-09-11 11:45:55 +1000) > 2 more questions about this pull-req and -stable. > > > commit 50f9464962fb41f04fd5f42e7ee2cb60942aba89 > Author: Daniel Henrique Barboza <dbarb...@ventanamicro.com> > Date: Thu Jul 20 10:24:23 2023 -0300 > > target/riscv/cpu.c: add zmmul isa string > > zmmul was promoted from experimental to ratified in commit 6d00ffad4e95. > Add a riscv,isa string for it. > > Fixes: 6d00ffad4e95 ("target/riscv: move zmmul out of the experimental > properties") > > Does this need to be picked for -stable (based on the "Fixes" tag)? > I don't know the full impact of this change (or lack thereof). > > > commit 4cc9f284d5971ecd8055d26ef74c23ef0be8b8f5 > Author: LIU Zhiwei <zhiwei_...@linux.alibaba.com> > Date: Sat Jul 29 11:16:18 2023 +0800 > > target/riscv: Fix page_check_range use in fault-only-first > > Commit bef6f008b98(accel/tcg: Return bool from page_check_range) converts > integer return value to bool type. However, it wrongly converted the use > of the API in riscv fault-only-first, where page_check_range < = 0, > should > be converted to !page_check_range. > > This one also catches an eye, the commit in question is in 8.1, and it is > a clear bugfix (from the patch anyway).
These two are also good candidates if it isn't too late. Alistair > > > I probably should stop making such questions and rely more on Cc: qemu-stable@ > instead. It just so happened that I had a closer look at this patchset/pullreq > while trying to cherry-pick already agreed-upon changes from there. > > So far, I picked the following changes for -stable from this pullreq: > > c255946e3d hw/char/riscv_htif: Fix printing of console characters on big > endian hosts > 058096f1c5 hw/char/riscv_htif: Fix the console syscall on big endian hosts > 50f9464962 target/riscv/cpu.c: add zmmul isa string > 4cc9f284d5 target/riscv: Fix page_check_range use in fault-only-first > eda633a534 target/riscv: Fix zfa fleq.d and fltq.d > e0922b73ba hw/intc: Fix upper/lower mtime write calculation > 9382a9eafc hw/intc: Make rtc variable names consistent > ae7d4d625c linux-user/riscv: Use abi type for target_ucontext > 9ff3140631 hw/riscv: virt: Fix riscv,pmu DT node path > 3a2fc23563 target/riscv: fix satp_mode_finalize() when satp_mode.supported = 0 > 4e3adce124 target/riscv/pmp.c: respect mseccfg.RLB for pmpaddrX changes > a7c272df82 target/riscv: Allocate itrigger timers only once > > Thanks, > > /mjt