[Qemu-devel] Why X grabs for absolute windows?

2013-09-18 Thread Burton, Ross
Hi, The documentation for qemu's input devices says: ‘tablet’ Pointer device that uses absolute coordinates (like a touchscreen). This means qemu is able to report the mouse position without having to grab the mouse. Also overrides the PS/2 mouse emulation when activated. However, qemu does do X

[Qemu-devel] [PATCH v3] kvm: add set_one_reg/get_one_reg helpers

2013-09-18 Thread Alexey Kardashevskiy
This adds QEMU wrappers for KVM_SET_ONE_REG/KVM_GET_ONE_REG ioctls. Signed-off-by: Alexey Kardashevskiy --- Changes: v3: * changed comments v2: * added Doc Comments * removed error_print --- include/sysemu/kvm.h | 21 + kvm-all.c| 18 ++ 2 files c

Re: [Qemu-devel] [PATCH v2] kvm: add set_one_reg/get_one_reg helpers

2013-09-18 Thread Peter Maydell
On 19 September 2013 11:08, Alexey Kardashevskiy wrote: > This adds QEMU wrappers for KVM_SET_ONE_REG/KVM_GET_ONE_REG ioctls. > > Signed-off-by: Alexey Kardashevskiy > --- > Changes: > v2: > * added Doc Comments > * removed error_print Thanks. The doc comments should go in the header file, not t

[Qemu-devel] [PATCH v2] kvm: add set_one_reg/get_one_reg helpers

2013-09-18 Thread Alexey Kardashevskiy
This adds QEMU wrappers for KVM_SET_ONE_REG/KVM_GET_ONE_REG ioctls. Signed-off-by: Alexey Kardashevskiy --- Changes: v2: * added Doc Comments * removed error_print --- include/sysemu/kvm.h | 4 kvm-all.c| 33 + 2 files changed, 37 insertions(+)

[Qemu-devel] [RFC] Enabling x2apic on most (all?) x86 CPU models

2013-09-18 Thread Eduardo Habkost
Hi, I would like to get your opinion on this: Currently we have x2apic enabled only on SandyBridge and Haswell CPU models because we try to keep the CPU models closer to real CPUs. However, x2apic improves performance by reducing the overhead of APIC accesses, and KVM can emulate it independently

[Qemu-devel] [PATCH] qemu-xen: make use of xenstore relative paths

2013-09-18 Thread Roger Pau Monne
Qemu has several hardcoded xenstore paths that are only valid on Dom0. Attempts to launch a Qemu instance (to act as a userspace backend for PV disks) will fail because Qemu is not able to access those paths when running on a domain different than Dom0. Instead make the xenstore paths relative to

Re: [Qemu-devel] QEMU SD card emulation problem

2013-09-18 Thread Andreas Färber
Hi, Am 17.09.2013 14:13, schrieb Taimoor Mirza: > I have been trying to use SD card emulation of QEMU by providing host > directory to QEMU to treat it as SD card. For this I am using option > "-sd fat:16:rw:/home/taimoor/temp/sdcard". This works fine for me and > I can see list of files in guest

Re: [Qemu-devel] [PATCH 1/2] linux-headers: update to 3.12-rc1

2013-09-18 Thread Eduardo Habkost
On Wed, Sep 18, 2013 at 04:41:44PM +0200, Andrew Jones wrote: > Signed-off-by: Andrew Jones Reviewed-by: Eduardo Habkost > --- > linux-headers/asm-x86/kvm_para.h | 1 + > linux-headers/linux/kvm.h| 1 + > linux-headers/linux/kvm_para.h | 1 + > linux-headers/linux/vfio.h | 3

Re: [Qemu-devel] [PATCH 2/2] qemu-kvm: paravirt: add feature kvm_pv_unhalt

2013-09-18 Thread Andreas Färber
Am 18.09.2013 16:41, schrieb Andrew Jones: > I don't know yet if want this feature on by default, so for now I'm > just adding support for "-cpu ...,+kvm_pv_unhalt". > > Signed-off-by: Andrew Jones > --- > target-i386/cpu.c | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > > diff --git

Re: [Qemu-devel] [PATCHv2 2/3] seccomp: adding command line support for blacklist

2013-09-18 Thread Paul Moore
On Wednesday, September 18, 2013 05:32:17 PM Daniel P. Berrange wrote: > On Wed, Sep 18, 2013 at 12:19:44PM -0400, Paul Moore wrote: > > On Wednesday, September 18, 2013 04:59:10 PM Daniel P. Berrange wrote: > > > On Wed, Sep 18, 2013 at 11:53:09AM -0400, Paul Moore wrote: > > > > On Wednesday, Sep

Re: [Qemu-devel] [PATCH v5 0/5] Do not set SO_REUSEADDR on Windows

2013-09-18 Thread Stefan Weil
Am 16.09.2013 17:10, schrieb Sebastian Ottlik: > On 16.09.2013 16:55, Paolo Bonzini wrote: >> Il 16/09/2013 16:23, Sebastian Ottlik ha scritto: >>> - Added the silent flag to socket_set_fast_reuse controlling error >>> reporting >>>One location where SO_REUSEADDR was set would report errors if

Re: [Qemu-devel] [PATCHv2 2/3] seccomp: adding command line support for blacklist

2013-09-18 Thread Corey Bryant
On 09/18/2013 12:32 PM, Daniel P. Berrange wrote: On Wed, Sep 18, 2013 at 12:19:44PM -0400, Paul Moore wrote: On Wednesday, September 18, 2013 04:59:10 PM Daniel P. Berrange wrote: On Wed, Sep 18, 2013 at 11:53:09AM -0400, Paul Moore wrote: On Wednesday, September 18, 2013 08:38:17 AM Daniel

Re: [Qemu-devel] [PATCH 2/2] qemu-kvm: paravirt: add feature kvm_pv_unhalt

