[Qemu-devel] [PATCH v13 08/12] hw/rx: RX Target hardware definition

2019-05-15 Thread Yoshinori Sato
rx62n - RX62N cpu. rx-virt - RX QEMU virtual target. Signed-off-by: Yoshinori Sato Tested-by: Philippe Mathieu-Daudé Reviewed-by: Philippe Mathieu-Daudé --- include/hw/rx/rx.h| 7 ++ include/hw/rx/rx62n.h | 94 hw/rx/rx-virt.c | 105 ++

[Qemu-devel] [PATCH v13 02/12] target/rx: TCG helper

2019-05-15 Thread Yoshinori Sato
Signed-off-by: Yoshinori Sato Reviewed-by: Richard Henderson --- target/rx/helper.h| 31 target/rx/helper.c| 148 target/rx/op_helper.c | 481 ++ 3 files changed, 660 insertions(+) create mode 100644

[Qemu-devel] [PATCH v13 05/12] hw/intc: RX62N interrupt controller (ICUa)

2019-05-15 Thread Yoshinori Sato
This implementation supported only ICUa. Hardware manual. https://www.renesas.com/us/en/doc/products/mpumcu/doc/rx_family/r01uh0033ej0140_rx62n.pdf Signed-off-by: Yoshinori Sato Reviewed-by: Alex Bennée --- include/hw/intc/rx_icu.h | 57 +++ hw/intc/rx_icu.c | 376

[Qemu-devel] [PATCH v13 11/12] qemu/bitops.h: Add extract8 and extract16

2019-05-15 Thread Yoshinori Sato
Signed-off-by: Yoshinori Sato Reviewed-by: Richard Henderson --- include/qemu/bitops.h | 38 ++ 1 file changed, 38 insertions(+) diff --git a/include/qemu/bitops.h b/include/qemu/bitops.h index 3f0926cf40..764f9d1ea0 100644 --- a/include/qemu/bitops.h +++

[Qemu-devel] [PATCH v13 04/12] target/rx: RX disassembler

2019-05-15 Thread Yoshinori Sato
Signed-off-by: Yoshinori Sato Reviewed-by: Richard Henderson Tested-by: Philippe Mathieu-Daudé --- include/disas/dis-asm.h |5 + target/rx/disas.c | 1480 +++ 2 files changed, 1485 insertions(+) create mode 100644 target/rx/disas.c diff

[Qemu-devel] [PATCH v13 03/12] target/rx: CPU definition

2019-05-15 Thread Yoshinori Sato
Signed-off-by: Yoshinori Sato Reviewed-by: Richard Henderson --- target/rx/cpu.h | 227 target/rx/cpu.c | 222 ++ target/rx/gdbstub.c | 112 ++

[Qemu-devel] [PATCH v13 00/12] Add RX archtecture support

2019-05-15 Thread Yoshinori Sato
Hello. This patch series is added Renesas RX target emulation. Add "Reviewed-by" for all changes. My git repository is bellow. git://git.pf.osdn.net/gitroot/y/ys/ysato/qemu.git tags/rx-20190514 Testing binaries bellow. u-boot Download - https://osdn.net/users/ysato/pf/qemu/dl/u-boot.bin.gz

[Qemu-devel] [PATCH v13 12/12] MAINTAINERS: Add RX

2019-05-15 Thread Yoshinori Sato
Signed-off-by: Yoshinori Sato Reviewed-by: Richard Henderson --- MAINTAINERS | 19 +++ 1 file changed, 19 insertions(+) diff --git a/MAINTAINERS b/MAINTAINERS index a73a61a546..ef6a02702e 100644 --- a/MAINTAINERS +++ b/MAINTAINERS @@ -272,6 +272,13 @@ F: include/hw/riscv/ F:

[Qemu-devel] [PATCH v13 10/12] hw/registerfields.h: Add 8bit and 16bit register macros.

2019-05-15 Thread Yoshinori Sato
Some RX peripheral using 8bit and 16bit registers. Added 8bit and 16bit APIs. Signed-off-by: Yoshinori Sato Reviewed-by: Richard Henderson --- include/hw/registerfields.h | 32 +++- 1 file changed, 31 insertions(+), 1 deletion(-) diff --git

[Qemu-devel] [PATCH v13 09/12] Add rx-softmmu

2019-05-15 Thread Yoshinori Sato
Signed-off-by: Yoshinori Sato Tested-by: Philippe Mathieu-Daudé Using only CONFIG_RX=y: Reviewed-by: Philippe Mathieu-Daudé Signed-off-by: Yoshinori Sato --- configure | 8 default-configs/rx-softmmu.mak | 3 +++ include/sysemu/arch_init.h | 1 +

Re: [Qemu-devel] [RFC] hw/core/bus.c: Only the main system bus can have no parent

2019-05-15 Thread Markus Armbruster
Peter Maydell writes: > In commit 80376c3fc2c38fdd453 in 2010 we added a workaround for > some qbus buses not being connected to qdev devices -- if the > bus has no parent object then we register a reset function which > resets the bus on system reset. > > Nearly a decade later, we have now no

Re: [Qemu-devel] [PATCH v2 11/13] tests/vm: netbsd autoinstall, using serial console

2019-05-15 Thread Kamil Rytarowski
On 10.05.2019 12:46, Gerd Hoffmann wrote: > Instead of fetching the prebuilt image from patchew download the install > iso and prepare the image locally. Install to disk, using the serial > console. Create qemu user, configure ssh login. Install packages > needed for qemu builds. > >

Re: [Qemu-devel] [Qemu-ppc] [PATCH v8 5/6] ppc: spapr: Enable FWNMI capability

