[Qemu-devel] [PATCH 5/5] hw/timer/exynos4210_mct: Remove unused defines

2017-03-01 Thread Krzysztof Kozlowski
Remove defines not used anywhere. Signed-off-by: Krzysztof Kozlowski --- hw/timer/exynos4210_mct.c | 3 --- 1 file changed, 3 deletions(-) diff --git a/hw/timer/exynos4210_mct.c b/hw/timer/exynos4210_mct.c index 4dd3e441e2e6..6069116942a4 100644 --- a/hw/timer/exynos4210_mct.c +++ b/hw/timer/ex

Re: [Qemu-devel] s390x failure: "tcg_handle_interrupt: assertion failed: (qemu_mutex_iothread_locked())"

2017-03-01 Thread Thomas Huth
On 01.03.2017 12:36, Alex Bennée wrote: > > Peter Maydell writes: > >> I got a make check failure on aarch64 host running a sparc64 test: >> >> >> TEST: tests/prom-env-test... (pid=13573) >> /sparc64/prom-env/sun4u: ** >> ERROR:/home/pm215/qemu/trans

[Qemu-devel] [PATCH 1/5] hw/intc/exynos4210_gic: Fix GIC memory mappings for secondary CPU

2017-03-01 Thread Krzysztof Kozlowski
Recent Linux kernel (tested next-20170224) was complaining about missing GIC mask and was unable to bring up secondary CPU: [    0.00] NR_IRQS:16 nr_irqs:16 16 [0.00] GIC CPU mask not found - kernel will fail to boot. ... [    0.400492] smp: Bringing up secondary CPUs .

[Qemu-devel] [PATCH 4/5] hw/timer/exynos4210_mct: Fix checkpatch style errors

2017-03-01 Thread Krzysztof Kozlowski
Fix checkpatch errors: 1. ERROR: spaces required around that '+' (ctx:VxV) 2. ERROR: spaces required around that '&' (ctx:VxV) No functional changes. Signed-off-by: Krzysztof Kozlowski --- hw/timer/exynos4210_mct.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/hw/timer

[Qemu-devel] [PATCH 2/5] hw/intc/exynos4210_gic: Use more meaningful name for local variable

2017-03-01 Thread Krzysztof Kozlowski
There are to SysBusDevice variables in exynos4210_gic_realize() function: one for the device itself and second for arm_gic device. Add a prefix "gic" to the second one so it will be easier to understand the code. While at it, put local uninitialized 'i' variable at the end, next to other uninitia

[Qemu-devel] [RFC 0/5] hw: arm: exynos: Bring up secondary CPU + CPUIDLE issue

