[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

[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] [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

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] 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 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] [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

[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 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 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 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 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 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 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 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 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

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

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] 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 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 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] 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] 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 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] [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] [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] [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] [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] [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] 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 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 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 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] [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(+),

[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 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.

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

[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] 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: >

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] [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] [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 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] [PATCH 11/12] nbd: switch to asynchronous operation

2011-09-09 Thread Nicholas Thomas
On 08/09/11 16:25, Paolo Bonzini wrote: > Signed-off-by: Paolo Bonzini > --- > block/nbd.c | 167 ++ > nbd.c |8 +++ > 2 files changed, 117 insertions(+), 58 deletions(-) > > diff --git a/block/nbd.c b/block/nbd.c > index 964caa8

Re: [Qemu-devel] [PATCH 00/12] nbd improvements

2011-09-09 Thread Nicholas Thomas
On 09/09/11 12:04, Kevin Wolf wrote: > Good to see agreement here. Do you think that Paolo's patches need to be > changed or can we do everything else on top? A few things have come up on a third read, actually. I'll respond in due course to the appropriate patch. > We do have some timer stubs i

[Qemu-devel] [PATCH] scsi-generic: fix get_stream_blocksize

2011-09-09 Thread Paolo Bonzini
The computation of the block size is completely broken; mode page 0 is vendor specific. Use the right SCSI command instead. Signed-off-by: Paolo Bonzini --- hw/scsi-generic.c |7 +++ 1 files changed, 3 insertions(+), 4 deletions(-) diff --git a/hw/scsi-generic.c b/hw/scsi-generic.c ind

[Qemu-devel] [PATCH] scsi: fix sign extension problems

2011-09-09 Thread Paolo Bonzini
When assigning a 32-bit value to cmd->xfer (which is 64-bits) it can be erroneously sign extended because the intermediate 32-bit computation is signed. Fix this by standardizing on the ld*_be_p functions. Signed-off-by: Paolo Bonzini --- hw/scsi-bus.c | 22 +++--- 1 files cha

Re: [Qemu-devel] [PATCH v8 3/4] block: add block timer and throttling algorithm

2011-09-09 Thread Marcelo Tosatti
On Thu, Sep 08, 2011 at 06:11:07PM +0800, Zhi Yong Wu wrote: > Note: > 1.) When bps/iops limits are specified to a small value such as 511 > bytes/s, this VM will hang up. We are considering how to handle this senario. You can increase the length of the slice, if the request is larger than s

Re: [Qemu-devel] CFQ I/O starvation problem triggered by RHEL6.0 KVM guests

2011-09-09 Thread Vivek Goyal
On Fri, Sep 09, 2011 at 06:00:28PM +0900, Takuya Yoshikawa wrote: [..] > > > > > - Even if there are close cooperators, these queues are merged and they > > are treated as single queue from slice point of view. So cooperating > > queues should be merged and get a single slice instead of star

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

2011-09-09 Thread 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: > qemu-img create -f add-cow -o backing_fi

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

2011-09-09 Thread Fabien Chouteau
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: >> >>> +} + +qemu_mod_timer(timer, *next); +}

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

2011-09-09 Thread Zhi Yong Wu
On Fri, Sep 9, 2011 at 9:54 PM, Stefan Hajnoczi wrote: > 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: Today, i did some basical I/O testing, and suddenly found

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

2011-09-09 Thread Kevin Wolf
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: Today, i did some basical I/O testing, and suddenly found that qemu

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

2011-09-09 Thread 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: >>> Today, i did some basical I/O testing, and suddenly found that qemu write >>> and rw speed is so low now, my qemu bina

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

2011-09-09 Thread Zhi Yong Wu
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: >> Today, i did some basical I/O testing, and suddenly found that qemu write >> and rw speed is so low now, my qemu binary is built on commit >> 344eecf6995f4a0ad1d887cec922f6806

Re: [Qemu-devel] CFQ I/O starvation problem triggered by RHEL6.0 KVM guests

2011-09-09 Thread Stefan Hajnoczi
On Fri, Sep 9, 2011 at 10:00 AM, Takuya Yoshikawa wrote: > Vivek Goyal wrote: > >> So you are using both RHEL 6.0 in both host and guest kernel? Can you >> reproduce the same issue with upstream kernels? How easily/frequently >> you can reproduce this with RHEL6.0 host. > > Guests were CentOS6.0.

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

