Re: [Qemu-devel] [Bug 1062589] [NEW] Xp guest disk is corrupted when the data size exceeds 4 GB

2012-10-08 Thread Stefan Hajnoczi
On Fri, Oct 05, 2012 at 10:30:15PM -, pil926 wrote: Host : - 2.6.30.10 i686 pentium3 i386 GNU/Linux ... Everything is running pretty good until data size on disk C exceeds 4 GB. I Tried many options before figuring out that the problem occurs when data size exceeds 4 GB. I tried with QEMU

Re: [Qemu-devel] buildbot failure in qemu on default_mingw32

2012-10-08 Thread Stefan Hajnoczi
On Sat, Oct 06, 2012 at 01:51:24AM +0200, q...@buildbot.b1-systems.de wrote: The Buildbot has detected a new failure on builder default_mingw32 while building qemu. Full details are available at: http://buildbot.b1-systems.de/qemu/builders/default_mingw32/builds/396 Buildbot URL:

Re: [Qemu-devel] [Qemu-trivial] [PATCH] sheepdog: use bool for boolean variables

2012-10-08 Thread Stefan Hajnoczi
On Sun, Oct 07, 2012 at 01:57:14AM +0900, MORITA Kazutaka wrote: This improves readability. Signed-off-by: MORITA Kazutaka morita.kazut...@lab.ntt.co.jp --- block/sheepdog.c | 70 +++--- 1 files changed, 35 insertions(+), 35 deletions(-)

Re: [Qemu-devel] [PATCH] target-sparc register window handling

2012-10-08 Thread Aurelien Jarno
On Sat, Oct 06, 2012 at 12:15:16PM +0200, Aurelien Jarno wrote: On Fri, Oct 05, 2012 at 05:00:04PM -0700, Richard Henderson wrote: This applies with or without the sparc-compare patch set I recently sent, and it works with the same set of tests. I've not had time to do true benchmarking

Re: [Qemu-devel] [Qemu-trivial] [PATCH] configure: Tidy up remnants of non-64-bit physaddrs

2012-10-08 Thread Stefan Hajnoczi
On Fri, Oct 05, 2012 at 05:33:22PM +0100, Peter Maydell wrote: Tidy up some remnants of code to support non-64-bit physaddrs which were accidentally omitted from commit 4be403c8. Signed-off-by: Peter Maydell peter.mayd...@linaro.org --- configure |2 -- 1 file changed, 2 deletions(-)

Re: [Qemu-devel] [Qemu-trivial] [PATCH] sheepdog: use bool for boolean variables

2012-10-08 Thread Stefan Hajnoczi
On Sun, Oct 07, 2012 at 01:57:14AM +0900, MORITA Kazutaka wrote: This improves readability. Signed-off-by: MORITA Kazutaka morita.kazut...@lab.ntt.co.jp --- block/sheepdog.c | 70 +++--- 1 files changed, 35 insertions(+), 35 deletions(-)

[Qemu-devel] Building QEMU with multiple CPU targets.

2012-10-08 Thread Peter Crosthwaite
Hi All, Im currently investigating the possibility of building QEMU with multiple CPU architectures active concurrently. That is, I have a binary with both an target-arm and target-microblaze and wish to run them as a heterogeneous multiprocessor platform. Given the recent QOM development in

Re: [Qemu-devel] [PATCH] versatile: Push lsi initialization to the end

2012-10-08 Thread Jan Kiszka
On 2012-10-06 04:13, Peter Maydell wrote: On 5 October 2012 19:01, Jan Kiszka jan.kis...@siemens.com wrote: I'm not a fan of this either, but the alternatives are way more complicated. We either need to rewrite the chardev subsystem, specifically how mux'ed devices are registered and how the

Re: [Qemu-devel] [PATCH 02/17] event_notifier: enable it to use pipes

2012-10-08 Thread Stefan Hajnoczi
On Tue, Sep 25, 2012 at 02:55:48PM +0200, Paolo Bonzini wrote: int event_notifier_init(EventNotifier *e, int active) { +int fds[2]; +int ret; + #ifdef CONFIG_EVENTFD -int fd = eventfd(!!active, EFD_NONBLOCK | EFD_CLOEXEC); -if (fd 0) -return -errno; -e-fd

Re: [Qemu-devel] [PATCH] Require spice-server 0.12.0 and spice-protocol 0.12.2

2012-10-08 Thread Gerd Hoffmann
On 10/04/12 19:37, Søren Sandmann wrote: From: Søren Sandmann Pedersen s...@redhat.com Set the default PCI revision to QXL_REVISION_STABLE_V12, and remove the various #ifs on various subversions of spice-server 0.11.x. Simliar patch is in spice-next (see

Re: [Qemu-devel] [PATCH] hw/arm_boot.c: bump initrd address (again) to accommodate large kernels

2012-10-08 Thread Peter Maydell
On 8 October 2012 01:09, Peter Crosthwaite peter.crosthwa...@petalogix.com wrote: Another completely different but workable solution is to dynamically determine a initrd (and dtb) location. Any reason why the bootloader can't track what memory real estate is in use by what pieces and just pick

Re: [Qemu-devel] [Bug 1062201] Re: qemu core dumped by kvm-spice

2012-10-08 Thread Tom Wijmenga
Hey Serge, thanks for your response. Running the debug module like you asked me to, the attached text file was generated. About the crashing before booting Windows: I have no idea. It starts the VM for about one or two seconds, then dumps the core and kills the VM. I'd like to point out it kills

[Qemu-devel] [PATCH 00/12] RFC: usb: input pipelining support and other speedups

2012-10-08 Thread Hans de Goede
This patchset started out as adding support for pipelining of input bulk transfers, although input pipelining is a bit tricky to do, there are 2 good reasons to do it: 1) Performance, I expect this to ie significantly speed-up reading from USB mass storage devices. 2) Currently reading from a

