Re: [PATCH v3 8/8] arm/aspeed: Replace mx25l25635e chip model

2022-07-24 Thread Andrew Jeffery
On Mon, 25 Jul 2022, at 16:02, Cédric Le Goater wrote: > On 7/25/22 04:08, Andrew Jeffery wrote: >> >> >> On Fri, 22 Jul 2022, at 16:06, Cédric Le Goater wrote: >>> aspeed_machine_witherspoon_class_init(ObjectClass *oc, void *data) >>> mc->desc = "OpenPOWER Witherspoon BMC (ARM1176

Re: [PATCH v3 8/8] arm/aspeed: Replace mx25l25635e chip model

2022-07-24 Thread Cédric Le Goater
On 7/25/22 04:08, Andrew Jeffery wrote: On Fri, 22 Jul 2022, at 16:06, Cédric Le Goater wrote: A mx25l25635f chip model is generally found on these machines. It's newer and uses 4B opcodes which is better to exercise the support in the Linux kernel. Signed-off-by: Cédric Le Goater --- hw/a

Re: [PATCH v5 0/1] target/riscv: Add Zihintpause support

2022-07-24 Thread Alistair Francis
On Mon, Jul 25, 2022 at 1:48 PM Dao Lu wrote: > > This patch adds RISC-V Zihintpause support. The extension is set to be enabled > by default and opcode has been added to insn32.decode. > > Added trans_pause to exit the TB and return to main loop. > > The change can also be found in: > https://git

[PATCH v5 1/1] target/riscv: Add Zihintpause support

2022-07-24 Thread Dao Lu
Added support for RISC-V PAUSE instruction from Zihintpause extension, enabled by default. Tested-by: Heiko Stuebner Reviewed-by: Alistair Francis Signed-off-by: Dao Lu --- target/riscv/cpu.c | 2 ++ target/riscv/cpu.h | 1 + target/riscv/insn32.deco

[PATCH v5 0/1] target/riscv: Add Zihintpause support

2022-07-24 Thread Dao Lu
This patch adds RISC-V Zihintpause support. The extension is set to be enabled by default and opcode has been added to insn32.decode. Added trans_pause to exit the TB and return to main loop. The change can also be found in: https://github.com/dlu42/qemu/tree/zihintpause_support_v1 Tested along

Re: [PATCH 00/16] Support VIRTIO_F_RING_RESET for virtio-net and vhost-user in virtio pci

2022-07-24 Thread Jason Wang
On Mon, Jul 25, 2022 at 10:34 AM Kangjie Xu wrote: > > Do you have any comments or suggestions about improvements to this patch > set? Will have a look and give feedback no later than the end of this week. Thanks > > > Regards, > > Kangjie > > 在 2022/7/18 19:16, Kangjie Xu 写道: > > The virtio qu

Re: [RFC] hw/nvme: Use irqfd to send interrupts

2022-07-24 Thread Jinhao Fan
at 3:36 AM, Stefan Hajnoczi wrote: > > > On Sun, Jul 24, 2022, 11:21 Jinhao Fan wrote: > at 9:29 PM, Stefan Hajnoczi wrote: > > > > > Nice, perf(1) is good for that. You can enable trace events and add > > kprobes/uprobes to record timestamps when specific functions are entered. > > > > T

Re: [PATCH 00/16] Support VIRTIO_F_RING_RESET for virtio-net and vhost-user in virtio pci

2022-07-24 Thread Kangjie Xu
Do you have any comments or suggestions about improvements to this patch set? Regards, Kangjie 在 2022/7/18 19:16, Kangjie Xu 写道: The virtio queue reset function has already been defined in the virtio spec 1.2. The relevant virtio spec information is here: https://github.com/oasis-tcs/v

Re: [PATCH v3 8/8] arm/aspeed: Replace mx25l25635e chip model

2022-07-24 Thread Andrew Jeffery
On Fri, 22 Jul 2022, at 16:06, Cédric Le Goater wrote: > A mx25l25635f chip model is generally found on these machines. It's > newer and uses 4B opcodes which is better to exercise the support in > the Linux kernel. > > Signed-off-by: Cédric Le Goater > --- > hw/arm/aspeed.c | 6 +++--- > 1 fi

Re: [PATCH] .cirrus.yml: Change winsymlinks to 'native'

2022-07-24 Thread Yonggang Luo
On Mon, Jul 25, 2022 at 9:24 AM Bin Meng wrote: > > On Wed, Jul 20, 2022 at 12:12 AM Bin Meng wrote: > > > > From: Bin Meng > > > > At present winsymlinks is set to 'nativestrict', and its behavior is: > > > > a) if native symlinks are enabled and exists, creates > > as a native Windows

Re: [PATCH] .cirrus.yml: Change winsymlinks to 'native'

2022-07-24 Thread Bin Meng
On Wed, Jul 20, 2022 at 12:12 AM Bin Meng wrote: > > From: Bin Meng > > At present winsymlinks is set to 'nativestrict', and its behavior is: > > a) if native symlinks are enabled and exists, creates > as a native Windows symlink; > b) else if native symlinks are not enabled or if doe

Re: [PATCH] hw/intc: sifive_plic: Fix multi-socket plic configuraiton

2022-07-24 Thread Alistair Francis
On Sat, Jul 23, 2022 at 7:22 PM Atish Patra wrote: > > Since commit 40244040a7ac, multi-socket configuration with plic is > broken as the hartid for second socket is calculated incorrectly. > The hartid stored in addr_config already includes the offset > for the base hartid for that socket. Adding

Re: [PATCH v2 2/2] tests/tcg/s390x: Test unaligned accesses to lowcore

