Re: [Qemu-devel] [PATCH 2/6] hw/m68k/mcf5206: Convert away from old_mmio

2018-06-01 Thread Thomas Huth
On 01.06.2018 16:12, Peter Maydell wrote: > Convert the mcf5206 device away from using the old_mmio field > of MemoryRegionOps. This device is used by the an5206 board. > > Signed-off-by: Peter Maydell > --- > hw/m68k/mcf5206.c | 48 +++ > 1 file chang

Re: [Qemu-devel] [PATCH 0/1] vmstatification of 0323 [for 2.8?]

2018-06-01 Thread Philippe Mathieu-Daudé
On 07/18/2016 02:00 PM, Dr. David Alan Gilbert wrote: > * Peter Maydell (peter.mayd...@linaro.org) wrote: >> On 18 July 2016 at 12:07, Dr. David Alan Gilbert (git) >> wrote: >>> From: "Dr. David Alan Gilbert" >>> >>> Hi, >>> Chipping away at register_vmstate's I thought I'd tackle one of the >>

Re: [Qemu-devel] Virtio-net drivers immune to Nethammer?

2018-06-01 Thread procmem
Michael S. Tsirkin: > On Fri, Jun 01, 2018 at 01:15:44PM +, procmem wrote: >> >> >> Stefan Hajnoczi: >>> On Mon, May 21, 2018 at 11:24:43PM +, procmem wrote: Hi I'm a privacy distro maintainer investigating the implications of the newly published nethammer attack [0] on KVM gue

Re: [Qemu-devel] Virtio-net drivers immune to Nethammer?

2018-06-01 Thread procmem
Dr. David Alan Gilbert: > Isn't the idea to misuse CAT to detect something about access patterns; > so it's not about it actually being related? AFAICT, CAT is directly responsible since the way it interacts with the cache allows it to introduce predictable bit flips that can wreak all kinds of

Re: [Qemu-devel] [PATCH 1/6] ahci: move PIO Setup FIS before transfer, fix it for ATAPI commands

2018-06-01 Thread John Snow
On 04/17/2018 11:39 AM, Paolo Bonzini wrote: > The PIO Setup FIS is written in the PIO:Entry state, which comes before > the ATA and ATAPI data transfer states. As a result, the PIO Setup FIS > interrupt is now raised before DMA ends for ATAPI commands, and tests have > to be adjusted. > Yes,

Re: [Qemu-devel] [RFC PATCH v1 0/2] QEMU binary instrumentation prototype

2018-06-01 Thread no-reply
Hi, This series seems to have some coding style problems. See output below for more information: Type: series Message-id: 152768551387.13955.7085403440722380428.stgit@pasha-ThinkPad-T60 Subject: [Qemu-devel] [RFC PATCH v1 0/2] QEMU binary instrumentation prototype === TEST SCRIPT BEGIN === #!/bi

Re: [Qemu-devel] [PATCH 6/6] ide: introduce ide_transfer_start_norecurse

2018-06-01 Thread John Snow
On 04/17/2018 11:39 AM, Paolo Bonzini wrote: > For the case where the end_transfer_func is also the caller of > ide_transfer_start, the mutual recursion can lead to unlimited > stack usage. Introduce a new version that can be used to change > tail recursion into a loop, and use it in trace_ide_

Re: [Qemu-devel] [PATCH 5/6] atapi: call ide_set_irq before ide_transfer_start

