[Qemu-devel] [PATCH v5 05/14] docker: Add test runner

2016-05-23 Thread Fam Zheng
It's better to have a launcher for all tests, to make it easier to initialize and manage the environment. If "DEBUG=1" a shell prompt will show up before the test runs. Reviewed-by: Alex Bennée Signed-off-by: Fam Zheng --- tests/docker/run | 58

[Qemu-devel] [PATCH v5 11/14] docker: Add travis tool

2016-05-23 Thread Fam Zheng
The script is not prefixed with test- so it won't run with "make docker-test", because it can take too long. Run it with "make docker-travis@ubuntu". Reviewed-by: Alex Bennée Signed-off-by: Fam Zheng --- tests/docker/travis| 21 + tests/docker/travis.py | 48 +++

[Qemu-devel] [PATCH v5 03/14] Makefile: Rules for docker testing

2016-05-23 Thread Fam Zheng
This adds a group of make targets to run docker tests, all are available in source tree without running ./configure. The usage is shown with "make docker". Besides the fixed ones, dynamic targets for building each image and running each test in each image are generated automatically by make, scan

[Qemu-devel] [PATCH v5 09/14] docker: Add clang test

2016-05-23 Thread Fam Zheng
The (currently partially commented out) configure options are suggested by John Snow . Signed-off-by: Fam Zheng --- tests/docker/test-clang | 26 ++ 1 file changed, 26 insertions(+) create mode 100755 tests/docker/test-clang diff --git a/tests/docker/test-clang b/tests/

[Qemu-devel] [PATCH v5 14/14] MAINTAINERS: Add tests/docker

2016-05-23 Thread Fam Zheng
Reviewed-by: Alex Bennée Signed-off-by: Fam Zheng --- MAINTAINERS | 7 +++ 1 file changed, 7 insertions(+) diff --git a/MAINTAINERS b/MAINTAINERS index 81e7fac..55f98b6 100644 --- a/MAINTAINERS +++ b/MAINTAINERS @@ -1616,3 +1616,10 @@ Build system architecture M: Daniel P. Berrange S: Od

[Qemu-devel] [PATCH v5 08/14] docker: Add full test

2016-05-23 Thread Fam Zheng
This builds all available targets. Signed-off-by: Fam Zheng --- tests/docker/test-full | 17 + 1 file changed, 17 insertions(+) create mode 100755 tests/docker/test-full diff --git a/tests/docker/test-full b/tests/docker/test-full new file mode 100755 index 000..fd9b798 ---

[Qemu-devel] [PATCH v5 10/14] docker: Add mingw test

2016-05-23 Thread Fam Zheng
Reviewed-by: Alex Bennée Signed-off-by: Fam Zheng --- tests/docker/test-mingw | 34 ++ 1 file changed, 34 insertions(+) create mode 100755 tests/docker/test-mingw diff --git a/tests/docker/test-mingw b/tests/docker/test-mingw new file mode 100755 index 000..

[Qemu-devel] [PATCH v5 04/14] docker: Add images

2016-05-23 Thread Fam Zheng
Reviewed-by: Alex Bennée Signed-off-by: Fam Zheng --- tests/docker/dockerfiles/centos6.docker | 6 ++ tests/docker/dockerfiles/fedora.docker | 7 +++ tests/docker/dockerfiles/ubuntu.docker | 11 +++ 3 files changed, 24 insertions(+) create mode 100644 tests/docker/dockerfile

[Qemu-devel] [PATCH v5 13/14] docker: Add EXTRA_CONFIGURE_OPTS

2016-05-23 Thread Fam Zheng
Whatever passed in this variable will be appended to all configure commands. Signed-off-by: Fam Zheng Reviewed-by: Alex Bennée --- tests/docker/Makefile.include | 3 +++ tests/docker/common.rc| 1 + 2 files changed, 4 insertions(+) diff --git a/tests/docker/Makefile.include b/tests/doc

[Qemu-devel] [PATCH v5 06/14] docker: Add common.rc

2016-05-23 Thread Fam Zheng
"requires" checks the "FEATURE" environment for specified prerequisits, and skip the execution of test if not found. "build_qemu" is the central routine to compile QEMU for tests to call. Reviewed-by: Alex Bennée Signed-off-by: Fam Zheng --- tests/docker/common.rc | 31

[Qemu-devel] [PATCH v5 12/14] docs: Add text for tests/docker in build-system.txt

2016-05-23 Thread Fam Zheng
Reviewed-by: Alex Bennée Signed-off-by: Fam Zheng --- docs/build-system.txt | 5 + 1 file changed, 5 insertions(+) diff --git a/docs/build-system.txt b/docs/build-system.txt index 5ea..2af1e66 100644 --- a/docs/build-system.txt +++ b/docs/build-system.txt @@ -438,6 +438,11 @@ top level

Re: [Qemu-devel] [PATCH v4 2/6] Migration: Split out ram part of qmp_query_migrate

2016-05-23 Thread Amit Shah
On (Tue) 17 May 2016 [13:55:51], Dr. David Alan Gilbert (git) wrote: > From: "Dr. David Alan Gilbert" > > The RAM section of qmp_query_migrate is reasonably complex > and repeated 3 times. Split it out into a helper. > > Signed-off-by: Dr. David Alan Gilbert > Reviewed-by: Eric Blake > Reviwe

Re: [Qemu-devel] [PATCH 1/1] migration: fix ram decompression race deadlock

