Re: [Qemu-devel] [PATCH V3 10/11] vcpu: introduce lockmap

2012-09-12 Thread liu ping fan
On Tue, Sep 11, 2012 at 8:41 PM, Avi Kivity wrote: > On 09/11/2012 03:24 PM, Avi Kivity wrote: >> On 09/11/2012 12:57 PM, Jan Kiszka wrote: >>> On 2012-09-11 11:44, liu ping fan wrote: On Tue, Sep 11, 2012 at 4:35 PM, Avi Kivity wrote: > On 09/11/2012 10:51 AM, Liu Ping Fan wrote: >>

Re: [Qemu-devel] [PATCH V3 10/11] vcpu: introduce lockmap

2012-09-12 Thread liu ping fan
On Tue, Sep 11, 2012 at 10:54 PM, Marcelo Tosatti wrote: > On Tue, Sep 11, 2012 at 03:41:15PM +0300, Avi Kivity wrote: >> On 09/11/2012 03:24 PM, Avi Kivity wrote: >> > On 09/11/2012 12:57 PM, Jan Kiszka wrote: >> >> On 2012-09-11 11:44, liu ping fan wrote: >> >>> On Tue, Sep 11, 2012 at 4:35 PM,

Re: [Qemu-devel] [PATCH V3 08/11] qom: introduce reclaimer to release obj in async

2012-09-12 Thread liu ping fan
On Tue, Sep 11, 2012 at 5:37 PM, Avi Kivity wrote: > On 09/11/2012 12:32 PM, liu ping fan wrote: >> On Tue, Sep 11, 2012 at 4:32 PM, Avi Kivity wrote: >>> On 09/11/2012 10:51 AM, Liu Ping Fan wrote: From: Liu Ping Fan DeviceState will be protected by refcnt from disappearing durin

Re: [Qemu-devel] [PATCH V3 01/11] atomic: introduce atomic operations

2012-09-12 Thread liu ping fan
On Tue, Sep 11, 2012 at 4:04 PM, Avi Kivity wrote: > On 09/11/2012 10:51 AM, Liu Ping Fan wrote: >> From: Liu Ping Fan >> >> If out of global lock, we will be challenged by SMP in low level, >> so need atomic ops. >> >> This file is a wrapper of GCC atomic builtin. >> >> Signed-off-by: Liu Ping F

Re: [Qemu-devel] [PATCH V3 01/11] atomic: introduce atomic operations

2012-09-12 Thread liu ping fan
On Tue, Sep 11, 2012 at 4:15 PM, Peter Maydell wrote: > On 11 September 2012 08:51, Liu Ping Fan wrote: >> + >> +/** >> + * * atomic_inc - increment atomic variable >> + * * @v: pointer of type Atomic >> + ** >> + * * Atomically increments @v by 1. >> + * */ > > Your editor has do

Re: [Qemu-devel] [PATCH 0/3] client monitors config support

2012-09-12 Thread Gerd Hoffmann
On 09/12/12 15:13, Alon Levy wrote: > - no addition of guest capabilities, use interrupt mask instead, ignore >0 or ~0 that are set by current windows driver. > - use crc to solve possible write while read. > - limit heads to 64, statically allocated on rom by host. > - some misc trace fixe

Re: [Qemu-devel] 'qemu-nbd' explicite flush to disk

2012-09-12 Thread Paolo Bonzini
Il 12/09/2012 23:28, Mark Trumpold ha scritto: > So, I've been experimenting with 'qemu-nbd --cache=writeback ..' > This nicely eliminates the 'checkpoint' issue; however, I have as > yet been unable to explicitely flush things to disk -- which I would like to > do just before a 'nilfs' snapshot.

Re: [Qemu-devel] Rethinking missed tick catchup

2012-09-12 Thread Paolo Bonzini
Il 12/09/2012 20:03, Clemens Kolbitsch ha scritto: > > not much that I can contribute to solving the problem, but I have a > bunch of VMs where this happens _every_ time I resume a snapshot (but > without hibernating). In case this could be a connected problem and > you need help testing a patch,

Re: [Qemu-devel] [PATCH] hw: Add support for new LSI Logic devices.

2012-09-12 Thread Paolo Bonzini
Il 12/09/2012 15:58, Anthony Liguori ha scritto: > Why would someone use this verses megasas vs. LSI vs virtio-scsi? LSI is dead. Compare it to IDE. virtio-scsi has the highest performance, but it is not supported on all guests. Compare it to virtio-blk. This vs. megasas is a good question; bo

[Qemu-devel] [PATCH v3 1/2] slirp: Handle more than 65535 blocks in TFTP transfers

2012-09-12 Thread Hervé Poussineau
RFC 1350 does not mention block count roll-over. However, a lot of TFTP servers implement it to be able to transmit big files, so do it also. Current block size is 512 bytes, so TFTP files were limited to 32 MB. Signed-off-by: Hervé Poussineau --- slirp/tftp.c | 24 ++-- s

[Qemu-devel] [PATCH v3 2/2] slirp: Implement TFTP Blocksize option

2012-09-12 Thread Hervé Poussineau
This option is described in RFC 1783. As this is only an optional field, we may ignore it in some situations and handle it in some others. However, MS Windows 2003 PXE boot client requests a block size of the MTU (most of the times 1472 bytes), and doesn't work if the option is not acknowledged (w

[Qemu-devel] [PATCH v3 0/2] slirp: tftp server improvements

2012-09-12 Thread Hervé Poussineau
These patches have already been sent in April 2011, and contain some fixes for the internal TFTP server. With these patches, MS Windows PE can be booted via PXE, and 32MB file limitation has been removed. This has been tested with MS Windows 2003 PXE boot client, PXELINUX and gPXE. Indentation