2011-09-09 Thread Alexander Graf
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: > >> >>> +} >>> + >>> +qemu_mod_timer(timer, *next); >>> +} >>> + >>> +static void booke_decr_cb(void *opaq

Re: [Qemu-devel] [PATCH 3/5] dma-helpers: rewrite completion/cancellation

2011-09-09 Thread Paolo Bonzini
On 09/09/2011 03:34 PM, Kevin Wolf wrote: So the release that we avoid is the release in the callback that may or may not be called indirectly by bdrv_aio_cancel. We always call dma_complete at the end of dma_aio_cancel so that it will be properly freed. In fact it may be worse than just the qe

[Qemu-devel] [PATCH 4/4] xen-mapcache: Fix rlimit set size.

2011-09-09 Thread stefano.stabellini
From: Anthony PERARD Previously, the address space soft limit was set mcache_max_size. So, before the mcache_max_size was reached by the mapcache, QEMU was killed for overuse of the virtual address space. This patch fix that by setting the soft limit the maximum than can have QEMU. So the soft a

[Qemu-devel] [PATCH 1/4] Introduce a new 'connected' xendev op called when Connected.

2011-09-09 Thread stefano.stabellini
From: John Haxby Rename the existing xendev 'connect' op to 'initialised' and introduce a new 'connected' op. This new op, if defined, is called when the backend is connected. Note that since there is no state transition this may be called more than once. Signed-off-by: John Haxby Signed-off-

[Qemu-devel] [PATCH 3/4] xen: use uint64_t instead of target_ulong in cpu_ioreq_move

2011-09-09 Thread stefano.stabellini
From: Stefano Stabellini cpu_ioreq_move might move 8 bytes at a time so we must make sure that the temporary variable can hold 8 bytes. Signed-off-by: Stefano Stabellini --- xen-all.c |2 +- 1 files changed, 1 insertions(+), 1 deletions(-) diff --git a/xen-all.c b/xen-all.c index 9eaeac1.

[Qemu-devel] [PATCH 2/4] Move the xenfb pointer handler to the connected method

2011-09-09 Thread stefano.stabellini
From: John Haxby Ensure that we read "request-abs-pointer" after the frontend has written it. This means that we will correctly set up an ansolute or relative pointer handler correctly. Signed-off-by: John Haxby Signed-off-by: Stefano Stabellini --- hw/xenfb.c | 21 - 1

Re: [Qemu-devel] [PATCH 3/5] dma-helpers: rewrite completion/cancellation

2011-09-09 Thread Kevin Wolf
Am 09.09.2011 15:12, schrieb Paolo Bonzini: > On 09/09/2011 02:59 PM, Kevin Wolf wrote: Also, I think it should be -EIO instead of -ENOMEM (even though it doesn't make any difference if we don't call the callback) >>> >>> If I understood the code correctly, dbs->io_func can only fail if i

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

2011-09-09 Thread Stefano Stabellini
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): git://xenbits.xen.org/people/sstabellini/qemu-dm.

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

2011-09-09 Thread Fabien Chouteau
On 09/09/2011 12:55, Alexander Graf wrote: > > On 09.09.2011, at 12:36, Fabien Chouteau wrote: > >> On 07/09/2011 21:59, Alexander Graf wrote: >>> >>> On 07.09.2011, at 16:41, Fabien Chouteau wrote: >>> On 06/09/2011 21:33, Alexander Graf wrote: > > > Am 01.09.2011 um 10:20 schri

Re: [Qemu-devel] RFC [v2]: vfio / device assignment -- layout of device fd files

2011-09-09 Thread Stuart Yoder
Meant to identify the changes in v2 of this proposal: v2: -removed PCI_INFO record type -removed PCI_BAR_INFO record type -PCI_CONFIG_SPACE is now a sub-record/property of a REGION -removed physical address from region and made it a subrecord/property of a REGION -added PCI_BAR_

Re: [Qemu-devel] [PATCH 3/5] dma-helpers: rewrite completion/cancellation

2011-09-09 Thread Paolo Bonzini
On 09/09/2011 02:59 PM, Kevin Wolf wrote: Also, I think it should be -EIO instead of -ENOMEM (even though it doesn't make any difference if we don't call the callback) If I understood the code correctly, dbs->io_func can only fail if it fails to get an AIOCB, which is basically out-of-memory.

[Qemu-devel] RFC [v2]: vfio / device assignment -- layout of device fd files

2011-09-09 Thread Stuart Yoder
Based on the discussions over the last couple of weeks I have updated the device fd file layout proposal and tried to specify it a bit more formally. === 1. Overview This specification describes the layout of device files used in t

Re: [Qemu-devel] [PATCH v1 0/6]: block: Add I/O status support

