Re: [Qemu-devel] [PATCH 0/3] make bh safe with hot-unplug

2013-06-24 Thread Paolo Bonzini
Il 25/06/2013 19:38, Liu Ping Fan ha scritto: > This series relies on refcnt of object used by bh callback to run against > unplug. > > Open issue: > Another choice may be rcu, but I think some issues are hard to resolve. > Using rcu, we have two choice: > when holding object refcnt, call qemu_

Re: [Qemu-devel] [PATCH 0/3] make bh safe with hot-unplug

2013-06-24 Thread liu ping fan
On Tue, Jun 25, 2013 at 2:24 PM, Paolo Bonzini wrote: > Il 25/06/2013 19:38, Liu Ping Fan ha scritto: >> This series relies on refcnt of object used by bh callback to run against >> unplug. >> >> Open issue: >> Another choice may be rcu, but I think some issues are hard to resolve. >> Using rcu,

Re: [Qemu-devel] [PATCH] tls.h: Enable TLS on FreeBSD

2013-06-24 Thread Paolo Bonzini
Il 24/06/2013 23:30, Peter Maydell ha scritto: > On 24 June 2013 22:15, Paolo Bonzini wrote: >> Il 24/06/2013 21:21, Ed Maste ha scritto: >>> Signed-off-by: Ed Maste >>> --- >>> I have had this in a local tree for some time, and it is needed by the >>> BSD-user work that is now being proposed. >>

[Qemu-devel] [PATCH 3/3] virtio-net: set referred object for virtio net's bh

2013-06-24 Thread Liu Ping Fan
Expose object to bh, so bh will pin virtio-net against unplugged in parallel Signed-off-by: Liu Ping Fan --- hw/net/virtio-net.c | 1 + 1 file changed, 1 insertion(+) diff --git a/hw/net/virtio-net.c b/hw/net/virtio-net.c index 1ea9556..7c0ded9 100644 --- a/hw/net/virtio-net.c +++ b/hw/net/virt

[Qemu-devel] [PATCH 2/3] QEMUBH: pin bh's referring object while scheduling

2013-06-24 Thread Liu Ping Fan
When out of biglock, object will be unplugged while its bh is scheduling, using refcnt on bh->object to pin the object. Signed-off-by: Liu Ping Fan --Another choice may be -- rcu_read_lock protect aio_bh_poll() qemu_bh_delete(); smp_wmb(); object_unref(obj) reclaim object after grace peri

[Qemu-devel] [PATCH 1/3] QEMUBH: introduce canceled member for bh

2013-06-24 Thread Liu Ping Fan
When dismissing the bh scheduling(delete/cancel/schedule), we need to reset bh->scheduled to zero and release the refcnt of object which is referred by bh(will introduced by next patch). Currently, the bh->scheduled will be reset to zero by many writers, so atomic ops should be involved in, which r

[Qemu-devel] [PATCH 0/3] make bh safe with hot-unplug

2013-06-24 Thread Liu Ping Fan
This series relies on refcnt of object used by bh callback to run against unplug. Open issue: Another choice may be rcu, but I think some issues are hard to resolve. Using rcu, we have two choice: when holding object refcnt, call qemu_bh_delete(); then after grace period, we can release. Or

[Qemu-devel] [PATCH v5] QEMUBH: make AioContext's bh re-entrant

2013-06-24 Thread Liu Ping Fan
BH will be used outside big lock, so introduce lock to protect between the writers, ie, bh's adders and deleter. The lock only affects the writers and bh's callback does not take this extra lock. Note that for the same AioContext, aio_bh_poll() can not run in parallel yet. Signed-off-by: Liu Ping

Re: [Qemu-devel] [PATCH 0/5] tcg-arm: Runtime detection of architecture

2013-06-24 Thread Richard Henderson
Ping 3. On 06/17/2013 08:56 AM, Richard Henderson wrote: > Ping. > > On 06/10/2013 11:42 AM, Richard Henderson wrote: >> Ping. >> >> >> On 06/06/2013 11:05 AM, Richard Henderson wrote: >>> Two prepatory generic tcg patches, to allow non-constant values for >>> the various TCG_TARGET_HAS_foo macro

Re: [Qemu-devel] [PATCH v2 0/4] tcg-arm: Implement tcg_register_jit

2013-06-24 Thread Richard Henderson
Ping 3. On 06/17/2013 08:54 AM, Richard Henderson wrote: > Ping. > > > On 06/10/2013 11:41 AM, Richard Henderson wrote: >> Ping. >> >> >> On 06/05/2013 10:29 AM, Richard Henderson wrote: >>> Changes v1-v2: >>> The suggestions for improvement I got from round 1 apply to all >>> of the hosts, not

Re: [Qemu-devel] [PATCH 1/4] tcg: Split rem requirement from div requirement

2013-06-24 Thread Richard Henderson
Ping 3. Pretty please? On 06/17/2013 08:53 AM, Richard Henderson wrote: > Ping. > > On 06/10/2013 11:39 AM, Richard Henderson wrote: >> Ping. >> >> On 05/30/2013 10:53 AM, Richard Henderson wrote: >>> There are several hosts with only a "div" insn. Remainder is computed >>> manually from the qu

Re: [Qemu-devel] [PATCH 21/26] kvmclock: use realize for kvmclock

2013-06-24 Thread Hu Tao
On Mon, Jun 24, 2013 at 11:01:54AM -0300, Eduardo Habkost wrote: > On Sat, Jun 22, 2013 at 04:50:33PM +0800, Hu Tao wrote: > > Cc: qemu-devel@nongnu.org > > Cc: "Andreas Färber" > > Cc: Paolo Bonzini > > Cc: Anthony Liguori > > Cc: Igor Mammedov > > Cc: Eduardo Habkost > > Signed-off-by: Hu Ta