2019-05-15 Thread Aravinda Prasad
On Thursday 16 May 2019 07:15 AM, David Gibson wrote: > On Tue, May 14, 2019 at 11:02:07AM +0530, Aravinda Prasad wrote: >> >> >> On Tuesday 14 May 2019 10:17 AM, David Gibson wrote: >>> On Mon, May 13, 2019 at 04:00:43PM +0530, Aravinda Prasad wrote: On Friday 10 May 2019 03:23

Re: [Qemu-devel] [Qemu-ppc] [PATCH v8 4/6] target/ppc: Build rtas error log upon an MCE

2019-05-15 Thread Aravinda Prasad
On Thursday 16 May 2019 07:17 AM, David Gibson wrote: > On Tue, May 14, 2019 at 10:36:17AM +0530, Aravinda Prasad wrote: >> >> >> On Tuesday 14 May 2019 10:10 AM, David Gibson wrote: >>> On Tue, May 14, 2019 at 09:56:41AM +0530, Aravinda Prasad wrote: On Tuesday 14 May 2019 05:38

Re: [Qemu-devel] [PATCH v12 00/12] Add RX archtecture support

2019-05-15 Thread Yoshinori Sato
On Thu, 16 May 2019 01:48:29 +0900, Richard Henderson wrote: > > On 5/13/19 11:14 PM, Yoshinori Sato wrote: > > This patch series is added Renesas RX target emulation. > > > > I fixed the ROM address because v11 was incorrect. > > > > My git repository is bellow. > >

Re: [Qemu-devel] [PATCH for-4.0.1] q35: Revert to kernel irqchip

2019-05-15 Thread Peter Xu
On Tue, May 14, 2019 at 02:22:03PM -0600, Alex Williamson wrote: > On Tue, 14 May 2019 13:03:31 -0600 > Alex Williamson wrote: > > > Commit b2fc91db8447 ("q35: set split kernel irqchip as default") changed > > the default for the pc-q35-4.0 machine type to use split irqchip, which > > turned out

Re: [Qemu-devel] [PATCH for-4.1 v2] q35: Revert to kernel irqchipQEM

2019-05-15 Thread Peter Xu
On Wed, May 15, 2019 at 07:23:13AM -0600, Alex Williamson wrote: > On Wed, 15 May 2019 14:15:03 +0800 > Peter Xu wrote: > > > On Tue, May 14, 2019 at 02:14:41PM -0600, Alex Williamson wrote: > > > Commit b2fc91db8447 ("q35: set split kernel irqchip as default") changed > > > the default for the

Re: [Qemu-devel] [Qemu-ppc] [PATCH v8 5/6] ppc: spapr: Enable FWNMI capability

2019-05-15 Thread David Gibson
On Tue, May 14, 2019 at 11:02:07AM +0530, Aravinda Prasad wrote: > > > On Tuesday 14 May 2019 10:17 AM, David Gibson wrote: > > On Mon, May 13, 2019 at 04:00:43PM +0530, Aravinda Prasad wrote: > >> > >> > >> On Friday 10 May 2019 03:23 PM, David Gibson wrote: > >>> On Fri, May 10, 2019 at

[Qemu-devel] [Bug 1828508] Re: qemu-img created VMDK files lead to "Unsupported or invalid disk type 7"