2018-06-01 Thread John Snow
On 04/17/2018 11:39 AM, Paolo Bonzini wrote: > The ATAPI_INT_REASON_IO interrupt is raised when I/O starts, but in the This one is benefit of the doubt for me. (I still can't quite track down our usage of the nsector register for this, it doesn't seem supported by cmd_packet in ATA8 ... It say

Re: [Qemu-devel] [PATCH v3 00/17] tcg: tb_lock removal redux v3

2018-06-01 Thread Emilio G. Cota
On Wed, May 30, 2018 at 16:05:14 -0700, Richard Henderson wrote: > On 05/30/2018 03:46 PM, Richard Henderson wrote: > > Thanks. Queued to tcg-next. > Hmph. Unqueued, at least for now. > > ERROR:/home/rth/work/qemu/qemu/accel/tcg/translate-all.c:615:page_unlock__debug: > assertion failed: (page_i

Re: [Qemu-devel] [PATCH 4/6] ide: make ide_transfer_stop idempotent

2018-06-01 Thread John Snow
On 04/17/2018 11:39 AM, Paolo Bonzini wrote: > There is code checking s->end_transfer_func and it was not taught about > ide_transfer_cancel. We can just use ide_transfer_stop because > s->end_transfer_func is only ever called in the DRQ phase. > > ide_transfer_cancel can then be removed, sinc

Re: [Qemu-devel] [PATCH 3/6] ide: call ide_cmd_done from ide_transfer_stop

2018-06-01 Thread John Snow
On 04/17/2018 11:39 AM, Paolo Bonzini wrote: > The code can simply be moved to the sole caller that has notify == true. > > Signed-off-by: Paolo Bonzini Reviewed-by: John Snow

Re: [Qemu-devel] [PATCH 2/6] ide: push end_transfer_func out of start_transfer callback, rename callback

2018-06-01 Thread John Snow
On 04/17/2018 11:39 AM, Paolo Bonzini wrote: > Now that end_transfer_func is a tail call in ahci_start_transfer, > formalize the fact that the callback (of which ahci_start_transfer is > the sole implementation) takes care of the transfer too: rename it to > pio_transfer and, if it is present, c

[Qemu-devel] [PATCH] target/sh4: Fix translator.c assertion failure for gUSA

2018-06-01 Thread Richard Henderson
The translator loop does not allow the tb_start hook to set dc->base.is_jmp; the only hook allowed to do that is translate_insn. Split the work between init_disas_context where we validate the gUSA parameters, and translate_insn where we emit code. Signed-off-by: Richard Henderson --- This fixe

Re: [Qemu-devel] [PATCH V6 0/7] nvdimm: guarantee persistence of QEMU writes to persistent memory

2018-06-01 Thread no-reply
Hi, This series failed build test on s390x host. Please find the details below. Type: series Message-id: 1527840629-18648-1-git-send-email-junyan...@gmx.com Subject: [Qemu-devel] [PATCH V6 0/7] nvdimm: guarantee persistence of QEMU writes to persistent memory === TEST SCRIPT BEGIN === #!/bin/ba

[Qemu-devel] [PULL 0/1] tcg-next patch queue

2018-06-01 Thread Richard Henderson
The following changes since commit 392fba9f583223786f844dce9b2e7f9a0ce0147a: Merge remote-tracking branch 'remotes/stsquad/tags/pull-travis-updates-010618-1' into staging (2018-06-01 17:32:30 +0100) are available in the Git repository at: https://github.com/rth7680/qemu.git tags/tcg-next-p

[Qemu-devel] [PULL 1/1] tcg: Pass tb and index to tcg_gen_exit_tb separately

2018-06-01 Thread Richard Henderson
Do the cast to uintptr_t within the helper, so that the compiler can type check the pointer argument. We can also do some more sanity checking of the index argument. Reviewed-by: Laurent Vivier Signed-off-by: Richard Henderson --- include/exec/gen-icount.h | 2 +- tcg/tcg-op.h

Re: [Qemu-devel] [PATCH v3 1/2] qapi: allow empty branches in flat unions

2018-06-01 Thread Eric Blake
On 05/30/2018 11:37 AM, Anton Nefedov wrote: It often happens that just a few discriminator values imply extra data in a flat union. Existing checks did not make possible to leave other values uncovered. Such cases had to be worked around by either stating a dummy (empty) type or introducing anot

Re: [Qemu-devel] [PATCH] iotests: Fix 219's timing

2018-06-01 Thread Eric Blake
On 06/01/2018 07:32 AM, Max Reitz wrote: 219 has two issues that may lead to sporadic failure, both of which are the result of issuing query-jobs too early after a job has been modified. This can then lead to different results based on whether the modification has taken effect already or not. F

Re: [Qemu-devel] [PATCH 1/2] linux-user: Implement signals for openrisc

2018-06-01 Thread Richard Henderson
On 06/01/2018 01:45 PM, Laurent Vivier wrote: > Le 31/05/2018 à 06:18, Richard Henderson a écrit : >> All of the existing code was boilerplate from elsewhere, >> and would crash the guest upon the first signal. >> >> Signed-off-by: Richard Henderson >> --- >> linux-user/openrisc/target_syscall.h

Re: [Qemu-devel] [PATCH 5/6] linux-user: Check is_hostfd in mmap syscalls

2018-06-01 Thread Laurent Vivier
Le 01/06/2018 à 00:49, Richard Henderson a écrit : > Signed-off-by: Richard Henderson > --- > linux-user/syscall.c | 9 + > 1 file changed, 9 insertions(+) > Reviewed-by: Laurent Vivier

Re: [Qemu-devel] [PATCH 4/6] linux-user: Check contains_hostfd in select syscalls

2018-06-01 Thread Laurent Vivier
Le 01/06/2018 à 00:49, Richard Henderson a écrit : > Signed-off-by: Richard Henderson > --- > linux-user/syscall.c | 10 ++ > 1 file changed, 10 insertions(+) Reviewed-by: Laurent Vivier

Re: [Qemu-devel] [PATCH 3/6] linux-user: Check is_hostfd in do_syscall

2018-06-01 Thread Laurent Vivier
Le 01/06/2018 à 00:49, Richard Henderson a écrit : > This is the vast majority of all fd inputs, but not all. > > Signed-off-by: Richard Henderson > --- > linux-user/syscall.c | 303 +++ > 1 file changed, 280 insertions(+), 23 deletions(-) Reviewed-by: La

Re: [Qemu-devel] [PATCH v2 00/16] block: Make blockdev-create a job and stable API

2018-06-01 Thread no-reply
Hi, This series seems to have some coding style problems. See output below for more information: Type: series Message-id: 20180529203910.7615-1-kw...@redhat.com Subject: [Qemu-devel] [PATCH v2 00/16] block: Make blockdev-create a job and stable API === TEST SCRIPT BEGIN === #!/bin/bash BASE=ba

Re: [Qemu-devel] [PATCH 1/2] linux-user: Implement signals for openrisc

2018-06-01 Thread Laurent Vivier
Le 31/05/2018 à 06:18, Richard Henderson a écrit : > All of the existing code was boilerplate from elsewhere, > and would crash the guest upon the first signal. > > Signed-off-by: Richard Henderson > --- > linux-user/openrisc/target_syscall.h | 23 +-- > linux-user/openrisc/signal.c | 2

Re: [Qemu-devel] [PATCH 2/3] pc-bios/s390-ccw/net: Add support for pxelinux-style config files

2018-06-01 Thread Farhan Ali
On 05/31/2018 11:21 PM, Thomas Huth wrote: Just a question do we want to clear cfgbuf here, before calling pxelinux_load_parse_cfg? That's theoretically not necessary. The buffer either gets populated with data, or the function errors out. The code also makes sure that there is a final NUL-ch

Re: [Qemu-devel] [PATCH 1/2] linux-user: Implement signals for openrisc

2018-06-01 Thread Richard Henderson
On 06/01/2018 12:59 PM, Laurent Vivier wrote: > Le 31/05/2018 à 06:18, Richard Henderson a écrit : >> All of the existing code was boilerplate from elsewhere, >> and would crash the guest upon the first signal. >> >> Signed-off-by: Richard Henderson >> --- >> linux-user/openrisc/target_syscall.h

Re: [Qemu-devel] [PATCH 3/3] pc-bios/s390-ccw/net: Try to load pxelinux.cfg file accoring to the UUID

2018-06-01 Thread Farhan Ali
Reviewed-by: Farhan Ali

Re: [Qemu-devel] [PULL 2/3] Fix libusb-1.0.22 deprecated libusb_set_debug with libusb_set_option

2018-06-01 Thread Philippe Mathieu-Daudé
Cc'ing qemu-stable since I can not build it without this patch cherry-picked. On 04/27/2018 07:20 AM, Gerd Hoffmann wrote: > From: John Thomson > > libusb-1.0.22 marked libusb_set_debug deprecated > it is replaced with > libusb_set_option(libusb_context, LIBUSB_OPTION_LOG_LEVEL, libusb_log_level

Re: [Qemu-devel] [PATCH 2/6] linux-user: Add host_fds and manipulators

2018-06-01 Thread Laurent Vivier
Le 01/06/2018 à 00:49, Richard Henderson a écrit : > This allows emulation of guest syscalls to reject > manipulations to fds used by the host. > > Signed-off-by: Richard Henderson > --- > linux-user/qemu.h | 30 ++ > linux-user/main.c | 27 ++-

Re: [Qemu-devel] [PATCH 1/6] gdbstub: Return the fd from gdbserver_start

2018-06-01 Thread Laurent Vivier
Le 01/06/2018 à 00:49, Richard Henderson a écrit : > This will allow us to protect gdbserver_fd from the guest. > > Signed-off-by: Richard Henderson Reviewed-by: Laurent Vivier

Re: [Qemu-devel] [PATCH 1/2] linux-user: Implement signals for openrisc

2018-06-01 Thread Laurent Vivier
Le 31/05/2018 à 06:18, Richard Henderson a écrit : > All of the existing code was boilerplate from elsewhere, > and would crash the guest upon the first signal. > > Signed-off-by: Richard Henderson > --- > linux-user/openrisc/target_syscall.h | 23 +-- > linux-user/openrisc/signal.c | 2

Re: [Qemu-devel] [PULL 50/69] hw/alpha/dp264: Use the TYPE_SMC37C669_SUPERIO

2018-06-01 Thread Richard Henderson
On 06/01/2018 11:51 AM, Emilio G. Cota wrote: > On Tue, Mar 13, 2018 at 23:47:00 +0100, Paolo Bonzini wrote: >> From: Philippe Mathieu-Daudé >> >> Signed-off-by: Philippe Mathieu-Daudé >> Message-Id: <20180308223946.26784-25-f4...@amsat.org> >> Signed-off-by: Paolo Bonzini >> --- >> default-con

[Qemu-devel] [Bug 1773753] Re: virsh start, after virsh managed save hangs and vm goes to paused state with qemu version v2.12.0-813-g5a5c383b13-dirty on powerpc

2018-06-01 Thread Fabiano Rosas
Could not reproduce with: qemu - v2.12.0-865-ge609fa7 host/guest kernel - 4.11 libvirt - 4.4.0 and qemu - v2.12.0-865-ge609fa7 host kernel - v4.17-rc7-22-g3d661e2 guest kernel - 4.11 libvirt - 4.4.0 So I'd say that this is fixed by: https://git.qemu.org/?p=qemu.git;a=commitdiff;h=0efc914 **

Re: [Qemu-devel] [RFC v2 00/12] vhost-user for input & GPU

2018-06-01 Thread no-reply
Hi, This series failed docker-mingw@fedora build test. Please find the testing commands and their output below. If you have Docker installed, you can probably reproduce it locally. Type: series Message-id: 20180601162749.27406-1-marcandre.lur...@redhat.com Subject: [Qemu-devel] [RFC v2 00/12] vh

Re: [Qemu-devel] [PATCH v3 15/22] target/arm: Add ARM_FEATURE_V7VE for v7 Virtualization Extensions

2018-06-01 Thread Aaron Lindsay
On Jun 01 16:59, Peter Maydell wrote: > On 1 June 2018 at 16:34, Aaron Lindsay wrote: > >> switch (extract32(id_isar0, 24, 4)) { > >> case 1: > >> set_feature(&features, ARM_FEATURE_THUMB_DIV); > >> break; > >> case 2: > >> set_feature(&features, ARM_FEATURE_ARM_DIV); > >> set_feat

Re: [Qemu-devel] [RFC PATCH 2/5] kbd-state: add hotkey registry

2018-06-01 Thread Programmingkid
> On Feb 21, 2018, at 12:08 PM, Gerd Hoffmann wrote: > > Add support to register hotkeys and to check whenever a given QKeyCode > combined with the current modifier state is a hotkey. A hotkey can be > any key combined with up to three modifier keys. I have finally reviewed the patches you se

Re: [Qemu-devel] Virtio-net drivers immune to Nethammer?

2018-06-01 Thread Dr. David Alan Gilbert
* Michael S. Tsirkin (m...@redhat.com) wrote: > On Fri, Jun 01, 2018 at 01:15:44PM +, procmem wrote: > > > > > > Stefan Hajnoczi: > > > On Mon, May 21, 2018 at 11:24:43PM +, procmem wrote: > > >> Hi I'm a privacy distro maintainer investigating the implications of the > > >> newly publish

Re: [Qemu-devel] [PULL 50/69] hw/alpha/dp264: Use the TYPE_SMC37C669_SUPERIO

2018-06-01 Thread Emilio G. Cota
On Tue, Mar 13, 2018 at 23:47:00 +0100, Paolo Bonzini wrote: > From: Philippe Mathieu-Daudé > > Signed-off-by: Philippe Mathieu-Daudé > Message-Id: <20180308223946.26784-25-f4...@amsat.org> > Signed-off-by: Paolo Bonzini > --- > default-configs/alpha-softmmu.mak | 3 +++ > hw/alpha/dp264.c

[Qemu-devel] [RFC] xen: Don't use memory_region_init_ram_nomigrate() in pci_assign_dev_load_option_rom()

2018-06-01 Thread Peter Maydell
The xen pci_assign_dev_load_option_rom() currently creates a RAM memory region with memory_region_init_ram_nomigrate(), and then manually registers it with vmstate_register_ram(). In fact for its only callsite, the 'owner' pointer we use for the init call and the '&dev->qdev' pointer we use for the

Re: [Qemu-devel] [PULL 56/56] hw: make virtio devices configurable via default-configs/

2018-06-01 Thread Eric Blake
On 06/01/2018 12:36 PM, Philippe Mathieu-Daudé wrote: On 06/01/2018 02:31 PM, Paolo Bonzini wrote: On 01/06/2018 19:26, Philippe Mathieu-Daudé wrote: On 06/01/2018 02:15 PM, Paolo Bonzini wrote: This is only half of the work, because the proxy devices (virtio-*-pci, virtio-*-ccw, etc.) are sti

Re: [Qemu-devel] unique (or otherwise) RAM block names

2018-06-01 Thread Dr. David Alan Gilbert
* Peter Maydell (peter.mayd...@linaro.org) wrote: > I was going through trying to fix up various devices that currently > fail to register RAM blocks for migration, or register the RAM > block globally rather than locally, and I ran into something > unexpected. > > We name RAM blocks in qemu_ram_s

Re: [Qemu-devel] [PULL 56/56] hw: make virtio devices configurable via default-configs/

2018-06-01 Thread Philippe Mathieu-Daudé
On 06/01/2018 02:31 PM, Paolo Bonzini wrote: > On 01/06/2018 19:26, Philippe Mathieu-Daudé wrote: >> On 06/01/2018 02:15 PM, Paolo Bonzini wrote: >>> This is only half of the work, because the proxy devices (virtio-*-pci, >>> virtio-*-ccw, etc.) are still included unconditionally. It is still a >>

[Qemu-devel] unique (or otherwise) RAM block names

2018-06-01 Thread Peter Maydell
I was going through trying to fix up various devices that currently fail to register RAM blocks for migration, or register the RAM block globally rather than locally, and I ran into something unexpected. We name RAM blocks in qemu_ram_set_idstr() like this: * if the user passed a device pointer,

Re: [Qemu-devel] [PULL 56/56] hw: make virtio devices configurable via default-configs/

2018-06-01 Thread Paolo Bonzini
On 01/06/2018 19:26, Philippe Mathieu-Daudé wrote: > On 06/01/2018 02:15 PM, Paolo Bonzini wrote: >> This is only half of the work, because the proxy devices (virtio-*-pci, >> virtio-*-ccw, etc.) are still included unconditionally. It is still a >> move in the right direction. >> >> Based-on: <201

Re: [Qemu-devel] [PATCH v5 1/2] qga: add mountpoint usage to GuestFilesystemInfo

2018-06-01 Thread Chen Hanxiao
在 2018-06-01 23:47:04,"Eric Blake" 写道: >On 06/01/2018 10:27 AM, Chen Hanxiao wrote: >> From: Chen Hanxiao >> >> This patch adds support for getting the usage of mounted >> filesystem. >> The usage of fs stored as used_bytes and total_bytes. >> It's very useful when we try to monitor guest's fi

Re: [Qemu-devel] [PULL 56/56] hw: make virtio devices configurable via default-configs/

2018-06-01 Thread Philippe Mathieu-Daudé
On 06/01/2018 02:15 PM, Paolo Bonzini wrote: > This is only half of the work, because the proxy devices (virtio-*-pci, > virtio-*-ccw, etc.) are still included unconditionally. It is still a > move in the right direction. > > Based-on: <20180522194943.24871-1-pbonz...@redhat.com> Is this tag use

Re: [Qemu-devel] [PATCH v5.1 2/2] qga-win: add driver path usage to GuestFilesystemInfo

2018-06-01 Thread Eric Blake
On 06/01/2018 11:57 AM, Chen Hanxiao wrote: From: Chen Hanxiao This patch adds support for getting the usage of windows driver path. The usage of fs stored as used_bytes and total_bytes. Cc: Michael Roth Signed-off-by: Chen Hanxiao --- v2: unpack usage as used-bytes and total-bytes @@

[Qemu-devel] [PULL 56/56] hw: make virtio devices configurable via default-configs/

2018-06-01 Thread Paolo Bonzini
This is only half of the work, because the proxy devices (virtio-*-pci, virtio-*-ccw, etc.) are still included unconditionally. It is still a move in the right direction. Based-on: <20180522194943.24871-1-pbonz...@redhat.com> Signed-off-by: Paolo Bonzini --- default-configs/arm-softmmu.mak

Re: [Qemu-devel] [Bug 1769189] Re: Issue with qemu 2.12.0 + SATA

2018-06-01 Thread Andrew Randrianasulu
For me issue was fixed with «[PATCH 2/3] ahci: fix PxCI register race» I additionally patched my qemu build with "[PATCH] e1000e: Do not auto-clear ICR bits which aren't set in EIAC" - so new network adapter for q35 machine works with KDE Neon liveDVD for example

[Qemu-devel] [PULL 55/56] hw: allow compiling out SCSI

2018-06-01 Thread Paolo Bonzini
Reviewed-by: Stefan Hajnoczi Signed-off-by: Paolo Bonzini --- default-configs/mips-softmmu-common.mak | 1 + default-configs/pci.mak | 1 + default-configs/riscv32-softmmu.mak | 1 + default-configs/riscv64-softmmu.mak | 1 + default-configs/s390x-softmmu.mak | 1 +

Re: [Qemu-devel] Mistake on man page

2018-06-01 Thread Daniel P . Berrangé
On Fri, Jun 01, 2018 at 04:47:55PM +0200, kvaps wrote: > Hi, here is mistake in man page for qemu-nbd: > > --- a/qemu-nbd.texi > +++ b/qemu-nbd.texi > @@ -1,2 +1,2 @@ > --- a/qemu-nbd.texi > -Don't exit on the last connection > +++ b/qemu-nbd.texi > +Don't exit on the lost connection It isn't a m

[Qemu-devel] [PULL 10/56] bsd-user: include "exec/exec-all.h" which provides mmap_lock/unlock"

2018-06-01 Thread Paolo Bonzini
Signed-off-by: Paolo Bonzini --- bsd-user/mmap.c | 1 + 1 file changed, 1 insertion(+) diff --git a/bsd-user/mmap.c b/bsd-user/mmap.c index 20cd29d145..17f4cd80aa 100644 --- a/bsd-user/mmap.c +++ b/bsd-user/mmap.c @@ -21,6 +21,7 @@ #include "qemu.h" #include "qemu-common.h" #include "bsd-mman

[Qemu-devel] [PULL v2 00/56] Misc patches for 2018-05-31

2018-06-01 Thread Paolo Bonzini
The following changes since commit 5a5c383b1373aeb6c87a0d6060f6c3dc7c53082b: Merge remote-tracking branch 'remotes/vivier2/tags/linux-user-for-2.13-pull-request' into staging (2018-05-25 10:04:36 +0100) are available in the Git repository at: git://github.com/bonzini/qemu.git tags/for-upst

[Qemu-devel] [PATCH v5.1 0/2] qga: report the usage of fs in guests

2018-06-01 Thread Chen Hanxiao
This series report the usage of guest's fs by "used_bytes" and "total_bytes" in guest-get-fsinfo. Chen Hanxiao (2): qga: add mountpoint usage info to GuestFilesystemInfo qga-win: add driver path usage to GuestFilesystemInfo qga/commands-posix.c | 18 ++ qga/commands-win32.c |

[Qemu-devel] Mistake on man page

2018-06-01 Thread kvaps
Hi, here is mistake in man page for qemu-nbd: --- a/qemu-nbd.texi +++ b/qemu-nbd.texi @@ -1,2 +1,2 @@ --- a/qemu-nbd.texi -Don't exit on the last connection +++ b/qemu-nbd.texi +Don't exit on the lost connection Regards -- kvaps

[Qemu-devel] [Bug 1774677] [NEW] -icount increases boot time by >10x

2018-06-01 Thread Andreas Gustafsson
Public bug reported: When I specify the -icount option, some guest operations such as booting a Linux kernel take more than 10 times longer than otherwise. For example, the following will boot Aboriginal Linux to the login prompt about 6 seconds on my system (using TCG, not KVM): wget http://la

Re: [Qemu-devel] [PATCH v5.1 1/2] qga: add mountpoint usage info to GuestFilesystemInfo

2018-06-01 Thread Eric Blake
On 06/01/2018 11:57 AM, Chen Hanxiao wrote: From: Chen Hanxiao This patch adds support for getting the usage of mounted filesystem. The usage of fs stored as used_bytes and total_bytes. It's very useful when we try to monitor guest's filesystem. Cc: Michael Roth Cc: Eric Blake Cc: Daniel P.

Re: [Qemu-devel] [PATCH 1/6] gdbstub: Return the fd from gdbserver_start

2018-06-01 Thread Richard Henderson
On 06/01/2018 01:59 AM, Peter Maydell wrote: > On 31 May 2018 at 23:49, Richard Henderson > wrote: >> This will allow us to protect gdbserver_fd from the guest. > > Ha, I hadn't realised we already had an internal-to-QEMU filedescriptor :-) Two, in fact, if you count qemu_logfile. ;-) r~

[Qemu-devel] [RFC v2 04/12] Add vhost-user-backend

2018-06-01 Thread Marc-André Lureau
Create a vhost-user-backend object that holds a connection to a vhost-user backend and can be referenced from virtio devices that support it. See later patches for input & gpu usage. A chardev can be specified to communicate with the vhost-user backend, ex: -chardev socket,id=char0,path=/tmp/foo.s

[Qemu-devel] [PULL 31/31] vhost-blk: turn on pre-defined RO feature bit

2018-06-01 Thread Michael S. Tsirkin
From: Changpeng Liu Read only feature shouldn't be negotiable, because if the backend device reported Read only feature supported, QEMU host driver shouldn't change backend's RO attribute. While here, also enable the vhost-user-blk test utility to test RO feature. Signed-off-by: Changpeng Liu R

[Qemu-devel] [PULL 30/31] ACPI testing: test NFIT platform capabilities

2018-06-01 Thread Michael S. Tsirkin
From: Ross Zwisler Add testing for the newly added NFIT Platform Capabilities Structure. Signed-off-by: Ross Zwisler Suggested-by: Igor Mammedov Reviewed-by: Michael S. Tsirkin Signed-off-by: Michael S. Tsirkin --- tests/bios-tables-test.c | 2 +- tests/acpi-test-data/pc/NFIT

[Qemu-devel] [PULL 29/31] nvdimm, acpi: support NFIT platform capabilities

2018-06-01 Thread Michael S. Tsirkin
From: Ross Zwisler Add a machine command line option to allow the user to control the Platform Capabilities Structure in the virtualized NFIT. This Platform Capabilities Structure was added in ACPI 6.2 Errata A. Signed-off-by: Ross Zwisler Reviewed-by: Michael S. Tsirkin Signed-off-by: Michae

[Qemu-devel] [PULL 24/31] colo: use local path for local headers

2018-06-01 Thread Michael S. Tsirkin
When pulling in headers that are in the same directory as the C file (as opposed to one in include/), we should use its relative path, without a directory. Reviewed-by: Zhang Chen Signed-off-by: Michael S. Tsirkin Reviewed-by: Philippe Mathieu-Daudé Tested-by: Philippe Mathieu-Daudé --- net/c

[Qemu-devel] [Bug 1772075] Re: Segmentation fault on aarch64 vm at powerdown

2018-06-01 Thread M0Rf30
QEMU_EFI.fd: http://snapshots.linaro.org/components/kernel/leg-virt-tianocore-edk2-upstream/latest/QEMU-AARCH64/RELEASE_GCC5/QEMU_EFI.fd sd-image: generated as this says: https://archlinuxarm.org/platforms/armv8/broadcom/raspberry-pi-3 (same error also with latest raspbian image) kernel image: ht

[Qemu-devel] [RFC v2 08/12] contrib: add vhost-user-input

2018-06-01 Thread Marc-André Lureau
Add a vhost-user input backend example, based on virtio-input-host device. It takes an evdev path as argument, and can be associated with a vhost-user-backend object, ex: -object vhost-user-backend,id=vuid,cmd="vhost-user-input /dev/input/event0" Signed-off-by: Marc-André Lureau --- contrib/lib

[Qemu-devel] [PULL 27/31] arch_init: sort architectures

2018-06-01 Thread Michael S. Tsirkin
Sort alphabetically. Will help us see if anything is missing (e.g. tile is not there now). Signed-off-by: Michael S. Tsirkin Reviewed-by: Laszlo Ersek --- arch_init.c | 16 1 file changed, 8 insertions(+), 8 deletions(-) diff --git a/arch_init.c b/arch_init.c index 9597218ced.

[Qemu-devel] [PATCH v5.1 2/2] qga-win: add driver path usage to GuestFilesystemInfo

2018-06-01 Thread Chen Hanxiao
From: Chen Hanxiao This patch adds support for getting the usage of windows driver path. The usage of fs stored as used_bytes and total_bytes. Cc: Michael Roth Signed-off-by: Chen Hanxiao --- v2: unpack usage as used-bytes and total-bytes qga/commands-win32.c | 12 1 file chan

[Qemu-devel] [RFC v2 07/12] libvhost-user: export vug_source_new

2018-06-01 Thread Marc-André Lureau
Simplify the creation of FD sources for other users. Signed-off-by: Marc-André Lureau --- contrib/libvhost-user/libvhost-user-glib.h | 3 +++ contrib/libvhost-user/libvhost-user-glib.c | 15 +++ 2 files changed, 14 insertions(+), 4 deletions(-) diff --git a/contrib/libvhost-user/li

[Qemu-devel] [PATCH v5.1 1/2] qga: add mountpoint usage info to GuestFilesystemInfo

2018-06-01 Thread Chen Hanxiao
From: Chen Hanxiao This patch adds support for getting the usage of mounted filesystem. The usage of fs stored as used_bytes and total_bytes. It's very useful when we try to monitor guest's filesystem. Cc: Michael Roth Cc: Eric Blake Cc: Daniel P. Berrangé Signed-off-by: Chen Hanxiao --- v2:

[Qemu-devel] [PULL 25/31] qga: use local path for local headers

2018-06-01 Thread Michael S. Tsirkin
When pulling in headers that are in the same directory as the C file (as opposed to one in include/), we should use its relative path, without a directory. Signed-off-by: Michael S. Tsirkin Reviewed-by: Philippe Mathieu-Daudé --- qga/channel-posix.c | 2 +- qga/channel-win32.c

[Qemu-devel] [RFC v2 00/12] vhost-user for input & GPU

2018-06-01 Thread Marc-André Lureau
Hi, vhost-user allows to drive a virtio device in a seperate process. After vhost-user-net, we have seen vhost-user-{scsi,blk,crypto} added more recently. This series, initially proposed 2 years ago (https://lists.gnu.org/archive/html/qemu-devel/2016-06/msg01905.html) contributes with vhost-user-

[Qemu-devel] [PULL 22/31] usb: use local path for local headers

2018-06-01 Thread Michael S. Tsirkin
When pulling in headers that are in the same directory as the C file (as opposed to one in include/), we should use its relative path, without a directory. Signed-off-by: Michael S. Tsirkin Reviewed-by: Philippe Mathieu-Daudé Tested-by: Philippe Mathieu-Daudé --- hw/usb/desc-msos.c

[Qemu-devel] [RFC v2 12/12] contrib: add vhost-user-gpu

2018-06-01 Thread Marc-André Lureau
Add a vhost-user gpu backend example, based on virtio-gpu/3d device. It is to be associated with a vhost-user-backend object, ex: -object vhost-user-backend,id=vug,cmd="vhost-user-gpu" TODO: - add/check multi-head support - add/check resize support - add missing options, such as virgl=off,rendern

[Qemu-devel] [PULL 23/31] migration: use local path for local headers

2018-06-01 Thread Michael S. Tsirkin
When pulling in headers that are in the same directory as the C file (as opposed to one in include/), we should use its relative path, without a directory. Signed-off-by: Michael S. Tsirkin Reviewed-by: Juan Quintela --- migration/block-dirty-bitmap.c | 2 +- migration/page_cache.c | 2

[Qemu-devel] [RFC v2 11/12] Add virtio-gpu vhost-user backend

2018-06-01 Thread Marc-André Lureau
Add to virtio-gpu devices a "vhost-user" property. When set, the associated vhost-user backend is used to handle the virtio rings. For now, a socketpair is created for the backend to share the rendering results with qemu via a simple VHOST_GPU protocol. Example usage: -object vhost-user-backend,i

[Qemu-devel] [PULL 15/31] ide: use local path for local headers

2018-06-01 Thread Michael S. Tsirkin
When pulling in headers that are in the same directory as the C file (as opposed to one in include/), we should use its relative path, without a directory. Signed-off-by: Michael S. Tsirkin Reviewed-by: Philippe Mathieu-Daudé Tested-by: Philippe Mathieu-Daudé --- hw/ide/ahci-allwinner.c | 2 +-

[Qemu-devel] [PULL 28/31] tests/.gitignore: add entry for generated file

2018-06-01 Thread Michael S. Tsirkin
From: Ross Zwisler After a "make check" we end up with the following: $ git status On branch master Your branch is up-to-date with 'origin/master'. Untracked files: (use "git add ..." to include in what will be committed) tests/test-block-backend nothing added to commit but untracke

[Qemu-devel] [RFC v2 10/12] vhost-user: add vhost_user_gpu_set_socket()

2018-06-01 Thread Marc-André Lureau
Add a new vhost-user message to give a unix socket for gpu updates to a vhost-user backend. Back when I started that work, I added a new GPU channel because the vhost-user protocol wasn't bidirectional. Since then, there is a vhost-user-slave channel for the slave to send requests to the master. W

[Qemu-devel] [PULL 20/31] vhost-scsi: drop an unused include

2018-06-01 Thread Michael S. Tsirkin
No reason for vhost-scsi to pull in migration headers directly. Signed-off-by: Michael S. Tsirkin Reviewed-by: Philippe Mathieu-Daudé Tested-by: Philippe Mathieu-Daudé --- hw/scsi/vhost-scsi-common.c | 1 - 1 file changed, 1 deletion(-) diff --git a/hw/scsi/vhost-scsi-common.c b/hw/scsi/vhost

[Qemu-devel] [PULL 21/31] sd: fix up include

2018-06-01 Thread Michael S. Tsirkin
include files shouldn't have the "include/" part, that is implied. Signed-off-by: Michael S. Tsirkin Reviewed-by: Philippe Mathieu-Daudé Tested-by: Philippe Mathieu-Daudé --- hw/sd/milkymist-memcard.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/hw/sd/milkymist-memcard.c

[Qemu-devel] [RFC v2 05/12] vhost-user: split vhost_user_read()

2018-06-01 Thread Marc-André Lureau
Split vhost_user_read(), so only header can be read with vhost_user_read_header(). Signed-off-by: Marc-André Lureau --- hw/virtio/vhost-user.c | 27 +++ 1 file changed, 19 insertions(+), 8 deletions(-) diff --git a/hw/virtio/vhost-user.c b/hw/virtio/vhost-user.c index cc

[Qemu-devel] [PULL 19/31] ppc: use local path for local headers

2018-06-01 Thread Michael S. Tsirkin
When pulling in headers that are in the same directory as the C file (as opposed to one in include/), we should use its relative path, without a directory. Signed-off-by: Michael S. Tsirkin Acked-by: David Gibson Reviewed-by: Philippe Mathieu-Daudé Tested-by: Philippe Mathieu-Daudé --- hw/ppc

[Qemu-devel] [PULL 16/31] ioapic: fix up includes

2018-06-01 Thread Michael S. Tsirkin
include files shouldn't have the "include/" part, that is implied. Also, drop an unused include. Signed-off-by: Michael S. Tsirkin Reviewed-by: Philippe Mathieu-Daudé Tested-by: Philippe Mathieu-Daudé --- hw/intc/ioapic.c | 3 +-- 1 file changed, 1 insertion(+), 2 deletions(-) diff --git a/h

[Qemu-devel] [PULL 26/31] ui: use local path for local headers

2018-06-01 Thread Michael S. Tsirkin
When pulling in headers that are in the same directory as the C file (as opposed to one in include/), we should use its relative path, without a directory. Signed-off-by: Michael S. Tsirkin Reviewed-by: Philippe Mathieu-Daudé --- ui/gtk.c | 2 +- ui/input-keymap.c | 2 +- ui/input-lega

[Qemu-devel] [PULL 18/31] rocker: drop an unused include

2018-06-01 Thread Michael S. Tsirkin
We don't use net/clients.h, drop that include. Signed-off-by: Michael S. Tsirkin Reviewed-by: Philippe Mathieu-Daudé Tested-by: Philippe Mathieu-Daudé --- hw/net/rocker/rocker_fp.c | 1 - 1 file changed, 1 deletion(-) diff --git a/hw/net/rocker/rocker_fp.c b/hw/net/rocker/rocker_fp.c index 27

[Qemu-devel] [PULL 13/31] trace: use local path for local headers

2018-06-01 Thread Michael S. Tsirkin
When pulling in headers that are in the same directory as the C file (as opposed to one in include/), we should use its relative path, without a directory. Signed-off-by: Michael S. Tsirkin Reviewed-by: Stefan Hajnoczi Reviewed-by: Philippe Mathieu-Daudé Tested-by: Philippe Mathieu-Daudé ---

[Qemu-devel] [RFC v2 09/12] Add vhost-input-pci

2018-06-01 Thread Marc-André Lureau
Use a vhost-user input backend. Usage: -object vhost-user-backend,id=vuid -device vhost-input-pci,vhost-user=vuid Signed-off-by: Marc-André Lureau --- hw/virtio/virtio-pci.h | 10 +++ include/hw/virtio/virtio-input.h | 14 hw/input/vhost-user-input.c | 124

[Qemu-devel] [PULL 17/31] e1000e: use local path for local headers

2018-06-01 Thread Michael S. Tsirkin
When pulling in headers that are in the same directory as the C file (as opposed to one in include/), we should use its relative path, without a directory. Signed-off-by: Michael S. Tsirkin Reviewed-by: Philippe Mathieu-Daudé Tested-by: Philippe Mathieu-Daudé --- hw/net/e1000e.c | 2 +- 1 file

[Qemu-devel] [PULL 14/31] display: use local path for local headers

2018-06-01 Thread Michael S. Tsirkin
When pulling in headers that are in the same directory as the C file (as opposed to one in include/), we should use its relative path, without a directory. Signed-off-by: Michael S. Tsirkin Reviewed-by: Philippe Mathieu-Daudé Tested-by: Philippe Mathieu-Daudé --- hw/display/bcm2835_fb.c | 2 +-

[Qemu-devel] [PULL 12/31] migration: drop an unused include

2018-06-01 Thread Michael S. Tsirkin
In the vmstate.h file, we just need a struct name. Use a forward declaration instead of an include, then adjust the one affected .c file to include the file that is no longer implicit from the header. Signed-off-by: Michael S. Tsirkin Reviewed-by: Eric Blake --- include/migration/vmstate.h | 2

[Qemu-devel] [RFC v2 06/12] vhost-user: add vhost_user_input_get_config()

2018-06-01 Thread Marc-André Lureau
Ask vhost user input backend the list of virtio_input_config. Signed-off-by: Marc-André Lureau --- include/hw/virtio/vhost-backend.h | 4 +++ hw/virtio/vhost-user.c| 59 +++ docs/interop/vhost-user.txt | 6 3 files changed, 69 insertions(+) d

[Qemu-devel] [PULL 09/31] block: use local path for local headers

2018-06-01 Thread Michael S. Tsirkin
When pulling in headers that are in the same directory as the C file (as opposed to one in include/), we should use its relative path, without a directory. Signed-off-by: Michael S. Tsirkin Reviewed-by: Philippe Mathieu-Daudé Tested-by: Philippe Mathieu-Daudé --- block/crypto.c | 2 +-

[Qemu-devel] [PULL 11/31] hppa: use local path for local headers

2018-06-01 Thread Michael S. Tsirkin
When pulling in headers that are in the same directory as the C file (as opposed to one in include/), we should use its relative path, without a directory. Signed-off-by: Michael S. Tsirkin Reviewed-by: Philippe Mathieu-Daudé Tested-by: Philippe Mathieu-Daudé --- hw/hppa/hppa_sys.h | 2 +- hw/

[Qemu-devel] [PULL 10/31] crypto: use local path for local headers

2018-06-01 Thread Michael S. Tsirkin
When pulling in headers that are in the same directory as the C file (as opposed to one in include/), we should use its relative path, without a directory. Signed-off-by: Michael S. Tsirkin Reviewed-by: Philippe Mathieu-Daudé Tested-by: Philippe Mathieu-Daudé Acked-by: Daniel P. Berrangé ---

[Qemu-devel] [RFC v2 03/12] vhost-user: wrap some read/write with retry handling

2018-06-01 Thread Marc-André Lureau
Signed-off-by: Marc-André Lureau --- hw/virtio/vhost-user.c | 15 --- 1 file changed, 12 insertions(+), 3 deletions(-) diff --git a/hw/virtio/vhost-user.c b/hw/virtio/vhost-user.c index ca554d4ff1..cc9298792d 100644 --- a/hw/virtio/vhost-user.c +++ b/hw/virtio/vhost-user.c @@ -869,7

[Qemu-devel] [PULL 02/31] vhost-user: allow slave to send fds via slave channel

2018-06-01 Thread Michael S. Tsirkin
From: Tiwei Bie Introduce VHOST_USER_PROTOCOL_F_SLAVE_SEND_FD protocol feature to allow slave to send at most 8 descriptors in each message to master via ancillary data using the slave channel. Suggested-by: Michael S. Tsirkin Signed-off-by: Tiwei Bie Reviewed-by: Michael S. Tsirkin Signed-of

[Qemu-devel] [PULL 07/31] osdep: add wait.h compat macros

2018-06-01 Thread Michael S. Tsirkin
Man page for WCOREDUMP says: WCOREDUMP(wstatus) returns true if the child produced a core dump. This macro should be employed only if WIFSIGNALED returned true. This macro is not specified in POSIX.1-2001 and is not available on some UNIX implementations (e.g., AIX, SunOS). Therefor

[Qemu-devel] [PULL 08/31] libqtest: fail if child coredumps

2018-06-01 Thread Michael S. Tsirkin
Right now tests report OK status if QEMU crashes during cleanup. Let's catch that case and fail the test. Signed-off-by: Michael S. Tsirkin Acked-by: Thomas Huth --- tests/libqtest.c | 9 - 1 file changed, 8 insertions(+), 1 deletion(-) diff --git a/tests/libqtest.c b/tests/libqtest.c

[Qemu-devel] [RFC v2 02/12] libvhost-user: exit by default on VHOST_USER_NONE

2018-06-01 Thread Marc-André Lureau
Since commit 2566378d6d13bf4d28c7770bdbda5f7682594bbe, libvhost-user no longer panics on disconnect (rc == 0), and instead silently ignores an invalid VHOST_USER_NONE message. Without extra work from the API user, this will simply busy-loop on HUP events. The obvious thing to do is to exit(0) inst

[Qemu-devel] [PULL 01/31] vhost: allow backends to filter memory sections

2018-06-01 Thread Michael S. Tsirkin
From: Tiwei Bie This patch introduces a vhost op for vhost backends to allow them to filter the memory sections that they can handle. Signed-off-by: Tiwei Bie Reviewed-by: Michael S. Tsirkin Signed-off-by: Michael S. Tsirkin --- include/hw/virtio/vhost-backend.h | 4 hw/virtio/vhost-us

  1   2   3   4   >