2016-05-23 Thread Amit Shah
Adding Liang Li to CC for his comments as the author of the feature. On (Fri) 13 May 2016 [10:27:01], Denis V. Lunev wrote: > From: Maxim Nestratov > > There is a race in between do_data_decompress and start_decompression. > > do_data_decompress() > while (!quit_decomp_thread) { > q

Re: [Qemu-devel] [PATCH 3/3] virtio-gpu: Warn if UI config will disable virgl

2016-05-23 Thread Gerd Hoffmann
Hi, > > We could turn the virgl switch into tristate (OnOffAuto), have it > > default to Auto, then fail (not warn) in case it is set to On without 3d > > support being available. > > > > Property settings are part of the migration format, right? Is there an easy > way to change the 'virgl' pr

Re: [Qemu-devel] [PATCH v1 5/6] scripts: ensure monitor socket has SO_REUSEADDR set

2016-05-23 Thread Amit Shah
On (Thu) 05 May 2016 [15:27:59], Daniel P. Berrange wrote: > If tests use a TCP based monitor socket, the connection will > go into a TIMED_WAIT state when the test exits. This will > randomly prevent the test from being re-run without a certain > time period. Set the SO_REUSEADDR flag on the socke

Re: [Qemu-devel] [PATCH 5/9] virtio-blk: multiqueue batch notify

2016-05-23 Thread Paolo Bonzini
On 23/05/2016 04:43, Fam Zheng wrote: > > The batch notification BH needs to know which virtqueues to notify when > > multiqueue is enabled. Use a bitmap to track the virtqueues that with > > pending notifications. > > This approach works great as long as VQs are in the same iothread. An > alte

Re: [Qemu-devel] [PATCH v2 0/3] Minor nmi cleanups

2016-05-23 Thread Paolo Bonzini
On 20/05/2016 18:28, Bandan Das wrote: > v2: > 2/4: Remove inject_nmi and call nmi_monitor_handle() > directly > 3/4: rm -rf > 4/4: Call nmi_monitor_handle and pass cpu index > > The primary change is a arch specific x86 nmi function > which can be called by the core nmi handler. > > Bandan Das

Re: [Qemu-devel] [PATCH 0/2] migration: Move qjson.[ch] to migration/, clean up

2016-05-23 Thread Amit Shah
On (Fri) 06 May 2016 [15:11:04], Markus Armbruster wrote: > Eric Blake writes: > > > On 05/04/2016 10:49 AM, Markus Armbruster wrote: > >> qjson.[ch] is a JSON writer used by migration. Eric proposed to > >> replace it by common code in his "Add qapi-to-JSON and clone visitors" > >> series. Dav

Re: [Qemu-devel] [PATCH] xen/blkif: avoid double access to any shared ring request fields

2016-05-23 Thread Stefano Stabellini
On Mon, 23 May 2016, Jan Beulich wrote: > Commit f9e98e5d7a ("xen/blkif: Avoid double access to > src->nr_segments") didn't go far enough: src->operation is also being > used twice. And nothing was done to prevent the compiler from using the > source side of the copy done by blk_get_request() (gran

Re: [Qemu-devel] [PATCH v2 0/9] live migration bug fix and refine

2016-05-23 Thread Amit Shah
I'll wait for Juan's review before picking this up. On (Thu) 05 May 2016 [15:32:50], Liang Li wrote: > This patch set fixed a bug which will block live migration and another > potential issue when using multi-thread (de)compression. > > The last patches try to refine the code and make the using o

Re: [Qemu-devel] [PATCH 5/9] virtio-blk: multiqueue batch notify

2016-05-23 Thread Fam Zheng
On Mon, 05/23 10:17, Paolo Bonzini wrote: > > > On 23/05/2016 04:43, Fam Zheng wrote: > > > The batch notification BH needs to know which virtqueues to notify when > > > multiqueue is enabled. Use a bitmap to track the virtqueues that with > > > pending notifications. > > > > This approach work

Re: [Qemu-devel] RFC: Make 'info snapshots' show all of snapshots with multiple devices info

2016-05-23 Thread Kevin Wolf
Am 22.05.2016 um 11:55 hat Lin Ma geschrieben: > Currently, the output of 'info snapshots' show fully available snapshots. > > In my opinion there are 2 disadvantages: > 1. It's opaque, hides some snapshot information to users. It's not convenient > if users want to know more about all of snapsho

Re: [Qemu-devel] [PATCH v5 5/6] acpi: Add IPMI table entries

2016-05-23 Thread Marcel Apfelbaum
On 05/22/2016 03:28 AM, Corey Minyard wrote: Thanks for all the comments. I didn't know about stubs, as there's nothing that currently uses it in hw directory, but it's easy enough to add. I did have two comment below: On 05/20/2016 04:53 AM, Igor Mammedov wrote: On Thu, 19 May 2016 10:24:01

[Qemu-devel] [PATCH] usb/ohci: Fix crash with when specifying too many num-ports

2016-05-23 Thread Thomas Huth
QEMU currently crashes when an OHCI controller is instantiated with too many ports, e.g. "-device pci-ohci,num-ports=100,masterbus=1". Thus add a proper check in usb_ohci_init() to make sure that we do not use more than OHCI_MAX_PORTS = 15 ports here. Ticket: https://bugs.launchpad.net/qemu/+bug/1

[Qemu-devel] KVM call for 2016-05-24

2016-05-23 Thread Juan Quintela
Hi Please, send any topic that you are interested in covering. At the end of Monday I will send an email with the agenda or the cancellation of the call, so hurry up. After discussions on the QEMU Summit, we are going to have always open a KVM call where you can add topics. Call details: By

Re: [Qemu-devel] [PATCH V3] tap: vhost busy polling support

2016-05-23 Thread Greg Kurz
On Thu, 7 Apr 2016 18:07:44 +0200 Greg Kurz wrote: > On Thu, 7 Apr 2016 12:56:24 +0800 > Jason Wang wrote: > > > This patch add the capability of basic vhost net busy polling which is > > supported by recent kernel. User could configure the maximum number of > > us that could be spent on busy

[Qemu-devel] [Bug 1581308] Re: ohci doesn't check the 'num-ports' property

2016-05-23 Thread thh
I was able to reproduce the crash, and proposed now a fix on the qemu- devel mailing list (see https://patchwork.ozlabs.org/patch/625092/ for details) -- You received this bug notification because you are a member of qemu- devel-ml, which is subscribed to QEMU. https://bugs.launchpad.net/bugs/158

[Qemu-devel] [Bug 1245724] Re: libfdt.a git compilation fail

2016-05-23 Thread thh
This should be working fine again with recent versions of QEMU, so I'm closing the bug. If you still have problems, feel free to open it again. ** Changed in: qemu Status: New => Fix Released -- You received this bug notification because you are a member of qemu- devel-ml, which is subscr

[Qemu-devel] [PATCH] docs/multi-thread-compression: Fix wrong command string

2016-05-23 Thread Wei Jiangang
s/info_migrate_capabilities/info migrate_capabilities Signed-off-by: Wei Jiangang --- docs/multi-thread-compression.txt | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/docs/multi-thread-compression.txt b/docs/multi-thread-compression.txt index 3d477c3..d0caaf7 100644 --- a/d

Re: [Qemu-devel] [PATCH v5 09/11] pci bridge dev: change msi property type

2016-05-23 Thread Marcel Apfelbaum
On 05/23/2016 05:22 AM, Cao jin wrote: On 05/17/2016 03:38 PM, Michael S. Tsirkin wrote: On Tue, May 17, 2016 at 03:39:14PM +0800, Cao jin wrote: Personally I don't consider this a big issue. I don't think many people specify msi=on. I also think so:) If so, the semantics won`t change

Re: [Qemu-devel] [PATCH v5 5/6] acpi: Add IPMI table entries

2016-05-23 Thread Igor Mammedov
On Sat, 21 May 2016 19:28:59 -0500 Corey Minyard wrote: > Thanks for all the comments. I didn't know about stubs, as > there's nothing that currently uses it in hw directory, but > it's easy enough to add. I did have two comment below: > > On 05/20/2016 04:53 AM, Igor Mammedov wrote: > > On Th

Re: [Qemu-devel] [PATCH v5 11/11] pci: Convert msi_init() to Error and fix callers to check it

2016-05-23 Thread Marcel Apfelbaum
On 05/17/2016 01:08 PM, Cao jin wrote: On 05/15/2016 09:41 PM, Marcel Apfelbaum wrote: diff --git a/hw/pci-bridge/pci_bridge_dev.c b/hw/pci-bridge/pci_bridge_dev.c index 9e31f0e..af71c98 100644 --- a/hw/pci-bridge/pci_bridge_dev.c +++ b/hw/pci-bridge/pci_bridge_dev.c @@ -53,6 +53,7 @@ stat

[Qemu-devel] RFC: do we need ordered dict?

2016-05-23 Thread Wei, Jiangang
Hi all, I found a problem about the output of 'query-version' As the qmp-command.hx said, it shoud be -> { "execute": "query-version" } <- { "return":{ "qemu":{ "major":0, "minor":11, "micro":5 }, "package":"" } } Act

Re: [Qemu-devel] GSoC 2016: Introductory email

2016-05-23 Thread Sergey Fedorov
On 20/05/16 15:51, Pranith Kumar wrote: > Hello all, > > I've been selected to participate in GSoC 2016 in the Qemu project. I will be > mentored by Alex Bennée and will be working on solving memory consistency > issues in the MTTCG project. In particular we are targeting to get ARM guests > workin

[Qemu-devel] [PATCH] 9p/fsdev: remove obsolete references to virtio

2016-05-23 Thread Greg Kurz
Most of the 9p code is now virtio agnostic. This patch does a final cleanup: - drop references to Virtio from the header comments - fix includes Also drop a couple of leading empty lines while here. Signed-off-by: Greg Kurz --- fsdev/file-op-9p.h |2 +- fsdev/qemu-fsdev-dummy.c |2

[Qemu-devel] [PATCH v5 0/4] QOM'ify hw/char devices

2016-05-23 Thread xiaoqiang zhao
This patch set trys to QOM'ify hw/char files, see commit messages for more details Thanks Paolo for your suggestions. Note: * CRIS axis_dev88 broad related test is passed and looks ok. * lm32 test cases by Michael is passed and looks good. Changes in v5: drop the call of qemu_char_get_next

[Qemu-devel] [PATCH v5 1/4] hw/char: QOM'ify escc.c

2016-05-23 Thread xiaoqiang zhao
* Drop the old SysBus init function and use instance_init * Call qemu_chr_add_handlers in the realize callback Signed-off-by: xiaoqiang zhao --- hw/char/escc.c | 30 +++--- 1 file changed, 19 insertions(+), 11 deletions(-) diff --git a/hw/char/escc.c b/hw/char/escc.c ind

Re: [Qemu-devel] [PULL 00/21] Machine Core queue, 2016-05-20

2016-05-23 Thread Peter Maydell
On 20 May 2016 at 21:28, Eduardo Habkost wrote: > The following changes since commit 65603e2fc18b48e6e55a3dd693669413141694ec: > > tci: do not include exec/exec-all.h (2016-05-20 15:07:46 +0100) > > are available in the git repository at: > > git://github.com/ehabkost/qemu.git tags/machine-pul

[Qemu-devel] [PATCH v5 2/4] hw/char: QOM'ify etraxfs_ser.c

2016-05-23 Thread xiaoqiang zhao
* Drop the old SysBus init function and use instance_init * Call qemu_chr_add_handlers in the realize callback * Use qdev chardev prop instead of qemu_char_get_next_serial * Add etraxfs_ser_create function to create etraxfs serial device Signed-off-by: xiaoqiang zhao --- hw/char/etraxfs_ser.c

[Qemu-devel] [PATCH v5 4/4] hw/char: QOM'ify lm32_uart.c

2016-05-23 Thread xiaoqiang zhao
* Drop the old SysBus init function and use instance_init * Call qemu_chr_add_handlers in the realize callback * Use qdev chardev prop instead of qemu_char_get_next_serial * Add lm32_uart_create function to create lm32 uart device Signed-off-by: xiaoqiang zhao --- hw/char/lm32_uart.c | 28

[Qemu-devel] [PATCH v5 3/4] hw/char: QOM'ify lm32_juart.c

2016-05-23 Thread xiaoqiang zhao
* Drop the old SysBus init function * Call qemu_chr_add_handlers in the realize callback * Use qdev chardev prop instead of qemu_char_get_next_serial Signed-off-by: xiaoqiang zhao --- hw/char/lm32_juart.c | 17 - hw/lm32/lm32.h | 2 ++ 2 files changed, 10 insertions(+), 9

[Qemu-devel] [PATCH] scsi: pvscsi: check command descriptor ring buffer size

2016-05-23 Thread P J P
From: Prasad J Pandit Vmware Paravirtual SCSI emulation uses command descriptors to process SCSI commands. These descriptors come with their ring buffers. A guest could set the ring buffer size to an arbitrary value leading to OOB access issue. Add check to avoid it. Reported-by: Li Qiang Signe

[Qemu-devel] Any topics for today's MTTCG sync-up call?

2016-05-23 Thread Alex Bennée
Hi, It's been a while since the last sync-up call. Have we got any topics to discuss today? Sergey and I (with a little Paolo) have spent some of last week delving into the locking hierarchy w.r.t to tb_lock vs mmap_lock to see if there is any simplification to be had. I'm not sure if this is a t

Re: [Qemu-devel] [PULL 00/20] Net patches

2016-05-23 Thread Peter Maydell
On 23 May 2016 at 03:13, Jason Wang wrote: > The following changes since commit 65603e2fc18b48e6e55a3dd693669413141694ec: > > tci: do not include exec/exec-all.h (2016-05-20 15:07:46 +0100) > > are available in the git repository at: > > https://github.com/jasowang/qemu.git tags/net-pull-reque

Re: [Qemu-devel] [PATCH v4 0/9] blockdev: (Nearly) free clean-up work

2016-05-23 Thread Kevin Wolf
Am 17.05.2016 um 16:41 hat Max Reitz geschrieben: > After a lot has been restructed in the block layer in the past, we can > now reap at least one of the fruits: Make bdrv_open() return a BDS! Thanks, applied to the block branch. Kevin

Re: [Qemu-devel] [PATCH v5 4/4] hw/char: QOM'ify lm32_uart.c

2016-05-23 Thread Michael Walle
Am 2016-05-23 12:24, schrieb xiaoqiang zhao: * Drop the old SysBus init function and use instance_init * Call qemu_chr_add_handlers in the realize callback * Use qdev chardev prop instead of qemu_char_get_next_serial * Add lm32_uart_create function to create lm32 uart device Signed-off-by: xiaoq

Re: [Qemu-devel] [PATCH] scsi: pvscsi: check command descriptor ring buffer size

2016-05-23 Thread Shmulik Ladkani
Hi, On Mon, 23 May 2016 16:18:05 +0530, ppan...@redhat.com wrote: > From: Prasad J Pandit > > Vmware Paravirtual SCSI emulation uses command descriptors to > process SCSI commands. These descriptors come with their ring > buffers. A guest could set the ring buffer size to an arbitrary > value le

Re: [Qemu-devel] [PATCH v5 3/4] hw/char: QOM'ify lm32_juart.c

2016-05-23 Thread Michael Walle
Am 2016-05-23 12:24, schrieb xiaoqiang zhao: * Drop the old SysBus init function * Call qemu_chr_add_handlers in the realize callback * Use qdev chardev prop instead of qemu_char_get_next_serial Signed-off-by: xiaoqiang zhao --- hw/char/lm32_juart.c | 17 - hw/lm32/lm32.h

Re: [Qemu-devel] [PATCH v5 3/4] hw/char: QOM'ify lm32_juart.c

2016-05-23 Thread xiaoqiang zhao
> 在 2016年5月23日,19:18,Michael Walle 写道: > > Am 2016-05-23 12:24, schrieb xiaoqiang zhao: >> * Drop the old SysBus init function >> * Call qemu_chr_add_handlers in the realize callback >> * Use qdev chardev prop instead of qemu_char_get_next_serial >> Signed-off-by: xiaoqiang zhao >> --- >> hw/c

Re: [Qemu-devel] [PATCH] scsi: pvscsi: check command descriptor ring buffer size

2016-05-23 Thread Dmitry Fleytman
Reviewed-by: Dmitry Fleytman > On 23 May 2016, at 14:16 PM, Shmulik Ladkani > wrote: > > Hi, > > On Mon, 23 May 2016 16:18:05 +0530, ppan...@redhat.com wrote: >> From: Prasad J Pandit >> >> Vmware Paravirtual SCSI emulation uses command descriptors to >> process SCSI commands. These descri

Re: [Qemu-devel] Any topics for today's MTTCG sync-up call?

2016-05-23 Thread alvise rigo
Hi Alex, I finally solved the issue I had, the branch is working well as far as I can say. The work I will share, in addition to making the LL/SC work mttcg-aware, extends the various TLB flushes calls with the query-based mechanism: the requesting CPU queries the flushes to the target CPUs and wa

[Qemu-devel] [PULL 1/7] virtio-gpu: check early scanout id

2016-05-23 Thread Gerd Hoffmann
From: Marc-André Lureau Before accessing the g->scanout array, in order to avoid potential out-of-bounds access. Signed-off-by: Marc-André Lureau Message-id: 1463653560-26958-2-git-send-email-marcandre.lur...@redhat.com Signed-off-by: Gerd Hoffmann --- hw/display/virtio-gpu.c | 19 +--

[Qemu-devel] [PULL 5/7] virtio-gpu: use VIRTIO_GPU_MAX_SCANOUTS

2016-05-23 Thread Gerd Hoffmann
From: Marc-André Lureau The value is defined in virtio_gpu.h already (changing from 4 to 16). Signed-off-by: Marc-André Lureau Message-id: 1463653560-26958-6-git-send-email-marcandre.lur...@redhat.com Signed-off-by: Gerd Hoffmann --- hw/display/virtio-gpu.c| 4 ++-- include/hw/virtio/

[Qemu-devel] [PULL 4/7] virtio-gpu: check max_outputs only

2016-05-23 Thread Gerd Hoffmann
From: Marc-André Lureau The scanout id should not be above the configured num_scanouts. Signed-off-by: Marc-André Lureau Message-id: 1463653560-26958-5-git-send-email-marcandre.lur...@redhat.com Signed-off-by: Gerd Hoffmann --- hw/display/virtio-gpu-3d.c | 4 ++-- hw/display/virtio-gpu.c|

[Qemu-devel] [PULL 3/7] virtio-gpu: check max_outputs value

2016-05-23 Thread Gerd Hoffmann
From: Marc-André Lureau The value must be less than VIRTIO_GPU_MAX_SCANOUT. Signed-off-by: Marc-André Lureau Message-id: 1463653560-26958-4-git-send-email-marcandre.lur...@redhat.com Signed-off-by: Gerd Hoffmann --- hw/display/virtio-gpu.c | 6 ++ 1 file changed, 6 insertions(+) diff --g

[Qemu-devel] [PULL 2/7] virtio-vga: propagate on gpu realized error

2016-05-23 Thread Gerd Hoffmann
From: Marc-André Lureau Signed-off-by: Marc-André Lureau Message-id: 1463653560-26958-3-git-send-email-marcandre.lur...@redhat.com Signed-off-by: Gerd Hoffmann --- hw/display/virtio-gpu-3d.c | 1 + hw/display/virtio-vga.c| 8 +++- 2 files changed, 8 insertions(+), 1 deletion(-) diff -

[Qemu-devel] [PULL 0/7] vga fixes

2016-05-23 Thread Gerd Hoffmann
branch 'remotes/ehabkost/tags/machine-pull-request' into staging (2016-05-23 10:30:41 +0100) are available in the git repository at: git://git.kraxel.org/qemu tags/pull-vga-20160523-1 for you to fetch changes up to 94ef4f337fb614f18b765a8e0e878a4c23cdedcd: vga: add sr_vbe register

[Qemu-devel] [PULL 7/7] vga: add sr_vbe register set

2016-05-23 Thread Gerd Hoffmann
Commit "fd3c136 vga: make sure vga register setup for vbe stays intact (CVE-2016-3712)." causes a regression. The win7 installer is unhappy because it can't freely modify vga registers any more while in vbe mode. This patch introduces a new sr_vbe register set. The vbe_update_vgaregs will fill s

[Qemu-devel] [PULL 6/7] virtio-gpu: fix ui idx check

2016-05-23 Thread Gerd Hoffmann
From: Marc-André Lureau Fix off-by-one value check (0 is the first scanout). Signed-off-by: Marc-André Lureau Message-id: 1463653560-26958-7-git-send-email-marcandre.lur...@redhat.com Signed-off-by: Gerd Hoffmann --- hw/display/virtio-gpu.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-

Re: [Qemu-devel] [PATCH v5 5/6] acpi: Add IPMI table entries

2016-05-23 Thread Corey Minyard
On 05/23/2016 05:01 AM, Igor Mammedov wrote: On Sat, 21 May 2016 19:28:59 -0500 Corey Minyard wrote: Thanks for all the comments. I didn't know about stubs, as there's nothing that currently uses it in hw directory, but it's easy enough to add. I did have two comment below: On 05/20/2016 04

Re: [Qemu-devel] [PATCH v5 11/11] pci: Convert msi_init() to Error and fix callers to check it

2016-05-23 Thread Cao jin
On 05/23/2016 06:06 PM, Marcel Apfelbaum wrote: On 05/17/2016 01:08 PM, Cao jin wrote: On 05/15/2016 09:41 PM, Marcel Apfelbaum wrote: That is why I am not quite sure about this device, msi has a relation with shpc. From its previous behaviour, it can be seen that it don`t treat 'on' as

Re: [Qemu-devel] Any topics for today's MTTCG sync-up call?

2016-05-23 Thread Alex Bennée
alvise rigo writes: > Hi Alex, > > I finally solved the issue I had, the branch is working well as far as I > can say. > The work I will share, in addition to making the LL/SC work mttcg-aware, > extends the various TLB flushes calls with the query-based mechanism: the > requesting CPU queries t

[Qemu-devel] [PATCH 2/7] dma-helpers: change BlockBackend to opaque value in DMAIOFunc

2016-05-23 Thread Paolo Bonzini
Callers of dma_blk_io have no way to pass extra data to the DMAIOFunc, because the original callback and opaque are gone by the time DMAIOFunc is called. On the other hand, the BlockBackend is usually derived from those extra data that you could pass to the DMAIOFunc (in the next patch, that would

[Qemu-devel] [PATCH 5/7] scsi-disk: add need_fua_emulation to SCSIDiskClass

2016-05-23 Thread Paolo Bonzini
scsi-block will be able to do FUA just by passing the request through to the LUN (which is also more efficient); there is no need to emulate it like we do for scsi-disk. Add a new method to distinguish this. Signed-off-by: Paolo Bonzini --- hw/scsi/scsi-disk.c | 9 +++-- 1 file changed, 7 i

[Qemu-devel] [PATCH v2 0/7] dma-helpers, scsi-block: use SG_IO for all I/O on scsi-block

2016-05-23 Thread Paolo Bonzini
scsi-block uses the block layer for reads and writes in order to avoid allocating bounce buffers as big as the transferred data. We know how to split a large transfer to multiple reads and writes, and thus we can use scsi-disk.c's existing code to do I/O in multiple chunks (for non-s/g SCSI hosts)

[Qemu-devel] [PATCH 6/7] scsi-disk: introduce scsi_disk_req_check_error

2016-05-23 Thread Paolo Bonzini
Commonize all the checks for canceled requests and errors. The next patch will add another case to check for, in order to handle passthrough commands. There is no semantic change here; the only nontrivial modification is in scsi_write_do_fua, where cancellation has been checked earlier by both ca

[Qemu-devel] [PATCH 1/7] dma-helpers: change interface to byte-based

2016-05-23 Thread Paolo Bonzini
Signed-off-by: Paolo Bonzini --- dma-helpers.c| 14 +++--- hw/block/nvme.c | 6 +++--- hw/ide/ahci.c| 6 -- hw/ide/core.c| 8 +--- hw/scsi/scsi-disk.c | 6 -- include/sysemu/dma.h | 6 +++--- trace-events | 2 +- 7 files changed, 27

[Qemu-devel] [PATCH 4/7] scsi-disk: introduce dma_readv and dma_writev

2016-05-23 Thread Paolo Bonzini
These are replacements for blk_aio_preadv and blk_aio_pwritev that allow customization of the data path. They reuse the DMA helpers' DMAIOFunc callback type, so that the same function can be used in either the QEMUSGList or the bounce-buffered case. This customization will be needed in the next p

[Qemu-devel] [PATCH 3/7] scsi-disk: introduce a common base class

2016-05-23 Thread Paolo Bonzini
This will be the place to add DMAIOFuncs in the next patch. There are also a couple DeviceClass members that can be moved to the abstract class's initialization function. Signed-off-by: Paolo Bonzini --- hw/scsi/scsi-disk.c | 36 ++-- 1 file changed, 22 insertion

Re: [Qemu-devel] Any topics for today's MTTCG sync-up call?

2016-05-23 Thread Claudio Fontana
On 23.05.2016 14:47, Alex Bennée wrote: > > alvise rigo writes: > >> Hi Alex, >> >> I finally solved the issue I had, the branch is working well as far as I >> can say. >> The work I will share, in addition to making the LL/SC work mttcg-aware, >> extends the various TLB flushes calls with the q

[Qemu-devel] [PATCH 7/7] scsi-block: always use SG_IO

2016-05-23 Thread Paolo Bonzini
Using pread/pwrite or io_submit has the advantage of eliminating the bounce buffer, but drops the SCSI status. This keeps the guest from seeing unit attention codes, as well as statuses such as RESERVATION CONFLICT. Because we know scsi-block operates on an SBC device we can still use the DMA hel

[Qemu-devel] [PULL 4/4] usb/ohci: Fix crash with when specifying too many num-ports

2016-05-23 Thread Gerd Hoffmann
From: Thomas Huth QEMU currently crashes when an OHCI controller is instantiated with too many ports, e.g. "-device pci-ohci,num-ports=100,masterbus=1". Thus add a proper check in usb_ohci_init() to make sure that we do not use more than OHCI_MAX_PORTS = 15 ports here. Ticket: https://bugs.launc

[Qemu-devel] [PULL 0/4] usb: add xen pvUSB backend

2016-05-23 Thread Gerd Hoffmann
ging (2016-05-23 10:30:41 +0100) are available in the git repository at: git://git.kraxel.org/qemu tags/pull-usb-20160523-1 for you to fetch changes up to d400fc018b326104d26d730e5cc8c36c1f662c34: usb/ohci: Fix crash with when specifying too many num-ports (2016-05-23 14:5

[Qemu-devel] [PULL 3/4] xen: add pvUSB backend

2016-05-23 Thread Gerd Hoffmann
From: Juergen Gross Add a backend for para-virtualized USB devices for xen domains. The backend is using host-libusb to forward USB requests from a domain via libusb to the real device(s) passed through. Signed-off-by: Juergen Gross Acked-by: Anthony PERARD Message-id: 1463062421-613-4-git-se

[Qemu-devel] [PULL 2/4] xen: write information about supported backends

2016-05-23 Thread Gerd Hoffmann
From: Juergen Gross Add a Xenstore directory for each supported pv backend. This will allow Xen tools to decide which backend type to use in case there are multiple possibilities. The information is added under /local/domain//device-model//backends before the "running" state is written to Xensto

[Qemu-devel] [PULL 1/4] xen: introduce dummy system device

2016-05-23 Thread Gerd Hoffmann
From: Juergen Gross Introduce a new dummy system device serving as parent for virtual buses. This will enable new pv backends to introduce virtual buses which are removable again opposed to system buses which are meant to stay once added. Signed-off-by: Juergen Gross Acked-by: Anthony PERARD R

Re: [Qemu-devel] Any topics for today's MTTCG sync-up call?

2016-05-23 Thread Alex Bennée
Claudio Fontana writes: > On 23.05.2016 14:47, Alex Bennée wrote: >> >> alvise rigo writes: >> >>> Hi Alex, >>> >>> I finally solved the issue I had, the branch is working well as far as I >>> can say. >>> The work I will share, in addition to making the LL/SC work mttcg-aware, >>> extends the

Re: [Qemu-devel] [PATCH v5 5/6] acpi: Add IPMI table entries

2016-05-23 Thread Igor Mammedov
On Mon, 23 May 2016 07:42:32 -0500 Corey Minyard wrote: > On 05/23/2016 05:01 AM, Igor Mammedov wrote: > > On Sat, 21 May 2016 19:28:59 -0500 > > Corey Minyard wrote: > > > >> Thanks for all the comments. I didn't know about stubs, as > >> there's nothing that currently uses it in hw director

[Qemu-devel] [PATCH] vnc: add configurable keyboard delay

2016-05-23 Thread Gerd Hoffmann
Limits the rate kbd events from the vnc server are forwarded to the guest, so input devices which are typically low-bandwidth can keep up even on bulky input. Signed-off-by: Gerd Hoffmann --- ui/vnc.c | 13 +++-- ui/vnc.h | 1 + 2 files changed, 12 insertions(+), 2 deletions(-) diff --

[Qemu-devel] [PATCH] virtio-gpu: add live migration support

2016-05-23 Thread Gerd Hoffmann
Store some additional state for cursor and resource backing storage, so we can write out and reload things. Implement vmsave+vmload for 2d mode. Continue blocking live migration in 3d/virgl mode. Signed-off-by: Gerd Hoffmann --- hw/display/virtio-gpu-3d.c | 2 +- hw/display/virtio-gpu.c

Re: [Qemu-devel] [PATCH] SDL2: add bgrx pixel format

2016-05-23 Thread Gerd Hoffmann
On Di, 2016-05-17 at 10:28 +0300, Pavel Dovgalyuk wrote: > This patch adds support of b8g8r8x8 pixel format for SDL2. > > Signed-off-by: Pavel Dovgalyuk > --- > ui/sdl2-2d.c |3 +++ > 1 file changed, 3 insertions(+) > > diff --git a/ui/sdl2-2d.c b/ui/sdl2-2d.c > index 9593006..8ab68d6 10064

Re: [Qemu-devel] vfio IGD assignment (was Re: [PATCH v6 0/8] Series short description)

2016-05-23 Thread Gerd Hoffmann
Hi, > UPT is probably the only thing Intel cares to support on Windows, yes, > but did you have any issues or try legacy mode on Windows? Just tried that with Win10 -- working fine. cheers, Gerd

Re: [Qemu-devel] [PATCH v5 5/6] acpi: Add IPMI table entries

2016-05-23 Thread Corey Minyard
On 05/23/2016 04:05 AM, Marcel Apfelbaum wrote: On 05/22/2016 03:28 AM, Corey Minyard wrote: Thanks for all the comments. I didn't know about stubs, as there's nothing that currently uses it in hw directory, but it's easy enough to add. I did have two comment below: On 05/20/2016 04:53 AM, Ig

Re: [Qemu-devel] [PATCH 00/10] RFCv3: vhost-user: simple reconnection support

2016-05-23 Thread Yuanhan Liu
On Sun, May 15, 2016 at 12:42:12PM +0300, Michael S. Tsirkin wrote: > > FYI, I have a follow up series (~20 patches, > > https://github.com/elmarco/qemu/tree/vhost-user-reconnect) doing > > mostly cleanups and extra checks for disconnection at run time. In > > particular, it should avoid some obvio

Re: [Qemu-devel] [PATCH] scsi: pvscsi: check command descriptor ring buffer size

2016-05-23 Thread Paolo Bonzini
On 23/05/2016 12:48, P J P wrote: > From: Prasad J Pandit > > Vmware Paravirtual SCSI emulation uses command descriptors to > process SCSI commands. These descriptors come with their ring > buffers. A guest could set the ring buffer size to an arbitrary > value leading to OOB access issue. Add ch

Re: [Qemu-devel] [QEMU PATCH v2] target-i386: Add Skylake-Client CPU model

2016-05-23 Thread Paolo Bonzini
On 20/05/2016 23:39, Eduardo Habkost wrote: > Introduce Skylake-Client CPU model, which inherits the features > from Broadwell and supports some additional features that are: > MPX, XSAVEC, and XGETBV1. > > Signed-off-by: Eduardo Habkost > Signed-off-by: Xiao Guangrong > Signed-off-by: Eduardo

Re: [Qemu-devel] [PATCH v5 5/6] acpi: Add IPMI table entries

2016-05-23 Thread Marcel Apfelbaum
On 05/23/2016 04:39 PM, Corey Minyard wrote: On 05/23/2016 04:05 AM, Marcel Apfelbaum wrote: On 05/22/2016 03:28 AM, Corey Minyard wrote: Thanks for all the comments. I didn't know about stubs, as there's nothing that currently uses it in hw directory, but it's easy enough to add. I did have

Re: [Qemu-devel] [PATCH] scripts/kvm/kvm_stat: fix tracepoint provider

2016-05-23 Thread Paolo Bonzini
On 20/05/2016 10:42, Laurent Vivier wrote: > tracepoint provider on ppc is broken since: > > fc116ef kvm_stat: Add RESET support for perf event ioctl > > because the value used with ioctl() to reset the event queue is > not valid on ppc. > > This patch defines the good value for ppc. > >

Re: [Qemu-devel] [PATCH v2 10/12] spice/gl: create dummy primary surface (RfC)

2016-05-23 Thread Gerd Hoffmann
Hi, Resuming to work on this after 2.6 freeze break ... > I have done some more testing and sent a series for spice-gtk fixing > display with gl scanout-only case. And a minor patch to spice server > to solve a cursor initialization when there is no canvas. Your series > works ok with that, onl

Re: [Qemu-devel] [PATCH] scripts/kvm/kvm_stat: fix tracepoint provider

2016-05-23 Thread Laurent Vivier
On 23/05/2016 15:51, Paolo Bonzini wrote: > > > On 20/05/2016 10:42, Laurent Vivier wrote: >> tracepoint provider on ppc is broken since: >> >> fc116ef kvm_stat: Add RESET support for perf event ioctl >> >> because the value used with ioctl() to reset the event queue is >> not valid on ppc.

[Qemu-devel] [PATCH RFC 0/2] enable iommu with -device

2016-05-23 Thread Marcel Apfelbaum
This is a proposal on how to create the iommu with '-device intel-iommu' instead of '-machine,iommu=on'. The device is part of the machine properties because we wanted to ensure it is created before any other PCI device. The alternative is to skip the bus_master_enable_region at the time the devi

[Qemu-devel] [PATCH RFC 1/2] hw/pci: delay bus_master_enable_region initialization

2016-05-23 Thread Marcel Apfelbaum
Skip bus_master_enable region creation on PCI devices init in order to be sure the IOMMU device (if present) would be created in advance. Add this memory region at machine_done time. Signed-off-by: Marcel Apfelbaum --- hw/i386/pc.c | 17 + hw/pci/pci.c | 22 ++

Re: [Qemu-devel] [PATCH v2 10/12] spice/gl: create dummy primary surface (RfC)

2016-05-23 Thread Marc-André Lureau
Hi On Mon, May 23, 2016 at 3:52 PM, Gerd Hoffmann wrote: > Hi, > > Resuming to work on this after 2.6 freeze break ... > >> I have done some more testing and sent a series for spice-gtk fixing >> display with gl scanout-only case. And a minor patch to spice server >> to solve a cursor initializ

[Qemu-devel] [PATCH RFC 2/2] hw/iommu: enable iommu with -device

2016-05-23 Thread Marcel Apfelbaum
Use the standard '-device iommu' instead of '-machine,iommu=on' to create the IOMMU device. Signed-off-by: Marcel Apfelbaum --- hw/core/machine.c | 20 hw/i386/intel_iommu.c | 17 + hw/i386/pc_q35.c | 1 + hw/pci-host/q35.c | 28

Re: [Qemu-devel] [PATCH RFC 1/2] hw/pci: delay bus_master_enable_region initialization

2016-05-23 Thread Paolo Bonzini
On 23/05/2016 16:01, Marcel Apfelbaum wrote: > Skip bus_master_enable region creation on PCI devices init > in order to be sure the IOMMU device (if present) would > be created in advance. Add this memory region at machine_done time. > > Signed-off-by: Marcel Apfelbaum > --- > hw/i386/pc.c

Re: [Qemu-devel] [PATCH 1/2] atomics: do not use __atomic primitives for RCU atomics

2016-05-23 Thread Paolo Bonzini
On 21/05/2016 22:42, Emilio G. Cota wrote: > Commit a0aa44b4 ("include/qemu/atomic.h: default to __atomic functions") > set all atomics to default (on recent GCC versions) to __atomic primitives. > > In the process, the atomic_rcu_read/set were converted to implement > consume/release semantics,

Re: [Qemu-devel] [PATCH RFC 1/2] hw/pci: delay bus_master_enable_region initialization

2016-05-23 Thread Marcel Apfelbaum
On 05/23/2016 05:08 PM, Paolo Bonzini wrote: On 23/05/2016 16:01, Marcel Apfelbaum wrote: Skip bus_master_enable region creation on PCI devices init in order to be sure the IOMMU device (if present) would be created in advance. Add this memory region at machine_done time. Signed-off-by: Marce

Re: [Qemu-devel] [PATCH RFC 1/2] hw/pci: delay bus_master_enable_region initialization

2016-05-23 Thread Paolo Bonzini
On 23/05/2016 16:22, Marcel Apfelbaum wrote: > On 05/23/2016 05:08 PM, Paolo Bonzini wrote: >> >> >> On 23/05/2016 16:01, Marcel Apfelbaum wrote: >>> Skip bus_master_enable region creation on PCI devices init >>> in order to be sure the IOMMU device (if present) would >>> be created in advance. A

Re: [Qemu-devel] [PULL 0/7] vga fixes

2016-05-23 Thread Peter Maydell
following changes since commit e081c24d30c1e7b29eb4450aa16d6a0da5782797: > > Merge remote-tracking branch 'remotes/ehabkost/tags/machine-pull-request' > into staging (2016-05-23 10:30:41 +0100) > > are available in the git repository at: > > > git://git.kraxel.org/

Re: [Qemu-devel] [PATCH v8 3/3] Add optionrom compatible with fw_cfg DMA version

2016-05-23 Thread Paolo Bonzini
On 11/05/2016 23:06, Richard W.M. Jones wrote: > From: Marc Marí > > This optionrom is based on linuxboot.S. > > Signed-off-by: Marc Marí > Signed-off-by: Richard W.M. Jones Hmm, I hadn't noticed that you added -m16. That breaks on even not-too-old GCC (such as 4.8 on RHEL/CentOS 7). Can y

  1   2   3   >