Re: [Qemu-devel] [v3 PATCH] target-mips: implement UserLocal Register

2014-05-29 Thread James Hogan
On 29/05/14 16:38, Peter Maydell wrote: On 29 May 2014 15:29, James Hogan james.ho...@imgtec.com wrote: On 29/05/14 14:26, Peter Maydell wrote: On 29 May 2014 14:22, James Hogan james.ho...@imgtec.com wrote: loadvm/migration will write Config3, possibly with a different value if the snapshot

Re: [Qemu-devel] [PATCH v2 2/3] e1000: allow command-line selection of card model

2014-05-29 Thread Gabriel L. Somlo
Hi Peter, Thanks for the QOM crash course ! So I did s/TYPE_E1000/TYPE_E1000_BASE/ as suggested by Stefan, and also moved PCI_VENDOR_ID_INTEL back into e1000_class_init() and away from e1000_devices[] as you suggested. That's going to be part of v3, as soon as we're done talking about QOM, see

Re: [Qemu-devel] [PATCH v4 05/10] trace: [tcg] Declare TCG tracing helper routines

2014-05-29 Thread Richard Henderson
On 05/29/2014 05:23 AM, Lluís Vilanova wrote: +'#ifdef HELPER_GEN_TRACE_PROXY', +'static inline void %(name)s(%(args)s)', +'{', +'%(code_new)s', +'%(proxy_name)s(%(tmp_names)s);', +'%(code_free)s', +

Re: [Qemu-devel] [PATCH v4 07/10] trace: [tcg] Include TCG-tracing helpers

2014-05-29 Thread Richard Henderson
On 05/29/2014 05:23 AM, Lluís Vilanova wrote: #include helper.h +#define HELPER_GEN_TRACE_PROXY 1 +#include trace/generated-helpers.h +#undef HELPER_GEN_TRACE_PROXY #include tcg-runtime.h Continued from patch 5... you don't need the HELPER_GEN_TRACE_PROXY define, just two include

Re: [Qemu-devel] [PATCH v4 10/10] trace: [tcg] Include TCG-tracing header on all targets

2014-05-29 Thread Richard Henderson
On 05/29/2014 05:23 AM, Lluís Vilanova wrote: Signed-off-by: Lluís Vilanova vilan...@ac.upc.edu --- target-alpha/translate.c |3 +++ target-arm/translate-a64.c|2 ++ target-arm/translate.c|3 +++ target-cris/translate.c |3 +++ target-i386/translate.c

Re: [Qemu-devel] [PATCH v3 05/34] add memdev backend infrastructure

2014-05-29 Thread Eric Blake
On 05/27/2014 07:01 AM, Igor Mammedov wrote: Provides framework for splitting host RAM allocation/ policies into a separate backend that could be used by devices. Initially only legacy RAM backend is provided, which uses memory_region_init_ram() allocator and compatible with every CLI

Re: [Qemu-devel] [PATCH] target-arm: add support for v8 SHA1 and SHA256 instructions

2014-05-29 Thread Peter Maydell
On 25 March 2014 16:27, Ard Biesheuvel ard.biesheu...@linaro.org wrote: This adds support for the SHA1 and SHA256 instructions that are available on some v8 implementations of Aarch32. Signed-off-by: Ard Biesheuvel ard.biesheu...@linaro.org Apologies for the incredibly late review; I was

[Qemu-devel] [qom/qom-cpu PATCH] i386: improve invtsc migration blocker error message