2013-09-18 Thread Eduardo Habkost
On Wed, Sep 18, 2013 at 04:41:45PM +0200, Andrew Jones wrote: > I don't know yet if want this feature on by default, so for now I'm > just adding support for "-cpu ...,+kvm_pv_unhalt". > > Signed-off-by: Andrew Jones Reviewed-by: Eduardo Habkost > --- > target-i386/cpu.c | 2 +- > 1 file chan

Re: [Qemu-devel] [PATCH v4 00/24] arm: ARM11MPCore+A9MPCore+A15MPCore QOM'ification

2013-09-18 Thread Andreas Färber
Am 13.09.2013 17:38, schrieb Peter Maydell: > On 11 September 2013 15:37, Andreas Färber wrote: >> From: Andreas Färber >> >> Hello Peter, >> >> This series fully QOM'ifies A9MPCore so that it can be embedded for Tegra2. >> It goes on to do the same for A15MPCore, which had previously been taken

Re: [Qemu-devel] [PATCHv2 2/3] seccomp: adding command line support for blacklist

2013-09-18 Thread Daniel P. Berrange
On Wed, Sep 18, 2013 at 12:19:44PM -0400, Paul Moore wrote: > On Wednesday, September 18, 2013 04:59:10 PM Daniel P. Berrange wrote: > > On Wed, Sep 18, 2013 at 11:53:09AM -0400, Paul Moore wrote: > > > On Wednesday, September 18, 2013 08:38:17 AM Daniel P. Berrange wrote: > > > > Libvirt does not

Re: [Qemu-devel] [PATCH v2 1/1] integrator: fix Linux boot failure by emulating dbg