[Qemu-devel] [PATCH v11 13/15] rdma: allow state transitions between other states besides ACTIVE

2013-06-24 Thread mrhines
From: "Michael R. Hines" This patch is in preparation for the next ones: Until now the MIG_STATE_SETUP state was not really a 'formal' state. It has been used as a 'zero' state and QEMU has been unconditionally transitioning into this state when the QMP migrate command was called. In preparation

[Qemu-devel] [PATCH v11 10/15] rdma: introduce capability x-rdma-pin-all

2013-06-24 Thread mrhines
From: "Michael R. Hines" This capability allows you to disable dynamic chunk registration for better throughput on high-performance links. For example, using an 8GB RAM virtual machine with all 8GB of memory in active use and the VM itself is completely idle using a 40 gbps infiniband link: 1.

[Qemu-devel] [PATCH v11 15/15] rdma: account for the time spent in MIG_STATE_SETUP through QMP

2013-06-24 Thread mrhines
From: "Michael R. Hines" Using the previous patches, we're now able to timestamp the SETUP state. Once we have this time, let the user know about it in the schema. Signed-off-by: Michael R. Hines --- hmp.c |4 include/migration/migration.h |1 + migration.c

[Qemu-devel] [PATCH v11 12/15] rdma: send pc.ram

2013-06-24 Thread mrhines
From: "Michael R. Hines" This takes advantages of the previous patches: 1. use the new QEMUFileOps hook 'save_page' 2. call out to the right accessor methods to invoke the iteration hooks defined in QEMUFileOps Reviewed-by: Paolo Bonzini Reviewed-by: Chegu Vinod Tested-by: Chegu Vinod Te

[Qemu-devel] [PATCH v11 06/15] rdma: export qemu_fflush()

2013-06-24 Thread mrhines
From: "Michael R. Hines" RDMA uses this to flush the control channel before sending its own message to handle page registrations. Reviewed-by: Paolo Bonzini Reviewed-by: Chegu Vinod Tested-by: Chegu Vinod Tested-by: Michael R. Hines Signed-off-by: Michael R. Hines --- include/migration/qem

[Qemu-devel] [PATCH v11 07/15] rdma: introduce ram_handle_compressed()

2013-06-24 Thread mrhines
From: "Michael R. Hines" This gives RDMA shared access to madvise() on the destination side when an entire chunk is found to be zero. Reviewed-by: Paolo Bonzini Reviewed-by: Chegu Vinod Tested-by: Chegu Vinod Tested-by: Michael R. Hines Signed-off-by: Michael R. Hines --- arch_init.c

[Qemu-devel] [PATCH v11 03/15] rdma: export yield_until_fd_readable()

2013-06-24 Thread mrhines
From: "Michael R. Hines" The RDMA event channel can be made non-blocking just like a TCP socket. Exporting this function allows us to yield so that the QEMU monitor remains available. Reviewed-by: Paolo Bonzini Reviewed-by: Chegu Vinod Tested-by: Chegu Vinod Tested-by: Michael R. Hines Signe

[Qemu-devel] [PATCH v11 09/15] rdma: new QEMUFileOps hooks

2013-06-24 Thread mrhines
From: "Michael R. Hines" These are the prototypes and implementation of new hooks that RDMA takes advantage of to perform dynamic page registration. An optional hook is also introduced for a custom function to be able to override the default save_page function. Also included are the prototypes

[Qemu-devel] [PATCH v11 08/15] rdma: introduce qemu_ram_foreach_block()

2013-06-24 Thread mrhines
From: "Michael R. Hines" This is used during RDMA initialization in order to transmit a description of all the RAM blocks to the peer for later dynamic chunk registration purposes. Reviewed-by: Paolo Bonzini Reviewed-by: Chegu Vinod Tested-by: Chegu Vinod Tested-by: Michael R. Hines Signed-o

[Qemu-devel] [PATCH v11 00/15] rdma: migration support

2013-06-24 Thread mrhines
From: "Michael R. Hines" Changes since v10: Detailed breakout of required state machine changes - Fix tags. Only #13, 14, and 15 need reviews. Rest are completed. - Patch #13: allow state transitions between other states besides ACTIVE - Patch #14: introduce MIG_STATE_NONE and change MIG_STATE_S

[Qemu-devel] [PATCH v11 01/15] rdma: add documentation

2013-06-24 Thread mrhines
From: "Michael R. Hines" docs/rdma.txt contains full documentation, wiki links, github url and contact information. Reviewed-by: Paolo Bonzini Reviewed-by: Chegu Vinod Tested-by: Chegu Vinod Tested-by: Michael R. Hines Signed-off-by: Michael R. Hines --- docs/rdma.txt | 415 ++

[Qemu-devel] [PATCH v11 04/15] rdma: export throughput w/ MigrationStats QMP

2013-06-24 Thread mrhines
From: "Michael R. Hines" This exposes throughput (in megabits/sec) through QMP. Reviewed-by: Paolo Bonzini Reviewed-by: Chegu Vinod Tested-by: Chegu Vinod Tested-by: Michael R. Hines Signed-off-by: Michael R. Hines --- hmp.c |8 ++-- include/migration/migrat

[Qemu-devel] [PATCH v11 14/15] rdma: introduce MIG_STATE_NONE and change MIG_STATE_SETUP state transition