2014-05-29 Thread Marcelo Tosatti
Improve migration blocker error message. Signed-off-by: Marcelo Tosatti mtosa...@redhat.com diff --git a/target-i386/kvm.c b/target-i386/kvm.c index f9ffa4b..cfe1271 100644 --- a/target-i386/kvm.c +++ b/target-i386/kvm.c @@ -711,7 +711,8 @@ int kvm_arch_init_vcpu(CPUState *cs) if (c

Re: [Qemu-devel] [PATCH v4 10/10] trace: [tcg] Include TCG-tracing header on all targets

2014-05-29 Thread Lluís Vilanova
Richard Henderson writes: On 05/29/2014 05:23 AM, Lluís Vilanova wrote: Signed-off-by: Lluís Vilanova vilan...@ac.upc.edu --- target-alpha/translate.c |3 +++ target-arm/translate-a64.c|2 ++ target-arm/translate.c|3 +++ target-cris/translate.c |3 +++

Re: [Qemu-devel] [PATCH v4 07/10] trace: [tcg] Include TCG-tracing helpers

2014-05-29 Thread Lluís Vilanova
Richard Henderson writes: On 05/29/2014 05:23 AM, Lluís Vilanova wrote: #include helper.h +#define HELPER_GEN_TRACE_PROXY 1 +#include trace/generated-helpers.h +#undef HELPER_GEN_TRACE_PROXY #include tcg-runtime.h Continued from patch 5... you don't need the HELPER_GEN_TRACE_PROXY

Re: [Qemu-devel] [PATCH] target-arm: add support for v8 SHA1 and SHA256 instructions

2014-05-29 Thread Ard Biesheuvel
On 29 May 2014 18:46, Peter Maydell peter.mayd...@linaro.org wrote: On 25 March 2014 16:27, Ard Biesheuvel ard.biesheu...@linaro.org wrote: This adds support for the SHA1 and SHA256 instructions that are available on some v8 implementations of Aarch32. Signed-off-by: Ard Biesheuvel

[Qemu-devel] [PATCH] block/iscsi: handle BUSY condition

2014-05-29 Thread Peter Lieven
this patch adds handling of BUSY status reponse from an iSCSI target. Currently, we fail with -EIO in case of SCSI_STATUS_BUSY while the obvious reaction would be to retry the operation after some time. The retry time is randomly choosen from a range with exponential growth increasing with each

[Qemu-devel] [v4 PATCH] target-mips: implement UserLocal Register

2014-05-29 Thread Petar Jovanovic
From: Petar Jovanovic petar.jovano...@imgtec.com From MIPS documentation (Volume III): UserLocal Register (CP0 Register 4, Select 2) Compliance Level: Recommended. The UserLocal register is a read-write register that is not interpreted by the hardware and conditionally readable via the RDHWR

Re: [Qemu-devel] [v3 PATCH] target-mips: implement UserLocal Register

2014-05-29 Thread Petar Jovanovic
I have just posted a patch copying CP0_Config1 into DisasContext, and moving existing accesses to this variable accordingly. This can be used as an example how to do it. Done in v4. Thanks. Regards, Petar From: Aurelien Jarno [aurel...@aurel32.net]

Re: [Qemu-devel] [PATCH] target-arm: add support for v8 VMULL.P64 instruction

2014-05-29 Thread Peter Maydell
On 27 March 2014 09:29, Ard Biesheuvel ard.biesheu...@linaro.org wrote: This adds support for the VMULL.P64 polynomial 64x64 to 128 bit multiplication instruction, which is an optional feature that is part of the v8 Crypto Extensions. Signed-off-by: Ard Biesheuvel ard.biesheu...@linaro.org

Re: [Qemu-devel] qemu 2.0, deadlock in block-commit

2014-05-29 Thread Marcin Gibuła
Please try disabling I/O limits on the drive and try again. Is there anything else I could try? I've captured trace of hanged VM with following events traced: bdrv_* paio_* thread_pool_* commit_* qcow2_* and debug code that prints requests from traced_requests in bdrv_requests_pending

Re: [Qemu-devel] [PATCH] block/iscsi: handle BUSY condition

2014-05-29 Thread Paolo Bonzini
Il 29/05/2014 19:31, Peter Lieven ha scritto: +static const unsigned iscsi_retry_times[] = {4, 16, 64, 256, 1024, 4096}; This is a pretty fast growth. +static void iscsi_retry_timer_expired(void *opaque) +{ +struct IscsiTask *iTask = opaque; +timer_del(iTask-retry_timer); This

[Qemu-devel] [PATCH 0/3] ARM A32/T32: implement remaining crypto insns

2014-05-29 Thread Peter Maydell
This patchset implements the remaining A32/T32 instructions in the ARM v8 Crypto Extensions: SHA1, SHA256 and VMULL.P64. The bulk of this is Ard's SHA patch with a few fixups from me; the VMULL.P64 patch is mine, based on the existing A64 implementation. I hope to have time to fill in the A64

[Qemu-devel] [PATCH 2/3] target-arm: Allow 3reg_wide undefreq to encode more bad size options

2014-05-29 Thread Peter Maydell
The current undefreq field in the neon_3reg_wide handling allows us to encode UNDEF if size != 0 and UNDEF if size == 0. This is no longer sufficient with the advent of 64-bit polynomial VMULL, which means we want to UNDEF if size == 1. Change the undefreq encoding to use separate bits for all of

[Qemu-devel] [PATCH 3/3] target-arm: add support for v8 VMULL.P64 instruction

2014-05-29 Thread Peter Maydell
Add support for the VMULL.P64 polynomial 64x64 to 128 bit multiplication instruction in the A32/T32 instruction sets; this is part of the v8 Crypto Extensions. To do this we have to move the neon_pmull_64_{lo,hi} helpers from helper-a64.c into neon_helper.c so they can be used by the AArch32

[Qemu-devel] [PATCH 1/3] target-arm: add support for v8 SHA1 and SHA256 instructions

2014-05-29 Thread Peter Maydell
From: Ard Biesheuvel ard.biesheu...@linaro.org This adds support for the SHA1 and SHA256 instructions that are available on some v8 implementations of Aarch32. Signed-off-by: Ard Biesheuvel ard.biesheu...@linaro.org [PMM: * rebase * fix bad indent * add a missing UNDEF check for Q!=1 in the

Re: [Qemu-devel] [qom/qom-cpu PATCH] i386: improve invtsc migration blocker error message

2014-05-29 Thread Eduardo Habkost
On Thu, May 29, 2014 at 01:55:14PM -0300, Marcelo Tosatti wrote: Improve migration blocker error message. Signed-off-by: Marcelo Tosatti mtosa...@redhat.com Reviewed-by: Eduardo Habkost ehabk...@redhat.com -- Eduardo

Re: [Qemu-devel] [PATCH] block/iscsi: handle BUSY condition

2014-05-29 Thread Peter Lieven
Am 29.05.2014 19:47, schrieb Paolo Bonzini: Il 29/05/2014 19:31, Peter Lieven ha scritto: +static const unsigned iscsi_retry_times[] = {4, 16, 64, 256, 1024, 4096}; This is a pretty fast growth. I know, I wanted to end up with a total retry period of approx. 10 seconds before failing. As we

Re: [Qemu-devel] [libvirt] [PATCHv2 3/4] qemu: fix RTC_CHANGE event for clock offset='variable' basis='utc'/

2014-05-29 Thread Marcelo Tosatti
On Fri, May 23, 2014 at 11:36:56PM +0200, Paolo Bonzini wrote: You can get the current time via the qom-get command, and then follow the same algorithm as QEMU: time_t seconds; if (rtc_date_offset == -1) { if (rtc_utc) { seconds = mktimegm(tm); } else

Re: [Qemu-devel] [PATCH v2 RESEND] vl.c: Unify MAX_CPUMASK_BITS and machine-max_cpus checks

2014-05-29 Thread Eduardo Habkost
Ping? Who can get this applied? Without this, the machine-max_cpus limit is ignored when using cpu-add. On Wed, May 14, 2014 at 04:18:42PM -0300, Eduardo Habkost wrote: If a given machine have max_cpus set, not just smp_cpus needs to be limited, but the total number of CPUs (considering CPU

Re: [Qemu-devel] [PATCH V9 3/4] raw-posix: Add full image preallocation option

2014-05-29 Thread Peter Lieven
Am 29.05.2014 03:55, schrieb Hu Tao: On Wed, May 28, 2014 at 07:36:26AM +0200, Markus Armbruster wrote: Peter Lieven p...@kamp.de writes: Am 27.05.2014 um 21:24 schrieb Eric Blake ebl...@redhat.com: On 05/27/2014 02:22 AM, Chen Fan wrote: From: Hu Tao hu...@cn.fujitsu.com This patch adds

Re: [Qemu-devel] [qom/qom-cpu PATCH] i386: invtsc migration blocker is redudant

2014-05-29 Thread Marcelo Tosatti
On Wed, May 28, 2014 at 01:25:38PM -0300, Eduardo Habkost wrote: On Wed, May 28, 2014 at 11:14:14AM +0200, Juan Quintela wrote: Eduardo Habkost ehabk...@redhat.com wrote: On Tue, May 27, 2014 at 11:39:20AM -0300, Marcelo Tosatti wrote: Migration blocker is redudant: blocking savevm is

Re: [Qemu-devel] [PATCH] block/iscsi: handle BUSY condition

2014-05-29 Thread Paolo Bonzini
Il 29/05/2014 20:41, Peter Lieven ha scritto: static inline unsigned geo_rand_range(double m, double n) { return exp((log(m) + (double)rand() / (RAND_MAX / (log(n) - log(m) + log(1)) + log(1; } Yes, exactly (except that log(1) is zero, and was only needed in your case

[Qemu-devel] Commit 5b6fb069378e breaks Solaris/OpenIndiana support (icount)

2014-05-29 Thread Sai Prajeeth
Git bisection tells that 5b6fb069378e61c45c577bbec3d7ef60367f7e4c is the first bad commit that breaks support for Solaris/OpenIndiana 32-bit x86 guest with icount 1. Seems like the option rom (kvmvapic.bin) is causing problems. Currently the workaround is to not load the option rom. Command:

[Qemu-devel] [PATCH 1/2] virtio-serial: report frontend connection state via monitor

2014-05-29 Thread Laszlo Ersek
Libvirt wants to know about the guest-side connection state of some virtio-serial ports (in particular the one(s) assigned to guest agent(s)). Introduce a new property that allows libvirt to request connection state reporting, and report the state via new monitor events. RHBZ:

[Qemu-devel] [PATCH 0/2] help libvirt know what's up with qga

2014-05-29 Thread Laszlo Ersek
In this series I try to implement the ideas that (I believe) were suggested by Gerd and Amit in https://bugzilla.redhat.com/show_bug.cgi?id=1080376. When the guest agent exits or dies (disconnects from the virtio-serial port), the backend (eg. a host-side unix domain socket) doesn't (in general,

[Qemu-devel] [PATCH 2/2] char: report frontend open/closed state in 'query-chardev'

2014-05-29 Thread Laszlo Ersek
In addition to the on-line reporting added in the previous patch, allow libvirt to query frontend state independently of events. Libvirt's path to identify the guest agent channel it cares about differs between the event added in the previous patch and the QMP response field added here. The event

Re: [Qemu-devel] [qom/qom-cpu PATCH] i386: invtsc migration blocker is redudant

2014-05-29 Thread Eduardo Habkost
On Thu, May 29, 2014 at 04:00:36PM -0300, Marcelo Tosatti wrote: On Wed, May 28, 2014 at 01:25:38PM -0300, Eduardo Habkost wrote: On Wed, May 28, 2014 at 11:14:14AM +0200, Juan Quintela wrote: Eduardo Habkost ehabk...@redhat.com wrote: On Tue, May 27, 2014 at 11:39:20AM -0300, Marcelo

Re: [Qemu-devel] [PATCH 1/1] Sun4m : TCX framebuffer hardware acceleration

2014-05-29 Thread Olivier Danet
On 25/05/2014 22:28, Mark Cave-Ayland wrote: On 25/05/14 14:20, Olivier Danet wrote: Here is the original patch, I have changed email settings since then, it should work better. Alas, I have not merged latest QEMU changes (your CG3/TCX patches), so it will probably not compile as-is...

Re: [Qemu-devel] [PATCH 0/2] help libvirt know what's up with qga

2014-05-29 Thread Eric Blake
On 05/29/2014 01:36 PM, Laszlo Ersek wrote: In this series I try to implement the ideas that (I believe) were suggested by Gerd and Amit in https://bugzilla.redhat.com/show_bug.cgi?id=1080376. When the guest agent exits or dies (disconnects from the virtio-serial port), the backend (eg. a

[Qemu-devel] [PATCH v2 2/8] linux-user: arm: set CPSR.E correctly for BE8 mode

2014-05-29 Thread Paolo Bonzini
Set it on startup and in signal handler frames. Signed-off-by: Paolo Bonzini pbonz...@redhat.com --- linux-user/main.c | 1 + linux-user/signal.c | 5 + 2 files changed, 6 insertions(+) diff --git a/linux-user/main.c b/linux-user/main.c index 882186e..2a04446 100644 ---

[Qemu-devel] [PATCH v2 1/8] linux-user: arm: fix coding style for some linux-user signal functions

2014-05-29 Thread Paolo Bonzini
Signed-off-by: Paolo Bonzini pbonz...@redhat.com --- linux-user/signal.c | 117 +++- 1 file changed, 60 insertions(+), 57 deletions(-) diff --git a/linux-user/signal.c b/linux-user/signal.c index 5b8a01f..c652829 100644 --- a/linux-user/signal.c

[Qemu-devel] [PATCH v2 3/8] linux-user: arm: pass env to get_user_code_*

2014-05-29 Thread Paolo Bonzini
This matches the idiom used by get_user_data_* in the next patch. Signed-off-by: Paolo Bonzini pbonz...@redhat.com --- linux-user/main.c | 20 +--- 1 file changed, 9 insertions(+), 11 deletions(-) diff --git a/linux-user/main.c b/linux-user/main.c index 2a04446..8eb910a 100644

[Qemu-devel] [PATCH v2 4/8] linux-user: arm: handle CPSR.E correctly in strex emulation

2014-05-29 Thread Paolo Bonzini
Now that CPSR.E is set correctly, prepare for when setend will be able to change it; bswap data in and out of strex manually by comparing bswap_code to CPSR.E (we do not have the luxury of using TCGMemOps). Signed-off-by: Paolo Bonzini pbonz...@redhat.com --- linux-user/main.c | 50

[Qemu-devel] [PATCH v2 7/8] target-arm: introduce be8 tbflag

2014-05-29 Thread Paolo Bonzini
The tb flag for be8 mode comes from the CPSR E bit. This will let us implement setend in the next patch. Signed-off-by: Paolo Bonzini pbonz...@redhat.com --- target-arm/cpu.h | 7 +++ target-arm/translate.c | 39 --- target-arm/translate.h | 1 +

[Qemu-devel] [PATCH v2 8/8] target-arm: implement setend

2014-05-29 Thread Paolo Bonzini
Since this is not a high-performance path, just use a helper to flip the E bit and force a lookup in the hash table since the flags have changed. Signed-off-by: Paolo Bonzini pbonz...@redhat.com --- target-arm/helper.h| 1 + target-arm/op_helper.c | 5 + target-arm/translate.c | 12

[Qemu-devel] [PATCH v2 6/8] target-arm: pass DisasContext to gen_aa32_ld*/st*

2014-05-29 Thread Paolo Bonzini
We'll need the DisasContext in the next patch to retrieve the desired endianness, so pass it as a whole to gen_aa32_ld*/st*. Unfortunately we cannot let those functions call get_mem_index, because of user-mode load/store instructions. Signed-off-by: Paolo Bonzini pbonz...@redhat.com ---

Re: [Qemu-devel] [PATCH 1/2] virtio-serial: report frontend connection state via monitor

2014-05-29 Thread Eric Blake
On 05/29/2014 01:36 PM, Laszlo Ersek wrote: Libvirt wants to know about the guest-side connection state of some virtio-serial ports (in particular the one(s) assigned to guest agent(s)). Introduce a new property that allows libvirt to request connection state reporting, and report the state

[Qemu-devel] [PATCH v2 5/8] target-arm: implement SCTLR.EE

2014-05-29 Thread Paolo Bonzini
Set CPSR.E to SCTLR.EE on exception, and use SCTLR.EE also to determine endianness for loads during TLB misses. Signed-off-by: Paolo Bonzini pbonz...@redhat.com --- target-arm/helper.c | 38 -- 1 file changed, 32 insertions(+), 6 deletions(-) diff --git

Re: [Qemu-devel] [PATCH 2/2] char: report frontend open/closed state in 'query-chardev'

2014-05-29 Thread Eric Blake
On 05/29/2014 01:36 PM, Laszlo Ersek wrote: In addition to the on-line reporting added in the previous patch, allow libvirt to query frontend state independently of events. Libvirt's path to identify the guest agent channel it cares about differs between the event added in the previous patch

Re: [Qemu-devel] [PATCH] block/iscsi: handle BUSY condition

2014-05-29 Thread Peter Lieven
Am 29.05.2014 21:17, schrieb Paolo Bonzini: Il 29/05/2014 20:41, Peter Lieven ha scritto: static inline unsigned geo_rand_range(double m, double n) { return exp((log(m) + (double)rand() / (RAND_MAX / (log(n) - log(m) + log(1)) + log(1; } Yes, exactly (except that

Re: [Qemu-devel] [PATCH 0/2] help libvirt know what's up with qga

2014-05-29 Thread Laszlo Ersek
On 05/29/14 21:47, Eric Blake wrote: virsh qemu-monitor-event $dom serport --no-case --regex --pretty --loop to set up a listening loop that will fire on all case-insensitive events matching the regex serport (basically, the two events added in this patch) that occur on $dom, until you get

Re: [Qemu-devel] [PATCH v2 4/8] linux-user: arm: handle CPSR.E correctly in strex emulation

2014-05-29 Thread Peter Maydell
On 29 May 2014 20:46, Paolo Bonzini pbonz...@redhat.com wrote: Now that CPSR.E is set correctly, prepare for when setend will be able to change it; bswap data in and out of strex manually by comparing bswap_code to CPSR.E (we do not have the luxury of using TCGMemOps). Signed-off-by: Paolo

Re: [Qemu-devel] [PATCH 0/2] help libvirt know what's up with qga

2014-05-29 Thread Eric Blake
On 05/29/2014 01:36 PM, Laszlo Ersek wrote: In this series I try to implement the ideas that (I believe) were suggested by Gerd and Amit in https://bugzilla.redhat.com/show_bug.cgi?id=1080376. When the guest agent exits or dies (disconnects from the virtio-serial port), the backend (eg. a

Re: [Qemu-devel] [PATCH v2 5/8] target-arm: implement SCTLR.EE

2014-05-29 Thread Peter Maydell
On 29 May 2014 20:46, Paolo Bonzini pbonz...@redhat.com wrote: Set CPSR.E to SCTLR.EE on exception, and use SCTLR.EE also to determine endianness for loads during TLB misses. Signed-off-by: Paolo Bonzini pbonz...@redhat.com --- target-arm/helper.c | 38 --

Re: [Qemu-devel] [PATCH] block/iscsi: handle BUSY condition

2014-05-29 Thread Peter Lieven
Am 29.05.2014 21:17, schrieb Paolo Bonzini: Il 29/05/2014 20:41, Peter Lieven ha scritto: static inline unsigned geo_rand_range(double m, double n) { return exp((log(m) + (double)rand() / (RAND_MAX / (log(n) - log(m) + log(1)) + log(1; } Yes, exactly (except that

Re: [Qemu-devel] [PATCH] tcg-i386: Fix win64 qemu store

2014-05-29 Thread Stefan Weil
Am 28.05.2014 23:24, schrieb Stefan Weil: Am 28.05.2014 23:04, schrieb Richard Henderson: The first non-register argument isn't placed at offset 0. Cc: Stefan Weil s...@weilnetz.de Signed-off-by: Richard Henderson r...@twiddle.net --- tcg/i386/tcg-target.c | 3 ++- 1 file changed, 2

[Qemu-devel] [PATCH v2 0/8] target-arm/linux-user-arm: implement setend, CPSR.E, SCTLR.EE

2014-05-29 Thread Paolo Bonzini
This is v2 of the series to implement CPSR.E. I've now included support for strex emulation and SCTLR.EE. I've now also tested system emulation better using an Arch Linux image. Using the Raspberry Pi optimized memcmp library and associated harness, QEMU 2.0 fails with Illegal instruction,

Re: [Qemu-devel] [PATCH 1/2] virtio-serial: report frontend connection state via monitor

2014-05-29 Thread Laszlo Ersek
On 05/29/14 22:05, Eric Blake wrote: On 05/29/2014 01:36 PM, Laszlo Ersek wrote: Libvirt wants to know about the guest-side connection state of some virtio-serial ports (in particular the one(s) assigned to guest agent(s)). Introduce a new property that allows libvirt to request connection

Re: [Qemu-devel] [PATCH v2 2/8] linux-user: arm: set CPSR.E correctly for BE8 mode

2014-05-29 Thread Peter Maydell
On 29 May 2014 20:46, Paolo Bonzini pbonz...@redhat.com wrote: Set it on startup and in signal handler frames. Signed-off-by: Paolo Bonzini pbonz...@redhat.com --- linux-user/main.c | 1 + linux-user/signal.c | 5 + 2 files changed, 6 insertions(+) diff --git a/linux-user/main.c

Re: [Qemu-devel] [PATCH 2/2] char: report frontend open/closed state in 'query-chardev'

2014-05-29 Thread Laszlo Ersek
On 05/29/14 22:12, Eric Blake wrote: On 05/29/2014 01:36 PM, Laszlo Ersek wrote: In addition to the on-line reporting added in the previous patch, allow libvirt to query frontend state independently of events. Libvirt's path to identify the guest agent channel it cares about differs between

Re: [Qemu-devel] [PATCH 0/2] help libvirt know what's up with qga

2014-05-29 Thread Laszlo Ersek
On 05/29/14 22:24, Eric Blake wrote: On 05/29/2014 01:36 PM, Laszlo Ersek wrote: In this series I try to implement the ideas that (I believe) were suggested by Gerd and Amit in https://bugzilla.redhat.com/show_bug.cgi?id=1080376. When the guest agent exits or dies (disconnects from the

Re: [Qemu-devel] [PATCH 1/2] virtio-serial: report frontend connection state via monitor

2014-05-29 Thread Eric Blake
On 05/29/2014 02:36 PM, Laszlo Ersek wrote: Emitting the events unconditionally shouldn't hurt anything, I think. I added the property for two reasons: - I was sure someone would ask for it. :) Someone still might :) But it may be a pre-mature optimization. - The property is

Re: [Qemu-devel] [PATCH 2/2] char: report frontend open/closed state in 'query-chardev'

2014-05-29 Thread Eric Blake
On 05/29/2014 02:43 PM, Laszlo Ersek wrote: +# backend (eg. with the chardev=... option) is in open or +# closed state (since 2.2) Why 2.2? Are you saying it is too late to make the 2.1 soft freeze? I thought that reviewers would immediately question the

[Qemu-devel] [PATCH 0/1] Support Archipelago as a QEMU block backend

2014-05-29 Thread Chrysostomos Nanakos
Hello team, this is a patch implementing support for a new storage layer, Archipelago [1][2]. We've been using Archipelago in our IaaS public cloud production environment for over a year now, along with Google Ganeti [3] and Ceph [4]. We are currently moving from accessing Archipelago through a

[Qemu-devel] [PATCH 1/1] block: Support Archipelago as a QEMU block backend

2014-05-29 Thread Chrysostomos Nanakos
VM Image on Archipelago volume is specified like this: file=archipelago:volumename[/mport=mapperd_port[:vport=vlmcd_port]] 'archipelago' is the protocol. 'mport' is the port number on which mapperd is listening. This is optional and if not specified, QEMU will make Archipelago to use the

Re: [Qemu-devel] [PATCH 0/2] help libvirt know what's up with qga

2014-05-29 Thread Eric Blake
On 05/29/2014 02:54 PM, Laszlo Ersek wrote: Thinking about it a bit more, would it help to make the frontend_open a tri-state? It would be nice to know the difference between a guest that has no idea how to use the channel (has never opened it in the past; possibly because we're still too

Re: [Qemu-devel] [PATCH 0/1] Support Archipelago as a QEMU block backend

2014-05-29 Thread Eric Blake
On 05/29/2014 05:14 AM, Chrysostomos Nanakos wrote: Hello team, this is a patch implementing support for a new storage layer, Archipelago [1][2]. We've been using Archipelago in our IaaS public cloud production environment for over a year now, along with Google Ganeti [3] and Ceph [4].

Re: [Qemu-devel] [PATCH v2 03/11] block: Add overlay BDS pointer into the BlockDriverState struct

2014-05-29 Thread Jeff Cody
On Tue, May 27, 2014 at 09:57:19AM -0600, Eric Blake wrote: On 05/27/2014 08:28 AM, Jeff Cody wrote: Now that node-names can reference an individual BlockDriverState without needing to navigate downwards from the 'device' level, in order to find the top-most image (active layer) we need a

Re: [Qemu-devel] [PATCH v3 3/9] spapr_pci: Introduce a finish_realize() callback

2014-05-29 Thread Alexander Graf
On 29.05.14 03:53, Alexey Kardashevskiy wrote: On 05/29/2014 01:11 AM, Alexander Graf wrote: On 27.05.14 07:36, Alexey Kardashevskiy wrote: The spapr-pci PHB initializes IOMMU for emulated devices only. The upcoming VFIO support will do it different. However both emulated and VFIO PHB types

Re: [Qemu-devel] [PATCH v2 2/3] e1000: allow command-line selection of card model

2014-05-29 Thread Peter Crosthwaite
On Fri, May 30, 2014 at 2:00 AM, Gabriel L. Somlo gso...@gmail.com wrote: Hi Peter, Thanks for the QOM crash course ! So I did s/TYPE_E1000/TYPE_E1000_BASE/ as suggested by Stefan, and also moved PCI_VENDOR_ID_INTEL back into e1000_class_init() and away from e1000_devices[] as you

[Qemu-devel] Question about hcd-ohci and address_space_translate

2014-05-29 Thread BALATON Zoltan
Hello, I'm trying to port a USB driver from coreboot to OpenBIOS to fix keyboard in qemu-system-ppc64 -M mac99. The driver sets up a HCCA struct in RAM and writes its address into the mmio mapped area of the card but then QEMU complains that it cannot read this struct and I don't understand

[Qemu-devel] [Bug 1324724] [NEW] make install fails on arm host

2014-05-29 Thread Peter Chubb
Public bug reported: I do: ./configure --target-list=arm-softmmu make sudo make install and see: install -d -m 0755 /usr/local/bin libtool --quiet --mode=install install -c -m 0755 qemu-ga qemu-nbd qemu-img qemu-io fsdev/virtfs-proxy-helper /usr/local/bin strip /usr/local/bin/qemu-ga

Re: [Qemu-devel] [PATCH] trace: Replace fprintf with error_report and print location

2014-05-29 Thread Alexey Kardashevskiy
On 05/29/2014 10:42 PM, Lluís Vilanova wrote: Alexey Kardashevskiy writes: This replaces fprintf(stderr) with error_report. This prints line number of the trace which does not exist or is not traceable. A little nit pick; it shows an error when some of the events in the list of events to

[Qemu-devel] [Bug 1324727] [NEW] qemu-system-arm segfaults without KVM on ARM

2014-05-29 Thread Peter Chubb
Public bug reported: I'm running on Odroid-XU, Debian Jessie armhf qemu built from today's head d7d3d6092cb7edc75dc49fb90c86dd5425ab4805 sudo qemu-system-arm -M vexpress-a15 -drive if=none,file=arm.img,cache=writeback,id=foo -device virtio-blk-device,drive=foo -netdev user,id=user.0 -device

[Qemu-devel] [Bug 1267520] Re: Keyboard input not working when the -k en-us argument is specified.

2014-05-29 Thread Rich Fletcher
I think I stumbled on this bug. I was using packer to generate qemu- based virtual machines, and identical configurations would fail (complaining about invalid keymaps when keystrokes were sent) depending on where the current directory was. It doesn't work if qemu is run from a directory

Re: [Qemu-devel] [PATCH] block/iscsi: handle BUSY condition

2014-05-29 Thread Paolo Bonzini
Il 29/05/2014 22:25, Peter Lieven ha scritto: Am 29.05.2014 21:17, schrieb Paolo Bonzini: Il 29/05/2014 20:41, Peter Lieven ha scritto: static inline unsigned geo_rand_range(double m, double n) { return exp((log(m) + (double)rand() / (RAND_MAX / (log(n) - log(m) + log(1)) +

<    1   2