2019-05-15 Thread Jake Mikelson
Hi, I'm running 5.5. I've been playing around with some of the options, and if I run the below, I end up with 2 files. qemu-img.exe convert "c:\test\AppD-VM01.vhd" -O vmdk -o adapter_type=lsilogic,subformat=monolithicFlat -p "c:\test\AppD- VM01.vmdk" The files I get are: AppD-VM01.vmdk (which

Re: [Qemu-devel] [PATCH v6 24/25] target/ppc: Use gen_io_start/end around DARN

2019-05-15 Thread David Gibson
On Fri, May 10, 2019 at 10:30:48AM -0700, Richard Henderson wrote: > Generating a random number counts as I/O, as it cannot be > replayed and produce the same results. > > Cc: David Gibson > Suggested-by: Peter Maydell > Signed-off-by: Richard Henderson Acked-by: David Gibson > --- >

Re: [Qemu-devel] [Qemu-ppc] [PATCH v8 4/6] target/ppc: Build rtas error log upon an MCE

2019-05-15 Thread David Gibson
On Tue, May 14, 2019 at 10:36:17AM +0530, Aravinda Prasad wrote: > > > On Tuesday 14 May 2019 10:10 AM, David Gibson wrote: > > On Tue, May 14, 2019 at 09:56:41AM +0530, Aravinda Prasad wrote: > >> > >> > >> On Tuesday 14 May 2019 05:38 AM, David Gibson wrote: > >>> On Mon, May 13, 2019 at

Re: [Qemu-devel] [PATCH] target/ppc: Set PSSCR_EC on cpu halt to prevent spurious wakeup

2019-05-15 Thread David Gibson
On Thu, May 16, 2019 at 10:57:44AM +1000, Suraj Jitindar Singh wrote: > The processor stop status and control register (PSSCR) is used to > control the power saving facilities of the thread. The exit criterion > bit (EC) is used to specify whether the thread should be woken by any > interrupt (EC

Re: [Qemu-devel] [PATCH] spapr/xive: Sanity checks of OV5 during CAS

2019-05-15 Thread David Gibson
On Wed, May 15, 2019 at 07:04:24PM +0200, Greg Kurz wrote: > If a machine is started with ic-mode=xive but the guest only knows > about XICS, eg. an RHEL 7.6 guest, the kernel panics. This is > expected but a bit unfortunate since the crash doesn't provide > much information for the end user to

Re: [Qemu-devel] [RFC v2 PATCH 2/3] spapr: Add NVDIMM device support

2019-05-15 Thread David Gibson
On Wed, May 15, 2019 at 12:30:07PM +0530, Shivaprasad G Bhat wrote: > Hi David, > > Thanks for the comments. Replies inline.. > > On 05/14/2019 07:52 AM, David Gibson wrote: > > On Mon, May 13, 2019 at 04:28:02AM -0500, Shivaprasad G Bhat wrote: > > > Add support for NVDIMM devices for sPAPR.

Re: [Qemu-devel] [PATCH v2 1/2] vfio/mdev: add version attribute for mdev device

2019-05-15 Thread Yan Zhao
On Tue, May 14, 2019 at 11:01:42PM +0800, Alex Williamson wrote: > On Tue, 14 May 2019 09:43:44 +0200 > Erik Skultety wrote: > > > On Tue, May 14, 2019 at 03:32:19AM -0400, Yan Zhao wrote: > > > On Tue, May 14, 2019 at 03:20:40PM +0800, Erik Skultety wrote: > > > > On Tue, May 14, 2019 at

[Qemu-devel] [PATCH] target/ppc: Set PSSCR_EC on cpu halt to prevent spurious wakeup

2019-05-15 Thread Suraj Jitindar Singh
The processor stop status and control register (PSSCR) is used to control the power saving facilities of the thread. The exit criterion bit (EC) is used to specify whether the thread should be woken by any interrupt (EC == 0) or only an interrupt enabled in the LPCR to wake the thread (EC == 1).

Re: [Qemu-devel] [Qemu-arm] [PATCH v2 1/1] target/arm: Fix vector operation segfault

2019-05-15 Thread Alistair Francis
On Wed, May 15, 2019 at 6:11 AM Alex Bennée wrote: > > > Alistair Francis writes: > > > Commit 89e68b575 "target/arm: Use vector operations for saturation" > > causes this abort() when booting QEMU ARM with a Cortex-A15: > > You may want to check your email settings because when I tried to apply

Re: [Qemu-devel] [PATCH for-4.1] vfio/common: Introduce vfio_set_irq_signaling helper

2019-05-15 Thread Alex Williamson
On Tue, 9 Apr 2019 17:58:31 +0200 Eric Auger wrote: > The code used to assign an interrupt index/subindex to an > eventfd is duplicated many times. Let's introduce an helper that > allows to set/unset the signaling for an ACTION_TRIGGER or > ACTION_UNMASK action. > > Signed-off-by: Eric Auger

Re: [Qemu-devel] [PATCH v9 1/7] libnvdimm: nd_region flush callback support

2019-05-15 Thread Dan Williams
On Tue, May 14, 2019 at 7:55 AM Pankaj Gupta wrote: > > This patch adds functionality to perform flush from guest > to host over VIRTIO. We are registering a callback based > on 'nd_region' type. virtio_pmem driver requires this special > flush function. For rest of the region types we are

[Qemu-devel] Unhandled Fault when trying to access PCIE [AARCH64/VIRT]

2019-05-15 Thread Adam Parker
Hello all, I am attempting to write PCIe drivers for my OP-TEE project against the Cortex-A57 using the virt machine. I am using the default packaged qemu, 3.0.93v3.1.0-rc3-dirty and v4.0.0-rc4. I have added the PCIE device to the page table of the OS and when I try to write to PIO (0xCF8) I

[Qemu-devel] [PULL 21/21] hw/net/ne2000: Extract the PCI device from the chipset common code

2019-05-15 Thread Paolo Bonzini
From: Philippe Mathieu-Daudé The ne2000.c file contains functions common the the ISA and PCI devices. To allow to build with one or another, extract the PCI specific part into a new file. This fix an issue where the NE2000_ISA Kconfig had to pull the full PCI core objects. Signed-off-by:

[Qemu-devel] [PULL 15/21] build: replace GENERATED_FILES by generated-files-y

2019-05-15 Thread Paolo Bonzini
From: Laurent Vivier When possible use generated-files-$(FLAG) to disable some targets (like KEYCODEMAP_FILES). Suggested-by: Paolo Bonzini Signed-off-by: Laurent Vivier Message-Id: <20190401141222.30034-3-lviv...@redhat.com> --- Makefile | 36

[Qemu-devel] [PULL 17/21] build: chardev is only needed for softmmu targets

2019-05-15 Thread Paolo Bonzini
From: Laurent Vivier Move the dependency from SUBDIR_RULES to SOFTMMU_SUBDIR_RULES Suggested-by: Paolo Bonzini Signed-off-by: Laurent Vivier Message-Id: <20190401141222.30034-5-lviv...@redhat.com> --- Makefile | 3 ++- Makefile.objs | 2 +- 2 files changed, 3 insertions(+), 2

[Qemu-devel] [PULL 16/21] configure: qemu-ga is only needed with softmmu targets

2019-05-15 Thread Paolo Bonzini
From: Laurent Vivier Remove it from the list of tools if --disable-system and --disable-tools are used as we don't need it for linux-user targets. Suggested-by: Paolo Bonzini [lv: I also disable it with disable-tools, not only with disable-system] Signed-off-by: Laurent Vivier Message-Id:

[Qemu-devel] [PULL 20/21] hw/char: Move multi-serial devices into separate file

2019-05-15 Thread Paolo Bonzini
From: Thomas Huth In our downstream distribution of QEMU, we'd like to ship the binary without the multi-serial PCI devices. To make this disablement easier, let's move the devices into a separate file and add a proper Kconfig- switch for these devices. Signed-off-by: Thomas Huth Message-Id:

[Qemu-devel] [PULL 19/21] ioapic: allow buggy guests mishandling level-triggered interrupts to make progress

2019-05-15 Thread Paolo Bonzini
From: Vitaly Kuznetsov It was found that Hyper-V 2016 on KVM in some configurations (q35 machine + piix4-usb-uhci) hangs on boot. Root-cause was that one of Hyper-V level-triggered interrupt handler performs EOI before fixing the cause of the interrupt. This results in IOAPIC keep re-raising the

[Qemu-devel] [PULL 18/21] build: don't build hardware objects with linux-user

2019-05-15 Thread Paolo Bonzini
From: Laurent Vivier Some objects are only needed for system emulation and tools. We can ignore them for the user mode case Update tests to run accordingly: conditionally build some tests on CONFIG_BLOCK. Some tests use components that are only built when softmmu or block tools are enabled,

[Qemu-devel] [PULL 14/21] trace: only include trace-event-subdirs when they are needed

2019-05-15 Thread Paolo Bonzini
From: Laurent Vivier Some directories are built only for softmmu targets, and the related trace-event-subdirs must do the same Signed-off-by: Laurent Vivier Reviewed-by: Stefan Hajnoczi Message-Id: <20190401141222.30034-2-lviv...@redhat.com> --- Makefile.objs | 12 +++- 1 file

[Qemu-devel] [PULL 08/21] memory: correct the comment to DIRTY_MEMORY_MIGRATION

2019-05-15 Thread Paolo Bonzini
From: Wei Yang The dirty bit is DIRTY_MEMORY_MIGRATION. Correct the comment. Signed-off-by: Wei Yang Message-Id: <20190426020927.25470-1-richardw.y...@linux.intel.com> Signed-off-by: Paolo Bonzini --- memory.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/memory.c

[Qemu-devel] [PULL 12/21] mips-fulong2e: obey -vga none

2019-05-15 Thread Paolo Bonzini
Do not create an ATI VGA if "-vga none" was passed on the command line. Cc: BALATON Zoltan Signed-off-by: Paolo Bonzini --- hw/mips/mips_fulong2e.c | 10 ++ 1 file changed, 6 insertions(+), 4 deletions(-) diff --git a/hw/mips/mips_fulong2e.c b/hw/mips/mips_fulong2e.c index

Re: [Qemu-devel] [PATCH v9 2/7] virtio-pmem: Add virtio pmem driver

2019-05-15 Thread David Hildenbrand
On 15.05.19 22:46, David Hildenbrand wrote: >> +vpmem->vdev = vdev; >> +vdev->priv = vpmem; >> +err = init_vq(vpmem); >> +if (err) { >> +dev_err(>dev, "failed to initialize virtio pmem vq's\n"); >> +goto out_err; >> +} >> + >> +

[Qemu-devel] [PULL 13/21] sun4m: obey -vga none

2019-05-15 Thread Paolo Bonzini
Do not create a TCX if "-vga none" was passed on the command line. Remove some dead code along the way to avoid big reindentation. Signed-off-by: Paolo Bonzini --- hw/sparc/sun4m.c | 6 ++ 1 file changed, 2 insertions(+), 4 deletions(-) diff --git a/hw/sparc/sun4m.c b/hw/sparc/sun4m.c

[Qemu-devel] [PULL 11/21] hw/i386/acpi: Assert a pointer is not null BEFORE using it

2019-05-15 Thread Paolo Bonzini
From: Philippe Mathieu-Daudé Commit 72c194f7e75c added a non-null check on the 'obj' pointer. Later, commit 500b11ea5095 added code which uses the 'obj' pointer _before_ the assertion check. Move the assertion _before_ the pointer use. Signed-off-by: Philippe Mathieu-Daudé Message-Id:

[Qemu-devel] [PULL 04/21] vl: Add missing descriptions to the VGA adapters list

2019-05-15 Thread Paolo Bonzini
From: Philippe Mathieu-Daudé Some VGA adapters do not contain an helpful description, this can be confusing: $ qemu-system-arm -M virt -vga help none std standard VGA cirrus Cirrus VGA (default) vmware VMWare SVGA xenfb Add a description

[Qemu-devel] [PULL 10/21] hw/i386/acpi: Add object_resolve_type_unambiguous to improve modularity

2019-05-15 Thread Paolo Bonzini
From: Philippe Mathieu-Daudé When building with CONFIG_Q35=n, we get: LINKx86_64-softmmu/qemu-system-x86_64 /usr/bin/ld: hw/i386/acpi-build.o: in function `acpi_get_misc_info': /source/qemu/hw/i386/acpi-build.c:243: undefined reference to `ich9_lpc_find' collect2: error: ld

[Qemu-devel] [PULL 09/21] hw/acpi/piix4: Move TYPE_PIIX4_PM to a public header

2019-05-15 Thread Paolo Bonzini
From: Philippe Mathieu-Daudé Move the TYPE_PIIX4_PM definition to the corresponding header, so other files can use it. Signed-off-by: Philippe Mathieu-Daudé Message-Id: <20190427144025.22880-2-phi...@redhat.com> Signed-off-by: Paolo Bonzini --- hw/acpi/piix4.c | 2 --

[Qemu-devel] [PULL 07/21] vl: fix -sandbox parsing crash when seccomp support is disabled

2019-05-15 Thread Paolo Bonzini
From: Marc-André Lureau $ ./x86_64-softmmu/qemu-system-x86_64 -sandbox off qemu-system-x86_64: -sandbox off: There is no option group 'sandbox' Segmentation fault Commit 5780760f5e ("seccomp: check TSYNC host capability") wrapped one use of the sandbox option group to produce a sensible error,

[Qemu-devel] [PULL 01/21] hw/input: Add a CONFIG_PS2 switch for the ps2.c file

2019-05-15 Thread Paolo Bonzini
From: Thomas Huth ps2.c only needs to be compiled if we are building pckbd.c or pl050.c. Signed-off-by: Thomas Huth Message-Id: <20190411182240.5957-1-th...@redhat.com> Tested-by: Philippe Mathieu-Daudé Reviewed-by: Philippe Mathieu-Daudé Signed-off-by: Paolo Bonzini --- hw/input/Kconfig

[Qemu-devel] [PULL 06/21] hvf: Add missing break statement

2019-05-15 Thread Paolo Bonzini
From: Chen Zhang via Qemu-devel In target/i386/hvf/hvf.c, a break statement was probably missing in `hvf_vcpu_exec()`, in handling EXIT_REASON_HLT. These lines seemed to be equivalent to `kvm_handle_halt()`. Signed-off-by: Chen Zhang Message-Id: <087f1d9c-109d-41d1-be2c-ce5d840c9...@me.com>

[Qemu-devel] [PULL 02/21] roms: assert if max rom size is less than the used size

2019-05-15 Thread Paolo Bonzini
From: Igor Mammedov It would ensure that we would notice attempt to write beyond the allocated buffer. In case of MemoryRegion backed ROM it's the host buffer and the guest RAM otherwise. assert can be triggered with: dd if=/dev/zero of=/tmp/blob bs=63k count=1 qemu-system-x86_64 `for i in

[Qemu-devel] [PULL 00/21] Misc patches for 2019-05-15

2019-05-15 Thread Paolo Bonzini
The following changes since commit e329ad2ab72c43b56df88b34954c2c7d839bb373: Merge remote-tracking branch 'remotes/rth/tags/pull-tcg-20190513' into staging (2019-05-14 10:08:47 +0100) are available in the git repository at: git://github.com/bonzini/qemu.git tags/for-upstream for you to

[Qemu-devel] [PULL 05/21] megasas: fix mapped frame size

2019-05-15 Thread Paolo Bonzini
From: Peter Lieven the current value of 1024 bytes (16 * MFI_FRAME_SIZE) we map is not enough to hold the maximum number of scatter gather elements we advertise. We actually need a maximum of 2048 bytes. This is 128 max sg elements * 16 bytes (sizeof (union mfi_sgl)). Cc:

[Qemu-devel] [PULL 03/21] Declare -realtime as deprecated

2019-05-15 Thread Paolo Bonzini
From: Thomas Huth The old -realtime mlock=on|off parameter does exactly the same as the new -overcommit mem-lock=on|off parameter. Additionally, "-realtime" does not activate any additional "realtime" capabilities as the name might indicate. We should avoid to confuse the users this way, so

Re: [Qemu-devel] [PATCH v9 4/7] dm: enable synchronous dax

2019-05-15 Thread Dan Williams
[ add Mike and dm-devel ] Mike, any concerns with the below addition to the device-mapper-dax implementation? On Tue, May 14, 2019 at 7:58 AM Pankaj Gupta wrote: > > This patch sets dax device 'DAXDEV_SYNC' flag if all the target > devices of device mapper support synchrononous DAX. If device

Re: [Qemu-devel] [PATCH v9 2/7] virtio-pmem: Add virtio pmem driver

2019-05-15 Thread Dan Williams
On Tue, May 14, 2019 at 8:25 AM Pankaj Gupta wrote: > > > > On 5/14/19 7:54 AM, Pankaj Gupta wrote: > > > diff --git a/drivers/virtio/Kconfig b/drivers/virtio/Kconfig > > > index 35897649c24f..94bad084ebab 100644 > > > --- a/drivers/virtio/Kconfig > > > +++ b/drivers/virtio/Kconfig > > > @@ -42,6

Re: [Qemu-devel] [PATCH v9 2/7] virtio-pmem: Add virtio pmem driver

2019-05-15 Thread David Hildenbrand
> + vpmem->vdev = vdev; > + vdev->priv = vpmem; > + err = init_vq(vpmem); > + if (err) { > + dev_err(>dev, "failed to initialize virtio pmem vq's\n"); > + goto out_err; > + } > + > + virtio_cread(vpmem->vdev, struct virtio_pmem_config, > +

[Qemu-devel] [PATCH v1 5/5] s390x/tcg: Implement VECTOR STRING RANGE COMPARE

2019-05-15 Thread David Hildenbrand
Crazy stuff. Implement it similar to VECTOR FIND ANY ELEMENT EQUAL. Signed-off-by: David Hildenbrand --- target/s390x/helper.h| 6 ++ target/s390x/insn-data.def | 2 + target/s390x/translate_vx.inc.c | 37 +++ target/s390x/vec_string_helper.c | 108

[Qemu-devel] [PATCH v1 4/5] s390x/tcg: Implement VECTOR ISOLATE STRING

2019-05-15 Thread David Hildenbrand
Signed-off-by: David Hildenbrand --- target/s390x/helper.h| 6 + target/s390x/insn-data.def | 2 ++ target/s390x/translate_vx.inc.c | 34 ++ target/s390x/vec_string_helper.c | 41 4 files changed, 83 insertions(+)

[Qemu-devel] [PATCH v1 0/5] s390x/tcg: Vector Instruction Support Part 3

2019-05-15 Thread David Hildenbrand
This is the third part of vector instruction support for s390x. It is based on part 2, which is will send a pull-request for to Conny soon. Part 1: Vector Support Instructions Part 2: Vector Integer Instructions Part 3: Vector String Instructions Part 4: Vector Floating-Point Instructions The

[Qemu-devel] [PATCH v1 3/5] s390x/tcg: Implement VECTOR FIND ELEMENT NOT EQUAL

2019-05-15 Thread David Hildenbrand
Similar to VECTOR FIND ELEMENT EQUAL, however the search also stops on any inequality. A match for inequality seems to have precedence over a match for zero, because both elements have to be zero. Signed-off-by: David Hildenbrand --- target/s390x/helper.h| 6

[Qemu-devel] [PATCH v1 1/5] s390x/tcg: Implement VECTOR FIND ANY ELEMENT EQUAL

2019-05-15 Thread David Hildenbrand
Complicated stuff. Provide two variants, one for the CC and one without the CC. The CC is returned via cpu_env. Signed-off-by: David Hildenbrand --- target/s390x/Makefile.objs | 2 +- target/s390x/helper.h| 8 +++ target/s390x/insn-data.def | 5 ++

[Qemu-devel] [PATCH v1 2/5] s390x/tcg: Implement VECTOR FIND ELEMENT EQUAL

2019-05-15 Thread David Hildenbrand
Implement it similar to VECTOR FIND ANY ELEMENT EQUAL. The zero-check seems to have precedence in case we have "data1 == data2 == 0". The description in the PoP is a little bi confusing. Signed-off-by: David Hildenbrand --- target/s390x/helper.h| 6 target/s390x/insn-data.def

[Qemu-devel] [PATCH 4/4] iotests: Make 245 faster and more reliable

2019-05-15 Thread Max Reitz
Sometimes, 245 fails for me because some stream job has already finished while the test expects it to still be active. (With -c none, it fails basically every time.) The most reliable way to fix this is to simply set auto_finalize=false so the job will remain in the block graph as long as we need

[Qemu-devel] [PATCH 2/4] iotests.py: Let assert_qmp() accept an array

2019-05-15 Thread Max Reitz
Sometimes we cannot tell which error message qemu will emit, and we do not care. With this change, we can then just pass an array of all possible messages to assert_qmp() and it will choose the right one. Signed-off-by: Max Reitz --- tests/qemu-iotests/iotests.py | 18 -- 1

[Qemu-devel] [PATCH 3/4] iotests.py: Fix VM.run_job

2019-05-15 Thread Max Reitz
log() is in the current module, there is no need to prefix it. In fact, doing so may make VM.run_job() unusable in tests that never use iotests.log() themselves. Signed-off-by: Max Reitz --- tests/qemu-iotests/iotests.py | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git

Re: [Qemu-devel] [PATCH v7 01/24] build: Link user-only with crypto-rng-obj-y

2019-05-15 Thread Daniel P . Berrangé
On Wed, May 15, 2019 at 09:38:00PM +0200, Laurent Vivier wrote: > On 15/05/2019 19:49, Daniel P. Berrangé wrote: > > On Wed, May 15, 2019 at 10:22:08AM -0700, Richard Henderson wrote: > >> On 5/15/19 9:53 AM, Daniel P. Berrangé wrote: > >>> On Tue, May 14, 2019 at 12:16:30PM -0700, Richard

[Qemu-devel] [PATCH 1/4] block: Improve "Block node is read-only" message

2019-05-15 Thread Max Reitz
This message does not make any sense when it appears as the response to making an R/W node read-only. We should detect that case and emit a different message, then. Signed-off-by: Max Reitz --- block.c | 17 - 1 file changed, 16 insertions(+), 1 deletion(-) diff --git

[Qemu-devel] [PATCH 0/4] iotests: Make 245 faster and more reliable

2019-05-15 Thread Max Reitz
245 is a bit flakey for me, because it uses block jobs that copy 1 MB of data but have a buffer size of 512 kB, so they may be done before the test gets to do the things it wants to do while the check is running. (Rate limiting doesn’t change this.) The boring way to fix this would be to increase

Re: [Qemu-devel] [PATCH v7 01/24] build: Link user-only with crypto-rng-obj-y

2019-05-15 Thread Laurent Vivier
On 15/05/2019 19:49, Daniel P. Berrangé wrote: > On Wed, May 15, 2019 at 10:22:08AM -0700, Richard Henderson wrote: >> On 5/15/19 9:53 AM, Daniel P. Berrangé wrote: >>> On Tue, May 14, 2019 at 12:16:30PM -0700, Richard Henderson wrote: For user-only, we require only the random number bits of

Re: [Qemu-devel] cpu.fail / MDS fixes

2019-05-15 Thread Stefan Priebe - Profihost AG
Am 15.05.19 um 19:54 schrieb Daniel P. Berrangé: > On Wed, May 15, 2019 at 07:13:56PM +0200, Stefan Priebe - Profihost AG wrote: >> Hello list, >> >> i've updated my host to kernel 4.19.43 and applied the following patch >> to my qemu 2.12.1: >> https://bugzilla.suse.com/attachment.cgi?id=798722

Re: [Qemu-devel] [PATCH] configure: Disable slirp if --disable-system

2019-05-15 Thread Aleksandar Markovic
On May 15, 2019 12:07 PM, "Peter Maydell" wrote: > > On Tue, 14 May 2019 at 20:16, Aleksandar Markovic > wrote: > > > > On May 13, 2019 11:14 PM, "Richard Henderson" < richard.hender...@linaro.org> > > wrote: > > > > > > On 5/11/19 5:47 AM, Aleksandar Markovic wrote: > > > > If no, the patch

Re: [Qemu-devel] cpu.fail / MDS fixes

2019-05-15 Thread Daniel P . Berrangé
On Wed, May 15, 2019 at 07:13:56PM +0200, Stefan Priebe - Profihost AG wrote: > Hello list, > > i've updated my host to kernel 4.19.43 and applied the following patch > to my qemu 2.12.1: > https://bugzilla.suse.com/attachment.cgi?id=798722 > > But my guest running 4.19.43 still says: >

Re: [Qemu-devel] [PATCH v9 00/27] gdbstub: Refactor command packets handler

2019-05-15 Thread Alex Bennée
Jon Doron writes: > This patch series refactors the old gdbstub command packets handler > with a new infrastructure which should ease extending and adding new > and missing gdb command packets. Jon, I've finished my review and things are looking pretty good. The code is a good clean-up and

[Qemu-devel] [PATCH 6/6] tests/hd-geo-test: Use qtest_init() instead of qtest_start()

2019-05-15 Thread Thomas Huth
qtest_start() + qtest_end() should be avoided, since they use the global_qtest variable that we want to get rid of in the long run. Use qtest_init() and qtest_quit() instead. Signed-off-by: Thomas Huth --- tests/hd-geo-test.c | 76 - 1 file changed,

Re: [Qemu-devel] [PATCH v9 26/27] gdbstub: Add support to read a MSR for KVM target

2019-05-15 Thread Alex Bennée
Jon Doron writes: > gdb> maint packet qqemu.kvm.Rdmsr:MsrIndex gdbserver already has a mechanism for exposing system registers see: commit 200bf5b7ffea635079cc05fdfb363372b9544ce7 Author: Abdallah Bouassida Date: Fri May 18 17:48:07 2018 +0100 for an example. As MSR's are very

Re: [Qemu-devel] [PATCH v7 01/24] build: Link user-only with crypto-rng-obj-y

2019-05-15 Thread Daniel P . Berrangé
On Wed, May 15, 2019 at 10:22:08AM -0700, Richard Henderson wrote: > On 5/15/19 9:53 AM, Daniel P. Berrangé wrote: > > On Tue, May 14, 2019 at 12:16:30PM -0700, Richard Henderson wrote: > >> For user-only, we require only the random number bits of the > >> crypto subsystem. > >> > >> We need to

[Qemu-devel] [PATCH 3/6] tests/numa-test: Use qtest_init() instead of qtest_start()

2019-05-15 Thread Thomas Huth
qtest_start() + qtest_end() should be avoided, since they use the global_qtest variable that we want to get rid of in the long run. Use qtest_init() and qtest_quit() instead. Signed-off-by: Thomas Huth --- tests/numa-test.c | 53 +++ 1 file changed,

[Qemu-devel] [PATCH 4/6] tests/qom-test: Use qtest_init() instead of qtest_start()

2019-05-15 Thread Thomas Huth
qtest_start() + qtest_end() should be avoided, since they use the global_qtest variable that we want to get rid of in the long run. Use qtest_init() and qtest_quit() instead. Signed-off-by: Thomas Huth --- tests/qom-test.c | 28 +--- 1 file changed, 13 insertions(+), 15

Re: [Qemu-devel] [PATCH v9 24/27] gdbstub: Add another handler for setting qemu.sstep

2019-05-15 Thread Alex Bennée
Jon Doron writes: > Follow GDB general query/set packet conventions, qemu.sstep can now > be set with the following command as well: > gdb> maint packet Qqemu.sstep:Value I;m not sure about exposing internal values to a protocol like this. Maybe text based flags would be better? > >

[Qemu-devel] [PATCH 5/6] tests/device-introspect: Use qtest_init() instead of qtest_start()

2019-05-15 Thread Thomas Huth
qtest_start() + qtest_end() should be avoided, since they use the global_qtest variable that we want to get rid of in the long run. Use qtest_init() and qtest_quit() instead. Signed-off-by: Thomas Huth --- tests/device-introspect-test.c | 85 ++ 1 file changed,

[Qemu-devel] [PATCH 2/6] tests/q35-test: Make test independent of global_qtest

2019-05-15 Thread Thomas Huth
Use a local QTestState variable, so that we can finally get rid of the undesired global_qtest variable in this file, too. Signed-off-by: Thomas Huth --- tests/q35-test.c | 39 ++- 1 file changed, 18 insertions(+), 21 deletions(-) diff --git

[Qemu-devel] [PATCH 0/6] Get rid of global_qtest in q35-, qom-, numa- and more tests

2019-05-15 Thread Thomas Huth
Here are some more patches that get rid of global_qtest and related functions in some of the qtests (hacked along the way while waiting for other compilation and test processes to finish). A global variable like global_qtest is very problematic in tests that track multiple test states (like

[Qemu-devel] [PATCH 1/6] tests/libqos: Get rid of global_qtest dependency in qvring_init()

2019-05-15 Thread Thomas Huth
Library functions should not depend on global_qtest functions like writew() and writeq(), so that they can also be used in tests that deal with multiple QTestStates at the same time (like migration tests). Signed-off-by: Thomas Huth --- tests/libqos/virtio-mmio.c | 2 +-

Re: [Qemu-devel] [PATCH v9 22/27] gdbstub: Implement generic query qemu.Supported

2019-05-15 Thread Alex Bennée
Jon Doron writes: > qemu.Supported query reply back with the supported qemu query/set > commands (commands are seperated with a semicolon from each other). > > gdb> maint packet qqemu.Supported > > Signed-off-by: Jon Doron > --- > gdbstub.c | 9 + > 1 file changed, 9 insertions(+) >

Re: [Qemu-devel] [PATCH 1/6] qemu-bridge-helper: Fix misuse of isspace()

2019-05-15 Thread Richard Henderson
On 5/15/19 9:55 AM, Markus Armbruster wrote: > Proposal: > > 1. Add qemu-bridge-helper.c to Jason's "Network device backends" > > 2. Deprecate -netdev tap parameter "helper" > > 3. Improve documentation of -netdev bridge > > 4. Create a manual page for qemu-bridge-helper that also covers >

Re: [Qemu-devel] [PATCH v9 21/27] gdbstub: Clear unused variables in gdb_handle_packet

2019-05-15 Thread Alex Bennée
Jon Doron writes: > Signed-off-by: Jon Doron Reviewed-by: Alex Bennée > --- > gdbstub.c | 11 ++- > 1 file changed, 2 insertions(+), 9 deletions(-) > > diff --git a/gdbstub.c b/gdbstub.c > index d678191705..8bdfae4b29 100644 > --- a/gdbstub.c > +++ b/gdbstub.c > @@ -2259,17

Re: [Qemu-devel] [PATCH v7 01/24] build: Link user-only with crypto-rng-obj-y

2019-05-15 Thread Richard Henderson
On 5/15/19 9:53 AM, Daniel P. Berrangé wrote: > On Tue, May 14, 2019 at 12:16:30PM -0700, Richard Henderson wrote: >> For user-only, we require only the random number bits of the >> crypto subsystem. >> >> We need to preserve --static linking, which for many recent Linux >> distributions precludes

Re: [Qemu-devel] [PATCH v9 20/27] gdbstub: Implement target halted (? pkt) with new infra

2019-05-15 Thread Alex Bennée
Jon Doron writes: > Signed-off-by: Jon Doron > --- > gdbstub.c | 36 ++-- > 1 file changed, 26 insertions(+), 10 deletions(-) > > diff --git a/gdbstub.c b/gdbstub.c > index 2fd0d66f4d..d678191705 100644 > --- a/gdbstub.c > +++ b/gdbstub.c > @@ -2239,13

[Qemu-devel] cpu.fail / MDS fixes

2019-05-15 Thread Stefan Priebe - Profihost AG
Hello list, i've updated my host to kernel 4.19.43 and applied the following patch to my qemu 2.12.1: https://bugzilla.suse.com/attachment.cgi?id=798722 But my guest running 4.19.43 still says: Vulnerable: Clear CPU buffers attempted, no microcode; SMT Host state unknown while the host says:

Re: [Qemu-devel] [PATCH v9 18/27] gdbstub: Implement generic query (q pkt) with new infra

2019-05-15 Thread Alex Bennée
Jon Doron writes: A bit more for the commit message here as there seems to be a fair amount going on. > Signed-off-by: Jon Doron > --- > gdbstub.c | 327 ++ > 1 file changed, 327 insertions(+) > > diff --git a/gdbstub.c b/gdbstub.c > index

Re: [Qemu-devel] [PATCH v9 17/27] gdbstub: Implement v commands with new infra

2019-05-15 Thread Alex Bennée
Jon Doron writes: > Signed-off-by: Jon Doron > --- > gdbstub.c | 170 +++--- > 1 file changed, 110 insertions(+), 60 deletions(-) > > diff --git a/gdbstub.c b/gdbstub.c > index 9b0556f8be..d56d0fd235 100644 > --- a/gdbstub.c > +++ b/gdbstub.c >

[Qemu-devel] [PATCH] spapr/xive: Sanity checks of OV5 during CAS

2019-05-15 Thread Greg Kurz
If a machine is started with ic-mode=xive but the guest only knows about XICS, eg. an RHEL 7.6 guest, the kernel panics. This is expected but a bit unfortunate since the crash doesn't provide much information for the end user to guess what's happening. Detect that during CAS and exit QEMU with a

Re: [Qemu-devel] [PATCH 1/6] qemu-bridge-helper: Fix misuse of isspace()

2019-05-15 Thread Markus Armbruster
Markus Armbruster writes: > Daniel P. Berrangé writes: > >> On Wed, May 15, 2019 at 08:34:17AM +0200, Markus Armbruster wrote: >>> Jason Wang writes: >>> >>> > On 2019/5/14 下午8:18, Markus Armbruster wrote: >>> >> Peter Maydell writes: >>> >> >>> >>> On Mon, 13 May 2019 at 14:21, Markus

Re: [Qemu-devel] [PATCH v9 16/27] gdbstub: Implement step (s pkt) with new infra

2019-05-15 Thread Alex Bennée
Jon Doron writes: > Signed-off-by: Jon Doron Reviewed-by: Alex Bennée > --- > gdbstub.c | 25 +++-- > 1 file changed, 19 insertions(+), 6 deletions(-) > > diff --git a/gdbstub.c b/gdbstub.c > index 9fe130f30d..9b0556f8be 100644 > --- a/gdbstub.c > +++ b/gdbstub.c > @@

Re: [Qemu-devel] [PATCH v9 15/27] gdbstub: Implement file io (F pkt) with new infra

2019-05-15 Thread Alex Bennée
Jon Doron writes: There is a bit more going on here than a simple conversion. I think we need some additional commentary about the format of the data coming back. > Signed-off-by: Jon Doron > --- > gdbstub.c | 62 +++ > 1 file changed, 40

Re: [Qemu-devel] [PATCH v7 01/24] build: Link user-only with crypto-rng-obj-y

2019-05-15 Thread Daniel P . Berrangé
On Tue, May 14, 2019 at 12:16:30PM -0700, Richard Henderson wrote: > For user-only, we require only the random number bits of the > crypto subsystem. > > We need to preserve --static linking, which for many recent Linux > distributions precludes using GnuTLS or GCrypt. Instead, use our >

Re: [Qemu-devel] [PATCH v7 01/24] build: Link user-only with crypto-rng-obj-y

2019-05-15 Thread Richard Henderson
On 5/15/19 9:42 AM, Laurent Vivier wrote: > On 14/05/2019 21:16, Richard Henderson wrote: >> For user-only, we require only the random number bits of the >> crypto subsystem. >> >> We need to preserve --static linking, which for many recent Linux >> distributions precludes using GnuTLS or GCrypt.

Re: [Qemu-devel] [PATCH v12 00/12] Add RX archtecture support

2019-05-15 Thread Richard Henderson
On 5/13/19 11:14 PM, Yoshinori Sato wrote: > This patch series is added Renesas RX target emulation. > > I fixed the ROM address because v11 was incorrect. > > My git repository is bellow. > git://git.pf.osdn.net/gitroot/y/ys/ysato/qemu.git tags/rx-20190514 > > Testing binaries bellow. > u-boot

Re: [Qemu-devel] [PATCH v7 01/24] build: Link user-only with crypto-rng-obj-y

2019-05-15 Thread Laurent Vivier
On 14/05/2019 21:16, Richard Henderson wrote: > For user-only, we require only the random number bits of the > crypto subsystem. > > We need to preserve --static linking, which for many recent Linux > distributions precludes using GnuTLS or GCrypt. Instead, use our > random-platform module

Re: [Qemu-devel] [PATCH v12 12/12] MAINTAINERS: Add RX

2019-05-15 Thread Richard Henderson
On 5/13/19 11:14 PM, Yoshinori Sato wrote: > Signed-off-by: Yoshinori Sato > --- > MAINTAINERS | 19 +++ > 1 file changed, 19 insertions(+) Reviewed-by: Richard Henderson r~

  1   2   3   >