2011-09-09 Thread Luiz Capitulino
On Thu, 1 Sep 2011 15:37:49 -0300 Luiz Capitulino wrote: > This series adds support to the block layer to keep track of devices' > I/O status. That information is also made available in QMP and HMP. ping? I'll have to rebase it anyway, but I'd like to know if the general approach is acceptable

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

2011-09-09 Thread Luiz Capitulino
On Tue, 06 Sep 2011 19:42:15 +0200 Lluís Vilanova wrote: > Luiz Capitulino writes: > > > On Tue, 06 Sep 2011 17:55:12 +0200 > > Jan Kiszka wrote: > > >> On 2011-09-06 15:14, Luiz Capitulino wrote: > >> > This commit could have been folded with the previous one, however > >> > doing it separate

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

2011-09-09 Thread Marcelo Tosatti
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 (0xf<<12) > >>> > >>

Re: [Qemu-devel] [PATCH 3/5] dma-helpers: rewrite completion/cancellation

2011-09-09 Thread Kevin Wolf
Am 09.09.2011 14:43, schrieb Paolo Bonzini: >>> @@ -120,9 +132,8 @@ static void dma_bdrv_cb(void *opaque, int ret) >>> dbs->acb = dbs->io_func(dbs->bs, dbs->sector_num, &dbs->iov, >>> dbs->iov.size / 512, dma_bdrv_cb, dbs); >>> if (!dbs->acb) { >>> -dm

Re: [Qemu-devel] [PATCH 3/5] dma-helpers: rewrite completion/cancellation

2011-09-09 Thread Paolo Bonzini
> > @@ -120,9 +132,8 @@ static void dma_bdrv_cb(void *opaque, int ret) > > dbs->acb = dbs->io_func(dbs->bs, dbs->sector_num, &dbs->iov, > > dbs->iov.size / 512, dma_bdrv_cb, dbs); > > if (!dbs->acb) { > > -dma_bdrv_unmap(dbs); > > -qemu_iovec_d

Re: [Qemu-devel] [PATCH 2/5] dma-helpers: allow including from target-independent code

2011-09-09 Thread Kevin Wolf
Am 07.09.2011 17:20, schrieb Paolo Bonzini: > Target-independent code cannot construct sglists, but it can take > them from the outside as a black box. Allow this. > > Signed-off-by: Paolo Bonzini I was hoping to find the use case for this somewhere in the rest of the series, but there doesn't

Re: [Qemu-devel] [PATCH 3/5] dma-helpers: rewrite completion/cancellation

2011-09-09 Thread Kevin Wolf
Am 07.09.2011 17:21, schrieb Paolo Bonzini: > This fixes various problems with completion/cancellation: > > * If DMA encounters a bounce buffer conflict, and the DMA operation is > canceled before the bottom half fires, bad things happen. > > * memory is not unmapped after cancellation, again cau

[Qemu-devel] cross compiler for MIPS64

2011-09-09 Thread Khansa Butt
Hi I'm working on MIPS64 emulation and I need cross compiler for MIPS64. would you please guide me in this regard?. Is there any cross compiler exist or I have to make it from scratch? thanks

Re: [Qemu-devel] [PATCH 2/5] dma-helpers: allow including from target-independent code

2011-09-09 Thread Paolo Bonzini
On 09/09/2011 01:39 PM, Kevin Wolf wrote: Am 07.09.2011 17:20, schrieb Paolo Bonzini: Target-independent code cannot construct sglists, but it can take them from the outside as a black box. Allow this. Signed-off-by: Paolo Bonzini I was hoping to find the use case for this somewhere in the r

Re: [Qemu-devel] [PATCH 1/5] dma-helpers: rename is_write to to_dev

2011-09-09 Thread Kevin Wolf
Am 07.09.2011 17:20, schrieb Paolo Bonzini: > Signed-off-by: Paolo Bonzini > --- > dma-helpers.c | 10 +- > 1 files changed, 5 insertions(+), 5 deletions(-) > > diff --git a/dma-helpers.c b/dma-helpers.c > index 4610ea0..ca97e14 100644 > --- a/dma-helpers.c > +++ b/dma-helpers.c > @@ -

Re: [Qemu-devel] [PATCH] ahci: Remove unused struct member

2011-09-09 Thread Kevin Wolf
Am 09.09.2011 11:43, schrieb Alexander Graf: > > On 09.09.2011, at 11:15, Kevin Wolf wrote: > >> Am 08.09.2011 21:44, schrieb Stefan Weil: >>> Member variable is_read is written, but never read >>> (contrary to its name). >> >> Nice commit message :-) >> >>> Kevin Wolf >>> Signed-off-by: Stefan

Re: [Qemu-devel] [PATCH 0/2] Make simpletrace work on Windows

2011-09-09 Thread Paolo Bonzini
On 09/09/2011 11:37 AM, Stefan Hajnoczi wrote: The 'simple' trace backend uses pthreads and does not work on Windows. These patches switch from pthreads to glib so that the code builds on all platforms supported by glib. Only one thing I'm unhappy about: the simpletrace write-out thread used to

[Qemu-devel] [RFC PATCH 2/3] hw/9pfs: Add synthetic file system support using 9p

2011-09-09 Thread Aneesh Kumar K.V
This patch create a synthetic file system with mount tag v_synth when -virtfs_synth command line option is specified in qemu. The synthetic file system can be mounted in guest using 9p using the below command line mount -t 9p -oversion=9p2000.L,trans=virtio v_synth Synthetic file system enabled

[Qemu-devel] [RFC PATCH 1/3] hw/9pfs: Abstract open state of fid to V9fsFidOpenState

2011-09-09 Thread Aneesh Kumar K.V
To implement synthetic file system in Qemu we may not really require file descriptor and Dir *. Make generic code use V9fsFidOpenState instead. Signed-off-by: Aneesh Kumar K.V --- fsdev/file-op-9p.h | 32 ++--- hw/9pfs/codir.c| 16 hw/9pfs/cofile.c

[Qemu-devel] [RFC PATCH 3/3] hw/9pfs: Add synth fs test

2011-09-09 Thread Aneesh Kumar K.V
Signed-off-by: Aneesh Kumar K.V --- hw/9pfs/virtio-9p-synth.c | 19 +++ 1 files changed, 19 insertions(+), 0 deletions(-) diff --git a/hw/9pfs/virtio-9p-synth.c b/hw/9pfs/virtio-9p-synth.c index 72a916c..cbf74e4 100644 --- a/hw/9pfs/virtio-9p-synth.c +++ b/hw/9pfs/virtio-9p-syn

[Qemu-devel] [RFC PATCH] Implement synthetic file system support in qemu using 9p

2011-09-09 Thread Aneesh Kumar K.V
This patchset implement a synthetic file system in qemu using 9p. The immediate goal is to get better error injection based testing for VirtFS server. But i guess other subsystem in qemu can use the framework to expose hypervisor data to guest. The framework also enables qemu subsytem to allow gu

Re: [Qemu-devel] [PATCH 00/12] nbd improvements

2011-09-09 Thread Kevin Wolf
Am 09.09.2011 12:50, schrieb Nicholas Thomas: > On Fri, 2011-09-09 at 12:29 +0200, Paolo Bonzini wrote: >> On 09/09/2011 11:00 AM, Kevin Wolf wrote: >>> There is anonther patch enabling AIO for NBD on the list [1], by >>> Nicholas Thomas (CCed), that lacked review so far. Can you guys please >>> re

Re: [Qemu-devel] [PATCH 00/12] nbd improvements

2011-09-09 Thread Paolo Bonzini
On 09/09/2011 12:50 PM, Nicholas Thomas wrote: > On the other hand, Nicholas's work includes timeout and reconnect. I'm > not sure how complicated it is to include it in my series, but probably > not much. With coroutines, preserving the list of outstanding I/O > requests is done implicitly

[Qemu-devel] [PATCH 6/8] hw/9pfs: Avoid unnecessary get_fid in v9fs_clunk

2011-09-09 Thread Aneesh Kumar K.V
Signed-off-by: Aneesh Kumar K.V --- hw/9pfs/virtio-9p.c | 20 ++-- 1 files changed, 10 insertions(+), 10 deletions(-) diff --git a/hw/9pfs/virtio-9p.c b/hw/9pfs/virtio-9p.c index 2a68953..e51df2a 100644 --- a/hw/9pfs/virtio-9p.c +++ b/hw/9pfs/virtio-9p.c @@ -417,7 +417,7 @@ sta

[Qemu-devel] [PATCH 2/8] hw/9pfs: Use read-write lock for protecting fid path.

2011-09-09 Thread Aneesh Kumar K.V
On rename we take the write lock and this ensure path doesn't change as we operate on them. Signed-off-by: Aneesh Kumar K.V --- hw/9pfs/codir.c| 18 ++- hw/9pfs/cofile.c | 32 +++-- hw/9pfs/cofs.c | 52 ++--- hw/9pfs/coxattr.c

[Qemu-devel] [PATCH 8/8] hw/9pfs: Add handle based fs driver

2011-09-09 Thread Aneesh Kumar K.V
Signed-off-by: Aneesh Kumar K.V --- Makefile.objs |8 +- fsdev/qemu-fsdev.c |1 + fsdev/qemu-fsdev.h |1 + hw/9pfs/virtio-9p-handle.c | 611 4 files changed, 617 insertions(+), 4 deletions(-) create mode 10064

[Qemu-devel] [PATCH] Implement file handle based fs driver

2011-09-09 Thread Aneesh Kumar K.V
This patchset implement a file handle based fs driver for VirtFS server Using file handle instead of names, simplify the rename handling in the server. The changes also enable us to handle renames on the host. Since we are not using pathnames, symlink related security issues are not present with ha

[Qemu-devel] [PATCH 5/8] hw/9pfs: Add fs driver specific details to fscontext

2011-09-09 Thread Aneesh Kumar K.V
Add a new context flag PATHNAME_FSCONTEXT and indicate whether the fs driver track fid using path names. Also add a private pointer that help us to track fs driver specific values in there Signed-off-by: Aneesh Kumar K.V --- fsdev/file-op-9p.h |6 hw/9pfs/codir.c|

[Qemu-devel] [PATCH 4/8] hw/9pfs: Add init callback to fs driver

2011-09-09 Thread Aneesh Kumar K.V
This call back can be used to do fs driver specific initialization. Signed-off-by: Aneesh Kumar K.V --- fsdev/file-op-9p.h |2 ++ hw/9pfs/virtio-9p-device.c |7 ++- hw/9pfs/virtio-9p-local.c |7 ++- 3 files changed, 14 insertions(+), 2 deletions(-) diff --git a/fsde

[Qemu-devel] [PATCH 1/8] hw/9pfs: Make v9fs_string* functions non-static

2011-09-09 Thread Aneesh Kumar K.V
We will use them later in other files Signed-off-by: Aneesh Kumar K.V --- hw/9pfs/virtio-9p.c | 10 +- hw/9pfs/virtio-9p.h |5 + 2 files changed, 10 insertions(+), 5 deletions(-) diff --git a/hw/9pfs/virtio-9p.c b/hw/9pfs/virtio-9p.c index d28edb7..dc3e32b 100644 --- a/hw/9pfs

Re: [Qemu-devel] [PATCH 00/12] nbd improvements

2011-09-09 Thread Nicholas Thomas
On Fri, 2011-09-09 at 12:29 +0200, Paolo Bonzini wrote: > On 09/09/2011 11:00 AM, Kevin Wolf wrote: > > There is anonther patch enabling AIO for NBD on the list [1], by > > Nicholas Thomas (CCed), that lacked review so far. Can you guys please > > review each others approach and then converge on a

[Qemu-devel] [PATCH 2/7] linux-user: Implement new ARM 64 bit cmpxchg kernel helper

2011-09-09 Thread riku . voipio
From: Dr. David Alan Gilbert linux-user: Implement new ARM 64 bit cmpxchg kernel helper Linux 3.1 will have a new kernel-page helper for ARM implementing 64 bit cmpxchg. Implement this helper in QEMU linux-user mode: * Provide kernel helper emulation for 64bit cmpxchg * Allow guest to object t

Re: [Qemu-devel] [PATCH 00/12] nbd improvements

2011-09-09 Thread Kevin Wolf
Am 09.09.2011 12:29, schrieb Paolo Bonzini: > On 09/09/2011 11:00 AM, Kevin Wolf wrote: >> There is anonther patch enabling AIO for NBD on the list [1], by >> Nicholas Thomas (CCed), that lacked review so far. Can you guys please >> review each others approach and then converge on a solution? I gue

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

2011-09-09 Thread Stefan Hajnoczi
On Fri, Sep 09, 2011 at 05:44:36PM +0800, Zhi Yong Wu wrote: > Today, i did some basical I/O testing, and suddenly found that qemu write and > rw speed is so low now, my qemu binary is built on commit > 344eecf6995f4a0ad1d887cec922f6806f91a3f8. > > Do qemu have regression? > > The testing data

[Qemu-devel] [PATCH 1/7] linux-user: Fix initialization of the heap contents when allocating new pages

2011-09-09 Thread riku . voipio
From: Cédric VINCENT Technically the new mmapped pages are already initialized to zero since they are anonymous, however we have to take care with the contents that come from the remaining part of the previous page: it may contains garbage data due to a previous heap usage (grown then shrunken).

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

2011-09-09 Thread Fabien Chouteau
On 07/09/2011 21:59, Alexander Graf wrote: > > On 07.09.2011, at 16:41, Fabien Chouteau wrote: > >> On 06/09/2011 21:33, Alexander Graf wrote: >>> >>> >>> Am 01.09.2011 um 10:20 schrieb Fabien Chouteau : >>> While working on the emulation of the freescale p2010 (e500v2) I realized that

  1   2   >