Re: [Qemu-devel] [PATCH 2/4] tcg/i386: Remove unused registers from tcg_target_call_iarg_regs

2012-09-12 Thread Stefan Weil
Am 12.09.2012 23:18, schrieb Peter Maydell: On 12 September 2012 21:44, Stefan Weil wrote: --- a/tcg/i386/tcg-target.c +++ b/tcg/i386/tcg-target.c @@ -75,9 +75,7 @@ static const int tcg_target_call_iarg_regs[] = { TCG_REG_R8, TCG_REG_R9, #else -TCG_REG_EAX, -TCG_REG_EDX, -

Re: [Qemu-devel] [PATCH 1/2] KVM: fix i8259 interrupt high to low transition logic

2012-09-12 Thread Matthew Ogilvie
On Wed, Sep 12, 2012 at 10:57:57AM +0200, Jan Kiszka wrote: > On 2012-09-12 10:51, Avi Kivity wrote: > > On 09/12/2012 11:48 AM, Jan Kiszka wrote: > >> On 2012-09-12 10:01, Avi Kivity wrote: > >>> On 09/10/2012 04:29 AM, Matthew Ogilvie wrote: > Intel's definition of "edge triggered" means: "a

Re: [Qemu-devel] [PATCH 3/4] tcg: Move tcg_target_get_call_iarg_regs_count to tcg.c

2012-09-12 Thread Stefan Weil
Am 12.09.2012 22:59, schrieb Aurelien Jarno: On Wed, Sep 12, 2012 at 10:44:37PM +0200, Stefan Weil wrote: The TCG targets no longer need individual implementations. Signed-off-by: Stefan Weil --- tcg/arm/tcg-target.c |6 -- tcg/hppa/tcg-target.c |6 -- tcg/i386/tcg-targe

Re: [Qemu-devel] [PATCH 1/9] ehci: Don't set seen to 0 when removing unseen queue-heads

2012-09-12 Thread Gerd Hoffmann
On 09/12/12 15:08, Hans de Goede wrote: > When removing unseen queue-heads from the async queue list, we should not > set the seen flag to 0, as this may cause them to be removed by > ehci_queues_rip_unused() during the next call to ehci_advance_async_state() > if the timer is late or running at a

Re: [Qemu-devel] [libvirt] [PATCH] snapshot: fix rollback failure in transaction mode

2012-09-12 Thread Guannan Ren
On 09/13/2012 01:47 AM, Eric Blake wrote: On 09/12/2012 09:22 AM, Guannan Ren wrote: After failure of qemu transaction command, the snapshot file still be there with non-zero in size. In order to unlink the file, the patch removes the file size checking. Can you give some exact steps to reprodu

Re: [Qemu-devel] [PATCH V2 2/6] libqblock type and structure defines

2012-09-12 Thread Eric Blake
On 09/12/2012 09:33 PM, Eric Blake wrote: >> OK ,then I think >> #if __GNUC__ >= 4 >> >> #else >> [warn name space pollution may happen] >> #endif >> would be better. > > It may be shorter, but it is definitely not better, at least not in the > current context of qemu. Using the short for

[Qemu-devel] [PATCH 01/13] ppc: Make kvm_arch_put_registers() put *all* the registers

2012-09-12 Thread David Gibson
At least when invoked with high enough 'level' arguments, kvm_arch_put_registers() is supposed to copy essentially all the cpu state as encoded in qemu's internal structures into the kvm state. Currently the ppc version does not do this - it never calls KVM_SET_SREGS, for example, and therefore ne

[Qemu-devel] [PATCH 06/13] pseries: Reset emulated PCI TCE tables on system reset

2012-09-12 Thread David Gibson
The emulated PCI host bridge on the pseries machine incorporates an IOMMU (PAPR TCE table). Currently the mappings in this IOMMU are not cleared when we reset the system. This patch fixes this bug. To do this it adds a new reset function to the IOMMU emulation code. The VIO devices already rese

[Qemu-devel] [PATCH 10/13] pseries: Remove XICS irq type enum type

2012-09-12 Thread David Gibson
Currently the XICS interrupt controller emulation uses a custom enum to specify whether a given interrupt is level-sensitive or message-triggered. This enum makes life awkward for saving the state, and isn't particularly useful since there are only two possibilities. This patch replaces the enum w

[Qemu-devel] [PATCH 07/13] pseries: Fix XICS reset

2012-09-12 Thread David Gibson
The XICS interrupt controller used on the pseries machine currently has no reset handler. We can get away with this under some circumstances, but it's not correct, and can cause failures if the XICS happens to be in the wrong state at the time of reset. This patch adds a hook to properly reset th

[Qemu-devel] [PATCH 02/13] pseries: Fix and cleanup CPU initialization and reset

2012-09-12 Thread David Gibson
The current pseries machine init function iterates over the CPUs at several points, doing various bits of initialization. This is messy; these can and should be merged into a single iteration doing all the necessary per cpu initialization. Worse, some of these initializations were setting up stat

[Qemu-devel] [PATCH 05/13] pseries: Clear TCE and signal state when resetting PAPR VIO devices

2012-09-12 Thread David Gibson
When we reset the system, the reset method for VIO bus devices resets the state of their request queue (if present) as it should. However it was not resetting the state of their TCE table (DMA translation) if present. It was also not resetting the state of the per-device signal mask set with H_VI

[Qemu-devel] [0/13] pseries patch queue

2012-09-12 Thread David Gibson
Hi Alex, Here's my current set of ready-to-merge pseries related patches. This includes the latest revisions of the patches to fix system reset which I posted before, plus a number of other small bugfixes and cleanups.

Re: [Qemu-devel] [PATCH V2 2/6] libqblock type and structure defines

2012-09-12 Thread Eric Blake
On 09/12/2012 09:24 PM, Wenchao Xia wrote: > 于 2012-9-12 20:59, Eric Blake 写道: >> On 09/11/2012 09:05 PM, Wenchao Xia wrote: Seriously? We require a C99-compliant compiler, which is required to treat the more compact version identically (all undefined names evaluate to 0 in the

[Qemu-devel] [PATCH 11/13] pseries: Remove never used flags field from spapr vio devices

2012-09-12 Thread David Gibson
The general device state structure for PAPR VIO emulated devices includes a 'flags' field which was never used. This patch removes it. Signed-off-by: David Gibson --- hw/spapr_vio.h |1 - 1 file changed, 1 deletion(-) diff --git a/hw/spapr_vio.h b/hw/spapr_vio.h index ea6aa43..acef65e 1006

Re: [Qemu-devel] [PATCH V2 2/6] libqblock type and structure defines

2012-09-12 Thread Wenchao Xia
于 2012-9-12 20:59, Eric Blake 写道: On 09/11/2012 09:05 PM, Wenchao Xia wrote: Seriously? We require a C99-compliant compiler, which is required to treat the more compact version identically (all undefined names evaluate to 0 in the preprocessor), and HACKING doesn't mandate that we spell out a d

[Qemu-devel] [PATCH 09/13] pseries: Remove C bitfields from xics code

2012-09-12 Thread David Gibson
The XICS interrupt controller emulation uses some C bitfield variables in its internal state structure. This makes like awkward for saving the state because we don't have easy VMSTATE helpers for bitfields. This patch removes the bitfields, instead using explicit bit masking in a single status va

[Qemu-devel] [PATCH 12/13] pseries: Rework implementation of TCE bypass

2012-09-12 Thread David Gibson
On the pseries machine the IOMMU (aka TCE tables) is always active for all PCI and VIO devices. Mostly to simplify the SLOF firmware, we implement an extension which allows the IOMMU to be temporarily disabled for certain devices. Currently this is implemented by setting the device's DMAContext p

[Qemu-devel] [PATCH 13/13] pseries: Fix semantics of RTAS int-on, int-off and set-xive functions

2012-09-12 Thread David Gibson
Currently the ibm,int-on and ibm,int-off RTAS functions are implemented as no-ops. This is because when implemented as specified in PAPR they caused Linux (which calls both int-on/off and set-xive) to end up with interrupts masked when they should not be. Since Linux's set-xive calls make the int

[Qemu-devel] [PATCH 08/13] pseries: Small cleanup to H_CEDE implementation

2012-09-12 Thread David Gibson
The H_CEDE hypercall implementation for the pseries machine doesn't trigger quite the right path in the main cpu exec loop. We should set exit_request to pop up one extra level and recheck state, and we should set the exception_index to EXCP_HLT (H_CEDE is roughly equivalent to the hlt instruction

[Qemu-devel] [PATCH 04/13] pseries: Add support for new KVM hash table control call

2012-09-12 Thread David Gibson
This adds support for then new "reset htab" ioctl which allows qemu to properly cleanup the MMU hash table when the guest is reset. With the corresponding kernel support, reset of a guest now works properly. This also paves the way for indicating a different size hash table to the kernel and for t

[Qemu-devel] [PATCH 03/13] pseries: Use new method to correct reset sequence

2012-09-12 Thread David Gibson
A number of things need to occur during reset of the PAPR paravirtualized platform in a specific order. For example, the hash table needs to be cleared before the CPUs are reset, so that they initialize their register state correctly, and the CPUs need to have their main reset called before we set

Re: [Qemu-devel] [RFC v3 ATCH 0/5] char: expose CirMemCharDriver and provide QMP interface

2012-09-12 Thread Anthony Liguori
"Daniel P. Berrange" writes: > On Wed, Sep 12, 2012 at 07:57:21PM +0800, Lei Li wrote: >> This RFC series attempts to convert the MemCharDriver to use a circular >> buffer for input and output, expose it to users by introducing QMP commands >> memchar_write and memchar_read and via the command li

Re: [Qemu-devel] Does TCG IR use static single assignment (SSA) form?

2012-09-12 Thread Wei-Ren Chen
> Excuse me for asking, does TCG-IR  use static single assignment (SSA) form? > > I just wanna know how to translate a register-based bytecode to TCG-IR. Sounds like you need to take a look at target-xxx/translate.c ? Regards, chenwj -- Wei-Ren Chen (陳韋任) Computer Systems Lab, Institute of I

[Qemu-devel] 'qemu-nbd' explicite flush to disk

2012-09-12 Thread Mark Trumpold
Hello, I posted a similar message to 'qemu-discuss', and was thinking my question may require a developer level of expertise. I will try to be as brief as possible. My context is using 'qemu-nbd' and 'qemu-img' to manage 'qcow2' loop filesystem images being hosted on a 'nilfs' filesystem. Thi

[Qemu-devel] [PATCH] Revert 455aa1e08 and c3767ed0eb

2012-09-12 Thread Anthony Liguori
commit c3767ed0eb5d0bb25fe409ae5dec06e3411ff1b6 qemu-char: (Re-)connect for tcp_chr_write() unconnected writing Has no hope of working because tcp_chr_connect() does not actually connect. 455aa1e08 just fixes the SEGV with server() but the attempt to connect a client socket is still compl

Re: [Qemu-devel] Enablig DLPAR capacity on QEMU pSeries

2012-09-12 Thread Benjamin Herrenschmidt
On Wed, 2012-09-12 at 17:53 +0200, Alexander Graf wrote: > On 09/12/2012 04:54 PM, Erlon Cruz wrote: > > Hi all, > > > > We are planning to implement DLPAR capacity on QEMU pSeries. As we > > What is DLPAR? Hotplug support? Yes. > > lack of experience in the internals of the arch we would like y

Re: [Qemu-devel] Enablig DLPAR capacity on QEMU pSeries

2012-09-12 Thread Alexander Graf
On 12.09.2012, at 22:56, Erlon Cruz wrote: > On Wed, Sep 12, 2012 at 12:53 PM, Alexander Graf wrote: >> On 09/12/2012 04:54 PM, Erlon Cruz wrote: >>> >>> Hi all, >>> >>> We are planning to implement DLPAR capacity on QEMU pSeries. As we >> >> >> What is DLPAR? Hotplug support? > > Yes, ba

Re: [Qemu-devel] [PATCH 2/4] tcg/i386: Remove unused registers from tcg_target_call_iarg_regs

2012-09-12 Thread Peter Maydell
On 12 September 2012 21:44, Stefan Weil wrote: > --- a/tcg/i386/tcg-target.c > +++ b/tcg/i386/tcg-target.c > @@ -75,9 +75,7 @@ static const int tcg_target_call_iarg_regs[] = { > TCG_REG_R8, > TCG_REG_R9, > #else > -TCG_REG_EAX, > -TCG_REG_EDX, > -TCG_REG_ECX > +/* 32 bit

[Qemu-devel] [PATCH 1/4] w64: Fix TCG helper functions with 5 arguments

2012-09-12 Thread Stefan Weil
TCG uses 6 registers for function arguments on 64 bit Linux hosts, but only 4 registers on W64 hosts. Commit 2999a0b20074a7e4a58f56572bb1436749368f59 increased the number of arguments for some important helper functions from 4 to 5 which triggered a bug for W64 hosts: QEMU aborts when executing he

[Qemu-devel] [PATCH 3/4] tcg: Move tcg_target_get_call_iarg_regs_count to tcg.c

2012-09-12 Thread Stefan Weil
The TCG targets no longer need individual implementations. Signed-off-by: Stefan Weil --- tcg/arm/tcg-target.c |6 -- tcg/hppa/tcg-target.c |6 -- tcg/i386/tcg-target.c |6 -- tcg/ia64/tcg-target.c |6 -- tcg/mips/tcg-target.c |6 -- tcg/ppc/tcg-targ

Re: [Qemu-devel] [PATCH 3/4] tcg: Move tcg_target_get_call_iarg_regs_count to tcg.c

2012-09-12 Thread Aurelien Jarno
On Wed, Sep 12, 2012 at 10:44:37PM +0200, Stefan Weil wrote: > The TCG targets no longer need individual implementations. > > Signed-off-by: Stefan Weil > --- > tcg/arm/tcg-target.c |6 -- > tcg/hppa/tcg-target.c |6 -- > tcg/i386/tcg-target.c |6 -- > tcg/ia64/tcg-ta

Re: [Qemu-devel] Enablig DLPAR capacity on QEMU pSeries

2012-09-12 Thread Erlon Cruz
On Wed, Sep 12, 2012 at 12:53 PM, Alexander Graf wrote: > On 09/12/2012 04:54 PM, Erlon Cruz wrote: >> >> Hi all, >> >> We are planning to implement DLPAR capacity on QEMU pSeries. As we > > > What is DLPAR? Hotplug support? Yes, basically the way PowerVM uses to dynamically add memory, cpu, and

[Qemu-devel] [PATCH 0/4] Fix and clean tcg_target_get_call_iarg_regs_count

2012-09-12 Thread Stefan Weil
Function tcg_target_get_call_iarg_regs_count was wrong for w64 hosts and can be simplified for all TCG targets: [PATCH 1/4] w64: Fix TCG helper functions with 5 arguments [PATCH 2/4] tcg/i386: Remove unused registers from [PATCH 3/4] tcg: Move tcg_target_get_call_iarg_regs_count to tcg.c [PATCH 4/

[Qemu-devel] [PATCH 2/4] tcg/i386: Remove unused registers from tcg_target_call_iarg_regs

2012-09-12 Thread Stefan Weil
32 bit x86 hosts don't need registers for helper function arguments because they use the default stack based calling convention. Removing the registers allows simpler code for function tcg_target_get_call_iarg_regs_count. Signed-off-by: Stefan Weil --- tcg/i386/tcg-target.c | 10 ++ 1

[Qemu-devel] [PATCH 4/4] tcg: Remove unused parameter from tcg_target_get_call_iarg_regs_count

2012-09-12 Thread Stefan Weil
Since commit 6a18ae2d2947532d5c26439548afa0481c4529f9, 'flags' is no longer used in tcg_target_get_call_iarg_regs_count. Signed-off-by: Stefan Weil --- tcg/tcg.c |4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/tcg/tcg.c b/tcg/tcg.c index 53316f6..0f0a8d0 100644 --- a/t

Re: [Qemu-devel] [PATCH v2 0/3] nonblocking connect address handling cleanup

2012-09-12 Thread Michael S. Tsirkin
On Wed, Sep 12, 2012 at 02:12:55PM +0300, Orit Wasserman wrote: > getaddrinfo can give us a list of addresses, but we only try to > connect to the first one. If that fails we never proceed to > the next one. This is common on desktop setups that often have ipv6 > configured but not actually workin

Re: [Qemu-devel] [libvirt] [PATCH] snapshot: fix rollback failure in transaction mode

2012-09-12 Thread Eric Blake
On 09/12/2012 12:33 PM, Jeff Cody wrote: > If QEMU creates the snapshot file, upon transaction failure it is > possible to have a newly created image file left, depending on when the > failure occurs. The running QEMU instance, however, will not be > affected. > > For instance, if we are perform

Re: [Qemu-devel] Rethinking missed tick catchup

2012-09-12 Thread Michael Roth
On Wed, Sep 12, 2012 at 07:30:08PM +0200, Stefan Weil wrote: > Am 12.09.2012 18:45, schrieb Gleb Natapov: > >On Wed, Sep 12, 2012 at 06:27:14PM +0200, Stefan Weil wrote: > >>Am 12.09.2012 15:54, schrieb Anthony Liguori: > >>>Hi, > >>> > >>>We've been running into a lot of problems lately with Windo

Re: [Qemu-devel] [PATCH] w64: Fix calls of TCG helper functions with 5 arguments

2012-09-12 Thread Stefan Weil
Am 12.09.2012 21:14, schrieb Aurelien Jarno: On Wed, Sep 12, 2012 at 07:12:47PM +0100, Peter Maydell wrote: On 12 September 2012 19:03, Stefan Weil wrote: diff --git a/tcg/i386/tcg-target.c b/tcg/i386/tcg-target.c index da17bba..43b5572 100644 --- a/tcg/i386/tcg-target.c +++ b/tcg/i386/tcg-tar

Re: [Qemu-devel] Rethinking missed tick catchup

2012-09-12 Thread Stefan Weil
Am 12.09.2012 20:13, schrieb Gleb Natapov: On Wed, Sep 12, 2012 at 07:30:08PM +0200, Stefan Weil wrote: I also meant host hibernation. Than I don't see how guest can handle the situation since it has no idea that it was stopped. Qemu has not idea about host hibernation either. The guest can c

Re: [Qemu-devel] [PATCH] w64: Fix calls of TCG helper functions with 5 arguments

2012-09-12 Thread Stefan Weil
Am 12.09.2012 21:14, schrieb Aurelien Jarno: On Wed, Sep 12, 2012 at 07:12:47PM +0100, Peter Maydell wrote: On 12 September 2012 19:03, Stefan Weil wrote: diff --git a/tcg/i386/tcg-target.c b/tcg/i386/tcg-target.c index da17bba..43b5572 100644 --- a/tcg/i386/tcg-target.c +++ b/tcg/i386/tcg-tar

Re: [Qemu-devel] [PATCH] socket: don't attempt to reconnect a TCP socket in server mode

2012-09-12 Thread Anthony Liguori
"Richard W.M. Jones" writes: > On Wed, Sep 05, 2012 at 02:01:36PM -0500, Anthony Liguori wrote: >> Commit c3767ed0eb5d0bb25fe409ae5dec06e3411ff1b6 introduced a possible SEGV >> when >> using a socket chardev with server=on because it assumes that all TCP sockets >> are in client mode. >> >> Thi

Re: [Qemu-devel] [PATCH] w64: Fix calls of TCG helper functions with 5 arguments

2012-09-12 Thread Aurelien Jarno
On Wed, Sep 12, 2012 at 07:12:47PM +0100, Peter Maydell wrote: > On 12 September 2012 19:03, Stefan Weil wrote: > > diff --git a/tcg/i386/tcg-target.c b/tcg/i386/tcg-target.c > > index da17bba..43b5572 100644 > > --- a/tcg/i386/tcg-target.c > > +++ b/tcg/i386/tcg-target.c > > @@ -118,7 +118,7 @@ s

Re: [Qemu-devel] [PATCH] socket: don't attempt to reconnect a TCP socket in server mode

2012-09-12 Thread Richard W.M. Jones
On Wed, Sep 05, 2012 at 02:01:36PM -0500, Anthony Liguori wrote: > Commit c3767ed0eb5d0bb25fe409ae5dec06e3411ff1b6 introduced a possible SEGV > when > using a socket chardev with server=on because it assumes that all TCP sockets > are in client mode. > > This patch adds a check to only reconnect

Re: [Qemu-devel] [libvirt] [PATCH] snapshot: fix rollback failure in transaction mode

2012-09-12 Thread Jeff Cody
On 09/12/2012 01:47 PM, Eric Blake wrote: > On 09/12/2012 09:22 AM, Guannan Ren wrote: >> After failure of qemu transaction command, the snapshot file still >> be there with non-zero in size. In order to unlink the file, the >> patch removes the file size checking. > > Can you give some exact step

Re: [Qemu-devel] [PATCH v2 3/4] target-i386: Allow changing of Hypervisor CPUIDs.

2012-09-12 Thread Marcelo Tosatti
The problem with integrating this is that it has little or no assurance from documentation. The Linux kernel source is a good source, then say "accordingly to VMWare guest support code in version xyz" in the changelog. Also extracting this information in a text file (or comment in the code) woul

Re: [Qemu-devel] Rethinking missed tick catchup

2012-09-12 Thread Anthony Liguori
Gleb Natapov writes: > On Wed, Sep 12, 2012 at 08:54:26AM -0500, Anthony Liguori wrote: >> >> Hi, >> >> We've been running into a lot of problems lately with Windows guests and >> I think they all ultimately could be addressed by revisiting the missed >> tick catchup algorithms that we use. Mi

Re: [Qemu-devel] Rethinking missed tick catchup

2012-09-12 Thread Gleb Natapov
On Wed, Sep 12, 2012 at 07:30:08PM +0200, Stefan Weil wrote: > Am 12.09.2012 18:45, schrieb Gleb Natapov: > >On Wed, Sep 12, 2012 at 06:27:14PM +0200, Stefan Weil wrote: > >>Am 12.09.2012 15:54, schrieb Anthony Liguori: > >>>Hi, > >>> > >>>We've been running into a lot of problems lately with Windo

Re: [Qemu-devel] [PATCH] w64: Fix calls of TCG helper functions with 5 arguments

2012-09-12 Thread Peter Maydell
On 12 September 2012 19:03, Stefan Weil wrote: > diff --git a/tcg/i386/tcg-target.c b/tcg/i386/tcg-target.c > index da17bba..43b5572 100644 > --- a/tcg/i386/tcg-target.c > +++ b/tcg/i386/tcg-target.c > @@ -118,7 +118,7 @@ static void patch_reloc(uint8_t *code_ptr, int type, > static inline int tc

Re: [Qemu-devel] Rethinking missed tick catchup

2012-09-12 Thread Clemens Kolbitsch
> On 2012-09-12 15:54, Anthony Liguori wrote: >> >> Hi, >> >> We've been running into a lot of problems lately with Windows guests and >> I think they all ultimately could be addressed by revisiting the missed >> tick catchup algorithms that we use. Mike and I spent a while talking >> about it yes

[Qemu-devel] [PATCH] w64: Fix calls of TCG helper functions with 5 arguments

2012-09-12 Thread Stefan Weil
TCG uses 6 registers for function arguments on 64 bit Linux hosts, but only 4 registers on W64 hosts. Commit 2999a0b20074a7e4a58f56572bb1436749368f59 increased the number of arguments for some important helper functions from 4 to 5 which triggered a bug for W64 hosts: QEMU aborts when executing he

Re: [Qemu-devel] [libvirt] [PATCH] snapshot: fix rollback failure in transaction mode

2012-09-12 Thread Eric Blake
On 09/12/2012 09:22 AM, Guannan Ren wrote: > After failure of qemu transaction command, the snapshot file still > be there with non-zero in size. In order to unlink the file, the > patch removes the file size checking. Can you give some exact steps to reproduce this, so that I know who is making t

Re: [Qemu-devel] Rethinking missed tick catchup

2012-09-12 Thread Stefan Weil
Am 12.09.2012 18:45, schrieb Gleb Natapov: On Wed, Sep 12, 2012 at 06:27:14PM +0200, Stefan Weil wrote: Am 12.09.2012 15:54, schrieb Anthony Liguori: Hi, We've been running into a lot of problems lately with Windows guests and I think they all ultimately could be addressed by revisiting the mi

Re: [Qemu-devel] Rethinking missed tick catchup

2012-09-12 Thread Luiz Capitulino
On Wed, 12 Sep 2012 08:54:26 -0500 Anthony Liguori wrote: > > Hi, > > We've been running into a lot of problems lately with Windows guests and > I think they all ultimately could be addressed by revisiting the missed > tick catchup algorithms that we use. Mike and I spent a while talking > abo

Re: [Qemu-devel] [PATCH] tcg: Fix MAX_OPC_PARAM_IARGS

2012-09-12 Thread Richard Henderson
On 09/12/2012 10:18 AM, Stefan Weil wrote: > DEF_HELPER_FLAGS_5 was added some time ago without adjusting > MAX_OPC_PARAM_IARGS. > > Fixing the definition becomes more important as QEMU is using > an increasing number of helper functions called with 5 arguments. > > Add also a comment to avoid fu

[Qemu-devel] [PATCH] tcg: Fix MAX_OPC_PARAM_IARGS

2012-09-12 Thread Stefan Weil
DEF_HELPER_FLAGS_5 was added some time ago without adjusting MAX_OPC_PARAM_IARGS. Fixing the definition becomes more important as QEMU is using an increasing number of helper functions called with 5 arguments. Add also a comment to avoid future problems when DEF_HELPER_FLAGS_6 will be added. Sig

Re: [Qemu-devel] TCG questions

2012-09-12 Thread Xin Tong
I have the code on http://code.google.com/p/qemu-trace/. I currently have memory trace, branch trace and some special instructions traces ready ( unverified though). we should discuss about what is the best way to do this btw. Xin On Wed, Sep 12, 2012 at 10:09 AM, Xin Tong wrote: > On Wed, Sep

Re: [Qemu-devel] TCG questions

2012-09-12 Thread Xin Tong
On Wed, Sep 12, 2012 at 6:14 AM, Lluís Vilanova wrote: > Xin Tong writes: > >> i do not know. could be similar. I am doing architecture research. i >> need traces of memory access for programming running under a full >> system environment, so i wrote this. > >> i do nto seem to be able to access t

Re: [Qemu-devel] Does TCG IR use static single assignment (SSA) form?

2012-09-12 Thread Richard Henderson
On 09/12/2012 02:45 AM, (Yichen Yang)楊逸臣 wrote: > Excuse me for asking, does TCG-IR use static single assignment (SSA) form? No. r~

Re: [Qemu-devel] Does TCG IR use static single assignment (SSA) form?

2012-09-12 Thread Peter Maydell
On 12 September 2012 10:45, (Yichen Yang)楊逸臣 wrote: > Excuse me for asking, does TCG-IR use static single assignment (SSA) form? No. The TCG IR is documented in tcg/README -- this should be enough to be able to translate something else into it. -- PMM

Re: [Qemu-devel] [PATCH v3 01/17] vga: rename pci_vga_init() into pci_std_vga_init()

2012-09-12 Thread Richard Henderson
On 09/11/2012 12:10 PM, Aurelien Jarno wrote: > This better explains what is this function about. Adjust all callers. > > Cc: Richard Henderson > Cc: Alexander Graf > Cc: Andreas Färber > Cc: David Gibson > Cc: Anthony Liguori > Acked-by: Blue Swirl > Acked-by: Andreas Färber > Signed-off-b

Re: [Qemu-devel] Windows VM slow boot

2012-09-12 Thread Richard Davies
Hi Mel - thanks for replying to my underhand bcc! Mel Gorman wrote: > I see that this is an old-ish bug but I did not read the full history. > Is it now booting faster than 3.5.0 was? I'm asking because I'm > interested to see if commit c67fe375 helped your particular case. Yes, I think 3.6.0-rc5

Re: [Qemu-devel] Rethinking missed tick catchup

2012-09-12 Thread Gleb Natapov
On Wed, Sep 12, 2012 at 06:27:14PM +0200, Stefan Weil wrote: > Am 12.09.2012 15:54, schrieb Anthony Liguori: > > > >Hi, > > > >We've been running into a lot of problems lately with Windows guests and > >I think they all ultimately could be addressed by revisiting the missed > >tick catchup algorith

Re: [Qemu-devel] Rethinking missed tick catchup

2012-09-12 Thread Stefan Weil
Am 12.09.2012 15:54, schrieb Anthony Liguori: Hi, We've been running into a lot of problems lately with Windows guests and I think they all ultimately could be addressed by revisiting the missed tick catchup algorithms that we use. Mike and I spent a while talking about it yesterday and I want

Re: [Qemu-devel] [PATCH 005/126] target-s390: Fix gdbstub

2012-09-12 Thread Richard Henderson
On 09/12/2012 08:54 AM, Alexander Graf wrote: > So the CC pseudo-register is never written to? They do handle a write to cc_regnum in s390_pseudo_register_write. They modify psw.mask as one would expect. r~

Re: [Qemu-devel] Rethinking missed tick catchup

2012-09-12 Thread Gleb Natapov
On Wed, Sep 12, 2012 at 06:06:47PM +0300, Gleb Natapov wrote: > On Wed, Sep 12, 2012 at 09:44:10AM -0500, Anthony Liguori wrote: > > Jan Kiszka writes: > > > > > On 2012-09-12 15:54, Anthony Liguori wrote: > > >> > > >> Hi, > > >> > > >> We've been running into a lot of problems lately with Win

Re: [Qemu-devel] [RFC v3 ATCH 0/5] char: expose CirMemCharDriver and provide QMP interface

2012-09-12 Thread Daniel P. Berrange
On Wed, Sep 12, 2012 at 07:57:21PM +0800, Lei Li wrote: > This RFC series attempts to convert the MemCharDriver to use a circular > buffer for input and output, expose it to users by introducing QMP commands > memchar_write and memchar_read and via the command line like the other > CharDriverStates

Re: [Qemu-devel] Windows VM slow boot

2012-09-12 Thread Mel Gorman
On Wed, Sep 12, 2012 at 11:56:59AM +0100, Richard Davies wrote: > [ adding linux-mm - previously at http://marc.info/?t=13451150943 ] > > Hi Rik, > I'm not Rik but hi anyway. > Since qemu-kvm 1.2.0 and Linux 3.6.0-rc5 came out, I thought that I would > retest with these. > Ok. 3.6.0-rc5 c

[Qemu-devel] Does TCG IR use static single assignment (SSA) form?

2012-09-12 Thread Yichen Yang
Dear all, Excuse me for asking, does TCG-IR use static single assignment (SSA) form? I just wanna know how to translate a register-based bytecode to TCG-IR. thanks :) Yichen

Re: [Qemu-devel] [PATCH 005/126] target-s390: Fix gdbstub

2012-09-12 Thread Alexander Graf
On 09/12/2012 05:11 PM, Richard Henderson wrote: On 09/12/2012 06:25 AM, Alexander Graf wrote: +case S390_PSWM_REGNUM: +env->psw.mask = tmpl; +env->cc_op = (tmpl >> 13) & 3; Are you sure this is correct? I thought gdbstub would just ignore the cc bits. Well... no it won't i

Re: [Qemu-devel] [RFC v3 ATCH 0/5] char: expose CirMemCharDriver and provide QMP interface

2012-09-12 Thread Avi Kivity
On 09/12/2012 02:57 PM, Lei Li wrote: > This RFC series attempts to convert the MemCharDriver to use a circular > buffer for input and output, expose it to users by introducing QMP commands > memchar_write and memchar_read and via the command line like the other > CharDriverStates. > > Serial port

Re: [Qemu-devel] Enablig DLPAR capacity on QEMU pSeries

2012-09-12 Thread Alexander Graf
On 09/12/2012 04:54 PM, Erlon Cruz wrote: Hi all, We are planning to implement DLPAR capacity on QEMU pSeries. As we What is DLPAR? Hotplug support? lack of experience in the internals of the arch we would like you guys to give us some design directions and confirm if we going in the right d

Re: [Qemu-devel] Rethinking missed tick catchup

2012-09-12 Thread Gleb Natapov
On Wed, Sep 12, 2012 at 05:42:58PM +0200, Jan Kiszka wrote: > On 2012-09-12 17:06, Gleb Natapov wrote: > Are there other issues with reinjection that people are aware of? Does > anything seem obviously wrong with the above? > >>> > >>> We should take the chance and design everything in a

Re: [Qemu-devel] Rethinking missed tick catchup

2012-09-12 Thread Jan Kiszka
On 2012-09-12 17:06, Gleb Natapov wrote: Are there other issues with reinjection that people are aware of? Does anything seem obviously wrong with the above? >>> >>> We should take the chance and design everything in a way that the HPET >>> can finally be (left) enabled. >> >> I thought

Re: [Qemu-devel] [PATCH] hw: Add support for new LSI Logic devices.

2012-09-12 Thread Avi Kivity
On 09/11/2012 08:00 PM, Don Slutz wrote: > Add LSI53C1030, SAS1068, SAS1068e. Based on code from "VirtualBox Open > Source Edition". > Based on QEMU MegaRAID SAS 8708EM2. > > This is a common VMware disk controller. > > SEABIOS change for booting is in the works. > > Tested with Fedora 16, 17.

Re: [Qemu-devel] Rethinking missed tick catchup

2012-09-12 Thread Gleb Natapov
On Wed, Sep 12, 2012 at 08:54:26AM -0500, Anthony Liguori wrote: > > Hi, > > We've been running into a lot of problems lately with Windows guests and > I think they all ultimately could be addressed by revisiting the missed > tick catchup algorithms that we use. Mike and I spent a while talking

Re: [Qemu-devel] [PATCH 005/126] target-s390: Fix gdbstub

2012-09-12 Thread Richard Henderson
On 09/12/2012 06:25 AM, Alexander Graf wrote: >> +case S390_PSWM_REGNUM: >> +env->psw.mask = tmpl; >> +env->cc_op = (tmpl >> 13) & 3; > > Are you sure this is correct? I thought gdbstub would just ignore the cc bits. Well... no it won't ignore the cc bits. But it would appear

Re: [Qemu-devel] Rethinking missed tick catchup

2012-09-12 Thread Gleb Natapov
On Wed, Sep 12, 2012 at 09:44:10AM -0500, Anthony Liguori wrote: > Jan Kiszka writes: > > > On 2012-09-12 15:54, Anthony Liguori wrote: > >> > >> Hi, > >> > >> We've been running into a lot of problems lately with Windows guests and > >> I think they all ultimately could be addressed by revisit

Re: [Qemu-devel] [PATCH 4/5] QAPI: Introduce memchar-read QMP command

2012-09-12 Thread Eric Blake
On 09/12/2012 05:57 AM, Lei Li wrote: > Signed-off-by: Lei Li > --- > hmp-commands.hx | 25 + > hmp.c| 18 ++ > hmp.h|1 + > qapi-schema.json | 27 +++ > qemu-char.c | 48

[Qemu-devel] Enablig DLPAR capacity on QEMU pSeries

2012-09-12 Thread Erlon Cruz
Hi all, We are planning to implement DLPAR capacity on QEMU pSeries. As we lack of experience in the internals of the arch we would like you guys to give us some design directions and confirm if we going in the right direction. Our first idea is: 1 - to patch 'spapr.c' so it can dynamically i

Re: [Qemu-devel] Rethinking missed tick catchup

2012-09-12 Thread Jan Kiszka
On 2012-09-12 16:44, Anthony Liguori wrote: > Jan Kiszka writes: > >> On 2012-09-12 15:54, Anthony Liguori wrote: >>> >>> Hi, >>> >>> We've been running into a lot of problems lately with Windows guests and >>> I think they all ultimately could be addressed by revisiting the missed >>> tick catch

Re: [Qemu-devel] Rethinking missed tick catchup

2012-09-12 Thread Anthony Liguori
Jan Kiszka writes: > On 2012-09-12 15:54, Anthony Liguori wrote: >> >> Hi, >> >> We've been running into a lot of problems lately with Windows guests and >> I think they all ultimately could be addressed by revisiting the missed >> tick catchup algorithms that we use. Mike and I spent a while

Re: [Qemu-devel] Rethinking missed tick catchup

2012-09-12 Thread Jan Kiszka
On 2012-09-12 15:54, Anthony Liguori wrote: > > Hi, > > We've been running into a lot of problems lately with Windows guests and > I think they all ultimately could be addressed by revisiting the missed > tick catchup algorithms that we use. Mike and I spent a while talking > about it yesterday

Re: [Qemu-devel] [PATCH v2 0/3] nonblocking connect address handling cleanup

2012-09-12 Thread Amos Kong
On 12/09/12 19:12, Orit Wasserman wrote: getaddrinfo can give us a list of addresses, but we only try to connect to the first one. If that fails we never proceed to the next one. This is common on desktop setups that often have ipv6 configured but not actually working. A simple way to reproduce

Re: [Qemu-devel] [PATCH] hw: Add support for new LSI Logic devices.

2012-09-12 Thread Anthony Liguori
Kevin Wolf writes: > Am 12.09.2012 01:50, schrieb Michael S. Tsirkin: >> On Tue, Sep 11, 2012 at 01:00:13PM -0400, Don Slutz wrote: >>> Add LSI53C1030, SAS1068, SAS1068e. Based on code from "VirtualBox Open >>> Source Edition". >>> Based on QEMU MegaRAID SAS 8708EM2. >>> >>> This is a common VM

[Qemu-devel] Rethinking missed tick catchup

2012-09-12 Thread Anthony Liguori
Hi, We've been running into a lot of problems lately with Windows guests and I think they all ultimately could be addressed by revisiting the missed tick catchup algorithms that we use. Mike and I spent a while talking about it yesterday and I wanted to take the discussion to the list to get som

[Qemu-devel] [PATCH 8/9] usb-redir: Revert usb-redir part of commit 93bfef4c

2012-09-12 Thread Hans de Goede
Commit 93bfef4c6e4b23caea9d51e1099d06433d8835a4 makes qemu-devices which report the qemu version string to the guest in some way use a qemu_get_version function which reports a machine-specific version string. However usb-redir does not expose the qemu version to the guest, only to the usbredir-ho

  1   2   >