2013-06-24 Thread mrhines
From: "Michael R. Hines" As described in the previous patch, until now, the MIG_STATE_SETUP state was not really a 'formal' state. It has been used as a 'zero' state (what we're calling 'NONE' here) and QEMU has been unconditionally transitioning into this state when the QMP migration command was

[Qemu-devel] [PATCH v11 02/15] rdma: introduce qemu_update_position()

2013-06-24 Thread mrhines
From: "Michael R. Hines" RDMA writes happen asynchronously, and thus the performance accounting also needs to be able to occur asynchronously. This allows anybody to call into savevm.c to update both f->pos as well as into arch_init.c to update the acct_info structure with up-to-date values when

[Qemu-devel] [PATCH v11 05/15] rdma: introduce qemu_file_mode_is_not_valid()

2013-06-24 Thread mrhines
From: "Michael R. Hines" QEMUFileRDMA also has read and write modes. This function is now shared to reduce code duplication. Reviewed-by: Paolo Bonzini Reviewed-by: Chegu Vinod Tested-by: Chegu Vinod Tested-by: Michael R. Hines Signed-off-by: Michael R. Hines --- include/migration/qemu-fil

Re: [Qemu-devel] [PATCH 21/26] kvmclock: use realize for kvmclock

2013-06-24 Thread Hu Tao
On Mon, Jun 24, 2013 at 12:14:07PM +0200, Igor Mammedov wrote: > On Sat, 22 Jun 2013 16:50:33 +0800 > Hu Tao wrote: > > > Cc: qemu-devel@nongnu.org > > Cc: "Andreas Färber" > > Cc: Paolo Bonzini > > Cc: Anthony Liguori > > Cc: Igor Mammedov > > Cc: Eduardo Habkost > > Signed-off-by: Hu Tao

Re: [Qemu-devel] [PATCH 01/26] ohci: use realize for ohci

2013-06-24 Thread Hu Tao
On Mon, Jun 24, 2013 at 04:17:28PM +1000, Peter Crosthwaite wrote: > Hi Hu, > > On Mon, Jun 24, 2013 at 4:11 PM, Hu Tao wrote: > > On Mon, Jun 24, 2013 at 03:54:31PM +1000, Peter Crosthwaite wrote: > >> Hi Hu, > >> > >> On Sat, Jun 22, 2013 at 6:50 PM, Hu Tao wrote: > >> > Cc: Gerd Hoffmann > >

[Qemu-devel] [PATCH 0/2] PPC: Newworld: Some uni-n hacks

2013-06-24 Thread Alexander Graf
Mac OS X has some internal insight into how a core99 style Mac is supposed to work. It relies on a few bits in the uni-north topology to exist. These patches fake things well enough to make at least 10.4 happy. With these as well as the previously sent patches and a few workarounds in the guest k

[Qemu-devel] [PATCH 1/2] PPC: Newworld: Add uninorth token register

2013-06-24 Thread Alexander Graf
Mac OS X expects the uninorth control register set to contain one register that always reads back what it writes in. Expose that. This is just a temporary hack. Eventually, we want to expose the uninorth (/uni-n in device tree) as a separate QOM device. Signed-off-by: Alexander Graf --- hw/ppc/

[Qemu-devel] [PATCH 2/2] PPC: Newworld: Add second uninorth control register set

2013-06-24 Thread Alexander Graf
Mac OS X requires a second uninorth register set to be mapped a few bytes above the first one. Let's just expose it to make it happy. Signed-off-by: Alexander Graf --- hw/ppc/mac_newworld.c | 4 1 file changed, 4 insertions(+) diff --git a/hw/ppc/mac_newworld.c b/hw/ppc/mac_newworld.c inde

Re: [Qemu-devel] Balloon Driver : Observation

2013-06-24 Thread Anthony Liguori
On Mon, Jun 24, 2013 at 8:31 PM, Saptarshi Sen wrote: > Hi, > > I am observing a strange phenomenon with balloon using KVM as hypervisor. > I set balloon value to an arbitratrilylow value so that the guest system is > out of memory. The VM freezes therafter. But if Iquery the balloon status > th

Re: [Qemu-devel] [PATCH V2] build: remove compile warning

2013-06-24 Thread Wenchao Xia
于 2013-6-24 22:44, Paolo Bonzini 写道: Il 22/06/2013 12:03, Stefan Weil ha scritto: Am 18.06.2013 12:13, schrieb Paolo Bonzini: Il 07/06/2013 14:17, Markus Armbruster ha scritto: diff --git a/util/iov.c b/util/iov.c index cc6e837..b91cfb9 100644 --- a/util/iov.c +++ b/util/iov.c @@ -146,7 +146,7

[Qemu-devel] Balloon Driver : Observation

2013-06-24 Thread Saptarshi Sen
Hi, I am observing a strange phenomenon with balloon using KVM as hypervisor. I set balloon value to an arbitratrilylow value so that the guest system is out of memory. The VM freezes therafter. But if Iquery the balloon status through qmp-shell, the qmp interface interacts with the balloon targ

[Qemu-devel] [PATCH] PPC: g3beige: Move secondary IDE bus to mac-io

2013-06-24 Thread Alexander Graf
On a real G3 Beige the secondary IDE bus lives on the mac-io chip, not on some random PCI device. Move it there to become more compatible. While at it, also clean up the IDE channel connection logic. Signed-off-by: Alexander Graf --- hw/ide/macio.c| 2 +- hw/misc/macio/macio.c | 95 +++

Re: [Qemu-devel] [PATCH] tls.h: Enable TLS on FreeBSD

2013-06-24 Thread Peter Maydell
On 24 June 2013 22:15, Paolo Bonzini wrote: > Il 24/06/2013 21:21, Ed Maste ha scritto: >> Signed-off-by: Ed Maste >> --- >> I have had this in a local tree for some time, and it is needed by the >> BSD-user work that is now being proposed. > > At this time, qemu/tls.h is really just for cpu_sing

Re: [Qemu-devel] [PATCH v10 14/14, resend] rdma: add setup time accounting to QMP statistics

2013-06-24 Thread Michael R. Hines
On 06/24/2013 10:41 AM, Anthony Liguori wrote: "Michael R. Hines" writes: On 06/24/2013 09:51 AM, Paolo Bonzini wrote: Il 24/06/2013 15:46, mrhi...@linux.vnet.ibm.com ha scritto: From: "Michael R. Hines" For very large virtual machines, pinning can take a long time. While this does not aff

[Qemu-devel] [PATCH v3] e600 core for MPC86xx processors

2013-06-24 Thread Julio Guerra
MPC86xx processors are based on the e600 core, which is not the case in qemu where it is based on the 7400 processor. This patch creates the e600 core and instantiates the MPC86xx processors based on it. Therefore, adding the high BATs, the SPRG 4..7 registers, which are e600-specific [1], and a H

Re: [Qemu-devel] Where to find the definition of RunState, RUN_STATE_IMMIGRATE

2013-06-24 Thread Paolo Bonzini
Il 24/06/2013 19:06, Yaodong Yang ha scritto: > Hi all, > > I'm a newer in QEMU-KVM. When I read the source code of > qemu-kvm-1.2.0, I can not find the definition of some types, like > RunState (in Vl.c file). I also checked the kvm-kmod-3.5, but I still > can not find the definition. Could anyon

Re: [Qemu-devel] [PATCH] tls.h: Enable TLS on FreeBSD

2013-06-24 Thread Paolo Bonzini
Il 24/06/2013 21:21, Ed Maste ha scritto: > Signed-off-by: Ed Maste > --- > I have had this in a local tree for some time, and it is needed by the > BSD-user work that is now being proposed. At this time, qemu/tls.h is really just for cpu_single_env, so I think this patch should be applied togeth

Re: [Qemu-devel] [PATCH v4 00/10] qemu-ga: fsfreeze on Windows using VSS

2013-06-24 Thread Laszlo Ersek
On 06/24/13 21:30, Tomoki Sekiyama wrote: > Any other comments for this series? It's been on my todo list since you posted it :( I was pleased by the design / summary you wrote up for my v3 questions, but I didn't trust myself to give an R-b based on "just" that. I'll try to do something about i

Re: [Qemu-devel] [PATCH 2/3] spapr: Fix compiler warning for some versions of gcc (h_remove)

2013-06-24 Thread Paolo Bonzini
Il 24/06/2013 22:35, Stefan Weil ha scritto: > Am 24.06.2013 22:27, schrieb Paolo Bonzini: >> Il 24/06/2013 19:48, Stefan Weil ha scritto: >>> i686-w64-mingw32-gcc (GCC) 4.6.3 from Debian wheezy reports this warning: >>> >>> hw/ppc/spapr_hcall.c:188:1: warning: >>> control reaches end of non-void

[Qemu-devel] Where to find the definition of RunState, RUN_STATE_IMMIGRATE

2013-06-24 Thread Yaodong Yang
Hi all, I'm a newer in QEMU-KVM. When I read the source code of qemu-kvm-1.2.0, I can not find the definition of some types, like RunState (in Vl.c file). I also checked the kvm-kmod-3.5, but I still can not find the definition. Could anyone give me some hints about where to find it. Thanks in

Re: [Qemu-devel] [PATCH 22/23] bsd-user: add more strace formating

2013-06-24 Thread Stacey Son
On Jun 24, 2013, at 12:41 PM, Peter Maydell wrote: > On 24 June 2013 03:03, Stacey Son wrote: >> This change add more strace formating for popular system calls. It also >> separates out *BSD and architecture dependent code. In addition, it >> changes TARGET_OS in configure (and in the associat

Re: [Qemu-devel] [PATCH 00/23] bsd-user: FreeBSD support for mips/mips64 and arm

2013-06-24 Thread Stacey Son
On Jun 24, 2013, at 3:07 PM, Anthony Liguori wrote: > Stacey Son writes: > >> On Jun 24, 2013, at 12:55 PM, Anthony Liguori wrote: >> >>> There is no humanly way 23k LOC can be reviewed in a single series. >> >> Yes, indeed, it is a lot of code. >> >>> Has this been maintained out of tree fo

Re: [Qemu-devel] [PATCH 17/23] bsd-user: add shims for thread related system calls

2013-06-24 Thread Stacey Son
On Jun 24, 2013, at 12:37 PM, Peter Maydell wrote: > On 24 June 2013 03:03, Stacey Son wrote: >> diff --git a/configure b/configure >> index ba90975..812ea3f 100755 >> --- a/configure >> +++ b/configure >> @@ -1438,6 +1438,15 @@ fi >> >> if test "$nptl" != "no" ; then >> cat > $TMPC <> +#ifde

[Qemu-devel] [PATCH] tls.h: Enable TLS on FreeBSD

2013-06-24 Thread Ed Maste
Signed-off-by: Ed Maste --- I have had this in a local tree for some time, and it is needed by the BSD-user work that is now being proposed. As an aside, an abstraction was recently proposed for Open vSwtich that can use any of _Thread_local, __thread, or pthread_getspecific() which may make a co

Re: [Qemu-devel] [PATCH 2/3] spapr: Fix compiler warning for some versions of gcc (h_remove)

2013-06-24 Thread Stefan Weil
Am 24.06.2013 22:27, schrieb Paolo Bonzini: > Il 24/06/2013 19:48, Stefan Weil ha scritto: >> i686-w64-mingw32-gcc (GCC) 4.6.3 from Debian wheezy reports this warning: >> >> hw/ppc/spapr_hcall.c:188:1: warning: >> control reaches end of non-void function [-Wreturn-type] >> >> Replacing the 4th cas

Re: [Qemu-devel] [PATCH 2/3] spapr: Fix compiler warning for some versions of gcc (h_remove)

2013-06-24 Thread Paolo Bonzini
Il 24/06/2013 19:48, Stefan Weil ha scritto: > i686-w64-mingw32-gcc (GCC) 4.6.3 from Debian wheezy reports this warning: > > hw/ppc/spapr_hcall.c:188:1: warning: > control reaches end of non-void function [-Wreturn-type] > > Replacing the 4th case REMOVE_HW (which is currently unused) by the def

[Qemu-devel] [RFC] i386: cpu: Always report power-of-2 maximum core IDs

2013-06-24 Thread Eduardo Habkost
(Note: as this patch is just a RFC, machine-type compatibility code is not included yet, as I first want to check if the solution I have implemented is acceptable) This is a bit tricky, so I am sending this as an RFC to get some feedback. Maybe somebody from Intel could help us figure this out?

Re: [Qemu-devel] [PATCH 03/23] bsd-user: additional seperation of OS and architecture dependent code

2013-06-24 Thread Stacey Son
On Jun 24, 2013, at 12:24 PM, Peter Maydell wrote: > On 24 June 2013 03:03, Stacey Son wrote: >> diff --git a/configure b/configure >> index ad32f87..749eba8 100755 >> --- a/configure >> +++ b/configure >> @@ -432,6 +432,7 @@ if test -z "$ARCH"; then >> fi >> >> # OS specific >> +TARGET_OS="" >

Re: [Qemu-devel] [PATCH 02/23] bsd-user: add initial support for mips/mips64

2013-06-24 Thread Stacey Son
On Jun 24, 2013, at 12:15 PM, Peter Maydell wrote: >> >> +#if defined(TARGET_MIPS) >> + >> +/* Compare to sys/mips/mips/trap.c */ >> + >> +void cpu_loop(CPUMIPSState *env) > > I suspect you'd do better in the long term to restructure > to pull cpu_loop out into a per-architecture source file >

Re: [Qemu-devel] [PATCH 00/23] bsd-user: FreeBSD support for mips/mips64 and arm

2013-06-24 Thread Anthony Liguori
Stacey Son writes: > On Jun 24, 2013, at 12:55 PM, Anthony Liguori wrote: > >> There is no humanly way 23k LOC can be reviewed in a single series. > > Yes, indeed, it is a lot of code. > >> Has this been maintained out of tree for some time? > > Yes, it has been a large, ugly patch set in the Fre

Re: [Qemu-devel] [PATCH v4 00/10] qemu-ga: fsfreeze on Windows using VSS

2013-06-24 Thread Tomoki Sekiyama
Any other comments for this series? On 6/6/13 11:06 , "Tomoki Sekiyama" wrote: >Hi, > >This patch series adds fsfreeze support for Windows qemu-guest-agent. > >changes from v3: > -[01/10] Use c++ instead of g++ in configureing C++ compiler if neither > $cross_prefix nor $CXX is specifie

[Qemu-devel] [Bug 1191606] Re: qemu crashes with iscsi initiator (libiscsi) when using virtio

2013-06-24 Thread Laszlo Ersek (Red Hat)
http://lists.nongnu.org/archive/html/qemu-devel/2013-06/msg03987.html ** Changed in: qemu Status: New => In Progress -- You received this bug notification because you are a member of qemu- devel-ml, which is subscribed to QEMU. https://bugs.launchpad.net/bugs/1191606 Title: qemu crashe

Re: [Qemu-devel] [PATCH v3] configure: Simplify alternate .text segment

2013-06-24 Thread Ed Maste
On 21 June 2013 22:10, Richard Henderson wrote: > For bsd-user and linux-user emulation modes QEMU needs to be linked at an > alternate .text segment address, so that it's out of the way of the guest > executable. Instead of including modified linker scripts for each arch, > just set the address

Re: [Qemu-devel] [PATCH 00/23] bsd-user: FreeBSD support for mips/mips64 and arm

2013-06-24 Thread Stacey Son
On Jun 24, 2013, at 12:55 PM, Anthony Liguori wrote: > There is no humanly way 23k LOC can be reviewed in a single series. Yes, indeed, it is a lot of code. > Has this been maintained out of tree for some time? Yes, it has been a large, ugly patch set in the FreeBSD qemu-devel port for some t

Re: [Qemu-devel] [PATCH 00/23] bsd-user: FreeBSD support for mips/mips64 and arm

2013-06-24 Thread Stacey Son
On Jun 24, 2013, at 12:49 PM, Peter Maydell wrote: > On 24 June 2013 03:03, Stacey Son wrote: >> This patch series add BSD user mode support for FreeBSD. In addition, >> it adds the necessary architecture dependent code for the mips, mips64, >> and arm targets. In its current state it can emul

Re: [Qemu-devel] [PATCH V3 06/10] NUMA: split out the common range parser

2013-06-24 Thread Bandan Das
Wanlong Gao writes: > Since cpus parser and hostnode parser have the common range parser > part, split it out to the common range parser to avoid the duplicate > code. > > Signed-off-by: Wanlong Gao > --- > vl.c | 89 > > 1 f

Re: [Qemu-devel] [PATCH V3 04/10] NUMA: parse guest numa nodes memory policy

2013-06-24 Thread Bandan Das
Wanlong Gao writes: > The memory policy setting format is like: > mem-policy={membind|interleave|preferred},mem-hostnode=[+|!]{all|N-N} > And we are adding this setting as a suboption of "-numa", > the memory policy then can be set like following: > -numa node,nodeid=0,mem=1024,cpus=0,mem-policy

[Qemu-devel] SPARC Leon3 Devices - Registers?

2013-06-24 Thread Kaila Thornton
I'm currently working on trying to add more devices to the SPARC Leon3 machine, but I'm running into a few problems. At the moment, I'm running QEMU on an i386 machine and targeting the generic leon3 board. I need to write some code that allows QEMU to gain access to the SPARC registers as my

Re: [Qemu-devel] [PATCH 6/8] qemu-img: use bdrv_write_zeroes to write zeroes

2013-06-24 Thread Peter Lieven
Am 24.06.2013 18:25, schrieb Paolo Bonzini: > Il 24/06/2013 18:17, Peter Lieven ha scritto: >> Am 24.06.2013 16:33, schrieb Paolo Bonzini: >>> Il 22/06/2013 22:58, Peter Lieven ha scritto: Signed-off-by: Peter Lieven --- qemu-img.c | 10 +++--- 1 file changed, 7 insertio

[Qemu-devel] [PATCH 21/23] bsd-user: add miscellaneous system call shims

2013-06-24 Thread Stacey Son
This change add System V semaphores/messages, uuid, FreeBSD cpu scheduling/affinity management, and other miscellaneous system call shims. A lot of these other system call shims are stubs for maybe future work. Signed-off-by: Stacey Son --- bsd-user/Makefile.objs |2 +- bsd-user/bsd-misc

[Qemu-devel] [PATCH 13/23] bsd-user: add shims for signal related system calls

2013-06-24 Thread Stacey Son
This change adds support for signal related system calls including sigaction(2), sigprocmask(2), sigpending(2), sigsuspend(2), and so on. The older, obsolete, system calls such as sigvec(2), sigblock(2), sigsetmask(2), and sigstack(2) are not supported. Signed-off-by: Stacey Son --- bsd-user/bs

[Qemu-devel] [PATCH 06/23] bsd-user: fix thread initialization and ELF addresses for mips/mips64

2013-06-24 Thread Stacey Son
Initialize all the registers correctly for mips/mips64 in init_thread(), use the correct ELF_START_MMAP for mips64, use the correct run-time linker, and clean up the code by eliminating some #if's. Also, fix all the checkpatch.pl warnings and errors. Signed-off-by: Stacey Son --- bsd-user/elflo

Re: [Qemu-devel] [PATCH V2] build: remove compile warning

2013-06-24 Thread Stefan Weil
Am 24.06.2013 16:44, schrieb Paolo Bonzini: > Il 22/06/2013 12:03, Stefan Weil ha scritto: >> I get this warning, too, when I run a normal cross compilation with >> MinGW-w64: >> >> util/iov.c:190:33: warning: ‘orig_len’ may be used uninitialized in this >> function [-Wuninitialized] >> >> My build

Re: [Qemu-devel] [PATCH 00/23] bsd-user: FreeBSD support for mips/mips64 and arm

2013-06-24 Thread Anthony Liguori
Stacey Son writes: > Hi all, > > This patch series add BSD user mode support for FreeBSD. In addition, > it adds the necessary architecture dependent code for the mips, mips64, > and arm targets. In its current state it can emulate most mips/mips64 > and arm target binaries on a x86 host in a s

[Qemu-devel] [PATCH 05/23] bsd-user: add bsd_binprm to TaskState for core dumping emulation

2013-06-24 Thread Stacey Son
Add the bsd_binprm (formerly linux_binprm) structure to TaskState so it can be used to support core dumping emulation support in the future. Also, make freebsd the default bsd_type, if compiled on a FreeBSD system. Signed-off-by: Stacey Son --- bsd-user/bsdload.c | 36 ++---

Re: [Qemu-devel] [PATCH 00/23] bsd-user: FreeBSD support for mips/mips64 and arm

2013-06-24 Thread Peter Maydell
On 24 June 2013 03:03, Stacey Son wrote: > This patch series add BSD user mode support for FreeBSD. In addition, > it adds the necessary architecture dependent code for the mips, mips64, > and arm targets. In its current state it can emulate most mips/mips64 > and arm target binaries on a x86 ho

[Qemu-devel] [PATCH 2/3] spapr: Fix compiler warning for some versions of gcc (h_remove)

2013-06-24 Thread Stefan Weil
i686-w64-mingw32-gcc (GCC) 4.6.3 from Debian wheezy reports this warning: hw/ppc/spapr_hcall.c:188:1: warning: control reaches end of non-void function [-Wreturn-type] Replacing the 4th case REMOVE_HW (which is currently unused) by the default case fixes this warning. The assertion is dead code

[Qemu-devel] [PATCH 3/3] spapr: Fix compiler warning for some versions of gcc (spapr_io_read)

2013-06-24 Thread Stefan Weil
i686-w64-mingw32-gcc (GCC) 4.6.3 from Debian wheezy reports this warning: hw/ppc/spapr_pci.c:454:1: warning: control reaches end of non-void function [-Wreturn-type] Adding a default case to the switch statement satisfies the compiler. This modification requires moving the assert statement. Sig

[Qemu-devel] [PATCH 0/3] spapr: Fix remaining compiler warnings

2013-06-24 Thread Stefan Weil
Hi Alex, this mini series fixes the remaining compiler warnings in my w32/w64 cross build environment on Debian wheezy. [PATCH 1/3] spapr: Use named enum for function remove_hpte [PATCH 2/3] spapr: Fix compiler warning for some versions of gcc [PATCH 3/3] spapr: Fix compiler warning for some vers

[Qemu-devel] [PATCH 1/3] spapr: Use named enum for function remove_hpte

2013-06-24 Thread Stefan Weil
The function returned a target_ulong which was made from unnamed enum values. The target_ulong was then assigned to an int variable which was used in a switch statement. Using a named enum in both cases makes reviews easier. Signed-off-by: Stefan Weil --- hw/ppc/spapr_hcall.c |8 1

Re: [Qemu-devel] [PATCH 22/23] bsd-user: add more strace formating

2013-06-24 Thread Peter Maydell
On 24 June 2013 03:03, Stacey Son wrote: > This change add more strace formating for popular system calls. It also > separates out *BSD and architecture dependent code. In addition, it > changes TARGET_OS in configure (and in the associated makefiles) to > what it should really be: HOST_OS. You

Re: [Qemu-devel] [PATCH 17/23] bsd-user: add shims for thread related system calls

2013-06-24 Thread Peter Maydell
On 24 June 2013 03:03, Stacey Son wrote: > diff --git a/configure b/configure > index ba90975..812ea3f 100755 > --- a/configure > +++ b/configure > @@ -1438,6 +1438,15 @@ fi > > if test "$nptl" != "no" ; then >cat > $TMPC < +#ifdef __FreeBSD__ > +#include > +int main(void) { > +#if !defined(

Re: [Qemu-devel] [PATCH 04/15] target-i386: cpu: convert 'level' to static property

2013-06-24 Thread Eduardo Habkost
On Wed, Jun 05, 2013 at 03:18:35PM +0200, Igor Mammedov wrote: > Signed-off-by: Igor Mammedov Reviewed-by: Eduardo Habkost > --- > target-i386/cpu.c | 20 +--- > 1 files changed, 1 insertions(+), 19 deletions(-) > > diff --git a/target-i386/cpu.c b/target-i386/cpu.c > index

Re: [Qemu-devel] [PATCH 10/23] bsd-user: add shims for memory management related syscalls

2013-06-24 Thread Peter Maydell
On 24 June 2013 03:03, Stacey Son wrote: > This change adds support for bsd memory management system calls including > mmap(2), munmap(2), mprotect(2), msync(2), etc. Also, it adds shared memory > management system calls. In addition, cleans up syscall.c a bit and fixes > checkpatch.pl warnings a

Re: [Qemu-devel] [PATCH 03/23] bsd-user: additional seperation of OS and architecture dependent code

2013-06-24 Thread Peter Maydell
On 24 June 2013 03:03, Stacey Son wrote: > diff --git a/configure b/configure > index ad32f87..749eba8 100755 > --- a/configure > +++ b/configure > @@ -432,6 +432,7 @@ if test -z "$ARCH"; then > fi > > # OS specific > +TARGET_OS="" > > case $targetos in > CYGWIN*) > @@ -457,6 +458,7 @@ FreeBSD

[Qemu-devel] [PATCH 07/23] bsd-user: find target executable in path when absolute path not given

2013-06-24 Thread Stacey Son
If the target executable's path is not absolute then this code will search the PATH to find it. Save the fullpath to put on to the stack for the runtime linker. Signed-off-by: Stacey Son --- bsd-user/bsdload.c | 84 +-- bsd-user/qemu.h|3

Re: [Qemu-devel] [PATCH 03/23] bsd-user: additional seperation of OS and architecture dependent code

2013-06-24 Thread Peter Maydell
On 24 June 2013 03:03, Stacey Son wrote: > This change moves more OS dependent code into *bsd directories. Also moves > architecture dependent code into the various arch directories. Add to > 'configure' $TARGET_OS so the path to the target OS can be include file search > path in the Makefile. >

Re: [Qemu-devel] [PATCH v10 14/14, resend] rdma: add setup time accounting to QMP statistics

2013-06-24 Thread Michael R. Hines
On 06/24/2013 10:02 AM, Paolo Bonzini wrote: Il 24/06/2013 15:55, Michael R. Hines ha scritto: Reviewed-by: Paolo Bonzini Please stop inventing Reviewed-by tags, or I will stop reviewing your patches. Paolo Inventing? I don't understand. I accumulated all of those tags from everybody - copy

Re: [Qemu-devel] [PATCH 23/23] bsd-user: add arm cpu support

2013-06-24 Thread Peter Maydell
On 24 June 2013 03:03, Stacey Son wrote: > This change adds support for the arm cpu target. In addition, it moves the > cpu dependent code out of main.c into the individual architecture directories > and cleans up main.c in general. Credit for much of this code: Olivier > Houchard. Rule of thum

Re: [Qemu-devel] [PATCH 02/23] bsd-user: add initial support for mips/mips64

2013-06-24 Thread Peter Maydell
On 24 June 2013 03:03, Stacey Son wrote: > Add the main cpu loop, cpu_loop(), for mips and mips64 architecture. Set the > cpu model. Add some stubs for future code. > > --- a/bsd-user/main.c > +++ b/bsd-user/main.c > @@ -2,6 +2,7 @@ > * qemu user main > * > * Copyright (c) 2003-2008 Fabri

[Qemu-devel] [PATCH 19/23] bsd-user: add shims for sysarch() and sysctl() system calls

2013-06-24 Thread Stacey Son
This change adds support for sysarch() and sysctl() system call emulation. sysarch() is both architecture and OS dependent. Therefore this change adds a handler for each architecture. sysctl() has a lot special cases that have to each decoded and handled individually. Signed-off-by: Stacey Son

Re: [Qemu-devel] [PATCH 07/15] target-i386: cpu: convert 'model-id' to static property

2013-06-24 Thread Eduardo Habkost
On Wed, Jun 05, 2013 at 03:18:38PM +0200, Igor Mammedov wrote: > * check "if (model_id == NULL)" looks unnecessary now, since all > builtin model-ids are not NULL and user shouldn't be able to set > it NULL (cpumodel string parsing code takes care of it, if feature > is specified as "model-id=" on

Re: [Qemu-devel] [PATCH 06/15] target-i386: cpu: convert 'vendor' to static property

2013-06-24 Thread Eduardo Habkost
On Wed, Jun 05, 2013 at 03:18:37PM +0200, Igor Mammedov wrote: > Signed-off-by: Igor Mammedov Reviewed-by: Eduardo Habkost > --- > target-i386/cpu.c | 31 --- > 1 files changed, 24 insertions(+), 7 deletions(-) > > diff --git a/target-i386/cpu.c b/target-i386/cpu

Re: [Qemu-devel] [PATCH 08/15] target-i386: cpu: convert 'tsc-frequency' to static property

2013-06-24 Thread Eduardo Habkost
On Wed, Jun 05, 2013 at 03:18:39PM +0200, Igor Mammedov wrote: > Signed-off-by: Igor Mammedov > --- > target-i386/cpu.c | 12 +--- > 1 files changed, 9 insertions(+), 3 deletions(-) > > diff --git a/target-i386/cpu.c b/target-i386/cpu.c > index 9f6fe06..ec6d33f 100644 > --- a/target-i3

Re: [Qemu-devel] [PATCH] RFC kvm irqfd: add directly mapped MSI IRQ support

2013-06-24 Thread Gleb Natapov
On Mon, Jun 24, 2013 at 10:17:41AM -0500, Anthony Liguori wrote: > Alexander Graf writes: > > > On 24.06.2013, at 16:31, Anthony Liguori wrote: > > > >> "Michael S. Tsirkin" writes: > >> > >>> On Mon, Jun 24, 2013 at 08:34:52AM -0500, Anthony Liguori wrote: > Gleb Natapov writes: >

[Qemu-devel] [PATCH 04/23] bsd-user: add bsd signal emulation

2013-06-24 Thread Stacey Son
Add bsd-user signal emulation code, fix name space confict with sigqueue, add arch dependent code for mips/mips64, and OS dependent definitions for FreeBSD. Signed-off-by: Stacey Son --- bsd-user/freebsd/target_os_signal.h |9 + bsd-user/i386/target_arch_signal.h| 39 ++ bsd-user/i38

[Qemu-devel] [PATCH 15/23] bsd-user: add shims for socket related system calls

2013-06-24 Thread Stacey Son
This change adds support for socket related system calls including socket, bind, accept, getsockopt, setsocketopt, etc. Like the other changes to add shims for system calls some are FreeBSD dependent. These shims are defined in freebsd/os-socket.h. Stubs to emulate these on other *BSDs are includ

[Qemu-devel] [PATCH 20/23] bsd-user: add shims for extended attributes system calls

2013-06-24 Thread Stacey Son
This change adds support for the extended attribute and access control list system calls. While NetBSD has the same extended attribute system calls it is unclear if OpenBSD does. Therefore, this calls were added as FreeBSD-only. Signed-off-by: Stacey Son --- bsd-user/Makefile.objs|2

Re: [Qemu-devel] [PATCH 02/15] target-i386: cpu: convert 'model' to static property

2013-06-24 Thread Eduardo Habkost
On Wed, Jun 05, 2013 at 03:18:33PM +0200, Igor Mammedov wrote: > Signed-off-by: Igor Mammedov Reviewed-by: Eduardo Habkost > --- > target-i386/cpu.c | 12 +--- > 1 files changed, 9 insertions(+), 3 deletions(-) > > diff --git a/target-i386/cpu.c b/target-i386/cpu.c > index c87cc9f..

[Qemu-devel] [PATCH 16/23] bsd-user: add shims for stat and file handle related syscalls

2013-06-24 Thread Stacey Son
This change adds support for status, file handle, and fcntl related system calls including stat(), statfs(), fhstatfs(), fcntl() and the many variants. Signed-off-by: Stacey Son --- bsd-user/Makefile.objs |3 +- bsd-user/freebsd/os-stat.c | 234 +++ bsd-user/freebsd/

[Qemu-devel] [PATCH 22/23] bsd-user: add more strace formating

2013-06-24 Thread Stacey Son
This change add more strace formating for popular system calls. It also separates out *BSD and architecture dependent code. In addition, it changes TARGET_OS in configure (and in the associated makefiles) to what it should really be: HOST_OS. Signed-off-by: Stacey Son --- Makefile.target

[Qemu-devel] [PATCH 09/23] bsd-user: refresh FreeBSD's system call numbers

2013-06-24 Thread Stacey Son
Update freebsd/syscall_nr.h with FreeBSD's latest system call numbers from -current (FreeBSD 10). Signed-off-by: Stacey Son --- bsd-user/freebsd/syscall_nr.h | 834 ++--- 1 files changed, 446 insertions(+), 388 deletions(-) diff --git a/bsd-user/freebsd/sysc

Re: [Qemu-devel] [PATCH 05/15] target-i386: cpu: convert 'xlevel' to static property

2013-06-24 Thread Eduardo Habkost
On Wed, Jun 05, 2013 at 03:18:36PM +0200, Igor Mammedov wrote: > Signed-off-by: Igor Mammedov Reviewed-by: Eduardo Habkost > --- > target-i386/cpu.c | 20 +--- > 1 files changed, 1 insertions(+), 19 deletions(-) > > diff --git a/target-i386/cpu.c b/target-i386/cpu.c > index

Re: [Qemu-devel] [PATCH 01/15] target-i386: cpu: convert 'family' to static property

2013-06-24 Thread Eduardo Habkost
On Wed, Jun 05, 2013 at 03:18:32PM +0200, Igor Mammedov wrote: > Signed-off-by: Igor Mammedov Reviewed-by: Eduardo Habkost > --- > target-i386/cpu.c | 17 ++--- > 1 files changed, 14 insertions(+), 3 deletions(-) > > diff --git a/target-i386/cpu.c b/target-i386/cpu.c > index 1a5

[Qemu-devel] [PATCH 01/23] bsd-user: initial code clean up

2013-06-24 Thread Stacey Son
Add license headers, #ifndef's for header files, and fix various style problem as reported by checkpatch.pl. Signed-off-by: Stacey Son --- bsd-user/bsd-mman.h |2 +- bsd-user/bsdload.c | 21 ++- bsd-user/elfload.c | 122 ++--

Re: [Qemu-devel] [PATCH 03/15] target-i386: cpu: convert 'stepping' to static property

2013-06-24 Thread Eduardo Habkost
On Wed, Jun 05, 2013 at 03:18:34PM +0200, Igor Mammedov wrote: > Signed-off-by: Igor Mammedov Reviewed-by: Eduardo Habkost > --- > target-i386/cpu.c | 12 +--- > 1 files changed, 9 insertions(+), 3 deletions(-) > > diff --git a/target-i386/cpu.c b/target-i386/cpu.c > index 5d379af.

  1   2   3   4   >