Re: [Qemu-devel] [PATCH 12/12] nbd: split requests

2011-09-09 Thread Nicholas Thomas
On 08/09/11 16:25, Paolo Bonzini wrote: > qemu-nbd has a limit of slightly less than 1M per request. Work > around this in the nbd block driver. > > Signed-off-by: Paolo Bonzini > --- > block/nbd.c | 52 ++-- > 1 files changed, 46 insertions(+),

Re: [Qemu-devel] [RESEND][PATCH] booke timers

2011-09-09 Thread Alexander Graf
On 09.09.2011, at 16:22, Fabien Chouteau wrote: > On 09/09/2011 15:46, Alexander Graf wrote: >> >> On 09.09.2011, at 15:27, Fabien Chouteau wrote: >> >>> On 09/09/2011 12:55, Alexander Graf wrote: On 09.09.2011, at 12:36, Fabien Chouteau wrote: >>> > +} >>>

Re: [Qemu-devel] [PATCH 11/12] nbd: switch to asynchronous operation

2011-09-09 Thread Paolo Bonzini
On 09/09/2011 04:52 PM, Nicholas Thomas wrote: I'm a bit unsure here, actually. So you lock a mutex, send a request, wait for a response, then unlock the mutex. Surely this code doesn't allow more than one request to be in flight at a time? No, it doesn't. It shouldn't be hard to do it though.

Re: [Qemu-devel] [PATCH] tcg/ppc64: Fix zero extension code generation bug for ppc64 host

2011-09-09 Thread malc
On Fri, 9 Sep 2011, David Gibson wrote: > From: Thomas Huth > > The ppc64 code generation backend uses an rldicr (Rotate Left Double > Immediate and Clear Right) instruction to implement zero extension of > a 32 bit quantity to a 64 bit quantity (INDEX_op_ext32u_i64). However > this is wrong -

Re: [Qemu-devel] Why qemu write/rw speed is so low?

2011-09-09 Thread Stefan Hajnoczi
On Fri, Sep 09, 2011 at 04:04:07PM +0200, Kevin Wolf wrote: > Am 09.09.2011 15:54, schrieb Stefan Hajnoczi: > > On Fri, Sep 9, 2011 at 2:48 PM, Zhi Yong Wu wrote: > >> On Fri, Sep 9, 2011 at 6:38 PM, Stefan Hajnoczi > >> wrote: > >>> On Fri, Sep 09, 2011 at 05:44:36PM +0800, Zhi Yong Wu wrote: >

[Qemu-devel] Question on kvm_clock working ...