2022-07-24 Thread Richard Henderson
On 7/23/22 05:06, Ilya Leoshkevich wrote: Add a small test to avoid regressions. Signed-off-by: Ilya Leoshkevich Acked-by: Richard Henderson r~ --- tests/tcg/s390x/Makefile.softmmu-target | 9 + tests/tcg/s390x/unaligned-lowcore.S | 19 +++ 2 files chang

Re: [PATCH v2 1/2] qapi: Add exit-failure PanicAction

2022-07-24 Thread Richard Henderson
On 7/23/22 05:06, Ilya Leoshkevich wrote: Currently QEMU exits with code 0 on both panic an shutdown. For tests it is useful to return 1 on panic, so that it counts as a test failure. Introduce a new exit-failure PanicAction that makes main() return EXIT_FAILURE. Tests can use -action panic=exit

Re: [PATCH] target/ppc: fix unreachable code in do_ldst_quad()

2022-07-24 Thread Richard Henderson
On 7/20/22 19:27, Daniel Henrique Barboza wrote: Coverity reports that commit fc34e81acd51 ("target/ppc: add macros to check privilege level") turned the following code unreachable: if (!prefixed && !(ctx->insns_flags2 & PPC2_LSQ_ISA207)) { /* lq and stq were privileged prior to V. 2.07 */

Re: [PATCH 4/4] semihosting: Fix handling of buffer in TARGET_SYS_TMPNAM

2022-07-24 Thread Richard Henderson
On 7/19/22 17:41, Peter Maydell wrote: The TARGET_SYS_TMPNAM implementation has two bugs spotted by Coverity: * confusion about whether 'len' has the length of the string including or excluding the terminating NUL means we lock_user() len bytes of memory but memcpy() len + 1 bytes * I

Deadlock between bdrv_drain_all_begin and prepare_mmio_access

2022-07-24 Thread Liang Yan
Hello All, I am facing a lock situation between main-loop thread 1 and vcpu thread 4 when doing a qmp snapshot. QEMU is running on 6.0.x, checked the upstream code and did not see any big change since between. Guest is a Windows 10 VM. Unfortunately, I could not get into the windows vm or rep

Re: [RFC] hw/nvme: Use irqfd to send interrupts

2022-07-24 Thread Stefan Hajnoczi
On Sun, Jul 24, 2022, 11:21 Jinhao Fan wrote: > at 9:29 PM, Stefan Hajnoczi wrote: > > > > > Nice, perf(1) is good for that. You can enable trace events and add > > kprobes/uprobes to record timestamps when specific functions are entered. > > > > Thanks Stefan, > > One last question: Currently w

Re: [PATCH 3/4] semihosting: Check for errors on SET_ARG()

2022-07-24 Thread Richard Henderson
On 7/19/22 17:41, Peter Maydell wrote: The SET_ARG() macro returns an error indication; we check this in the TARGET_SYS_GET_CMDLINE case but not when we use it in implementing TARGET_SYS_ELAPSED. Check for and handle the errors via the do_fault codepath, and update the comment documenting the SE

Re: [PATCH 2/4] semihosting: Don't copy buffer after console_write()

2022-07-24 Thread Richard Henderson
On 7/19/22 17:41, Peter Maydell wrote: The console_write() semihosting function outputs guest data from a buffer; it doesn't update that buffer. It therefore doesn't need to pass a length value to unlock_user(), but can pass 0, meaning "do not copy any data back to the guest memory". Signed-off

Re: [PATCH 1/4] semihosting: Don't return negative values on qemu_semihosting_console_write() failure

2022-07-24 Thread Richard Henderson
On 7/19/22 17:41, Peter Maydell wrote: The documentation comment for qemu_semihosting_console_write() says * Returns: number of bytes written -- this should only ever be short * on some sort of i/o error. and the callsites rely on this. However, the implementation code path which sends cons

Re: [PATCH v6 3/5] target/riscv: smstateen check for fcsr

2022-07-24 Thread Mayuresh Chitale
On Fri, 2022-07-22 at 09:42 +0800, Weiwei Li wrote: > 在 2022/7/21 下午11:31, Mayuresh Chitale 写道: > > If smstateen is implemented and sstateen0.fcsr is clear then the > > floating point operations must return illegal instruction > > exception. > > > > Signed-off-by: Mayuresh Chitale > > --- > > t

Re: [PATCH v6 1/5] target/riscv: Add smstateen support

2022-07-24 Thread Mayuresh Chitale
On Fri, 2022-07-22 at 08:31 +0800, Weiwei Li wrote: > 在 2022/7/21 下午11:31, Mayuresh Chitale 写道: > > 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 corre

Re: [RFC] hw/nvme: Use irqfd to send interrupts

2022-07-24 Thread Jinhao Fan
at 9:29 PM, Stefan Hajnoczi wrote: > > Nice, perf(1) is good for that. You can enable trace events and add > kprobes/uprobes to record timestamps when specific functions are entered. > Thanks Stefan, One last question: Currently we can achieve hundreds of KIOPS. That means perf can easily cap

[PATCH v15 04/10] vhost: introduce new VhostOps vhost_set_config_call

2022-07-24 Thread Cindy Lu
This patch introduces new VhostOps vhost_set_config_call. This function allows the qemu to set the config event fd to kernel driver. Signed-off-by: Cindy Lu --- include/hw/virtio/vhost-backend.h | 3 +++ 1 file changed, 3 insertions(+) diff --git a/include/hw/virtio/vhost-backend.h b/include/h

[PATCH v15 10/10] virtio-pci: add support for configure interrupt

2022-07-24 Thread Cindy Lu
Add process to handle the configure interrupt, The function's logic is the same with vq interrupt.Add extra process to check the configure interrupt Signed-off-by: Cindy Lu --- hw/virtio/virtio-pci.c | 110 - include/hw/virtio/virtio-pci.h | 4 +- 2 file