2013-09-18 Thread Andreas Färber
Am 18.09.2013 16:31, schrieb alex.ben...@linaro.org: > From: Alex Bennée > > Commit 9b8c69243 broke the ability to boot the kernel as the value > returned by unassigned_mem_read returned non-zero and left the kernel > looping forever waiting for it to change (see integrator_led_set in > the kerne

Re: [Qemu-devel] [PATCHv2 2/3] seccomp: adding command line support for blacklist

2013-09-18 Thread Daniel P. Berrange
On Wed, Sep 18, 2013 at 11:53:09AM -0400, Paul Moore wrote: > On Wednesday, September 18, 2013 08:38:17 AM Daniel P. Berrange wrote: > > Libvirt does not want to be in the business of creating seccomp syscall > > filters for QEMU. As mentioned before, IMHO that places an unacceptable > > burden on

Re: [Qemu-devel] [PATCHv2 2/3] seccomp: adding command line support for blacklist

2013-09-18 Thread Paul Moore
On Wednesday, September 18, 2013 08:38:17 AM Daniel P. Berrange wrote: > Libvirt does not want to be in the business of creating seccomp syscall > filters for QEMU. As mentioned before, IMHO that places an unacceptable > burden on libvirt to know about the syscalls each a particular version > of QE

Re: [Qemu-devel] [PATCH v2 1/1] integrator: fix Linux boot failure by emulating dbg

2013-09-18 Thread Alex Bennée
afaer...@suse.de writes: > Am 18.09.2013 16:31, schrieb alex.ben...@linaro.org: > > Looks okay in general, some minor nits below: Thanks, I shall hopefully get those sorted out in the next week. -- Alex Bennée

Re: [Qemu-devel] [PATCHv2 2/3] seccomp: adding command line support for blacklist

2013-09-18 Thread Paul Moore
On Wednesday, September 18, 2013 04:59:10 PM Daniel P. Berrange wrote: > On Wed, Sep 18, 2013 at 11:53:09AM -0400, Paul Moore wrote: > > On Wednesday, September 18, 2013 08:38:17 AM Daniel P. Berrange wrote: > > > Libvirt does not want to be in the business of creating seccomp syscall > > > filters

Re: [Qemu-devel] [Question] memory: AddressSpace backed by single IO MemoryRegion

2013-09-18 Thread Paolo Bonzini
Il 17/09/2013 10:29, Peter Maydell ha scritto: > On 16 September 2013 18:18, Paolo Bonzini wrote: >> Il 16/09/2013 19:11, Marcel Apfelbaum ha scritto: > memory_region_init_io(my_reg, owner, my_ops, my_obj, "my region", > INT64_MAX); >> >> This is 2^63-1, not 2^64-1. You need UINT64_M

Re: [Qemu-devel] [Xen-devel] [PATCH] libxl: Remove qxl support for the 4.3 release

2013-09-18 Thread Jan Beulich
>>> On 18.09.13 at 17:26, Fabio Fantoni wrote: > Il 18/09/2013 16:30, Jan Beulich ha scritto: > On 18.09.13 at 16:12, Fabio Fantoni wrote: >>> Il 18/09/2013 14:42, Jan Beulich ha scritto: >>> On 18.09.13 at 14:29, Fabio Fantoni wrote: > I not see the MMIO error on logs, what must I d

Re: [Qemu-devel] [Xen-devel] [PATCH] libxl: Remove qxl support for the 4.3 release

2013-09-18 Thread Fabio Fantoni
Il 18/09/2013 16:30, Jan Beulich ha scritto: On 18.09.13 at 16:12, Fabio Fantoni wrote: Il 18/09/2013 14:42, Jan Beulich ha scritto: On 18.09.13 at 14:29, Fabio Fantoni wrote: I not see the MMIO error on logs, what must I do to see it or any similar debug information? Tell me if I must do ot

Re: [Qemu-devel] Merging the quorum block driver

2013-09-18 Thread Benoît Canet
Le Tuesday 17 Sep 2013 à 14:44:13 (+0200), Stefan Hajnoczi a écrit : > Hi Benoit, > Kevin and I had a chance to chat face-to-face and we discussed what > concrete changes are necessary to merge quorum (without solving all the > other block layers problems at once). > > I think quorum could be merg

Re: [Qemu-devel] [PATCH v4 08/33] tcg-aarch64: Introduce tcg_fmt_Rdnm and tcg_fmt_Rdnm_lsl

2013-09-18 Thread Claudio Fontana
On 18.09.2013 16:54, Richard Henderson wrote: > On 09/18/2013 01:24 AM, Claudio Fontana wrote: >> If you _really_ want to strip the right shift functionality away for some >> reason, could you state it? > > (1) It will never never ever be used. > > (2) I do not find + for LSR, - for LSL intuitiv

Re: [Qemu-devel] [PATCH 2/2] rdma: simplify qemu_rdma_register_and_get_keys()

2013-09-18 Thread Michael R. Hines
On 09/03/2013 10:32 PM, Isaku Yamahata wrote: Signed-off-by: Isaku Yamahata --- migration-rdma.c | 23 +++ 1 file changed, 7 insertions(+), 16 deletions(-) diff --git a/migration-rdma.c b/migration-rdma.c index db5a908..941c07e 100644 --- a/migration-rdma.c +++ b/migrat

Re: [Qemu-devel] [PATCH v10 5/8] module: implement module loading

2013-09-18 Thread Paolo Bonzini
Il 18/09/2013 16:44, Richard Henderson ha scritto: > On 09/18/2013 04:45 AM, Paolo Bonzini wrote: >> But on Mac OS X a bundle (module loaded with dlopen) cannot have a >> dependency on another bundle, afaik. > > The documentation for NSAddImage suggests that they can: > >> NSADDIMAGE_OPTION_WITH_

Re: [Qemu-devel] [PATCH 1/3] block: Additional info string in ImageInfo and BDI

2013-09-18 Thread Stefan Hajnoczi
On Thu, Sep 05, 2013 at 02:05:13PM +0200, Max Reitz wrote: > diff --git a/block.c b/block.c > index 26639e8..9fd9f3a 100644 > --- a/block.c > +++ b/block.c > @@ -1921,7 +1921,7 @@ void bdrv_round_to_clusters(BlockDriverState *bs, > int64_t *cluster_sector_num, >

Re: [Qemu-devel] [PATCH v4 08/33] tcg-aarch64: Introduce tcg_fmt_Rdnm and tcg_fmt_Rdnm_lsl

2013-09-18 Thread Richard Henderson
On 09/18/2013 01:24 AM, Claudio Fontana wrote: > If you _really_ want to strip the right shift functionality away for some > reason, could you state it? (1) It will never never ever be used. (2) I do not find + for LSR, - for LSL intuitive at all. If we're to have them at all, better to pass

Re: [Qemu-devel] [PATCH v10 5/8] module: implement module loading

2013-09-18 Thread Richard Henderson
On 09/18/2013 04:45 AM, Paolo Bonzini wrote: > But on Mac OS X a bundle (module loaded with dlopen) cannot have a > dependency on another bundle, afaik. The documentation for NSAddImage suggests that they can: > NSADDIMAGE_OPTION_WITH_SEARCHING > With this option, the image_name passed for the li

[Qemu-devel] [PATCH 0/2] kvm: add kvm_pv_unhalt

2013-09-18 Thread Andrew Jones
pvticketlocks are in v3.12-rc1, teach qemu-kvm about it. Benchmarking is still in progress to determine if we can turn it on by default. So leaving it off for now. Andrew Jones (2): linux-headers: update to 3.12-rc1 qemu-kvm: paravirt: add feature kvm_pv_unhalt linux-headers/asm-x86/kvm_para

[Qemu-devel] R: qemu-nbd segmentation fault

2013-09-18 Thread Mario DE CHENNO
This time crashed just issuing a mount command: Starting program: /vmstore/vmtools/qemu-nbd-from1.6.0 /vmstore/playground/archivioweb.img [Thread debugging using libthread_db enabled] [New Thread 0x755e3700 (LWP 7626)] [New Thread 0x747dd700 (LWP 7645)] [New Thread 0x73fdc700 (LWP 764

[Qemu-devel] [PATCH 1/2] linux-headers: update to 3.12-rc1

2013-09-18 Thread Andrew Jones
Signed-off-by: Andrew Jones --- linux-headers/asm-x86/kvm_para.h | 1 + linux-headers/linux/kvm.h| 1 + linux-headers/linux/kvm_para.h | 1 + linux-headers/linux/vfio.h | 38 ++ 4 files changed, 41 insertions(+) diff --git a/linux-headers/as

[Qemu-devel] [PATCH 2/2] qemu-kvm: paravirt: add feature kvm_pv_unhalt

2013-09-18 Thread Andrew Jones
I don't know yet if want this feature on by default, so for now I'm just adding support for "-cpu ...,+kvm_pv_unhalt". Signed-off-by: Andrew Jones --- target-i386/cpu.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/target-i386/cpu.c b/target-i386/cpu.c index 42c5de034ecc8..

Re: [Qemu-devel] [PATCH] .travis.yml: basic compile and check recipes

2013-09-18 Thread Alex Bennée
stefa...@gmail.com writes: > > Looks useful. To get more code coverage, flesh out the build > environment: Thanks. Am I dreaming or are there some system image tests somewhere as well? -- Alex Bennée

Re: [Qemu-devel] [ANNOUNCE] OSv, an new operating system for the cloud, v0.01

2013-09-18 Thread humble devassy
Indeed.. OSv++ ... Gave a try already and shared it via http://humblec.com/osv-best-os-cloud-workloads-release-announcement/ On Wed, Sep 18, 2013 at 10:33 AM, Xiao Guangrong < xiaoguangr...@linux.vnet.ibm.com> wrote: > Sounds fantastic, will play it. :) > > On 09/17/2013 03:22 AM, Pekka Enber

Re: [Qemu-devel] [ANNOUNCE] OSv, an new operating system for the cloud, v0.01

2013-09-18 Thread Dor Laor
On Wed, Sep 18, 2013 at 3:43 PM, Zhi Yong Wu wrote: > any doc that describes its archtecture or interval? > > You can find some of the design principles here [1] and here [2]. We're starting to make the wiki thicker too [3] The rest is documented in c++ for the moment ;) [1] http://www.osv.io/d

[Qemu-devel] [PATCH v2 0/0] integrator: fix Linux boot failure by emulating dbg

2013-09-18 Thread alex . bennee
Hi, I mistakenly appended this to the previous patch revision. I'm now sending for further review. Since v1: - Updated with Peter Maydell's review comments. -- Alex Bennée

[Qemu-devel] [PATCH v2 1/1] integrator: fix Linux boot failure by emulating dbg

2013-09-18 Thread alex . bennee
From: Alex Bennée Commit 9b8c69243 broke the ability to boot the kernel as the value returned by unassigned_mem_read returned non-zero and left the kernel looping forever waiting for it to change (see integrator_led_set in the kernel code). Relying on a varying implementation detail is incorrect

[Qemu-devel] [PATCH v2 1/1] integrator: fix Linux boot failure by emulating dbg

2013-09-18 Thread alex . bennee
From: Alex Bennée Commit 9b8c69243 broke the ability to boot the kernel as the value returned by unassigned_mem_read returned non-zero and left the kernel looping forever waiting for it to change (see integrator_led_set in the kernel code). Relying on a varying implementation detail is incorrect

Re: [Qemu-devel] [PATCH 1/2] rdma: constify ram_chunk_{index, start, end}

2013-09-18 Thread Michael R. Hines
On 09/03/2013 10:32 PM, Isaku Yamahata wrote: Signed-off-by: Isaku Yamahata --- migration-rdma.c |8 +--- 1 file changed, 5 insertions(+), 3 deletions(-) diff --git a/migration-rdma.c b/migration-rdma.c index e71c10a..db5a908 100644 --- a/migration-rdma.c +++ b/migration-rdma.c @@ -5

Re: [Qemu-devel] [PATCH v4 04/33] tcg-aarch64: Hoist common argument loads in tcg_out_op

2013-09-18 Thread Claudio Fontana
On 17.09.2013 16:27, Richard Henderson wrote: > On 09/17/2013 01:01 AM, Claudio Fontana wrote: >> I don't have anything against it in principle, but just adding c0 and c1, >> which iirc should cover all uses, would be fine by me. > > Not really. > > There are 6 potential args[] values, 5 of which

Re: [Qemu-devel] Problems with QEMU sdcard while using glib 2.33.8

2013-09-18 Thread Stefan Hajnoczi
On Thu, Sep 05, 2013 at 04:02:43PM +0500, Taimoor Mirza wrote: > I am using released 1.5.0 version from http://wiki.qemu.org/Download. > I think it should be same as commit ID > "295d81c62414a63c625fa2e78175573d4b3f5ba4" > > I have observed some interesting behavior. This problem does not come > i

Re: [Qemu-devel] [RFC PATCH 0/3] move global bdrv_states out of block.c

2013-09-18 Thread Stefan Hajnoczi
On Tue, Aug 27, 2013 at 03:49:22PM +0800, Wenchao Xia wrote: > In order to support multiple caller from different thread, global > inside block layer should be carefully treated. bdrv_states represent > a group of bds* which is now used by qemu, so it is a user concept which > should be managed by

Re: [Qemu-devel] [PATCH v4 04/33] tcg-aarch64: Hoist common argument loads in tcg_out_op

2013-09-18 Thread Richard Henderson
On 09/18/2013 01:10 AM, Claudio Fontana wrote: > On 17.09.2013 16:27, Richard Henderson wrote: >> On 09/17/2013 01:01 AM, Claudio Fontana wrote: >>> I don't have anything against it in principle, but just adding c0 and c1, >>> which iirc should cover all uses, would be fine by me. >> >> Not really.

Re: [Qemu-devel] [PATCH v3 0/4] timers thread-safe stuff

2013-09-18 Thread Stefan Hajnoczi
On Tue, Aug 27, 2013 at 11:20:59AM +0800, Liu Ping Fan wrote: > Saw the Alex's patches has been merged, rebase mine onto his. > > v3: > 1. rename seqlock_read_check as seqlock_read_retry > 2. Document timerlist were protected by BQL, and discard private lock > around "qemu_event_wait(tl->ev)"

Re: [Qemu-devel] [ANNOUNCE] OSv, an new operating system for the cloud, v0.01

2013-09-18 Thread Zhi Yong Wu
Thanks for your pointers, Dor. On Wed, Sep 18, 2013 at 8:59 PM, Dor Laor wrote: > On Wed, Sep 18, 2013 at 3:43 PM, Zhi Yong Wu wrote: >> >> any doc that describes its archtecture or interval? >> > > You can find some of the design principles here [1] and here [2]. > We're starting to make the wi

Re: [Qemu-devel] [PATCH v4 0/3] qemu-timer: make qemu_timer_mod_ns() and qemu_timer_del() thread-safe

2013-09-18 Thread Stefan Hajnoczi
On Thu, Sep 12, 2013 at 11:02:17AM +0200, Stefan Hajnoczi wrote: > v4: > * Rebased & retested for easy review and merge > * No code changes > > v3: > * Squashed Paolo's fixes and added his patch to avoid locking in > timer_pending() > > v2: > * Rebased onto qemu.git/master following the merg

Re: [Qemu-devel] [PATCH 0/2] osdep: warn if open(O_DIRECT) on fails with EINVAL

2013-09-18 Thread Stefan Hajnoczi
On Thu, Aug 22, 2013 at 11:29:01AM +0200, Stefan Hajnoczi wrote: > After trying out a few approaches, here is what I think is the simplest viable > way to print a user-friendly warning if opening a file O_DIRECT fails with > EINVAL. This happens on Linux tmpfs. > > We don't really know why we got

[Qemu-devel] [PULL 5/6] QMP: Update qmp-spec.txt

2013-09-18 Thread Luiz Capitulino
Simplify the text, fix some of the examples. Signed-off-by: Luiz Capitulino Reviewed-by: Eric Blake --- docs/qmp/qmp-spec.txt | 65 ++- 1 file changed, 28 insertions(+), 37 deletions(-) diff --git a/docs/qmp/qmp-spec.txt b/docs/qmp/qmp-spec.txt i

Re: [Qemu-devel] [PATCH v2 00/38] Delay destruction of memory regions to instance_finalize

2013-09-18 Thread Paolo Bonzini
Il 18/09/2013 15:19, Peter Maydell ha scritto: > > There's nothing magic, really. Both PV and real devices have been doing > > it forever by placing some registers in RAM instead of MMIO, and > > communicating synchronization points via interrupts and doorbell registers. > > Sure, but that's a har

[Qemu-devel] [PULL 4/6] QMP: Update README file

2013-09-18 Thread Luiz Capitulino
Drop unneeded info, fix some of the examples and rename QEMU Monitor Protocol to QEMU Machine Protocol. Signed-off-by: Luiz Capitulino Reviewed-by: Eric Blake --- docs/qmp/README | 99 - 1 file changed, 49 insertions(+), 50 deletions(-) d

Re: [Qemu-devel] [PATCH 2/2] osdep: warn if open(O_DIRECT) on fails with EINVAL

2013-09-18 Thread Stefan Hajnoczi
On Fri, Sep 06, 2013 at 04:32:43PM -0400, Jeff Cody wrote: > On Thu, Aug 22, 2013 at 11:29:03AM +0200, Stefan Hajnoczi wrote: > > Print a warning when opening a file O_DIRECT fails with EINVAL. This > > saves users a lot of time trying to figure out the EINVAL error, which > > is typical when atte

Re: [Qemu-devel] [PATCH v2 00/38] Delay destruction of memory regions to instance_finalize

2013-09-18 Thread Peter Maydell
On 18 September 2013 22:11, Paolo Bonzini wrote: > Il 18/09/2013 13:56, Peter Maydell ha scritto: >>> > But does guest code actually care? In many cases, I suspect that >>> > sticking a smp_rmb() in the read side of "unlocked" register accesses, >>> > and a smp_wmb() in the write side, will do ju

Re: [Qemu-devel] Hibernate and qemu-nbd

2013-09-18 Thread Stefan Hajnoczi
On Tue, Sep 17, 2013 at 07:10:44AM -0700, Mark Trumpold wrote: > I am using the kernel functionality directly with the commands: > echo platform >/sys/power/disk > echo disk >/sys/power/state > > The following appears in dmesg when I attempt to hibernate: > > =

Re: [Qemu-devel] [PATCH v2 00/38] Delay destruction of memory regions to instance_finalize

2013-09-18 Thread Paolo Bonzini
Il 18/09/2013 13:56, Peter Maydell ha scritto: >> > But does guest code actually care? In many cases, I suspect that >> > sticking a smp_rmb() in the read side of "unlocked" register accesses, >> > and a smp_wmb() in the write side, will do just fine. And add a >> > compatibility property to plac

[Qemu-devel] [PULL 3/6] QMP: QMP/ -> docs/qmp/

2013-09-18 Thread Luiz Capitulino
Signed-off-by: Luiz Capitulino Reviewed-by: Eric Blake --- {QMP => docs/qmp}/README | 0 {QMP => docs/qmp}/qmp-events.txt | 0 {QMP => docs/qmp}/qmp-spec.txt | 0 3 files changed, 0 insertions(+), 0 deletions(-) rename {QMP => docs/qmp}/README (100%) rename {QMP => docs/qmp}/qmp-even

[Qemu-devel] [PULL 2/6] QMP: fix qmp-commands.txt generation path

2013-09-18 Thread Luiz Capitulino
This file should be generated in the BUILD_DIR, as all other docs. Signed-off-by: Luiz Capitulino Reviewed-by: Eric Blake --- Makefile | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) diff --git a/Makefile b/Makefile index 362fe3e..aca548d 100644 --- a/Makefile +++ b/Makefile @@ -65

Re: [Qemu-devel] [RFC] Policy for supported hosts/platforms

2013-09-18 Thread Christian Berendt
Sorry. Looks like I missed a mail. I'll create the account tomorrow. Who should receive the credentials. You, Ed? On 09/18/2013 02:57 PM, Ed Maste wrote: > I was originally going to host the FreeBSD 9 buildbot myself (in a VM) > but I now have an offer for permanent amd64/x86-64 FreeBSD 9 and > Fr

[Qemu-devel] [PULL 6/6] QMP: qmp-events.txt: alphabetical order fix and other minor changes

2013-09-18 Thread Luiz Capitulino
Signed-off-by: Luiz Capitulino Reviewed-by: Eric Blake --- docs/qmp/qmp-events.txt | 34 +- 1 file changed, 17 insertions(+), 17 deletions(-) diff --git a/docs/qmp/qmp-events.txt b/docs/qmp/qmp-events.txt index 4b24ec9..6b87e97 100644 --- a/docs/qmp/qmp-events.tx

[Qemu-devel] [PULL 1/6] QMP: add scripts/qmp

2013-09-18 Thread Luiz Capitulino
Populate it with all scripts stored in QMP/. Also fixes trailing whitespaces in qmp-shell and qmp.py. Signed-off-by: Luiz Capitulino Reviewed-by: Eric Blake --- {QMP => scripts/qmp}/qemu-ga-client | 0 {QMP => scripts/qmp}/qmp| 0 {QMP => scripts/qmp}/qmp-shell | 2 +- {QMP =>

[Qemu-devel] [PULL 0/6] QMP queue

2013-09-18 Thread Luiz Capitulino
The following changes since commit 6c2679fc19560699679200fb42ab4659bcbe7f79: Merge remote-tracking branch 'kiszka/queues/slirp' into staging (2013-09-17 10:01:24 -0500) are available in the git repository at: git://repo.or.cz/qemu/qmp-unstable.git queue/qmp for you to fetch changes up to

Re: [Qemu-devel] [PATCH] rdma: clean up of qemu_rdma_cleanup()

2013-09-18 Thread Juan Quintela
Isaku Yamahata wrote: > - It can't be determined by RDMAContext::cm_id != NULL if the connection > is established or not. > - RDMAContext::cm_id is leaked and not destroyed because it is set to NULL > too early. > - RDMAContext::qp is created by rdma_create_qp() so that it should be > destroy

Re: [Qemu-devel] [PATCH 2/2] rdma: simplify qemu_rdma_register_and_get_keys()

2013-09-18 Thread Juan Quintela
Isaku Yamahata wrote: > Signed-off-by: Isaku Yamahata I don't understand this bit: > +uint8_t *chunk_start = ram_chunk_start(block, chunk); > +uint8_t *chunk_end = ram_chunk_end(block, chunk); Here we are getting just the length of a chunk > sge.length = length; > >

Re: [Qemu-devel] [PATCH] Extend qemu-ga's 'guest-info' command to expose flag 'success-response'

2013-09-18 Thread Eric Blake
On 09/18/2013 01:23 AM, Mark Wu wrote: > Now we have several qemu-ga commands not returning response on success. > It has been documented in qga/qapi-schema.json already. This patch exposes > the 'success-response' flag by extending 'guest-info' command. With this > change, the clients can handle t

Re: [Qemu-devel] [RFC] Policy for supported hosts/platforms

2013-09-18 Thread Ed Maste
I was originally going to host the FreeBSD 9 buildbot myself (in a VM) but I now have an offer for permanent amd64/x86-64 FreeBSD 9 and FreeBSD 10 buildbots. It seems Christian is not available; is there anyone else who can set up a buildslave account (ideally both FreeBSD 9 amd64 and FreeBSD 10 a

Re: [Qemu-devel] [PATCH] Extend qemu-ga's 'guest-info' command to expose flag 'success-response'

2013-09-18 Thread Luiz Capitulino
On Wed, 18 Sep 2013 15:23:58 +0800 Mark Wu wrote: > Now we have several qemu-ga commands not returning response on success. > It has been documented in qga/qapi-schema.json already. This patch exposes > the 'success-response' flag by extending 'guest-info' command. With this > change, the clients

Re: [Qemu-devel] [PATCH 1/2] rdma: constify ram_chunk_{index, start, end}

2013-09-18 Thread Juan Quintela
Isaku Yamahata wrote: > Signed-off-by: Isaku Yamahata Reviewed-by: Juan Quintela

Re: [Qemu-devel] qemu-nbd segmentation fault

2013-09-18 Thread Stefan Hajnoczi
On Tue, Sep 17, 2013 at 02:53:51PM +0200, ing. Mario De Chenno wrote: > I cannot use qemu-nbd to write files to a qcow2 disk image. It always exit > after a while with a segmentation fault. Hi Mario, Thanks for providing the strace. Is it possible for you to post a backtrace? The backtrace shows

Re: [Qemu-devel] [ANNOUNCE] OSv, an new operating system for the cloud, v0.01

2013-09-18 Thread Zhi Yong Wu
any doc that describes its archtecture or interval? On Tue, Sep 17, 2013 at 3:22 AM, Pekka Enberg wrote: > Hello, > > We're proud to announce release 0.01 of OSv, a new operating system > for running applications on virtual machines. OSv is free software, > released under the BSD license, and you

Re: [Qemu-devel] [PATCH v5 0/5] Do not set SO_REUSEADDR on Windows

2013-09-18 Thread Stefan Hajnoczi
On Mon, Sep 16, 2013 at 04:23:44PM +0200, Sebastian Ottlik wrote: > This patchset disables most uses of SO_REUSEADDR on Windows and replaces it > with > calls to the new function socket_set_fast_reuse. On Windows systems the > default > behaviour is equivalent to SO_REUSEADDR on other operating s

Re: [Qemu-devel] [PATCH] .travis.yml: basic compile and check recipes

2013-09-18 Thread Stefan Hajnoczi
On Fri, Sep 13, 2013 at 04:20:22PM +0100, alex.ben...@linaro.org wrote: > From: Alex Bennée > > While QEMU already has various continuous integration set-ups in > buildbot and commercial Jenkins setups there is some value in supporting > travis-ci as well. It is well integrated into GitHub work f

Re: [Qemu-devel] Q35 FreeBSD install status

2013-09-18 Thread Bret Ketchum
For those following along. The mptable I presented was an old table, updating to SeaBIOS 1.7.3 fixed that but still no interrupts. However, it appears FreeBSD has trouble parsing the ACPI tables presented via q35-acpi-dsdt.aml as it does not create the sysresource devices associated with A

Re: [Qemu-devel] [PATCH] migration: ram_handle_compressed

2013-09-18 Thread Michael R. Hines
On 09/18/2013 06:07 AM, Isaku Yamahata wrote: ram_handle_compressed() should be aware size > TARGET_PAGE_SIZE migration-rdma can call it with larger size. Signed-off-by: Isaku Yamahata --- arch_init.c | 21 ++--- 1 file changed, 14 insertions(+), 7 deletions(-) diff --git

Re: [Qemu-devel] [PATCH] migration: ram_handle_compressed

2013-09-18 Thread Juan Quintela
Isaku Yamahata wrote: > ram_handle_compressed() should be aware size > TARGET_PAGE_SIZE > migration-rdma can call it with larger size. > > Signed-off-by: Isaku Yamahata > --- > arch_init.c | 21 ++--- > 1 file changed, 14 insertions(+), 7 deletions(-) > > diff --git a/arch_init

Re: [Qemu-devel] [PATCH v2 00/38] Delay destruction of memory regions to instance_finalize

2013-09-18 Thread Peter Maydell
On 18 September 2013 20:26, Paolo Bonzini wrote: > But does guest code actually care? In many cases, I suspect that > sticking a smp_rmb() in the read side of "unlocked" register accesses, > and a smp_wmb() in the write side, will do just fine. And add a > compatibility property to place a devic

Re: [Qemu-devel] [PATCH v3 1/2] block: allow live commit of active image

2013-09-18 Thread Paolo Bonzini
Il 04/09/2013 14:35, Stefan Hajnoczi ha scritto: > Changing from 512 KB to 64 KB can affect performance. 8 times as many > iops may be issued to copy data. > > Also, the image's cluster size should really be taken into account. > Otherwise additional inefficiency will be suffered when we populate

Re: [Qemu-devel] [PATCH v10 5/8] module: implement module loading

2013-09-18 Thread Paolo Bonzini
Il 17/09/2013 07:40, Richard Henderson ha scritto: > On 09/16/2013 06:29 PM, Fam Zheng wrote: >>> Link spice-qemu.char.so against spice-core.so. The DT_NEEDED entry will be >>> recorded, and ld.so will do the right thing. >>> >>> Anything else sounds way too much like Not Invented Here. >>> >> How

Re: [Qemu-devel] [PATCH v2 0/7] smbios cleanup & nicer defaults for type 1

2013-09-18 Thread Markus Armbruster
[Note cc: Andreas] "Michael S. Tsirkin" writes: > On Fri, Aug 16, 2013 at 03:18:27PM +0200, arm...@redhat.com wrote: >> From: Markus Armbruster >> >> This gets rid of one of the last get_param_value() users, makes >> multiple -smbios work sanely, cleans up the gross side effect in >> qemu_uuid

Re: [Qemu-devel] [PATCH v2 00/38] Delay destruction of memory regions to instance_finalize

2013-09-18 Thread Paolo Bonzini
Il 18/09/2013 10:41, Michael S. Tsirkin ha scritto: > On Wed, Sep 18, 2013 at 09:40:19AM +0200, Paolo Bonzini wrote: >> Il 18/09/2013 07:48, Michael S. Tsirkin ha scritto: >>> So I think the fix is actually obeying ordering rules, >>> that is know that write is in progress >>> and flush on read. >>

Re: [Qemu-devel] [PATCH] kvm: add set_one_reg/get_one_reg

2013-09-18 Thread Paolo Bonzini
Il 18/09/2013 10:08, Alexey Kardashevskiy ha scritto: > On 09/18/2013 05:59 PM, Peter Maydell wrote: >> On 18 September 2013 16:46, Alexey Kardashevskiy wrote: >>> On 09/18/2013 05:14 PM, Peter Maydell wrote: On 18 September 2013 05:21, Alexey Kardashevskiy wrote: > This adds QEMU wrappe

[Qemu-devel] [PATCH 2/2] vmdk: fix cluster size check for flat extents

2013-09-18 Thread Fam Zheng
We use the extent size as cluster size for flat extents (where no L1/L2 table is allocated so it's safe). So don't check the cluster size for flat case in opening. Otherwise flat extent opening is broken: # qemu-img create -f vmdk -o subformat=monolithicFlat /tmp/a.vmdk 100G Formatting '

[Qemu-devel] [PATCH 1/2] block: don't lose data from last incomplete sector

2013-09-18 Thread Fam Zheng
To read the last sector that is not aligned to sector boundary, current code for growable backends, since commit 893a8f6 "block: Produce zeros when protocols reading beyond end of file", drops the data and directly returns zeroes. That is incorrect. Signed-off-by: Fam Zheng --- block.c | 2 +- 1

[Qemu-devel] [PATCH 0/2] block: two fixes for image opening

2013-09-18 Thread Fam Zheng
This fixes two bugs of image opening. The vmdk fix also applies to 1.6. Fam Zheng (2): block: don't lose data from last incomplete sector vmdk: fix cluster size check for flat extents block.c | 2 +- block/vmdk.c | 2 +- 2 files changed, 2 insertions(+), 2 deletions(-) -- 1.8.3.1

[Qemu-devel] [PATCH 03/11] s390/sclpquiesce: Add code to support live migration

2013-09-18 Thread Christian Borntraeger
From: Heinz Graalfs This patch adds the necessary life migration pieces to sclpquiesce by using the vmstate_register. Signed-off-by: Heinz Graalfs Signed-off-by: Christian Borntraeger --- hw/s390x/sclpquiesce.c | 13 - 1 file changed, 12 insertions(+), 1 deletion(-) diff --git a/

[Qemu-devel] [PATCH 08/11] s390/eventfacility: remove unused event_type variable

2013-09-18 Thread Christian Borntraeger
The event_type variable is never used. Get rid of it. Signed-off-by: Christian Borntraeger --- hw/char/sclpconsole.c | 1 - hw/s390x/sclpquiesce.c| 2 -- include/hw/s390x/event-facility.h | 1 - 3 files changed, 4 deletions(-) diff --git a/hw/char/sclpconsole.c b/hw/char

[Qemu-devel] [PATCH 02/11] s390/sclpconsole: Add code to support live migration for sclpconsole

2013-09-18 Thread Christian Borntraeger
From: Heinz Graalfs This patch adds the necessary life migration pieces to the sclp code by using vmstate_register. Signed-off-by: Heinz Graalfs Signed-off-by: Christian Borntraeger --- hw/char/sclpconsole.c | 19 ++- 1 file changed, 18 insertions(+), 1 deletion(-) diff --git

[Qemu-devel] [PATCH 04/11] s390/event-facility: Add code to support live migration

2013-09-18 Thread Christian Borntraeger
From: Heinz Graalfs This patch adds the necessary life migration pieces to the event facility by using the vmstate_register. Signed-off-by: Heinz Graalfs Signed-off-by: Christian Borntraeger --- hw/s390x/event-facility.c | 13 - 1 file changed, 12 insertions(+), 1 deletion(-) dif

[Qemu-devel] [PATCH 09/11] s390/eventfacility: allow childs to handle more than 1 event type

2013-09-18 Thread Christian Borntraeger
Currently all handlers (quiesce, console) only handle one event type. Some drivers will handle multiple (compatible) event types. Rework the code accordingly. Signed-off-by: Christian Borntraeger --- hw/char/sclpconsole.c | 6 +++--- hw/s390x/event-facility.c | 2 +- hw/s390x

[Qemu-devel] [PATCH 11/11] s390/sclplmconsole: Add support for SCLP line-mode console

2013-09-18 Thread Christian Borntraeger
From: Heinz Graalfs Add simple support for SCLP line-mode also known as operating system messages. This can be added in addition to or instead of the SCLP full screen console with -device sclplmconsole. Signed-off-by: Heinz Graalfs Signed-off-by: Christian Borntraeger --- hw/char/Makefile.obj

[Qemu-devel] [PATCH 00/11] sclp related fixes and sclp line mode console

2013-09-18 Thread Christian Borntraeger
Alex, here is a bunch of fixes/changes for sclp/eventfacility followed by a move of the ebcdic conversion into a stand-alone header file and the sclp line mode console. Christian Borntraeger (3): s390/eventfacility: Fix receive/send masks s390/eventfacility: remove unused event_type variable

[Qemu-devel] [PATCH 01/11] s390/sclpconsole: modify definition of input buffer

2013-09-18 Thread Christian Borntraeger
From: Heinz Graalfs To use VMState for migration, we need to adapt some sclp code: - allocate console buffer as part of the console - change semantic of sclpconsole offset fields Signed-off-by: Heinz Graalfs Signed-off-by: Christian Borntraeger --- hw/char/sclpconsole.c | 54 +++

[Qemu-devel] [PATCH 06/11] s390/eventfacility: fix multiple Read Event Data sources

2013-09-18 Thread Christian Borntraeger
From: Ralf Hoppe Make the handler for SCLP Read Event Data deal with notifications for multiple sources correctly. Signed-off-by: Ralf Hoppe Reviewed-by: Thomas Huth Signed-off-by: Christian Borntraeger [split bigger patch into smaller independent chunks] --- hw/s390x/event-facility.c | 6 ++

[Qemu-devel] [PATCH 10/11] s390/ebcdic: Move conversion tables to header file

2013-09-18 Thread Christian Borntraeger
From: Heinz Graalfs Move conversion tables to header file. - In SCLP line mode processing EBCDIC/ASCII conversion is needed. - An additional EBCDIC to ASCII conversion function is added. Signed-off-by: Heinz Graalfs Signed-off-by: Christian Borntraeger --- include/hw/s390x/ebcdic.h | 1

[Qemu-devel] [PATCH 05/11] s390/sclp: add reset() functions

2013-09-18 Thread Christian Borntraeger
From: Heinz Graalfs Add reset() functions for event-facility, sclpconsole, and sclpquiesce. The reset() functions perform variable initialization at IPL and e.g. when monitor system_reset is called. Signed-off-by: Heinz Graalfs Reviewed-by: Thomas Huth Signed-off-by: Christian Borntraeger ---

[Qemu-devel] [PATCH 07/11] s390/eventfacility: Fix receive/send masks

2013-09-18 Thread Christian Borntraeger
Currently we announce interchanged receive/send masks. This did not trigger a bug, since the sclp console has the same masks for send/receive and the Linux guest does not check the sclp mask for simple events like quiesce. With other event users like the sclp line mode console, we will have differe

[Qemu-devel] [PATCH] migration: ram_handle_compressed

2013-09-18 Thread Isaku Yamahata
ram_handle_compressed() should be aware size > TARGET_PAGE_SIZE migration-rdma can call it with larger size. Signed-off-by: Isaku Yamahata --- arch_init.c | 21 ++--- 1 file changed, 14 insertions(+), 7 deletions(-) diff --git a/arch_init.c b/arch_init.c index e47e139..64c81b0

Re: [Qemu-devel] [PATCH v3 1/2] block: allow live commit of active image

2013-09-18 Thread Stefan Hajnoczi
On Wed, Sep 18, 2013 at 11:32:31AM +0800, Fam Zheng wrote: > On Wed, 09/04 14:35, Stefan Hajnoczi wrote: > > On Thu, Aug 15, 2013 at 04:14:06PM +0800, Fam Zheng wrote: > > > diff --git a/block/commit.c b/block/commit.c > > > index 2227fc2..b5e024b 100644 > > > --- a/block/commit.c > > > +++ b/block

Re: [Qemu-devel] [RFC] [PATCHv10 00/31] aio / timers: Add AioContext timers and use ppoll

2013-09-18 Thread Paolo Bonzini
Il 18/09/2013 11:02, Alex Bligh ha scritto: > Paolo, > > On 18 Sep 2013, at 09:23, Alex Bligh wrote: > >>> Yes, that was my understanding too. Can we do it for 1.7? > > Whilst we are changing the calling semantics, do you think > qemu_coroutine_yield() should also run the timers for the > aio_c

Re: [Qemu-devel] [RFC] [PATCHv10 00/31] aio / timers: Add AioContext timers and use ppoll

2013-09-18 Thread Alex Bligh
Paolo, On 18 Sep 2013, at 09:23, Alex Bligh wrote: >> Yes, that was my understanding too. Can we do it for 1.7? Whilst we are changing the calling semantics, do you think qemu_coroutine_yield() should also run the timers for the aio_context? IE should timers always be deferred to the next qemu_

  1   2   >