2011-09-09 Thread al pat
We are doing an experiment with kvm-clock to validate its effectiveness, particularly when running NTP on the host to make sure the host’s clock stays properly sync. Our observations leads us to a few unanswered questions, including the possibility of a bug (our our misunderstanding of how kvm_cloc

Re: [Qemu-devel] [PATCH 12/12] nbd: split requests

2011-09-09 Thread Paolo Bonzini
On 09/09/2011 04:52 PM, Nicholas Thomas wrote: > +/* qemu-nbd has a limit of slightly less than 1M per request. For safety, > + * transfer at most 512K per request. */ > +#define NBD_MAX_SECTORS 1024 As far as I'm aware, the limit of 1MiB - header size is common to all NBD servers. I'm not

Re: [Qemu-devel] [PATCH 11/12] nbd: switch to asynchronous operation

2011-09-09 Thread Paolo Bonzini
On 09/09/2011 05:03 PM, Paolo Bonzini wrote: I'm a bit unsure here, actually. So you lock a mutex, send a request, wait for a response, then unlock the mutex. Surely this code doesn't allow more than one request to be in flight at a time? No, it doesn't. It shouldn't be hard to do it though.

[Qemu-devel] Envie de variété pour votre pause café?

2011-09-09 Thread Tassimo Pro
Si ce message ne s'affiche pas correctement, visualisez la version en ligne. Une solution simple : 1 machine = 12 boissons, en un seul clic ! Des recettes de qualité : 5 cafés, 3 boissons gourmandes, 3 thés et 1 infusion, de savoureuses recettes cautionnées par de grandes marques. Un

Re: [Qemu-devel] [PATCH] hw/lan9118.c: Convert to MemoryRegion

2011-09-09 Thread Peter Maydell
Ping? On 25 August 2011 18:59, Peter Maydell wrote: > Signed-off-by: Peter Maydell > --- > Another device I needed to convert so I could connect it to omap_gpmc > for an omap3 board (in this case overo). > >  hw/lan9118.c |   29 +++-- >  1 files changed, 11 insertions(+),

Re: [Qemu-devel] [PATCH 0/2] omap_intc: convert to MemoryRegion, qdev

2011-09-09 Thread Peter Maydell
Ping? -- PMM On 31 August 2011 16:55, Peter Maydell wrote: > This patchset converts the omap_intc device to MemoryRegion and qdev. > > The MemoryRegion conversion is fairly straightforward; the slight > ugliness of using get_system_memory() is just so it can be pulled > out as a separate patch f

Re: [Qemu-devel] About hotplug multifunction

2011-09-09 Thread Alex Williamson
On Fri, 2011-09-09 at 10:32 +0300, Michael S. Tsirkin wrote: > On Fri, Sep 09, 2011 at 03:08:21AM -0400, Amos Kong wrote: > > Hello all, > > > > I'm working on hotplug pci multifunction. > > > > 1. qemu cmdline: > > ./x86_64-softmmu/qemu-system-x86_64 -snapshot -m 2000 > > /home/kvm_autotest_r

Re: [Qemu-devel] [PATCH v2] Fix X86 CPU topology in KVM mode

2011-09-09 Thread Anthony Liguori
On 09/08/2011 12:33 AM, bharata@gmail.com wrote: From: Bharata B Rao apic id returned to guest kernel in ebx for cpuid(function=1) depends on CPUX86State->cpuid_apic_id which gets populated after the cpuid information is cached in the host kernel. This results in broken CPU topology in guest

Re: [Qemu-devel] About hotplug multifunction

2011-09-09 Thread Isaku Yamahata
pci/pcie hot plug needs clean up for multifunction hotplug in long term. Only single function device case works. Multifunction case is broken somwehat. Especially the current acpi based hotplug should be replaced by the standardized hot plug controller in long term. On Fri, Sep 09, 2011 at 03:08:2

Re: [Qemu-devel] [PATCH] pc_init: Fail on bad kernel

2011-09-09 Thread Anthony Liguori
On 09/03/2011 02:35 PM, Sasha Levin wrote: When providing QEMU with a bad '-kernel' parameter, such as a file which is not really a kernel, QEMU will attempt to allocate a huge amount of memory and fail either with "Failed to allocate memory: Cannot allocate memory" or a GLib error: "GLib-ERROR *

Re: [Qemu-devel] [PATCH] KVM: emulate lapic tsc deadline timer for hvm

2011-09-09 Thread Liu, Jinsong
Marcelo Tosatti wrote: > On Fri, Sep 09, 2011 at 01:12:51AM +0800, Liu, Jinsong wrote: > --- a/arch/x86/include/asm/msr-index.h > +++ b/arch/x86/include/asm/msr-index.h > @@ -229,6 +229,8 @@ > #define MSR_IA32_APICBASE_ENABLE (1<<11) > #define MSR_IA32_APICBASE_BASE

Re: [Qemu-devel] [PATCH] KVM: emulate lapic tsc deadline timer for hvm

2011-09-09 Thread Liu, Jinsong
Marcelo Tosatti wrote: > On Fri, Sep 09, 2011 at 01:12:51AM +0800, Liu, Jinsong wrote: > --- a/arch/x86/include/asm/msr-index.h > +++ b/arch/x86/include/asm/msr-index.h > @@ -229,6 +229,8 @@ > #define MSR_IA32_APICBASE_ENABLE (1<<11) > #define MSR_IA32_APICBASE_BASE

Re: [Qemu-devel] [PULL 0/4] xen: old bug fixes

2011-09-09 Thread Anthony Liguori
On 09/09/2011 08:39 AM, Stefano Stabellini wrote: Hi Anthony, I have few not very exiting xen bug fixes laying around that haven't been applied to master yet. None of them are new, some of them are more than an year old. Please pull (git branch based on 07ff2c4475df77e38a31d50ee7f3932631806c15)

Re: [Qemu-devel] [PULL 0/8] Trivial patches for September 3 to 9 2011

2011-09-09 Thread Anthony Liguori
On 09/09/2011 04:24 AM, Stefan Hajnoczi wrote: The following changes since commit 07ff2c4475df77e38a31d50ee7f3932631806c15: Merge remote-tracking branch 'origin/master' into staging (2011-09-08 09:25:36 -0500) are available in the git repository at: ssh://repo.or.cz/srv/git/qemu/stefanh

Re: [Qemu-devel] [PATCH] qemu_vmalloc: align properly for transparent hugepages and KVM

2011-09-09 Thread Anthony Liguori
On 09/05/2011 03:07 AM, Avi Kivity wrote: To make good use of transparent hugepages, KVM requires that guest-physical and host-virtual addresses share the low 21 bits (as opposed to just the low 12 bits normally required). Adjust qemu_vmalloc() to honor that requirement. Ignore it for small reg

Re: [Qemu-devel] [PATCH 0/3] Fix build failures.

2011-09-09 Thread Anthony Liguori
On 09/07/2011 08:02 AM, Gerd Hoffmann wrote: Hi, This patch series fixes a bunch of build failures. please apply, Gerd Applied all. Thanks. Regards, Anthony Liguori Gerd Hoffmann (3): Don't use g_thread_get_initialized. Use hex instead of binary. vns/tls: don't use depric

Re: [Qemu-devel] About hotplug multifunction

2011-09-09 Thread Marcelo Tosatti
On Fri, Sep 09, 2011 at 10:05:01AM -0700, Alex Williamson wrote: > On Fri, 2011-09-09 at 10:32 +0300, Michael S. Tsirkin wrote: > > On Fri, Sep 09, 2011 at 03:08:21AM -0400, Amos Kong wrote: > > > Hello all, > > > > > > I'm working on hotplug pci multifunction. > > > > > > 1. qemu cmdline: > >

Re: [Qemu-devel] [PATCH] curses: fix garbling when chtype != long

2011-09-09 Thread Anthony Liguori
On 09/07/2011 02:44 PM, Devin J. Pohly wrote: From: "Devin J. Pohly" Qemu currently assumes that chtype is typedef'd to unsigned long, but this is not necessarily the case (ncurses, for instance, can configure this at build-time). This patch uses the predefined chtype if qemu is configured for

Re: [Qemu-devel] [PATCH 2/2] build: fix race with creating qapi-generated

2011-09-09 Thread Anthony Liguori
On 09/07/2011 06:40 PM, Michael Roth wrote: Since qapi-generated/ is a global QEMU include path, we need to make sure it is created before anything is compiled, so do this in the configure phase rather than via the Makefile. Signed-off-by: Michael Roth Applied. Thanks. Regards, Anthony Ligu

Re: [Qemu-devel] [PATCH 1/2] balloon: Disassociate handlers from balloon device on unplug

2011-09-09 Thread Anthony Liguori
On 09/09/2011 04:00 AM, Amit Shah wrote: When a balloon device gets unplugged, allow the balloon handlers to be freed. Signed-off-by: Amit Shah Applied all. Thanks. Regards, Anthony Liguori --- balloon.c | 10 ++ balloon.h |1 + 2 files changed, 11 insertions(+), 0 delet

Re: [Qemu-devel] [PATCH] KVM: emulate lapic tsc deadline timer for hvm

2011-09-09 Thread Marcelo Tosatti
On Sat, Sep 10, 2011 at 02:11:36AM +0800, Liu, Jinsong wrote: > Marcelo Tosatti wrote: > > On Fri, Sep 09, 2011 at 01:12:51AM +0800, Liu, Jinsong wrote: > > --- a/arch/x86/include/asm/msr-index.h > > +++ b/arch/x86/include/asm/msr-index.h > > @@ -229,6 +229,8 @@ > > #define MSR_IA3

Re: [Qemu-devel] [PATCH] KVM: emulate lapic tsc deadline timer for hvm

2011-09-09 Thread Liu, Jinsong
>> >> My question is, which kvm_get_msrs/kvm_put_msrs routine be used by >> live migration, the routine in target-i386/kvm.c, or in >> kvm/libkvm/libkvm-x86.c? They both have ioctl >> KVM_GET_MSR_INDEX_LIST/ KVM_GET_MSRS/ KVM_SET_MSRS, but I'm not >> clear their purpose/usage difference. > >

Re: [Qemu-devel] [PATCH V8 02/14] Add TPM (frontend) hardware interface (TPM TIS) to Qemu

2011-09-09 Thread Paul Moore
On Wednesday, August 31, 2011 10:35:53 AM Stefan Berger wrote: > Index: qemu-git/hw/tpm_tis.c > === > --- /dev/null > +++ qemu-git/hw/tpm_tis.c > @@ -0,0 +1,841 @@ > +/* > + * tpm_tis.c - QEMU emulator for a 1.2 TPM with TIS interface

[Qemu-devel] [PATCH v5 0/9]: Introduce the RunState type

2011-09-09 Thread Luiz Capitulino
It replaces the VMSTOP macros and allows us to drop some global variables. Additionally, the problem with issuing 'cont' when the VM is in bad state is addressed and we make the current state available in QMP and HMP. ** IMPORTANT: patch 04/09 is a new addition, it needs careful review. This is

[Qemu-devel] [PATCH 7/9] Monitor/QMP: Don't allow cont on bad VM state

2011-09-09 Thread Luiz Capitulino
We have two states where issuing cont before system_reset can cause problems: RSTATE_SHUTDOWN (when -no-shutdown is used) and RSTATE_PANICKED (which only happens with kvm). This commit fixes that by doing the following when state is RSTATE_SHUTDOWN or RSTATE_PANICKED: 1. returning an error to th

[Qemu-devel] [PATCH 2/9] Replace the VMSTOP macros with a proper state type

2011-09-09 Thread Luiz Capitulino
Today, when notifying a VM state change with vm_state_notify(), we pass a VMSTOP macro as the 'reason' argument. This is not ideal because the VMSTOP macros tell why qemu stopped and not exactly what the current VM state is. One example to demonstrate this problem is that vm_start() calls vm_state

[Qemu-devel] [PATCH 3/9] RunState: Add additional states

2011-09-09 Thread Luiz Capitulino
Currently, only vm_start() and vm_stop() change the VM state. That's, the state is only changed when starting or stopping the VM. This commit adds the runstate_set() function, which makes it possible to also do state transitions when the VM is stopped or running. Additional states are also added

[Qemu-devel] [PATCH 9/9] HMP: info status: Print the VM state

2011-09-09 Thread Luiz Capitulino
Today our printf format for the "info status" command is: VM status: %s Where the string can be "running", "running (single step mode)" or "paused". This commit extends it to: VM status: %s (%s) The second string corresponds to the "status" field as returned by the query-status QMP command

[Qemu-devel] [PATCH 1/9] Move vm_state_notify() prototype from cpus.h to sysemu.h

2011-09-09 Thread Luiz Capitulino
It's where all the state handling functions prototypes are located. Signed-off-by: Luiz Capitulino --- cpus.h |1 - sysemu.h |1 + 2 files changed, 1 insertions(+), 1 deletions(-) diff --git a/cpus.h b/cpus.h index f42b54e..5885885 100644 --- a/cpus.h +++ b/cpus.h @@ -15,7 +15,6 @@ vo

[Qemu-devel] [PATCH 5/9] Drop the incoming_expected global variable

2011-09-09 Thread Luiz Capitulino
Test against RSTATE_IN_MIGRATE instead. Please, note that the RSTATE_IN_MIGRATE state is only set when all the initial VM setup is done, while 'incoming_expected' was set right in the beginning when parsing command-line options. Shouldn't be a problem as far as I could check. Signed-off-by: Luiz

[Qemu-devel] [PATCH 4/9] runstate_set(): Check for valid transitions

2011-09-09 Thread Luiz Capitulino
This commit could have been folded with the previous one, however doing it separately will allow for easy bisect and revert if needed. Checking and testing all valid transitions wasn't trivial, chances are this will need broader testing to become more stable. This is a transition table as suggest

[Qemu-devel] [PATCH 6/9] Drop the vm_running global variable

2011-09-09 Thread Luiz Capitulino
Use runstate_is_running() instead, which is introduced by this commit. Signed-off-by: Luiz Capitulino --- cpus.c|9 - gdbstub.c |4 ++-- hw/etraxfs_dma.c |2 +- hw/kvmclock.c |2 +- hw/virtio.c |2 +- migration.c |2 +- monitor

[Qemu-devel] [PATCH 8/9] QMP: query-status: Introduce 'status' key

2011-09-09 Thread Luiz Capitulino
This new key reports the current VM status to clients. Please, check the documentation being added in this commit for more details. Signed-off-by: Luiz Capitulino --- monitor.c |3 +-- qmp-commands.hx | 19 ++- sysemu.h|1 + vl.c| 23

Re: [Qemu-devel] [PATCH V8 03/14] Add persistent state handling to TPM TIS frontend driver

2011-09-09 Thread Paul Moore
On Wednesday, August 31, 2011 10:35:54 AM Stefan Berger wrote: > Index: qemu-git/hw/tpm_tis.c > === > --- qemu-git.orig/hw/tpm_tis.c > +++ qemu-git/hw/tpm_tis.c > @@ -6,6 +6,8 @@ > * Author: Stefan Berger > * David Safford

[Qemu-devel] qemu virtIO blocking operation - question

2011-09-09 Thread Sinha, Ani
Hey guys, So I am writing a virtIO driver for a device that supports blocking calls like poll() etc. Now the front end paravirtualized driver mirrors the request to the backend "host" qemu process that then does the actual call on the host kernel on behalf of the guest. Now my question is, when

Re: [Qemu-devel] [PATCH] support add-cow file format

2011-09-09 Thread Dong Xu Wang
于Fri 09 Sep 2011 10:27:26 PM CST,Kevin Wolf写到: Am 09.09.2011 07:48, schrieb Dong Xu Wang: As raw file format does not support backing_file and copy on write feature, so I add COW to it to support backing_file option. I store dirty bitmap in an add-cow file. When executed, it looks like this: q

[Qemu-devel] [PATCH] Add iSCSI support for QEMU

2011-09-09 Thread Ronnie Sahlberg
List, Please find a patch that adds built-in iSCSI support to QEMU when built and linked against the multiplatform iscsi initiator library at git://github.com/sahlberg/libiscsi.git All previous comments and suggestions have been addressed in this patch. I and others have done extensive testi

[Qemu-devel] [PATCH] This patch adds a new block driver : iSCSI

2011-09-09 Thread Ronnie Sahlberg
This provides built-in support for iSCSI to QEMU. This has the advantage that the iSCSI devices need not be made visible to the host, which is useful if you have very many virtual machines and very many iscsi devices. It also has the benefit that non-root users of QEMU can access iSCSI devices a

[Qemu-devel] buildbot failure in qemu on s390-next_i386_debian_6_0

2011-09-09 Thread qemu
The Buildbot has detected a new failure on builder s390-next_i386_debian_6_0 while building qemu. Full details are available at: http://buildbot.b1-systems.de/qemu/builders/s390-next_i386_debian_6_0/builds/26 Buildbot URL: http://buildbot.b1-systems.de/qemu/ Buildslave for this Build: yuzuki B

<    1   2