[Qemu-devel] [PATCH 02/12] usb-redir: Add support for 32 bits bulk packet length

2012-10-08 Thread Hans de Goede
And also switch to using usb_ep_set_pipeline(), rather then modifying the ep directly. This is a preparation patch for adding input queuing support. Signed-off-by: Hans de Goede hdego...@redhat.com --- configure | 2 +- hw/usb/redirect.c | 20 +--- 2 files changed, 18

[Qemu-devel] [PATCH 08/12] ehci: Get rid of packet tbytes field

2012-10-08 Thread Hans de Goede
This field is used in some places to track the tbytes field of the token, but in other places the field is used directly, use it directly everywhere for consistency. Signed-off-by: Hans de Goede hdego...@redhat.com --- hw/usb/hcd-ehci.c | 20 +--- 1 file changed, 9 insertions(+),

Re: [Qemu-devel] [patch 1/7] cirrus_vga: allow configurable vram size

2012-10-08 Thread Gerd Hoffmann
On 10/05/12 19:51, Marcelo Tosatti wrote: Allow RAM size to be configurable for cirrus, to allow migration compatibility from qemu-kvm. Signed-off-by: Marcelo Tosatti mtosa...@redhat.com Acked-by: Gerd Hoffmann kra...@redhat.com cheers, Gerd

[Qemu-devel] [PATCH 03/12] usb-host-linux: Only enabling pipeling for output endpoints

2012-10-08 Thread Hans de Goede
With the upcoming input pipelining support, large input packets may get submitted to the device, which require special handling when the packets ends up being split again by usb-host-linux due to usbfs limitations. The exact demands for properly handling larger split input transfers is explained

[Qemu-devel] [PATCH 11/12] ehci: Improve latency of interrupt delivery and async schedule scanning

2012-10-08 Thread Hans de Goede
While doing various performance tests of reading from USB mass storage devices I noticed the following:: 1) When an async handled packet completes, we don't immediately report an interrupt to the guest, instead we wait for the frame-timer to run and report it from there 2) If 1) has been

[Qemu-devel] [PATCH 04/12] usb: Add support for input pipelining

2012-10-08 Thread Hans de Goede
Currently we effectively only do pipelining for output endpoints, since controllers will stop filling the queue for a packet with stop the queue on a short read semantics enabled. This causes large input transfers to get split into multiple packets, which comes with a serious performance penalty.

[Qemu-devel] [PATCH 06/12] uhci: Move checks to continue queuing to uhci_fill_queue()

2012-10-08 Thread Hans de Goede
Rather then having a special check to start queuing after the first packet, and then another check for the other packets in uhci_fill_queue(), simply check the previous packet beforehand in uhci_fill_queue() Signed-off-by: Hans de Goede hdego...@redhat.com --- hw/usb/hcd-uhci.c | 10 +++---

Re: [Qemu-devel] bug in popcnt emulation with some register operand(s)?

2012-10-08 Thread malc
On Mon, 8 Oct 2012, Andriy Gapon wrote: I am running Qemu (plain, no kvm, etc) on an AMD 10h machine that provides popcnt instruction. Qemu advertises availability of pocnt to a guest as well. What I see in the guest that popcnt 0x20(%r12),%r8 instruction actually placed its result into

Re: [Qemu-devel] Ping [PATCH 0/2] Add TPCI200 and IP-Octal 232 IndustryPack emulation