2017-03-01 Thread Krzysztof Kozlowski
Hi, The first patch fixes GIC and brings the secondary CPU. Unfortunately, this brings up to light an annoying issue with CPUIDLE driver. Overview of the problem === On Exynos4210, by default Linux kernel uses cpuidle driver which tries to enter low power mode, called AFTR (

Re: [Qemu-devel] [PATCH v5 1/4] Provide support for the CUSE TPM

2017-03-01 Thread Dr. David Alan Gilbert
* Michael S. Tsirkin (m...@redhat.com) wrote: > On Wed, Mar 01, 2017 at 06:06:02PM +, Dr. David Alan Gilbert wrote: > > * Michael S. Tsirkin (m...@redhat.com) wrote: > > > On Wed, Mar 01, 2017 at 05:38:23PM +, Daniel P. Berrange wrote: > > > > On Wed, Mar 01, 2017 at 12:25:46PM -0500, Stefa

Re: [Qemu-devel] intermittent make check failure: "tcg_handle_interrupt: assertion failed: (qemu_mutex_iothread_locked())"

2017-03-01 Thread Thomas Huth
On 01.03.2017 16:19, Alex Bennée wrote: > > Mark Cave-Ayland writes: > >> On 01/03/17 12:41, Alex Bennée wrote: >> Here's the backtrace from my test case of qemu-system-sparc running on an x86_64 host that I posted yesterday: $ gdb --args ./qemu-system-sparc -cdrom

Re: [Qemu-devel] [PATCH v5 1/4] Provide support for the CUSE TPM

2017-03-01 Thread Michael S. Tsirkin
On Wed, Mar 01, 2017 at 06:11:17PM +, Daniel P. Berrange wrote: > On Wed, Mar 01, 2017 at 07:58:36PM +0200, Michael S. Tsirkin wrote: > > On Wed, Mar 01, 2017 at 05:38:23PM +, Daniel P. Berrange wrote: > > > On Wed, Mar 01, 2017 at 12:25:46PM -0500, Stefan Berger wrote: > > > > On 03/01/201

[Qemu-devel] Regarding Gsoc17

2017-03-01 Thread Mohammad Sameer Malik
Hi i am Mohammad Sameer Malik a Btech undergraduate in Electronics and Communication at IIIT Delhi and i want to be a part of GSOC 17 as a student and want to chose Qemu as my organisation i have strong web development experience. Skills which i have used are C,Python,HTML/HTML5, CSS/CSS5, javas

Re: [Qemu-devel] [PATCH v5 1/4] Provide support for the CUSE TPM

2017-03-01 Thread Daniel P. Berrange
On Wed, Mar 01, 2017 at 07:58:36PM +0200, Michael S. Tsirkin wrote: > On Wed, Mar 01, 2017 at 05:38:23PM +, Daniel P. Berrange wrote: > > On Wed, Mar 01, 2017 at 12:25:46PM -0500, Stefan Berger wrote: > > > On 03/01/2017 12:16 PM, Michael S. Tsirkin wrote: > > > > On Wed, Mar 01, 2017 at 12:12:

Re: [Qemu-devel] [PATCH v5 1/4] Provide support for the CUSE TPM

2017-03-01 Thread Michael S. Tsirkin
On Wed, Mar 01, 2017 at 06:06:02PM +, Dr. David Alan Gilbert wrote: > * Michael S. Tsirkin (m...@redhat.com) wrote: > > On Wed, Mar 01, 2017 at 05:38:23PM +, Daniel P. Berrange wrote: > > > On Wed, Mar 01, 2017 at 12:25:46PM -0500, Stefan Berger wrote: > > > > On 03/01/2017 12:16 PM, Michae

Re: [Qemu-devel] [PATCH v5 1/4] Provide support for the CUSE TPM

2017-03-01 Thread Dr. David Alan Gilbert
* Michael S. Tsirkin (m...@redhat.com) wrote: > On Wed, Mar 01, 2017 at 05:38:23PM +, Daniel P. Berrange wrote: > > On Wed, Mar 01, 2017 at 12:25:46PM -0500, Stefan Berger wrote: > > > On 03/01/2017 12:16 PM, Michael S. Tsirkin wrote: > > > > On Wed, Mar 01, 2017 at 12:12:34PM -0500, Stefan Ber

Re: [Qemu-devel] [PATCH v5 1/4] Provide support for the CUSE TPM

2017-03-01 Thread Michael S. Tsirkin
On Wed, Mar 01, 2017 at 05:20:13PM +, Daniel P. Berrange wrote: > > > > > QEMU's defined the vhost-user ABI/API and delegated > > > > > impl to something else. > > > > The vhost ABI isn't easy to maintain at all though. So I would not > > > > commit to that lightly without a good reason. > > >

Re: [Qemu-devel] [PULL v4 00/19] KVM and cpu-exec patches for 2.9 soft freeze

2017-03-01 Thread Peter Maydell
On 28 February 2017 at 16:13, Paolo Bonzini wrote: > The following changes since commit d992f2f1368ceb92e6bfd8efece174110f4236ff: > > Merge remote-tracking branch 'remotes/artyom/tags/pull-sun4v-20170226' into > staging (2017-02-26 22:40:23 +) > > are available in the git repository at: > >

[Qemu-devel] [PATCH v3] virtio: guard vring access when setting notification

2017-03-01 Thread Cornelia Huck
Switching to vring caches exposed an existing bug in virtio_queue_set_notification(): We can't access vring structures if they have not been set up yet. This may happen, for example, for virtio-blk devices with multiple queues: The code will try to switch notifiers for every queue, but the guest ma

Re: [Qemu-devel] [PATCH v5 1/4] Provide support for the CUSE TPM

2017-03-01 Thread Michael S. Tsirkin
On Wed, Mar 01, 2017 at 05:38:23PM +, Daniel P. Berrange wrote: > On Wed, Mar 01, 2017 at 12:25:46PM -0500, Stefan Berger wrote: > > On 03/01/2017 12:16 PM, Michael S. Tsirkin wrote: > > > On Wed, Mar 01, 2017 at 12:12:34PM -0500, Stefan Berger wrote: > > > > On 03/01/2017 12:02 PM, Michael S.

Re: [Qemu-devel] [PATCH] virtio: guard vring access when setting notification

2017-03-01 Thread Cornelia Huck
On Wed, 1 Mar 2017 19:19:40 +0200 "Michael S. Tsirkin" wrote: > On Wed, Mar 01, 2017 at 06:14:52PM +0100, Cornelia Huck wrote: > > On Wed, 1 Mar 2017 19:04:56 +0200 > > "Michael S. Tsirkin" wrote: > > > > > On Wed, Mar 01, 2017 at 06:00:44PM +0100, Cornelia Huck wrote: > > > > On Wed, 1 Mar 201

Re: [Qemu-devel] [PULL 0/1] seabios: update to 1.10.2 release

2017-03-01 Thread Peter Maydell
On 1 March 2017 at 13:27, Stefan Hajnoczi wrote: > On Tue, Feb 28, 2017 at 9:06 AM, Gerd Hoffmann wrote: >> Our seabios.git mirror @ qemu.org seems to not sync properly. The >> commit isn't there even though it was pushed a few days ago already >> (only the tag was pushed today). John, can you

Re: [Qemu-devel] [PATCH v5 1/4] Provide support for the CUSE TPM

2017-03-01 Thread Daniel P. Berrange
On Wed, Mar 01, 2017 at 12:25:46PM -0500, Stefan Berger wrote: > On 03/01/2017 12:16 PM, Michael S. Tsirkin wrote: > > On Wed, Mar 01, 2017 at 12:12:34PM -0500, Stefan Berger wrote: > > > On 03/01/2017 12:02 PM, Michael S. Tsirkin wrote: > > > > On Wed, Mar 01, 2017 at 04:31:04PM +, Daniel P. B

Re: [Qemu-devel] [PATCH v5 1/4] Provide support for the CUSE TPM

2017-03-01 Thread Daniel P. Berrange
On Wed, Mar 01, 2017 at 12:12:34PM -0500, Stefan Berger wrote: > On 03/01/2017 12:02 PM, Michael S. Tsirkin wrote: > > On Wed, Mar 01, 2017 at 04:31:04PM +, Daniel P. Berrange wrote: > > > On Wed, Mar 01, 2017 at 06:22:45PM +0200, Michael S. Tsirkin wrote: > > > > On Wed, Mar 01, 2017 at 09:50:

Re: [Qemu-devel] [PATCH v5 1/4] Provide support for the CUSE TPM

2017-03-01 Thread Stefan Berger
On 03/01/2017 12:16 PM, Michael S. Tsirkin wrote: On Wed, Mar 01, 2017 at 12:12:34PM -0500, Stefan Berger wrote: On 03/01/2017 12:02 PM, Michael S. Tsirkin wrote: On Wed, Mar 01, 2017 at 04:31:04PM +, Daniel P. Berrange wrote: On Wed, Mar 01, 2017 at 06:22:45PM +0200, Michael S. Tsirkin wr

Re: [Qemu-devel] [PATCH v5 1/4] Provide support for the CUSE TPM

2017-03-01 Thread Daniel P. Berrange
On Wed, Mar 01, 2017 at 07:16:01PM +0200, Michael S. Tsirkin wrote: > On Wed, Mar 01, 2017 at 12:12:34PM -0500, Stefan Berger wrote: > > On 03/01/2017 12:02 PM, Michael S. Tsirkin wrote: > > > On Wed, Mar 01, 2017 at 04:31:04PM +, Daniel P. Berrange wrote: > > > > On Wed, Mar 01, 2017 at 06:22:

Re: [Qemu-devel] [PATCH] virtio: guard vring access when setting notification

2017-03-01 Thread Michael S. Tsirkin
On Wed, Mar 01, 2017 at 06:14:52PM +0100, Cornelia Huck wrote: > On Wed, 1 Mar 2017 19:04:56 +0200 > "Michael S. Tsirkin" wrote: > > > On Wed, Mar 01, 2017 at 06:00:44PM +0100, Cornelia Huck wrote: > > > On Wed, 1 Mar 2017 18:50:34 +0200 > > > "Michael S. Tsirkin" wrote: > > > > > Yes all these

Re: [Qemu-devel] [PATCH v2] virtio: guard vring access when setting notification

2017-03-01 Thread Cornelia Huck
On Wed, 1 Mar 2017 19:07:14 +0200 "Michael S. Tsirkin" wrote: > On Wed, Mar 01, 2017 at 05:56:56PM +0100, Cornelia Huck wrote: > > Switching to vring caches exposed an existing bug in > > virtio_queue_set_notification(): We can't access vring structures > > if they have not been set up yet. This

Re: [Qemu-devel] [PATCH 4/5] configure: detect presence of libxendevicemodel

2017-03-01 Thread Anthony PERARD
On Thu, Feb 23, 2017 at 02:53:54PM +, Paul Durrant wrote: > This patch adds code in configure to set CONFIG_XEN_CTRL_INTERFACE_VERSION > to a new value of 490 if libxendevicemodel is present in the build > environment. > > Signed-off-by: Paul Durrant > --- > Cc: Stefano Stabellini > Cc: Anth

Re: [Qemu-devel] [PATCH v5 1/4] Provide support for the CUSE TPM

2017-03-01 Thread Michael S. Tsirkin
On Wed, Mar 01, 2017 at 12:12:34PM -0500, Stefan Berger wrote: > On 03/01/2017 12:02 PM, Michael S. Tsirkin wrote: > > On Wed, Mar 01, 2017 at 04:31:04PM +, Daniel P. Berrange wrote: > > > On Wed, Mar 01, 2017 at 06:22:45PM +0200, Michael S. Tsirkin wrote: > > > > On Wed, Mar 01, 2017 at 09:50:

Re: [Qemu-devel] [PATCH] virtio: guard vring access when setting notification

2017-03-01 Thread Cornelia Huck
On Wed, 1 Mar 2017 19:04:56 +0200 "Michael S. Tsirkin" wrote: > On Wed, Mar 01, 2017 at 06:00:44PM +0100, Cornelia Huck wrote: > > On Wed, 1 Mar 2017 18:50:34 +0200 > > "Michael S. Tsirkin" wrote: > > > Yes all these callbacks complicate code to the point it's barely > > > readable. I really do

Re: [Qemu-devel] [PATCH v5 1/4] Provide support for the CUSE TPM

2017-03-01 Thread Stefan Berger
On 03/01/2017 12:02 PM, Michael S. Tsirkin wrote: On Wed, Mar 01, 2017 at 04:31:04PM +, Daniel P. Berrange wrote: On Wed, Mar 01, 2017 at 06:22:45PM +0200, Michael S. Tsirkin wrote: On Wed, Mar 01, 2017 at 09:50:38AM -0500, Stefan Berger wrote: I had already proposed a linked-in version be

Re: [Qemu-devel] [PATCH v2] virtio: guard vring access when setting notification

2017-03-01 Thread Michael S. Tsirkin
On Wed, Mar 01, 2017 at 05:56:56PM +0100, Cornelia Huck wrote: > Switching to vring caches exposed an existing bug in > virtio_queue_set_notification(): We can't access vring structures > if they have not been set up yet. This may happen, for example, > for virtio-blk devices with multiple queues:

Re: [Qemu-devel] [PATCH] virtio: guard vring access when setting notification

2017-03-01 Thread Michael S. Tsirkin
On Wed, Mar 01, 2017 at 06:00:44PM +0100, Cornelia Huck wrote: > On Wed, 1 Mar 2017 18:50:34 +0200 > "Michael S. Tsirkin" wrote: > > > On Wed, Mar 01, 2017 at 05:15:54PM +0100, Cornelia Huck wrote: > > > On Wed, 1 Mar 2017 17:55:24 +0200 > > > "Michael S. Tsirkin" wrote: > > > > > > > On Tue, F

Re: [Qemu-devel] [PATCH v5 1/4] Provide support for the CUSE TPM

2017-03-01 Thread Michael S. Tsirkin
On Wed, Mar 01, 2017 at 04:31:04PM +, Daniel P. Berrange wrote: > On Wed, Mar 01, 2017 at 06:22:45PM +0200, Michael S. Tsirkin wrote: > > On Wed, Mar 01, 2017 at 09:50:38AM -0500, Stefan Berger wrote: > > > I had already proposed a linked-in version before I went to the > > > out-of-process >

Re: [Qemu-devel] [PATCH] virtio: guard vring access when setting notification

2017-03-01 Thread Cornelia Huck
On Wed, 1 Mar 2017 18:50:34 +0200 "Michael S. Tsirkin" wrote: > On Wed, Mar 01, 2017 at 05:15:54PM +0100, Cornelia Huck wrote: > > On Wed, 1 Mar 2017 17:55:24 +0200 > > "Michael S. Tsirkin" wrote: > > > > > On Tue, Feb 28, 2017 at 04:24:11PM +0100, Cornelia Huck wrote: > > > > Switching to vrin

Re: [Qemu-devel] [PATCH v5 1/4] Provide support for the CUSE TPM

2017-03-01 Thread Dr. David Alan Gilbert
* Daniel P. Berrange (berra...@redhat.com) wrote: > On Wed, Mar 01, 2017 at 06:22:45PM +0200, Michael S. Tsirkin wrote: > > On Wed, Mar 01, 2017 at 09:50:38AM -0500, Stefan Berger wrote: > > > I had already proposed a linked-in version before I went to the > > > out-of-process > > > design. Anthon

[Qemu-devel] [PATCH v2] virtio: guard vring access when setting notification

2017-03-01 Thread Cornelia Huck
Switching to vring caches exposed an existing bug in virtio_queue_set_notification(): We can't access vring structures if they have not been set up yet. This may happen, for example, for virtio-blk devices with multiple queues: The code will try to switch notifiers for every queue, but the guest ma

Re: [Qemu-devel] [PATCH 1/1] virtio: fallback from irqfd to non-irqfd notify

2017-03-01 Thread Michael S. Tsirkin
On Wed, Mar 01, 2017 at 05:08:39PM +0100, Halil Pasic wrote: > > > On 03/01/2017 03:29 PM, Paolo Bonzini wrote: > > > > > > On 01/03/2017 14:22, Halil Pasic wrote: > >> Here a trace: > >> > >> 135871@1488304024.512533:virtio_blk_req_complete req 0x2aa6b117e10 status 0 > >> 135871@1488304024.512

Re: [Qemu-devel] [PATCH] virtio: guard vring access when setting notification

2017-03-01 Thread Michael S. Tsirkin
On Wed, Mar 01, 2017 at 05:15:54PM +0100, Cornelia Huck wrote: > On Wed, 1 Mar 2017 17:55:24 +0200 > "Michael S. Tsirkin" wrote: > > > On Tue, Feb 28, 2017 at 04:24:11PM +0100, Cornelia Huck wrote: > > > Switching to vring caches exposed an existing bug in > > > virtio_queue_set_notification(): W

Re: [Qemu-devel] Windows Pipe

2017-03-01 Thread Jiahuan Zhang
I found the problem! The pipe server and client should be both ready, and then data transfer can start. The windows pipe implementation is well-done! Sorry for bothering... On 1 March 2017 at 16:24, Jiahuan Zhang wrote: > I think the reading failure in 2 is because there is no writing to the >

Re: [Qemu-devel] intermittent make check failure: "tcg_handle_interrupt: assertion failed: (qemu_mutex_iothread_locked())"

2017-03-01 Thread Peter Maydell
On 1 March 2017 at 15:19, Alex Bennée wrote: > I haven't managed to run into the intermittent trigger yet. How do I run > the test directly? > > make tests/prom-env-test > > Just makes the test case and > > ./tests/prom-env-test > ** > ERROR:tests/libqtest.c:589:qtest_get_arch: assertion faile

Re: [Qemu-devel] [PATCH v5 1/4] Provide support for the CUSE TPM

2017-03-01 Thread Daniel P. Berrange
On Wed, Mar 01, 2017 at 06:22:45PM +0200, Michael S. Tsirkin wrote: > On Wed, Mar 01, 2017 at 09:50:38AM -0500, Stefan Berger wrote: > > I had already proposed a linked-in version before I went to the > > out-of-process > > design. Anthony's concerns back then were related to the code not being >

Re: [Qemu-devel] [PULL v2 0/5] Block patches for 2.9

2017-03-01 Thread Peter Maydell
On 28 February 2017 at 16:34, Jeff Cody wrote: > The following changes since commit c8c0a1a784cdf70ecea50e93213137c6c89337a7: > > Merge remote-tracking branch 'remotes/cody/tags/block-pull-request' into > staging (2017-02-28 13:41:03 +) > > are available in the git repository at: > > g...

Re: [Qemu-devel] intermittent make check failure: "tcg_handle_interrupt: assertion failed: (qemu_mutex_iothread_locked())"

2017-03-01 Thread Mark Cave-Ayland
On 01/03/17 15:19, Alex Bennée wrote: > Actually looking at where cpu_check_irq is I've pushed the BQL wraps > higher up: > > > https://github.com/stsquad/qemu/commit/f52e01bf4d3415ce4fad54cbb491f6c30a8e1903 > > The reasoning is HW emulation code can expect to be run under the BQL. > Anything

Re: [Qemu-devel] [PATCH v5 1/4] Provide support for the CUSE TPM

2017-03-01 Thread Michael S. Tsirkin
On Wed, Mar 01, 2017 at 09:50:38AM -0500, Stefan Berger wrote: > I had already proposed a linked-in version before I went to the out-of-process > design. Anthony's concerns back then were related to the code not being > trusted > and a segfault in the code could bring down all of QEMU. That we hav

Re: [Qemu-devel] [PATCH RFC] block: Tolerate existing writers on read only BdrvChild

2017-03-01 Thread Kevin Wolf
Am 01.03.2017 um 17:10 hat Fam Zheng geschrieben: > On Wed, 03/01 16:16, Kevin Wolf wrote: > > > I'm not sure about this because: 1) this is intrusive from a user PoV, > > > many > > > scripts and upper layer tools will stop working; > > > > Are you sure? I don't expect user scripts or even prope

Re: [Qemu-devel] [PATCH 3/5] xen: create wrappers for all other uses of xc_hvm_XXX() functions

2017-03-01 Thread Paul Durrant
> -Original Message- > From: Anthony PERARD [mailto:anthony.per...@citrix.com] > Sent: 01 March 2017 16:14 > To: Paul Durrant > Cc: xen-de...@lists.xenproject.org; qemu-devel@nongnu.org; Stefano > Stabellini ; Paolo Bonzini ; > Richard Henderson ; Eduardo Habkost > ; Michael S. Tsirkin >

Re: [Qemu-devel] [PATCH] virtio: guard vring access when setting notification

2017-03-01 Thread Cornelia Huck
On Wed, 1 Mar 2017 17:55:24 +0200 "Michael S. Tsirkin" wrote: > On Tue, Feb 28, 2017 at 04:24:11PM +0100, Cornelia Huck wrote: > > Switching to vring caches exposed an existing bug in > > virtio_queue_set_notification(): We can't access vring structures > > if they have not been set up yet. This

Re: [Qemu-devel] [PATCH 3/5] xen: create wrappers for all other uses of xc_hvm_XXX() functions

2017-03-01 Thread Anthony PERARD
On Thu, Feb 23, 2017 at 02:53:53PM +, Paul Durrant wrote: > This patch creates inline wrapper functions in xen_common.h for all open > coded calls to xc_hvm_XXX() functions outside of xen_common.h so that use > of xen_xc can be made implicit. This again is in preparation for the move > to using

Re: [Qemu-devel] [PATCH v5 1/4] Provide support for the CUSE TPM

2017-03-01 Thread Daniel P. Berrange
On Wed, Mar 01, 2017 at 10:40:13AM -0500, Stefan Berger wrote: > On 03/01/2017 10:18 AM, Daniel P. Berrange wrote: > > This goes back to the question Dave mentions above ? Ignoring the control > > channel aspect temporarily, can the CUSE TPM support the exact same ioctl > > interface as the existin

Re: [Qemu-devel] [PATCH RFC] block: Tolerate existing writers on read only BdrvChild

2017-03-01 Thread Fam Zheng
On Wed, 03/01 16:16, Kevin Wolf wrote: > > I'm not sure about this because: 1) this is intrusive from a user PoV, many > > scripts and upper layer tools will stop working; > > Are you sure? I don't expect user scripts or even proper management > tools to use qemu-io on running VMs. I do expect tha

Re: [Qemu-devel] [PATCH] iotests: Update 049 output

2017-03-01 Thread Fam Zheng
On Wed, 03/01 16:45, Kevin Wolf wrote: > Am 01.03.2017 um 16:26 hat Fam Zheng geschrieben: > > 75cdcd155 improved the error message, which affects the reference > > output. Let's sync it up. > > > > Note that previously '1kilobyte' was treated like '1k', but now it's an > > error. Looking at the c

Re: [Qemu-devel] [PATCH 1/1] virtio: fallback from irqfd to non-irqfd notify

2017-03-01 Thread Halil Pasic
On 03/01/2017 03:29 PM, Paolo Bonzini wrote: > > > On 01/03/2017 14:22, Halil Pasic wrote: >> Here a trace: >> >> 135871@1488304024.512533:virtio_blk_req_complete req 0x2aa6b117e10 status 0 >> 135871@1488304024.512541:virtio_notify_irqfd vdev 0x2aa6b0e19d8 vq >> 0x2aa6b4c0870 >> 135871@1488304

Re: [Qemu-devel] [PATCH v5 1/4] Provide support for the CUSE TPM

2017-03-01 Thread Stefan Berger
On 03/01/2017 10:24 AM, Marc-André Lureau wrote: Hi On Wed, Mar 1, 2017 at 6:50 PM Stefan Berger mailto:stef...@linux.vnet.ibm.com>> wrote: On 03/01/2017 09:17 AM, Marc-André Lureau wrote: Hi On Wed, Mar 1, 2017 at 5:26 PM Stefan Berger mailto:stef...@us.ibm.com>> wrote:

Re: [Qemu-devel] [PATCH] virtio: guard vring access when setting notification

2017-03-01 Thread Michael S. Tsirkin
On Tue, Feb 28, 2017 at 04:24:11PM +0100, Cornelia Huck wrote: > Switching to vring caches exposed an existing bug in > virtio_queue_set_notification(): We can't access vring structures > if they have not been set up yet. This may happen, for example, > for virtio-blk devices with multiple queues:

Re: [Qemu-devel] [PATCH] iotests: Update 049 output

2017-03-01 Thread Kevin Wolf
Am 01.03.2017 um 16:26 hat Fam Zheng geschrieben: > 75cdcd155 improved the error message, which affects the reference > output. Let's sync it up. > > Note that previously '1kilobyte' was treated like '1k', but now it's an > error. Looking at the commit in question, this appears to be intended. >

[Qemu-devel] [PULL v3 2/3] audio: make audio poll timer deterministic

2017-03-01 Thread Gerd Hoffmann
From: Pavel Dovgalyuk This patch changes resetting strategy of the audio polling timer. It does not change expiration time if the timer is already set. This patch is needed to make this timer deterministic and to use execution record/replay for audio devices. audio_reset_timer is used in the fun

[Qemu-devel] [PULL v3 1/3] replay: add record/replay for audio passthrough

2017-03-01 Thread Gerd Hoffmann
From: Pavel Dovgalyuk This patch adds recording and replaying audio data. Is saves synchronization information for audio out and inputs from the microphone. v2: removed unneeded whitespace change Signed-off-by: Pavel Dovgalyuk Message-id: 20170202055054.4848.94901.st...@pasha-isp.lan02.inno [

[Qemu-devel] [PULL v3 0/3] audio patch queue

2017-03-01 Thread Gerd Hoffmann
repository at: git://git.kraxel.org/qemu tags/pull-audio-20170301-1 for you to fetch changes up to bcf19777df78193f7cdb108a55db44fd4f20d5b5: audio/sdlaudio: Allow audio playback with SDL2 (2017-03-01 15:12:03 +0100) audio: replay

Re: [Qemu-devel] [PATCH] iotests: Update 049 output

2017-03-01 Thread Eric Blake
On 03/01/2017 09:26 AM, Fam Zheng wrote: > 75cdcd155 improved the error message, which affects the reference > output. Let's sync it up. > > Note that previously '1kilobyte' was treated like '1k', but now it's an > error. Looking at the commit in question, this appears to be intended. > > Signed-

[Qemu-devel] [PULL v3 3/3] audio/sdlaudio: Allow audio playback with SDL2

2017-03-01 Thread Gerd Hoffmann
From: Thomas Huth When compiling with SDL2, the semaphore trick used in sdlaudio.c does not work - QEMU locks up completely in this case. To avoid the hang and get at least some audio playback up and running (it's a little bit crackling, but better than nothing), we can use the SDL locking functi

Re: [Qemu-devel] [PATCH v5 1/4] Provide support for the CUSE TPM

2017-03-01 Thread Stefan Berger
On 03/01/2017 10:18 AM, Daniel P. Berrange wrote: On Wed, Mar 01, 2017 at 08:25:43AM -0500, Stefan Berger wrote: "Daniel P. Berrange" wrote on 03/01/2017 07:54:14 AM: From: "Daniel P. Berrange" To: Stefan Berger Cc: "Dr. David Alan Gilbert" , Stefan Berger/ Watson/IBM@IBMUS, "m...@redhat.co

Re: [Qemu-devel] [PATCH] migration: allow clearing migration string parameters

2017-03-01 Thread Eric Blake
On 03/01/2017 08:48 AM, Daniel P. Berrange wrote: >> >> So now we have a dilemma: do we special case "" here, to be stuck with >> it even if we later add nullable-string support later, or do we go all >> the way to nullable-string support now? We've missed soft freeze for >> 2.9, so my vote is th

[Qemu-devel] [PATCH] iotests: Update 049 output

2017-03-01 Thread Fam Zheng
75cdcd155 improved the error message, which affects the reference output. Let's sync it up. Note that previously '1kilobyte' was treated like '1k', but now it's an error. Looking at the commit in question, this appears to be intended. Signed-off-by: Fam Zheng --- tests/qemu-iotests/049.out | 14

[Qemu-devel] [PATCH v1] target/ppc: rewrite f[n]m[add, sub] using float64_muladd

2017-03-01 Thread Nikunj A Dadhania
Use the softfloat api for fused multiply-add. Also, generate VXISI using a helper function by computing intermediate result. Signed-off-by: Nikunj A Dadhania --- v0: * Use MADD/MSUB_FLAGS as used by VSX instructions * Introduce helper float64_madd_set_vxisi() --- target/ppc/fpu_helper.c | 218

Re: [Qemu-devel] Windows Pipe

2017-03-01 Thread Jiahuan Zhang
I think the reading failure in 2 is because there is no writing to the pipe from the frontend (serial port). So i wonder if the windows pipe implementation is complete. I mean dual-way pipe. >From the codes, I sense not. If somebody knows please inform me. 🙏 Thanks a lot! On 1 March 2017 at 16:1

Re: [Qemu-devel] [PATCH v5 1/4] Provide support for the CUSE TPM

2017-03-01 Thread Marc-André Lureau
Hi On Wed, Mar 1, 2017 at 6:50 PM Stefan Berger wrote: On 03/01/2017 09:17 AM, Marc-André Lureau wrote: Hi On Wed, Mar 1, 2017 at 5:26 PM Stefan Berger wrote: "Daniel P. Berrange" wrote on 03/01/2017 07:54:14 AM: > > On Wed, Mar 01, 2017 at 07:25:28AM -0500, Stefan Berger wrote: > > On 06/

Re: [Qemu-devel] [PATCH 2/5] xen: rename xen_modified_memory() to xen_hvm_modified_memory()

2017-03-01 Thread Anthony PERARD
On Thu, Feb 23, 2017 at 02:53:52PM +, Paul Durrant wrote: > This patch is a purely cosmetic change that avoids a name collision in > a subsequent patch. > > Signed-off-by: Paul Durrant Reviewed-by: Anthony PERARD -- Anthony PERARD

Re: [Qemu-devel] [PATCH 1/5] xen: make use of xen_xc implicit in xen_common.h inlines

2017-03-01 Thread Anthony PERARD
On Thu, Feb 23, 2017 at 02:53:51PM +, Paul Durrant wrote: > Doing this will make the transition to using the new libxendevicemodel > interface less intrusive on the callers of these functions, since using > the new library will require a change of handle. > > NOTE: The patch also moves the 'ex

Re: [Qemu-devel] intermittent make check failure: "tcg_handle_interrupt: assertion failed: (qemu_mutex_iothread_locked())"

2017-03-01 Thread Alex Bennée
Mark Cave-Ayland writes: > On 01/03/17 12:41, Alex Bennée wrote: > >>> Here's the backtrace from my test case of qemu-system-sparc running on >>> an x86_64 host that I posted yesterday: >>> >>> >>> $ gdb --args ./qemu-system-sparc -cdrom >>> /home/build/src/qemu/image/sparc32/debian-40r4a-sparc-

Re: [Qemu-devel] [PATCH v5 1/4] Provide support for the CUSE TPM

2017-03-01 Thread Daniel P. Berrange
On Wed, Mar 01, 2017 at 08:25:43AM -0500, Stefan Berger wrote: > "Daniel P. Berrange" wrote on 03/01/2017 07:54:14 > AM: > > > From: "Daniel P. Berrange" > > To: Stefan Berger > > Cc: "Dr. David Alan Gilbert" , Stefan Berger/ > > Watson/IBM@IBMUS, "m...@redhat.com" , "qemu- > > de...@nongnu.or

Re: [Qemu-devel] [PATCH RFC] block: Tolerate existing writers on read only BdrvChild

2017-03-01 Thread Kevin Wolf
Am 01.03.2017 um 13:39 hat Fam Zheng geschrieben: > On Wed, 03/01 10:49, Kevin Wolf wrote: > > Am 01.03.2017 um 09:15 hat Fam Zheng geschrieben: > > > In an ideal world, read-write access to an image is inherently > > > exclusive, because we cannot guarantee other readers and writers a > > > consis

Re: [Qemu-devel] Windows Pipe

2017-03-01 Thread Jiahuan Zhang
What I have done is two things. 1. I create a NamedPipeClient.exe to open the pipe and write a string into it. QEMU indicates the string is recieved from the pipe. 2. I create another NamedPipeClient.exe to open the pipe and read the data in the pipe. In qemu, I run a test.bin as kernel with vers

Re: [Qemu-devel] intermittent make check failure: "tcg_handle_interrupt: assertion failed: (qemu_mutex_iothread_locked())"

2017-03-01 Thread Mark Cave-Ayland
On 01/03/17 12:41, Alex Bennée wrote: >> Here's the backtrace from my test case of qemu-system-sparc running on >> an x86_64 host that I posted yesterday: >> >> >> $ gdb --args ./qemu-system-sparc -cdrom >> /home/build/src/qemu/image/sparc32/debian-40r4a-sparc-netinst.iso -boot d >> GNU gdb (GDB)

Re: [Qemu-devel] Windows Pipe

2017-03-01 Thread Marc-André Lureau
Hi On Wed, Mar 1, 2017 at 6:42 PM Jiahuan Zhang wrote: > Hi, > > I agree with you about win_chr_poll(). But what I wonder is how to write > data into the pipe for the host to get. I see "writeFile()" in > "win_chr_write()", but have no idea how to use it. Or I have to write > something for this

Re: [Qemu-devel] [PATCH v5 1/4] Provide support for the CUSE TPM

2017-03-01 Thread Stefan Berger
On 03/01/2017 09:17 AM, Marc-André Lureau wrote: Hi On Wed, Mar 1, 2017 at 5:26 PM Stefan Berger > wrote: "Daniel P. Berrange" mailto:berra...@redhat.com>> wrote on 03/01/2017 07:54:14 AM: > > On Wed, Mar 01, 2017 at 07:25:28AM -0500, Stefan Berger wr

Re: [Qemu-devel] [PATCH] migration: allow clearing migration string parameters

2017-03-01 Thread Daniel P. Berrange
On Wed, Mar 01, 2017 at 08:36:03AM -0600, Eric Blake wrote: > On 03/01/2017 06:32 AM, Daniel P. Berrange wrote: > > Some of the migration parameters are strings, which default to NULL, > > eg tls_hostname and tls_creds. > > > > The mgmt app will set the tls_creds parameter on both source and targe

Re: [Qemu-devel] [PATCH] migration: allow clearing migration string parameters

2017-03-01 Thread Eric Blake
On 03/01/2017 06:32 AM, Daniel P. Berrange wrote: > Some of the migration parameters are strings, which default to NULL, > eg tls_hostname and tls_creds. > > The mgmt app will set the tls_creds parameter on both source and target > QEMU instances, in order to trigger use of TLS for migration. > >

Re: [Qemu-devel] Windows Pipe

2017-03-01 Thread Jiahuan Zhang
Hi, I agree with you about win_chr_poll(). But what I wonder is how to write data into the pipe for the host to get. I see "writeFile()" in "win_chr_write()", but have no idea how to use it. Or I have to write something for this writing. In this case, I don't know what the right buffer pointer for

Re: [Qemu-devel] [PATCH] Removed header qemu-common.h from path.c

2017-03-01 Thread no-reply
Hi, This series failed automatic build test. Please find the testing commands and their output below. If you have docker installed, you can probably reproduce it locally. Message-id: 20170301142320.8328-1-shah.sura...@gmail.com Type: series Subject: [Qemu-devel] [PATCH] Removed header qemu-commo

Re: [Qemu-devel] [PULL 00/28] 9p CVE-2016-9602 fixes 2017-02-28 for 2.9 soft freeze

2017-03-01 Thread Peter Maydell
On 28 February 2017 at 10:30, Greg Kurz wrote: > The following changes since commit 9b9fbe8a4e9eec9072ee2697a6af5912785f: > > Merge remote-tracking branch 'remotes/kraxel/tags/pull-ui-20170227-1' into > staging (2017-02-27 19:19:46 +) > > are available in the git repository at: > > ht

Re: [Qemu-devel] [PATCH 1/1] virtio: fallback from irqfd to non-irqfd notify

2017-03-01 Thread Paolo Bonzini
On 01/03/2017 14:22, Halil Pasic wrote: > Here a trace: > > 135871@1488304024.512533:virtio_blk_req_complete req 0x2aa6b117e10 status 0 > 135871@1488304024.512541:virtio_notify_irqfd vdev 0x2aa6b0e19d8 vq > 0x2aa6b4c0870 > 135871@1488304024.522607:virtio_blk_req_complete req 0x2aa6b118980 statu

Re: [Qemu-devel] [PATCH] Removed header qemu-common.h from path.c

2017-03-01 Thread no-reply
Hi, This series failed build test on s390x host. Please find the details below. Message-id: 20170301142320.8328-1-shah.sura...@gmail.com Type: series Subject: [Qemu-devel] [PATCH] Removed header qemu-common.h from path.c === TEST SCRIPT BEGIN === #!/bin/bash # Testing script will be invoked und

Re: [Qemu-devel] Windows Pipe

2017-03-01 Thread Marc-André Lureau
Hi On Wed, Mar 1, 2017 at 5:16 PM Jiahuan Zhang wrote: > Dear QEMU developers, > > I want the host-guest communication for QEMU. I want to achieve > it via serial port redirection to a pipe. My host is windows and guest is > Linux. QEMU is in version 2.8.50 > > I looked into char-pipe.c and char

[Qemu-devel] [PATCH] Removed header qemu-common.h from path.c

2017-03-01 Thread Suramya Shah
Signed-off-by: Suramya Shah --- util/path.c | 3 +-- 1 file changed, 1 insertion(+), 2 deletions(-) diff --git a/util/path.c b/util/path.c index 5479f76..46a6098 100644 --- a/util/path.c +++ b/util/path.c @@ -6,8 +6,7 @@ #include "qemu/osdep.h" #include #include -#include "qemu-common.h" -#

Re: [Qemu-devel] [PATCH] Add inst_dirty_pages_rate in 'info migrate'

2017-03-01 Thread Eric Blake
On 03/01/2017 02:19 AM, Chao Fan wrote: > Auto-converge aims to accelerate migration by slowing down the > generation of dirty pages. But user doesn't know how to determine the > throttle value, so, a new item "inst-dirty-pages-rate" in "info migrate" > would be helpful for user's determination. >

Re: [Qemu-devel] [PATCH v5 1/4] Provide support for the CUSE TPM

2017-03-01 Thread Marc-André Lureau
Hi On Wed, Mar 1, 2017 at 5:26 PM Stefan Berger wrote: > "Daniel P. Berrange" wrote on 03/01/2017 07:54:14 > AM: > > > From: "Daniel P. Berrange" > > To: Stefan Berger > > Cc: "Dr. David Alan Gilbert" , Stefan Berger/ > > Watson/IBM@IBMUS, "m...@redhat.com" , "qemu- > > de...@nongnu.org" , "S

[Qemu-devel] [PULL 1/2] q35: Improve sample configuration files

2017-03-01 Thread Gerd Hoffmann
From: Andrea Bolognani Instead of having a single sample configuration file, we now have several: * q35-emulated.cfg documents the default devices QEMU adds to a q35 guest and the additional devices that are pretty much guaranteed to be present in a physical q35-based machine; *

[Qemu-devel] [PULL 0/2] docs: update sample configuration files

2017-03-01 Thread Gerd Hoffmann
tags/pull-docs-20170301-1 for you to fetch changes up to 166d434685ed6da5db88aca0c0599114e0c54462: mach-virt: Provide sample configuration files (2017-03-01 14:40:40 +0100) docs: update sample conf

Re: [Qemu-devel] [PULL 0/2] Docker patches

2017-03-01 Thread Peter Maydell
On 28 February 2017 at 12:44, Fam Zheng wrote: > The following changes since commit 6181478f6395cdd9d6ffd99623d0c9f39ea53606: > > Merge remote-tracking branch 'remotes/stefanha/tags/block-pull-request' > into staging (2017-02-28 08:46:03 +) > > are available in the git repository at: > >

[Qemu-devel] [PULL 2/2] mach-virt: Provide sample configuration files

2017-03-01 Thread Gerd Hoffmann
From: Andrea Bolognani These are very much like the sample configuration files for q35, and can be used both as documentation and as a starting point for creating your own guest. Two sample configuration files are provided: * mach-virt-graphical.cfg can be used to start a fully-featured (

Re: [Qemu-devel] [PATCH 1/1] virtio: fallback from irqfd to non-irqfd notify

2017-03-01 Thread Halil Pasic
On 03/01/2017 01:54 PM, Michael S. Tsirkin wrote: > On Wed, Mar 01, 2017 at 12:50:04PM +0100, Halil Pasic wrote: >> The commits 03de2f527 "virtio-blk: do not use vring in dataplane" and >> 9ffe337c08 "virtio-blk: always use dataplane path if ioeventfd is active" >> changed how notifications are

Re: [Qemu-devel] [PULL 0/1] seabios: update to 1.10.2 release

2017-03-01 Thread Stefan Hajnoczi
On Tue, Feb 28, 2017 at 9:06 AM, Gerd Hoffmann wrote: > Our seabios.git mirror @ qemu.org seems to not sync properly. The > commit isn't there even though it was pushed a few days ago already > (only the tag was pushed today). John, can you have a look please? > master branch, *-stable branches

Re: [Qemu-devel] [PATCH v5 1/4] Provide support for the CUSE TPM

2017-03-01 Thread Stefan Berger
"Daniel P. Berrange" wrote on 03/01/2017 07:54:14 AM: > From: "Daniel P. Berrange" > To: Stefan Berger > Cc: "Dr. David Alan Gilbert" , Stefan Berger/ > Watson/IBM@IBMUS, "m...@redhat.com" , "qemu- > de...@nongnu.org" , "SERBAN, CRISTINA" > , "Xu, Quan" , > "silviu.vlasce...@gmail.com" , >

Re: [Qemu-devel] [PATCH 1/1] virtio: fallback from irqfd to non-irqfd notify

2017-03-01 Thread Halil Pasic
On 03/01/2017 01:57 PM, Paolo Bonzini wrote: > > > On 01/03/2017 12:50, Halil Pasic wrote: >> The commits 03de2f527 "virtio-blk: do not use vring in dataplane" and >> 9ffe337c08 "virtio-blk: always use dataplane path if ioeventfd is active" >> changed how notifications are done for virtio-blk

[Qemu-devel] Windows Pipe

2017-03-01 Thread Jiahuan Zhang
Dear QEMU developers, I want the host-guest communication for QEMU. I want to achieve it via serial port redirection to a pipe. My host is windows and guest is Linux. QEMU is in version 2.8.50 I looked into char-pipe.c and char-win.c, and find that the current QEMU only supports reading data on t

Re: [Qemu-devel] [PULL 00/27] migration queue

2017-03-01 Thread Dr. David Alan Gilbert
* Peter Maydell (peter.mayd...@linaro.org) wrote: > On 28 February 2017 at 12:40, Dr. David Alan Gilbert (git) > wrote: > > From: "Dr. David Alan Gilbert" > > > > The following changes since commit 6181478f6395cdd9d6ffd99623d0c9f39ea53606: > > > > Merge remote-tracking branch 'remotes/stefanha/

Re: [Qemu-devel] [PATCH v3 1/3] qemu-options: explain disk I/O throttling options

2017-03-01 Thread Alberto Garcia
On Wed 01 Mar 2017 12:50:24 PM CET, Stefan Hajnoczi wrote: > The disk I/O throttling options have been listed for a long time but > never explained on the QEMU man page. > > Suggested-by: Nini Gu > Cc: Alberto Garcia > Signed-off-by: Stefan Hajnoczi Reviewed-by: Alberto Garcia Berto

Re: [Qemu-devel] [PULL 00/27] migration queue

2017-03-01 Thread Peter Maydell
On 28 February 2017 at 12:40, Dr. David Alan Gilbert (git) wrote: > From: "Dr. David Alan Gilbert" > > The following changes since commit 6181478f6395cdd9d6ffd99623d0c9f39ea53606: > > Merge remote-tracking branch 'remotes/stefanha/tags/block-pull-request' > into staging (2017-02-28 08:46:03 +0

Re: [Qemu-devel] [PULL v4 00/19] KVM and cpu-exec patches for 2.9 soft freeze

2017-03-01 Thread Peter Maydell
On 1 March 2017 at 12:29, Paolo Bonzini wrote: > The following changes since commit d992f2f1368ceb92e6bfd8efece174110f4236ff: > > Merge remote-tracking branch 'remotes/artyom/tags/pull-sun4v-20170226' into > staging (2017-02-26 22:40:23 +) > > are available in the git repository at: > > g

Re: [Qemu-devel] [PULL for-2.9 0/5] s390x: network boot

2017-03-01 Thread Peter Maydell
On 28 February 2017 at 11:31, Cornelia Huck wrote: > The following changes since commit 6181478f6395cdd9d6ffd99623d0c9f39ea53606: > > Merge remote-tracking branch 'remotes/stefanha/tags/block-pull-request' > into staging (2017-02-28 08:46:03 +) > > are available in the git repository at: >

Re: [Qemu-devel] [PATCH 1/1] virtio: fallback from irqfd to non-irqfd notify

2017-03-01 Thread Paolo Bonzini
On 01/03/2017 12:50, Halil Pasic wrote: > The commits 03de2f527 "virtio-blk: do not use vring in dataplane" and > 9ffe337c08 "virtio-blk: always use dataplane path if ioeventfd is active" > changed how notifications are done for virtio-blk substantially. Due to a > race condition interrupts are

<    1   2   3   >