2012-10-08 Thread Alberto Garcia
On Sun, Oct 07, 2012 at 12:19:07PM +0200, Avi Kivity wrote: So you could easily have struct IPackDevice { DeviceState qdev; int32_t slot; /* IRQ objects for the IndustryPack INT0# and INT1# */ qemu_irq *irq; MemoryRegion io_space; MemoryRegion

[Qemu-devel] [PATCH 05/12] uhci: Properly unmap packets on cancel / invalid pid

2012-10-08 Thread Hans de Goede
Packets with an invalid pid, or which were cancelled have usb_packet_map() called on them on init, but not usb_packet_unmap() before being freed. Signed-off-by: Hans de Goede hdego...@redhat.com --- hw/usb/hcd-uhci.c | 2 ++ 1 file changed, 2 insertions(+) diff --git a/hw/usb/hcd-uhci.c

[Qemu-devel] [PATCH 10/12] ehci: Add support for input queuing

2012-10-08 Thread Hans de Goede
Signed-off-by: Hans de Goede hdego...@redhat.com --- hw/usb/hcd-ehci.c | 31 +-- 1 file changed, 29 insertions(+), 2 deletions(-) diff --git a/hw/usb/hcd-ehci.c b/hw/usb/hcd-ehci.c index d9d4918..444db15 100644 --- a/hw/usb/hcd-ehci.c +++ b/hw/usb/hcd-ehci.c @@

Re: [Qemu-devel] [PATCH 2/2] Adding BAR0 for e500 PCI controller

2012-10-08 Thread Bhushan Bharat-R65777
Which device's initialization function you are talking about? static const TypeInfo e500_host_bridge_info = { .name = e500-host-bridge, .parent= TYPE_PCI_DEVICE, .instance_size = sizeof(PCIDevice), .class_init= e500_host_bridge_class_init, }; This

[Qemu-devel] [PATCH 07/12] uhci: Add support for input queuing

2012-10-08 Thread Hans de Goede
Signed-off-by: Hans de Goede hdego...@redhat.com --- hw/usb/hcd-uhci.c | 31 +-- 1 file changed, 25 insertions(+), 6 deletions(-) diff --git a/hw/usb/hcd-uhci.c b/hw/usb/hcd-uhci.c index 124d43a..63f2161 100644 --- a/hw/usb/hcd-uhci.c +++ b/hw/usb/hcd-uhci.c @@ -678,6

[Qemu-devel] [PATCH 09/12] ehci: Set int flag on a short input packet

2012-10-08 Thread Hans de Goede
According to 4.15.1.2 an interrupt must be raised when a short packet is received. If we don't do this it may take a significant time for the guest to notice a short trasnfer has completed, since only the last td will have its IOC flag set, and a short transfer may complete in an earlier packet.

[Qemu-devel] [PATCH 01/12] usb-redir: When a packet contains data on a stall, ignore the stall

2012-10-08 Thread Hans de Goede
It is possbile for bulk packets to transfer some of the data and to then stall. ATM our usb core allows us to return either data, or an error, not both. For now return the data rather then the stall when this happens, counting on further packets to detect the stall. In the future we should fix the

[Qemu-devel] CPU utilization between physical CPU and virtual CPU in KVM

2012-10-08 Thread Dennis Chen
Hi All, I am confused by the following observed scenario: In my 4-CPU (KVM supported, 2 core with 2 thread for each) host machine box, I create only one VM with 3-vCPU through virsh/libvirt tools and also I pin this VM process to the physical processor 3. I guess the CPU utilization for the

[Qemu-devel] [PATCH 12/12] ehci: Speed up the timer of raising int from the async schedule

2012-10-08 Thread Hans de Goede
Often the guest will queue up new packets in response to a packet, in the async schedule with its IOC flag set, completing. By speeding up the frame-timer, we notice these new packets earlier. This increases the speed (MB/s) of a Linux guest reading from a USB mass storage device by a factor of

Re: [Qemu-devel] [PATCH 2/2] Adding BAR0 for e500 PCI controller

2012-10-08 Thread Alexander Graf
On 08.10.2012, at 10:23, Bhushan Bharat-R65777 wrote: Which device's initialization function you are talking about? static const TypeInfo e500_host_bridge_info = { .name = e500-host-bridge, .parent= TYPE_PCI_DEVICE, .instance_size = sizeof(PCIDevice), .class_init

[Qemu-devel] [PATCH v10 04/14] target-mips-ase-dsp: Add branch instructions

2012-10-08 Thread Jia Liu
Add MIPS ASE DSP Branch instructions. Signed-off-by: Jia Liu pro...@gmail.com --- target-mips/translate.c | 36 1 file changed, 36 insertions(+) diff --git a/target-mips/translate.c b/target-mips/translate.c index b023d6f..f1e5bb0 100644 ---

[Qemu-devel] [PATCH v10 10/14] target-mips-ase-dsp: Add compare-pick instructions

2012-10-08 Thread Jia Liu
Add MIPS ASE DSP Compare-Pick instructions. Signed-off-by: Jia Liu pro...@gmail.com --- target-mips/dsp_helper.c | 233 + target-mips/helper.h | 52 +++ target-mips/translate.c | 372 ++ 3 files changed, 657

[Qemu-devel] [PATCH v10 11/14] target-mips-ase-dsp: Add DSP accumulator instructions

2012-10-08 Thread Jia Liu
Add MIPS ASE DSP Accumulator and DSPControl Access instructions. Signed-off-by: Jia Liu pro...@gmail.com --- target-mips/dsp_helper.c | 609 ++ target-mips/helper.h | 35 +++ target-mips/translate.c | 355 +++ 3 files

[Qemu-devel] [PATCH v10 12/14] target-mips-ase-dsp: Add MIPS DSP processors

2012-10-08 Thread Jia Liu
Add 74kf and mips64dspr2-generic-cpu model for test. Signed-off-by: Jia Liu pro...@gmail.com --- target-mips/translate_init.c | 52 ++ 1 file changed, 52 insertions(+) diff --git a/target-mips/translate_init.c b/target-mips/translate_init.c index

[Qemu-devel] [PATCH v10 14/14] target-mips-ase-dsp: Change TODO file

2012-10-08 Thread Jia Liu
Delete DSP r1 DSP r2 from TODO file. Signed-off-by: Jia Liu pro...@gmail.com --- target-mips/TODO |2 -- 1 file changed, 2 deletions(-) diff --git a/target-mips/TODO b/target-mips/TODO index 2a3546f..15d67cd 100644 --- a/target-mips/TODO +++ b/target-mips/TODO @@ -6,8 +6,6 @@ General -

[Qemu-devel] [PATCH v10 05/14] target-mips-ase-dsp: Add load instructions

2012-10-08 Thread Jia Liu
Add MIPS ASE DSP Load instructions. Signed-off-by: Jia Liu pro...@gmail.com --- target-mips/translate.c | 89 +++ 1 file changed, 89 insertions(+) diff --git a/target-mips/translate.c b/target-mips/translate.c index f1e5bb0..7f08700 100644 ---

Re: [Qemu-devel] bug in popcnt emulation with some register operand(s)?

2012-10-08 Thread Andriy Gapon
on 08/10/2012 10:52 malc said the following: On Mon, 8 Oct 2012, Andriy Gapon wrote: I am running Qemu (plain, no kvm, etc) on an AMD 10h machine that provides popcnt instruction. Qemu advertises availability of pocnt to a guest as well. What I see in the guest that popcnt 0x20(%r12),%r8

[Qemu-devel] [PATCH v10 03/14] target-mips-ase-dsp: Use correct acc value to index cpu_HI/cpu_LO rather than using a fix number

2012-10-08 Thread Jia Liu
Use correct acc value to index cpu_HI/cpu_LO rather than using a fix number. Signed-off-by: Jia Liu pro...@gmail.com --- target-mips/translate.c | 122 --- 1 file changed, 95 insertions(+), 27 deletions(-) diff --git a/target-mips/translate.c

[Qemu-devel] [PATCH v10 02/14] target-mips-ase-dsp: Add dsp resources access check

2012-10-08 Thread Jia Liu
Add MIPS ASE DSP resources access check. Signed-off-by: Jia Liu pro...@gmail.com --- linux-user/main.c |6 ++ target-mips/cpu.h | 23 +-- target-mips/helper.c|3 +++ target-mips/translate.c | 23 +++ 4 files changed, 53

[Qemu-devel] [PATCH v10 01/14] target-mips-ase-dsp: Add internal functions

2012-10-08 Thread Jia Liu
Add internal functions using by MIPS ASE DSP instructions. Signed-off-by: Jia Liu pro...@gmail.com --- target-mips/Makefile.objs |2 +- target-mips/dsp_helper.c | 1086 + 2 files changed, 1087 insertions(+), 1 deletion(-) create mode 100644

[Qemu-devel] [PATCH v10 07/14] target-mips-ase-dsp: Add GPR-based shift instructions

2012-10-08 Thread Jia Liu
Add MIPS ASE DSP GPR-Based Shift instructions. Signed-off-by: Jia Liu pro...@gmail.com --- target-mips/dsp_helper.c | 256 target-mips/helper.h | 38 ++ target-mips/translate.c | 328 ++ 3 files

[Qemu-devel] [PATCH v10 09/14] target-mips-ase-dsp: Add bit/manipulation instructions

2012-10-08 Thread Jia Liu
Add MIPS ASE DSP Bit/Manipulation instructions. Signed-off-by: Jia Liu pro...@gmail.com --- target-mips/dsp_helper.c | 55 +++ target-mips/helper.h |7 ++ target-mips/translate.c | 231 ++ 3 files changed, 293 insertions(+) diff

Re: [Qemu-devel] [PATCH 1/1] kvmclock: fix guest stop notification

2012-10-08 Thread Amit Shah
On (Sun) 30 Sep 2012 [20:05:16], Marcelo Tosatti wrote: On Thu, Sep 20, 2012 at 09:46:41AM -0300, Marcelo Tosatti wrote: On Thu, Sep 20, 2012 at 01:55:20PM +0530, Amit Shah wrote: Commit f349c12c0434e29c79ecde89029320c4002f7253 added the guest stop notification, but it did it in a way

Re: [Qemu-devel] [PATCH 1/1] kvmclock: fix guest stop notification

2012-10-08 Thread Amit Shah
On (Sun) 30 Sep 2012 [21:50:07], Amos Kong wrote: - Original Message - On Thu, Sep 20, 2012 at 09:46:41AM -0300, Marcelo Tosatti wrote: On Thu, Sep 20, 2012 at 01:55:20PM +0530, Amit Shah wrote: Commit f349c12c0434e29c79ecde89029320c4002f7253 added the guest stop In

[Qemu-devel] [PATCH] vfio: Fix BAR-VFIODevice translation in vfio_bar_read/write

2012-10-08 Thread Jan Kiszka
DO_UPCAST is supposed to translate from the first member of a struct to that struct, not from arbitrary ones. And it (usually) breaks the build when neglecting this rule. Use container_of to fix the build breakage and likely also the runtime behavior. Signed-off-by: Jan Kiszka

[Qemu-devel] [PULL 0/8] spice patch queue

2012-10-08 Thread Gerd Hoffmann
Hi, Here comes the spice patch queue. It raises the minimal required spice-server version to 0.12 and the qxl device revision to 4. It also brings a collection of bugfixes. please pull, Gerd The following changes since commit 4bb26682f70a5f626cad3e0ac82bf4b6252ea7a4: Merge branch

[Qemu-devel] [PATCH 7/8] spice: raise requirement to 0.12

2012-10-08 Thread Gerd Hoffmann
With the next qemu version (1.3) we are going to bump the qxl device revision to 4. The new features available require a recent spice-server version, so raise up the bar. Otherwise we would end up with different qxl revisions depending on the spice-server version installed, which would be a

[Qemu-devel] [PATCH 1/8] qxl: always update displaysurface on resize

2012-10-08 Thread Gerd Hoffmann
Don't try to be clever and skip displaysurface reinitialization in case the size hasn't changed. Other parameters might have changed nevertheless, for example depth or stride, resulting in rendering being broken then. Trigger: boot linux guest with vesafb, start X11, make sure both vesafb and

[Qemu-devel] [PATCH 5/8] hw/qxl: fix condition for exiting guest_bug

2012-10-08 Thread Gerd Hoffmann
From: Alon Levy al...@redhat.com Reported and suggested by Paolo Bonzini, thanks. Signed-off-by: Alon Levy al...@redhat.com Signed-off-by: Gerd Hoffmann kra...@redhat.com --- hw/qxl.c |2 +- 1 files changed, 1 insertions(+), 1 deletions(-) diff --git a/hw/qxl.c b/hw/qxl.c index

Re: [Qemu-devel] Building QEMU with multiple CPU targets.

2012-10-08 Thread Peter Maydell
On 8 October 2012 07:39, Peter Crosthwaite peter.crosthwa...@petalogix.com wrote: Im currently investigating the possibility of building QEMU with multiple CPU architectures active concurrently. That is, I have a binary with both an target-arm and target-microblaze and wish to run them as a

Re: [Qemu-devel] RFC: making QMP query-block more useful

2012-10-08 Thread Daniel P. Berrange
On Fri, Oct 05, 2012 at 05:52:35PM -0600, Eric Blake wrote: Right now, 'query-block' has no way to filter to a single device, but conversely, for each device, it shows only the first backing file, rather than the entire backing chain. Jeff and I were lamenting this fact on IRC while debugging

[Qemu-devel] [PATCH 6/8] hw/qxl: qxl_dirty_surfaces: use uintptr_t

2012-10-08 Thread Gerd Hoffmann
From: Alon Levy al...@redhat.com As suggested by Paolo Bonzini, to avoid possible integer overflow issues. Signed-off-by: Alon Levy al...@redhat.com Signed-off-by: Gerd Hoffmann kra...@redhat.com --- hw/qxl.c |4 ++-- 1 files changed, 2 insertions(+), 2 deletions(-) diff --git a/hw/qxl.c

[Qemu-devel] [PATCH 2/8] qxl/update_area_io: cleanup invalid parameters handling

2012-10-08 Thread Gerd Hoffmann
From: Michael Tokarev m...@tls.msk.ru This cleans up two additions of almost the same code in commits 511b13e2c9 and ccc2960d654. While at it, make error paths consistent (always use 'break' instead of 'return'). Signed-off-by: Michael Tokarev m...@tls.msk.ru Cc: Dunrong Huang

Re: [Qemu-devel] [PATCH] hw/qxl: qxl_dirty_surfaces: use uintptr_t

2012-10-08 Thread Gerd Hoffmann
On 10/07/12 17:03, Alon Levy wrote: As suggested by Paolo Bonzini, to avoid possible integer overflow issues. Patch added to spice patch queue, likewise the other two qxl fixes. thanks, Gerd

[Qemu-devel] [PATCH 8/8] qxl: set default revision to 4

2012-10-08 Thread Gerd Hoffmann
Set qxl pci revision to 4 (for pc-1.3+) so guests know spice-server 0.12 features are available. Signed-off-by: Gerd Hoffmann kra...@redhat.com --- hw/pc_piix.c |8 hw/qxl.h |5 - 2 files changed, 8 insertions(+), 5 deletions(-) diff --git a/hw/pc_piix.c b/hw/pc_piix.c

Re: [Qemu-devel] Building QEMU with multiple CPU targets.

2012-10-08 Thread Evgeny Voevodin
On 10/08/2012 02:54 PM, Peter Maydell wrote: On 8 October 2012 07:39, Peter Crosthwaite peter.crosthwa...@petalogix.com wrote: Im currently investigating the possibility of building QEMU with multiple CPU architectures active concurrently. That is, I have a binary with both an target-arm and

[Qemu-devel] [PATCH 4/8] hw/qxl: exit on failure to register qxl interface

2012-10-08 Thread Gerd Hoffmann
From: Alon Levy al...@redhat.com This prevents a segfault later on when the device reset handler tries to access a NULL ssd.worker since interface_attach_worker has not been called. Signed-off-by: Alon Levy al...@redhat.com Signed-off-by: Gerd Hoffmann kra...@redhat.com --- hw/qxl.c |6

Re: [Qemu-devel] EHCI USB regression in 1.2.0 - ehci_state_fetchqtd() asserting

2012-10-08 Thread Hans de Goede
Hi, On 10/02/2012 05:26 PM, Shawn Starr wrote: On Monday, September 24, 2012 10:38:57 AM Shawn Starr wrote: On Monday, September 24, 2012 04:36:15 PM Hans de Goede wrote: Hi, Hello, Reopening this issue with usb-host stalling now ehci warning: guest updated active QH USBDEVFS_DISCARDURB:

[Qemu-devel] [PATCH 3/8] qxl: fix range check for rev3 io commands.

2012-10-08 Thread Gerd Hoffmann
Enables QXL_IO_FLUSH_SURFACES_ASYNC and QXL_IO_FLUSH_RELEASE which are part of the qxl rev3 feature set. Signed-off-by: Gerd Hoffmann kra...@redhat.com --- hw/qxl.c |2 +- 1 files changed, 1 insertions(+), 1 deletions(-) diff --git a/hw/qxl.c b/hw/qxl.c index 3c82c2a..eb7707c 100644 ---

Re: [Qemu-devel] [RFC PATCH 00/17] Support for multiple AIO contexts

2012-10-08 Thread Stefan Hajnoczi
On Tue, Sep 25, 2012 at 02:55:46PM +0200, Paolo Bonzini wrote: This series removes the globals from async.c/aio-posix.c so that multiple AIO contexts (mini event loops) can be added. Right now, all block devices still use qemu_bh_new, but switching them to aio_bh_new would let you associate

[Qemu-devel] [PATCH v2] target-i386: initialize APIC at CPU level

2012-10-08 Thread Igor Mammedov
(L)APIC is a part of cpu [1] so move APIC initialization inside of x86_cpu object. Since cpu_model and override flags currently specify whether APIC should be created or not, APIC creationinitialization is moved into x86_cpu_apic_init() which is called from x86_cpu_realize(). [1] - all x86 cpus

[Qemu-devel] [PATCH 1/4] rtl8139: implement 8139cp link status

2012-10-08 Thread Stefan Hajnoczi
From: Jason Wang jasow...@redhat.com Add a link status chang callback and change the link status bit in BMSR MSR accordingly. Tested in Linux/Windows guests. The link status bit of MediaStatus is infered from BasicModeStatus, they are inverse. nc.link_down could not be migrated, this patch

[Qemu-devel] [PULL 0/4] Net patches

2012-10-08 Thread Stefan Hajnoczi
The following changes since commit 4bb26682f70a5f626cad3e0ac82bf4b6252ea7a4: Merge branch 'master' of git.qemu.org:/pub/git/qemu (2012-10-07 18:42:18 +) are available in the git repository at: git://github.com/stefanha/qemu.git net for you to fetch changes up to

[Qemu-devel] [PATCH 2/4] e1000: update nc.link_down in e1000_post_load()

2012-10-08 Thread Stefan Hajnoczi
From: Amos Kong ak...@redhat.com This patch introduced e1000_post_load(), it will be called in the end of migration. nc.link_down could not be migrated, this patch updates link_down in e1000_post_load() to keep it coincident with real link status. Signed-off-by: Amos Kong ak...@redhat.com

[Qemu-devel] [PATCH 4/4] net: consolidate NetClientState header files into one

2012-10-08 Thread Stefan Hajnoczi
From: Paolo Bonzini pbonz...@redhat.com This patch doesn't seem much useful alone, I must admit. However, it makes sense as part of the upcoming directory reorganization, where I want to have include/net/tap.h as the net-hw interface for tap. Then having both net/tap.h and include/net/tap.h

[Qemu-devel] [PATCH 3/4] virtio-net: update nc.link_down in virtio_net_load()

2012-10-08 Thread Stefan Hajnoczi
From: Amos Kong ak...@redhat.com nc.link_down could not be migrated, this patch updates link_down in virtio_post_load() to keep it coincident with real link status. Signed-off-by: Amos Kong ak...@redhat.com Signed-off-by: Stefan Hajnoczi stefa...@gmail.com --- hw/virtio-net.c | 5 + 1 file

Re: [Qemu-devel] [PATCH v1 04/23] xen_pt: use separate MemoryListeners for memory and I/O

2012-10-08 Thread Stefano Stabellini
On Sun, 7 Oct 2012, Avi Kivity wrote: Using an unfiltered memory listener will cause regions to be reported fails multiple times if we have more than two address spaces. Use a separate listener for memory and I/O, and utilize MemoryListener's address space filtering to fix this.

[Qemu-devel] [PATCH 04/14] i8254: convert PIO to new memory api read/write

2012-10-08 Thread Alexander Graf
Signed-off-by: Alexander Graf ag...@suse.de --- hw/i8254.c | 20 +++- 1 files changed, 11 insertions(+), 9 deletions(-) diff --git a/hw/i8254.c b/hw/i8254.c index 77bd5e8..cea8d13 100644 --- a/hw/i8254.c +++ b/hw/i8254.c @@ -111,7 +111,8 @@ static void

[Qemu-devel] [PATCH 11/14] vmport: convert PIO to new memory api read/write

2012-10-08 Thread Alexander Graf
Signed-off-by: Alexander Graf ag...@suse.de --- hw/vmport.c | 21 - 1 files changed, 12 insertions(+), 9 deletions(-) diff --git a/hw/vmport.c b/hw/vmport.c index a4f52ee..77dc8d6 100644 --- a/hw/vmport.c +++ b/hw/vmport.c @@ -54,7 +54,8 @@ void vmport_register(unsigned

[Qemu-devel] [PATCH 02/14] virtio-pci: convert PIO to new memory api read/write

2012-10-08 Thread Alexander Graf
Signed-off-by: Alexander Graf ag...@suse.de --- hw/virtio-pci.c | 126 +- 1 files changed, 49 insertions(+), 77 deletions(-) diff --git a/hw/virtio-pci.c b/hw/virtio-pci.c index 400f3c2..e28254f 100644 --- a/hw/virtio-pci.c +++

[Qemu-devel] [PATCH 03/14] es1370: convert PIO to new memory api read/write

2012-10-08 Thread Alexander Graf
Signed-off-by: Alexander Graf ag...@suse.de --- hw/es1370.c | 46 -- 1 files changed, 36 insertions(+), 10 deletions(-) diff --git a/hw/es1370.c b/hw/es1370.c index e34234c..308f2c2 100644 --- a/hw/es1370.c +++ b/hw/es1370.c @@ -908,18 +908,44 @@

[Qemu-devel] [PATCH 01/14] ac97: convert PIO to new memory api read/write

2012-10-08 Thread Alexander Graf
Signed-off-by: Alexander Graf ag...@suse.de --- hw/ac97.c | 109 +--- 1 files changed, 89 insertions(+), 20 deletions(-) diff --git a/hw/ac97.c b/hw/ac97.c index 0f561fa..c1cc3dd 100644 --- a/hw/ac97.c +++ b/hw/ac97.c @@ -1226,32 +1226,101

[Qemu-devel] [PATCH 07/14] pc port92: convert PIO to new memory api read/write

2012-10-08 Thread Alexander Graf
Signed-off-by: Alexander Graf ag...@suse.de --- hw/pc.c | 19 +++ 1 files changed, 11 insertions(+), 8 deletions(-) diff --git a/hw/pc.c b/hw/pc.c index 7e7e0e2..9eb1230 100644 --- a/hw/pc.c +++ b/hw/pc.c @@ -422,7 +422,8 @@ typedef struct Port92State { qemu_irq *a20_out;

[Qemu-devel] [PATCH 10/14] serial: convert PIO to new memory api read/write

2012-10-08 Thread Alexander Graf
Signed-off-by: Alexander Graf ag...@suse.de --- hw/serial.c | 31 ++- 1 files changed, 18 insertions(+), 13 deletions(-) diff --git a/hw/serial.c b/hw/serial.c index a421d1e..abae1e7 100644 --- a/hw/serial.c +++ b/hw/serial.c @@ -28,6 +28,7 @@ #include pc.h

[Qemu-devel] [PATCH 12/14] xen_platform: convert PIO to new memory api read/write

2012-10-08 Thread Alexander Graf
Signed-off-by: Alexander Graf ag...@suse.de --- hw/xen_platform.c | 48 ++-- 1 files changed, 38 insertions(+), 10 deletions(-) diff --git a/hw/xen_platform.c b/hw/xen_platform.c index 956dbfe..171f8ff 100644 --- a/hw/xen_platform.c +++

Re: [Qemu-devel] [PATCH 04/12] usb: Add support for input pipelining

2012-10-08 Thread Gerd Hoffmann
On 10/08/12 09:51, Hans de Goede wrote: Currently we effectively only do pipelining for output endpoints, since controllers will stop filling the queue for a packet with stop the queue on a short read semantics enabled. This causes large input transfers to get split into multiple packets,

[Qemu-devel] [PATCH 08/14] pckbd: convert PIO to new memory api read/write

2012-10-08 Thread Alexander Graf
Signed-off-by: Alexander Graf ag...@suse.de --- hw/pckbd.c | 48 +++- 1 files changed, 27 insertions(+), 21 deletions(-) diff --git a/hw/pckbd.c b/hw/pckbd.c index 69857ba..2c78f7d 100644 --- a/hw/pckbd.c +++ b/hw/pckbd.c @@ -194,7 +194,8 @@ static

Re: [Qemu-devel] EHCI USB regression in 1.2.0 - ehci_state_fetchqtd() asserting

2012-10-08 Thread Johannes Stezenbach
Hi Hans, On Mon, Oct 08, 2012 at 01:27:28PM +0200, Hans de Goede wrote: On 10/02/2012 05:26 PM, Shawn Starr wrote: Reopening this issue with usb-host stalling now ehci warning: guest updated active QH USBDEVFS_DISCARDURB: Invalid argument USBDEVFS_DISCARDURB: Invalid argument husb:

[Qemu-devel] [PATCH 13/14] PPC: e500: Map PIO space into core memory region

2012-10-08 Thread Alexander Graf
On PPC, we don't have PIO. So usually PIO space behind a PCI bridge is accessible via MMIO. Do this mapping explicitly by mapping the PIO space of our PCI bus into a memory region that lives in memory space. Signed-off-by: Alexander Graf ag...@suse.de --- hw/ppc/e500.c|3 +--

Re: [Qemu-devel] [PATCH v2] qom: Reject attempts to add a property that already exists

2012-10-08 Thread Peter Maydell
Ping! -- PMM On 7 September 2012 14:55, Peter Maydell peter.mayd...@linaro.org wrote: Reject attempts to add a property to an object if one of that name already exists. This is always a bug in the caller; this is merely diagnosing it gracefully rather than behaving oddly later.

Re: [Qemu-devel] [PATCH 10/14] qxl: add QXL_IO_MONITORS_CONFIG_ASYNC

2012-10-08 Thread Paolo Bonzini
Il 07/10/2012 11:35, Alon Levy ha scritto: Also, please consider adding the library version in the spice-protocol header files, so that you do not need CONFIG_QXL_IO_MONITORS_CONFIG_ASYNC at all. spice-protocol is a separate repository from spice library. Yes, I mean adding to the

[Qemu-devel] [PATCH 09/14] rtl8139: convert PIO to new memory api read/write

2012-10-08 Thread Alexander Graf
Signed-off-by: Alexander Graf ag...@suse.de --- hw/rtl8139.c | 78 ++--- 1 files changed, 36 insertions(+), 42 deletions(-) diff --git a/hw/rtl8139.c b/hw/rtl8139.c index b7c82ee..6be6883 100644 --- a/hw/rtl8139.c +++ b/hw/rtl8139.c @@

Re: [Qemu-devel] Building QEMU with multiple CPU targets.

2012-10-08 Thread Andreas Färber
Hi Peter, Am 08.10.2012 08:39, schrieb Peter Crosthwaite: Im currently investigating the possibility of building QEMU with multiple CPU architectures active concurrently. That is, I have a binary with both an target-arm and target-microblaze and wish to run them as a heterogeneous

[Qemu-devel] [PATCH 00/14] Remove old_portio users for memory region PIO mapping

2012-10-08 Thread Alexander Graf
When running on PowerPC, we don't have native PIO support. There are a few hacks around to enable PIO access on PowerPC nevertheless. The most typical one is the isa-mmio device. It takes MMIO requests and converts them to PIO requests on the (QEMU internal) PIO bus. This however is not how real

[Qemu-devel] [PATCH 05/14] m48t59: convert PIO to new memory api read/write

2012-10-08 Thread Alexander Graf
Signed-off-by: Alexander Graf ag...@suse.de --- hw/m48t59.c | 24 ++-- 1 files changed, 14 insertions(+), 10 deletions(-) diff --git a/hw/m48t59.c b/hw/m48t59.c index dd6cb37..389f6c9 100644 --- a/hw/m48t59.c +++ b/hw/m48t59.c @@ -27,6 +27,7 @@ #include sysemu.h #include

[Qemu-devel] [PATCH 14/14] PPC: pseries: Remove hack for PIO window

2012-10-08 Thread Alexander Graf
Now that all users of old_portio are gone, we can remove the hack that enabled us to support them. Signed-off-by: Alexander Graf ag...@suse.de --- hw/spapr_pci.c | 44 +--- hw/spapr_pci.h |2 +- 2 files changed, 2 insertions(+), 44 deletions(-) diff

[Qemu-devel] [PATCH 06/14] mc146818rtc: convert PIO to new memory api read/write

2012-10-08 Thread Alexander Graf
Signed-off-by: Alexander Graf ag...@suse.de --- hw/mc146818rtc.c | 19 +++ 1 files changed, 11 insertions(+), 8 deletions(-) diff --git a/hw/mc146818rtc.c b/hw/mc146818rtc.c index 332a77d..670f826 100644 --- a/hw/mc146818rtc.c +++ b/hw/mc146818rtc.c @@ -383,7 +383,8 @@ static

Re: [Qemu-devel] [PATCH v2] qom: Reject attempts to add a property that already exists

2012-10-08 Thread Anthony Liguori
Peter Maydell peter.mayd...@linaro.org writes: Ping! This is wrong. Container properties are added by the user. You will turn a gracefully failure (during hotplug) into an abort(). Please limit this to static properties as they are not added by a user. Regards, Anthony Liguori -- PMM

Re: [Qemu-devel] [PATCH v2] qom: Reject attempts to add a property that already exists

2012-10-08 Thread Peter Crosthwaite
On Mon, Oct 8, 2012 at 11:29 PM, Anthony Liguori aligu...@us.ibm.com wrote: Peter Maydell peter.mayd...@linaro.org writes: Ping! This is wrong. Container properties are added by the user. You will turn a gracefully failure (during hotplug) into an abort(). Can we just populate errp with

Re: [Qemu-devel] [PATCH v2] qom: Reject attempts to add a property that already exists

2012-10-08 Thread Peter Maydell
On 8 October 2012 14:29, Anthony Liguori aligu...@us.ibm.com wrote: This is wrong. Container properties are added by the user. You will turn a gracefully failure (during hotplug) into an abort(). No, it's turning a bug into an abort -- we don't handle trying to create two identically named

Re: [Qemu-devel] EHCI USB regression in 1.2.0 - ehci_state_fetchqtd() asserting

2012-10-08 Thread Hans de Goede
Hi, On 10/08/2012 03:01 PM, Johannes Stezenbach wrote: Hi Hans, On Mon, Oct 08, 2012 at 01:27:28PM +0200, Hans de Goede wrote: On 10/02/2012 05:26 PM, Shawn Starr wrote: Reopening this issue with usb-host stalling now ehci warning: guest updated active QH USBDEVFS_DISCARDURB: Invalid

[Qemu-devel] [Bug 1062201] Re: qemu core dumped by kvm-spice

2012-10-08 Thread Serge Hallyn
@Tom, you are running an experimental PPA containing unstable usb redirection code. Your best bets are to (a) try with the quantal or upstream package, (b) if you don't really need the experimental usb redirection, use the precise packages, or (c) contact the owner of the PPA. Qemu 1.1 (which

Re: [Qemu-devel] [RFC PATCH 00/17] Support for multiple AIO contexts

2012-10-08 Thread Paolo Bonzini
Il 08/10/2012 13:39, Stefan Hajnoczi ha scritto: This series looks useful - it compartmentalizes aio.c so there can be multiple event loops. In order to get a performance benefit (hooking up virtio-blk ioeventfd to a non-QEMU mutex thread) we need two more things: 1. Block layer association

Re: [Qemu-devel] [PATCH v2] qom: Reject attempts to add a property that already exists

2012-10-08 Thread Peter Maydell
On 8 October 2012 14:38, Peter Crosthwaite peter.crosthwa...@petalogix.com wrote: Can we just populate errp with a nice meaningful error (perhaps the contents of that printf), then the caller can decide if failure is tolerable? I would find this approach more plausible if it wasn't that so

Re: [Qemu-devel] [PATCH 2/2] Adding BAR0 for e500 PCI controller

2012-10-08 Thread Andreas Färber
Am 08.10.2012 10:50, schrieb Alexander Graf: On 08.10.2012, at 10:23, Bhushan Bharat-R65777 wrote: @@ -307,6 +313,16 @@ static const VMStateDescription vmstate_ppce500_pci = { #include exec-memory.h +static int e500_pcihost_bridge_initfn(PCIDevice *d) { +bharat *b = DO_UPCAST(bharat,

Re: [Qemu-devel] Using PCI config space to indicate config location

2012-10-08 Thread Anthony Liguori
Rusty Russell ru...@rustcorp.com.au writes: (Topic updated, cc's trimmed). Anthony Liguori aligu...@us.ibm.com writes: Rusty Russell ru...@rustcorp.com.au writes: 4) The only significant change to the spec is that we use PCI capabilities, so we can have infinite feature bits. (see

Re: [Qemu-devel] [PATCH] kvm: Set default accelerator to kvm if the host supports it

2012-10-08 Thread Andreas Färber
Am 05.10.2012 04:24, schrieb Alexander Graf: On 05.10.2012, at 04:17, Anthony Liguori wrote: Alexander Graf ag...@suse.de writes: On 03.10.2012, at 22:26, Peter Maydell wrote: On 3 October 2012 21:01, Blue Swirl blauwir...@gmail.com wrote: On Mon, Oct 1, 2012 at 4:20 PM, Anthony Liguori

Re: [Qemu-devel] [PATCH] kvm: Set default accelerator to kvm if the host supports it

2012-10-08 Thread Alexander Graf
On 08.10.2012, at 16:03, Andreas Färber wrote: Am 05.10.2012 04:24, schrieb Alexander Graf: On 05.10.2012, at 04:17, Anthony Liguori wrote: Alexander Graf ag...@suse.de writes: On 03.10.2012, at 22:26, Peter Maydell wrote: On 3 October 2012 21:01, Blue Swirl blauwir...@gmail.com

Re: [Qemu-devel] [PATCH 1/2] usb-redir: Change usbredir_open_chardev into usbredir_create_parser

2012-10-08 Thread Gerd Hoffmann
On 09/27/12 17:25, Hans de Goede wrote: As we need to create the parser at more places. Both patches added to usb patch queue. thanks, Gerd

  1   2   3   >