Re: [Qemu-devel] [PATCH 3/7] dataplane: add virtqueue vring code

2012-11-16 Thread Stefan Hajnoczi
On Fri, Nov 16, 2012 at 8:48 AM, Christian Borntraeger borntrae...@de.ibm.com wrote: On 15/11/12 16:19, Stefan Hajnoczi wrote: +#include trace.h +#include hw/dataplane/vring.h + +/* Map target physical address to host address + */ +static inline void *phys_to_host(Vring *vring, hwaddr

Re: [Qemu-devel] [PATCH 1/7] raw-posix: add raw_get_aio_fd() for virtio-blk-data-plane

2012-11-16 Thread Paolo Bonzini
Il 16/11/2012 07:15, Stefan Hajnoczi ha scritto: So why not just make a string 'filename' property and open it directly in virtio-blk-data-plane? Then it's at least clear to the user and management tools what the device is capable of doing. There are some benefits to raw_get_aio_fd():

Re: [Qemu-devel] [PATCH] hw/virtio-serial-bus: post_load send_event when vm is running

2012-11-16 Thread Amit Shah
On (Wed) 14 Nov 2012 [15:09:07], Alon Levy wrote: Add a new timer based on vm_clock for 1 ns in the future from post_load to do the event send in case host_connected differs between migration source and target. RHBZ: 867366 Signed-off-by: Alon Levy al...@redhat.com ---

[Qemu-devel] TCP based PCIE request forwarding

2012-11-16 Thread lementec fabien
Hi, I am a software engineer who works in an electronic group. Using QEMU to emulate devices allows me to start writing and testing LINUX software before the device is actually available. In the group, we are mostly working with XILINX FPGAs, communicating with the host via PCIE. The devices are

Re: [Qemu-devel] [PATCH] iscsi: fix deadlock during login

2012-11-16 Thread Peter Lieven
Am 16.11.2012 08:44, schrieb Paolo Bonzini: Il 15/11/2012 19:28, Peter Lieven ha scritto: I dont know if we should switch to use synchronous code here. It is much nicer if all code is async. bdrv_open is generally synchronous, so I think Peter's patch is ok. if all is sync wouldn't it be best

Re: [Qemu-devel] [PATCH 1/2] Buildsystem fix distclean error in pixman

2012-11-16 Thread Peter Maydell
On 16 November 2012 05:05, Wenchao Xia xiaw...@linux.vnet.ibm.com wrote: Currently if pixman have no config.log inside, make file still try to clean it resulting error. This patch fix it. - test -f pixman/config.log make -C pixman distclean + @if test -f pixman/config.log; \ +

Re: [Qemu-devel] [PATCH v2 05/39] fdsets: use weak aliases instead of qemu-tool.c/qemu-user.c

2012-11-16 Thread Paolo Bonzini
Il 15/11/2012 23:18, Stefan Weil ha scritto: Am 15.11.2012 21:52, schrieb Paolo Bonzini: Il 15/11/2012 19:01, Stefan Weil ha scritto: Hi Paolo, this patch breaks QEMU on 32 and 64 bit hosts, native and with Wine. It's easy to reproduce the SIGSEGV crash: just add a -snapshot option.

[Qemu-devel] QEMU hangs when shutdown windows7 guest with virtio-serial drivers installed

2012-11-16 Thread Dunrong Huang
Hi all: I meet a weird problem: If I I boot QEMU with virtio-serial being enabled and assign only one cpu to windows VM, when I shutdown VM, QEMU hangs with using 100% cpu. the command line I use is: ~/usr/bin/qemu-system-x86_64 -enable-kvm -m 1024 -smp 1 -device virtio-serial-pci -chardev

Re: [Qemu-devel] [PATCH 1/2] Buildsystem fix distclean error in pixman

2012-11-16 Thread Wenchao Xia
于 2012-11-16 17:27, Peter Maydell 写道: On 16 November 2012 05:05, Wenchao Xia xiaw...@linux.vnet.ibm.com wrote: Currently if pixman have no config.log inside, make file still try to clean it resulting error. This patch fix it. - test -f pixman/config.log make -C pixman distclean +

[Qemu-devel] [PATCH V9 0/8] libqblock qemu block layer library

2012-11-16 Thread Wenchao Xia
These patches introduce libqblock API, make libqblock.la and make check-libqblock could build this library. Functionalities: 1 create a new image. 2 sync access of an image. 3 basic image information retrieving such as backing file. 4 detect if a sector is allocated in an image. Supported

[Qemu-devel] [PATCH V9 6/8] libqblock type defines

2012-11-16 Thread Wenchao Xia
This patch contains type and macro defines used in APIs, one file for public usage, one for libqblock internal usage. Signed-off-by: Wenchao Xia xiaw...@linux.vnet.ibm.com --- libqblock/libqblock-internal.h | 75 libqblock/libqblock-types.h| 252

[Qemu-devel] [PATCH V9 5/8] libqblock build system

2012-11-16 Thread Wenchao Xia
Libqblock was placed in new directory ./libqblock, libtool will build dynamic library there, source files of block layer remains in ./block. So block related source code will generate 3 sets of binary, first is old ones used in qemu, second and third are non PIC and PIC ones in ./libqblock.

[Qemu-devel] [PATCH V9 7/8] libqblock API

2012-11-16 Thread Wenchao Xia
This patch contains the major APIs in the library. For ABI some reserved members were used. Important APIs: 1 QBlockContext. This structure was used to retrieve errors, every thread must create one first. 2 QBlockState. It stands for an block image object. 3 QBlockStaticInfo. It contains

[Qemu-devel] [PATCH V9 8/8] libqblock test example

2012-11-16 Thread Wenchao Xia
In this example, first it will create some qcow2 images, then try get information including backing file relationship, then it will do sync IO on the image. Signed-off-by: Wenchao Xia xiaw...@linux.vnet.ibm.com --- tests/libqblock/libqblock-qcow2.c | 390 -

Re: [Qemu-devel] [PATCH 1/2] Buildsystem fix distclean error in pixman

2012-11-16 Thread Peter Maydell
On 16 November 2012 10:08, Wenchao Xia xiaw...@linux.vnet.ibm.com wrote: 于 2012-11-16 17:27, Peter Maydell 写道: On 16 November 2012 05:05, Wenchao Xia xiaw...@linux.vnet.ibm.com wrote: - test -f pixman/config.log make -C pixman distclean + @if test -f pixman/config.log; \ +

Re: [Qemu-devel] [PATCH 1/2] Buildsystem fix distclean error in pixman

2012-11-16 Thread Paolo Bonzini
Il 16/11/2012 10:27, Peter Maydell ha scritto: On 16 November 2012 05:05, Wenchao Xia xiaw...@linux.vnet.ibm.com wrote: Currently if pixman have no config.log inside, make file still try to clean it resulting error. This patch fix it. - test -f pixman/config.log make -C pixman

Re: [Qemu-devel] [PATCH V9 2/8] Buildsystem clean tests directory clearly

2012-11-16 Thread Peter Maydell
On 16 November 2012 10:12, Wenchao Xia xiaw...@linux.vnet.ibm.com wrote: Currently make clean only clean tests/tcg and hard to extend. This patch added command make check-clean, which clean all generated files used in tests. With this command root Makefile do not care tests clean method any

Re: [Qemu-devel] [PATCH 2/2] Buildsystem clean tests directory clearly

2012-11-16 Thread Paolo Bonzini
Il 16/11/2012 06:05, Wenchao Xia ha scritto: Currently make clean only clean tests/tcg and hard to extend. This patch added command make check-clean, which clean all generated files used in tests. With this command root Makefile do not care tests clean method any more, it simply calls the

Re: [Qemu-devel] usb: uhci: Look up queue by address, not token

2012-11-16 Thread Gerd Hoffmann
On 11/15/12 16:40, Jan Kiszka wrote: Hi Hans, On 2012-11-15 16:19, Hans de Goede wrote: Hi Jan, I just saw your $subject patch in Gerd's usb-next tree, and I've a question about it. The token should be enough to uniquely identify a device + ep, and unless a guest uses multiple qhs for a

Re: [Qemu-devel] [PATCH 1/2] Buildsystem fix distclean error in pixman

2012-11-16 Thread Peter Maydell
On 16 November 2012 10:23, Paolo Bonzini pbonz...@redhat.com wrote: Il 16/11/2012 10:27, Peter Maydell ha scritto: On 16 November 2012 05:05, Wenchao Xia xiaw...@linux.vnet.ibm.com wrote: - test -f pixman/config.log make -C pixman distclean + @if test -f pixman/config.log; \ +

Re: [Qemu-devel] [PATCH 1/2] Buildsystem fix distclean error in pixman

2012-11-16 Thread Wenchao Xia
于 2012-11-16 18:16, Peter Maydell 写道: On 16 November 2012 10:08, Wenchao Xia xiaw...@linux.vnet.ibm.com wrote: 于 2012-11-16 17:27, Peter Maydell 写道: On 16 November 2012 05:05, Wenchao Xia xiaw...@linux.vnet.ibm.com wrote: - test -f pixman/config.log make -C pixman distclean + @if

Re: [Qemu-devel] [PATCH 2/2] Buildsystem clean tests directory clearly

2012-11-16 Thread Peter Maydell
On 16 November 2012 10:23, Paolo Bonzini pbonz...@redhat.com wrote: Il 16/11/2012 06:05, Wenchao Xia ha scritto: - $(MAKE) -C tests/tcg clean + MAKEFILES=./tests/Makefile $(MAKE) check-clean What is MAKEFILES? make uses it as a list of additional makefiles to be read, as if they were

Re: [Qemu-devel] [PATCH V9 2/8] Buildsystem clean tests directory clearly

2012-11-16 Thread Wenchao Xia
于 2012-11-16 18:23, Peter Maydell 写道: On 16 November 2012 10:12, Wenchao Xia xiaw...@linux.vnet.ibm.com wrote: Currently make clean only clean tests/tcg and hard to extend. This patch added command make check-clean, which clean all generated files used in tests. With this command root

Re: [Qemu-devel] [PATCH] iscsi: fix deadlock during login

2012-11-16 Thread Kevin Wolf
Am 15.11.2012 17:37, schrieb Paolo Bonzini: Il 15/11/2012 17:13, ronnie sahlberg ha scritto: On Thu, Nov 15, 2012 at 7:54 AM, Paolo Bonzini pbonz...@redhat.com wrote: Il 15/11/2012 15:57, ronnie sahlberg ha scritto: I dont know if we should switch to use synchronous code here. It is much

Re: [Qemu-devel] [PATCH 1/2] Buildsystem fix distclean error in pixman

2012-11-16 Thread Gerd Hoffmann
On 11/16/12 11:26, Peter Maydell wrote: On 16 November 2012 10:23, Paolo Bonzini pbonz...@redhat.com wrote: Il 16/11/2012 10:27, Peter Maydell ha scritto: On 16 November 2012 05:05, Wenchao Xia xiaw...@linux.vnet.ibm.com wrote: - test -f pixman/config.log make -C pixman distclean +

[Qemu-devel] [PATCH V9 2/8] Buildsystem clean tests directory clearly

2012-11-16 Thread Wenchao Xia
Currently make clean only clean tests/tcg and hard to extend. This patch added command make check-clean, which clean all generated files used in tests. With this command root Makefile do not care tests clean method any more, it simply calls the command to do it, so any more clean script could be

Re: [Qemu-devel] slow virtio network with vhost=on and multiple cores

2012-11-16 Thread Dietmar Maurer
I only tested with RHEL6.3 kernel on host. can you check if there is a difference on interrupt delivery between those two? cat /proc/interrupts should be sufficient after some traffic has flown. While trying to reproduce the bug, we just detected that it depends on the hardware

[Qemu-devel] [PATCH V9 3/8] Buildsystem move qapi generation to Makefile.objs

2012-11-16 Thread Wenchao Xia
Sub Makefile may need qapi generated files, so move them to Makefile.objs, In this way Makfile.objs control file auto generation. Signed-off-by: Wenchao Xia xiaw...@linux.vnet.ibm.com --- Makefile | 22 -- Makefile.objs | 22 ++ 2 files changed,

Re: [Qemu-devel] [Spice-devel] QEMU hangs when shutdown windows7 guest with virtio-serial drivers installed

2012-11-16 Thread Christophe Fergeau
On Fri, Nov 16, 2012 at 06:02:33PM +0800, Dunrong Huang wrote: I meet a weird problem: If I I boot QEMU with virtio-serial being enabled and assign only one cpu to windows VM, when I shutdown VM, QEMU hangs with using 100% cpu. What version of virtio-serial were you using? The old

Re: [Qemu-devel] [PATCH 1/2] Buildsystem fix distclean error in pixman

2012-11-16 Thread Wenchao Xia
于 2012-11-16 18:23, Paolo Bonzini 写道: Il 16/11/2012 10:27, Peter Maydell ha scritto: On 16 November 2012 05:05, Wenchao Xia xiaw...@linux.vnet.ibm.com wrote: Currently if pixman have no config.log inside, make file still try to clean it resulting error. This patch fix it. - test -f

[Qemu-devel] [PATCH V9 1/8] Buildsystem fix distclean error in pixman

2012-11-16 Thread Wenchao Xia
Currently if pixman have no config.log inside, make file still try to clean it resulting error. This patch fix it. Signed-off-by: Wenchao Xia xiaw...@linux.vnet.ibm.com --- Makefile |5 - 1 files changed, 4 insertions(+), 1 deletions(-) diff --git a/Makefile b/Makefile index

Re: [Qemu-devel] [PATCH 1/2] Buildsystem fix distclean error in pixman

2012-11-16 Thread Wenchao Xia
于 2012-11-16 18:38, Gerd Hoffmann 写道: On 11/16/12 11:26, Peter Maydell wrote: On 16 November 2012 10:23, Paolo Bonzini pbonz...@redhat.com wrote: Il 16/11/2012 10:27, Peter Maydell ha scritto: On 16 November 2012 05:05, Wenchao Xia xiaw...@linux.vnet.ibm.com wrote: - test -f

[Qemu-devel] [PATCH V9 4/8] block export function path_has_protocol

2012-11-16 Thread Wenchao Xia
This function is needed in other module, so export it. There is already some patch on mail-list try export it, If that patch was applied, pls ignore this one. Signed-off-by: Wenchao Xia xiaw...@linux.vnet.ibm.com --- block.c |2 +- block.h |2 ++ 2 files changed, 3 insertions(+), 1

Re: [Qemu-devel] [PATCH V9 3/8] Buildsystem move qapi generation to Makefile.objs

2012-11-16 Thread Paolo Bonzini
Il 16/11/2012 11:12, Wenchao Xia ha scritto: Sub Makefile may need qapi generated files, so move them to Makefile.objs, In this way Makfile.objs control file auto generation. No objection in principle, but why are you doing this? Does it fix anything? Paolo Signed-off-by: Wenchao Xia

Re: [Qemu-devel] [PATCH V9 2/8] Buildsystem clean tests directory clearly

2012-11-16 Thread Paolo Bonzini
Il 16/11/2012 11:31, Wenchao Xia ha scritto: 于 2012-11-16 18:23, Peter Maydell 写道: On 16 November 2012 10:12, Wenchao Xia xiaw...@linux.vnet.ibm.com wrote: Currently make clean only clean tests/tcg and hard to extend. This patch added command make check-clean, which clean all generated

Re: [Qemu-devel] [PATCH V9 3/8] Buildsystem move qapi generation to Makefile.objs

2012-11-16 Thread Wenchao Xia
于 2012-11-16 18:54, Paolo Bonzini 写道: Il 16/11/2012 11:12, Wenchao Xia ha scritto: Sub Makefile may need qapi generated files, so move them to Makefile.objs, In this way Makfile.objs control file auto generation. No objection in principle, but why are you doing this? Does it fix anything?

Re: [Qemu-devel] slow virtio network with vhost=on and multiple cores

2012-11-16 Thread Alexandre DERUMIER
While trying to reproduce the bug, we just detected that it depends on the hardware (mainboard) you run on. Sigh :-/ Hi, I can reproduce the bug on all my dell servers,differents generation (R710 (intel),R815 (amd), 2950 (intel). They all use broadcom bnx2 network card (don't know if it can

Re: [Qemu-devel] [PATCH V9 2/8] Buildsystem clean tests directory clearly

2012-11-16 Thread Wenchao Xia
于 2012-11-16 18:56, Paolo Bonzini 写道: Il 16/11/2012 11:31, Wenchao Xia ha scritto: 于 2012-11-16 18:23, Peter Maydell 写道: On 16 November 2012 10:12, Wenchao Xia xiaw...@linux.vnet.ibm.com wrote: Currently make clean only clean tests/tcg and hard to extend. This patch added command make

Re: [Qemu-devel] [PATCH V9 5/8] libqblock build system

2012-11-16 Thread Paolo Bonzini
Il 16/11/2012 11:12, Wenchao Xia ha scritto: Libqblock was placed in new directory ./libqblock, libtool will build dynamic library there, source files of block layer remains in ./block. So block related source code will generate 3 sets of binary, first is old ones used in qemu, second and

Re: [Qemu-devel] [PATCH V9 3/8] Buildsystem move qapi generation to Makefile.objs

2012-11-16 Thread Paolo Bonzini
Il 16/11/2012 11:58, Wenchao Xia ha scritto: Sub Makefile may need qapi generated files, so move them to Makefile.objs, In this way Makfile.objs control file auto generation. No objection in principle, but why are you doing this? Does it fix anything? It seems tools or block objects

Re: [Qemu-devel] [PATCH V9 2/8] Buildsystem clean tests directory clearly

2012-11-16 Thread Paolo Bonzini
Il 16/11/2012 12:01, Wenchao Xia ha scritto: ifneq ($(wildcard config-host.mak),) include $(SRC_PATH)/tests/Makefile endif to -include $(SRC_PATH)/tests/Makefile ? yes, but original author seems not include tests/Makefile in this condition on purpose, so using MAKEFILES to keep it

[Qemu-devel] [PATCH 7/7] target-mips: implement DSP (d)append sub-class with TCG

2012-11-16 Thread Aurelien Jarno
DSP instruction from the (d)append sub-class can be implemented with TCG. Use a different function for these instructions are they are quite different from compare-pick sub-class. Fix BALIGN instruction for negative value, where the value should be zero-extended before being shift to the right.

Re: [Qemu-devel] [Spice-devel] QEMU hangs when shutdown windows7 guest with virtio-serial drivers installed

2012-11-16 Thread Dunrong Huang
Thanks for your quick reply. 2012/11/16 Christophe Fergeau cferg...@redhat.com: On Fri, Nov 16, 2012 at 06:02:33PM +0800, Dunrong Huang wrote: I meet a weird problem: If I I boot QEMU with virtio-serial being enabled and assign only one cpu to windows VM, when I shutdown VM, QEMU hangs with

Re: [Qemu-devel] usb: uhci: Look up queue by address, not token

2012-11-16 Thread Jan Kiszka
On 2012-11-16 11:25, Gerd Hoffmann wrote: On 11/15/12 16:40, Jan Kiszka wrote: Hi Hans, On 2012-11-15 16:19, Hans de Goede wrote: Hi Jan, I just saw your $subject patch in Gerd's usb-next tree, and I've a question about it. The token should be enough to uniquely identify a device + ep,

Re: [Qemu-devel] [Spice-devel] QEMU hangs when shutdown windows7 guest with virtio-serial drivers installed

2012-11-16 Thread Dunrong Huang
2012/11/16 Marian Krcmarik mkrcm...@redhat.com: - Original Message - From: Christophe Fergeau cferg...@redhat.com To: Dunrong Huang riegama...@gmail.com Cc: Amit Shah amit.s...@redhat.com, spice-devel spice-de...@lists.freedesktop.org, qemu-devel qemu-devel@nongnu.org Sent:

[Qemu-devel] [PATCH 1/7] target-mips: fix DSP loads with rd = 0

2012-11-16 Thread Aurelien Jarno
When rd is 0, which still need to do the actually load to possibly generate a TLB exception. Signed-off-by: Aurelien Jarno aurel...@aurel32.net --- target-mips/translate.c |5 - 1 file changed, 5 deletions(-) diff --git a/target-mips/translate.c b/target-mips/translate.c index

Re: [Qemu-devel] [Spice-devel] QEMU hangs when shutdown windows7 guest with virtio-serial drivers installed

2012-11-16 Thread Christophe Fergeau
Hey, On Fri, Nov 16, 2012 at 07:22:58PM +0800, Dunrong Huang wrote: 2012/11/16 Christophe Fergeau cferg...@redhat.com: On Fri, Nov 16, 2012 at 06:02:33PM +0800, Dunrong Huang wrote: I meet a weird problem: If I I boot QEMU with virtio-serial being enabled and assign only one cpu to

[Qemu-devel] [PATCH 5/7] target-mips: use DSP unions for unary DSP operators

2012-11-16 Thread Aurelien Jarno
This allow to reduce the number of macros. Signed-off-by: Aurelien Jarno aurel...@aurel32.net --- target-mips/dsp_helper.c | 124 -- 1 file changed, 42 insertions(+), 82 deletions(-) diff --git a/target-mips/dsp_helper.c b/target-mips/dsp_helper.c

[Qemu-devel] [PATCH 4/7] target-mips: use DSP unions for binary DSP operators

2012-11-16 Thread Aurelien Jarno
This allow to reduce the number of macros. Signed-off-by: Aurelien Jarno aurel...@aurel32.net --- target-mips/dsp_helper.c | 384 ++ 1 file changed, 116 insertions(+), 268 deletions(-) diff --git a/target-mips/dsp_helper.c b/target-mips/dsp_helper.c

[Qemu-devel] [PATCH 6/7] target-mips: use DSP unions for reduction add instructions

2012-11-16 Thread Aurelien Jarno
Signed-off-by: Aurelien Jarno aurel...@aurel32.net --- target-mips/dsp_helper.c | 32 +++- 1 file changed, 15 insertions(+), 17 deletions(-) diff --git a/target-mips/dsp_helper.c b/target-mips/dsp_helper.c index 3bd2d35..474c249 100644 --- a/target-mips/dsp_helper.c

[Qemu-devel] [PATCH 3/7] target-mips: add unions to access DSP elements

2012-11-16 Thread Aurelien Jarno
Instead of playing with bit shifting, add two unions (one for 32-bit values, one for 64-bit ones) to access all the DSP elements with the correct type. This make the code easier to read and less error prone, and allow GCC to vectorize the code in some cases. Signed-off-by: Aurelien Jarno

[Qemu-devel] [PATCH 0/7] target-mips: DSP ASE fixes and cleanup

2012-11-16 Thread Aurelien Jarno
This patch series contains some fixes and cleanup following the merge of the DSP ASE patches. Aurelien Jarno (7): target-mips: fix DSP loads with rd = 0 target-mips: generate a reserved instruction exception on CPU without DSP target-mips: add unions to access DSP elements target-mips:

Re: [Qemu-devel] TCP based PCIE request forwarding

2012-11-16 Thread Stefan Hajnoczi
On Fri, Nov 16, 2012 at 9:39 AM, lementec fabien fabien.lemen...@gmail.com wrote: I am a software engineer who works in an electronic group. Using QEMU to emulate devices allows me to start writing and testing LINUX software before the device is actually available. In the group, we are mostly

Re: [Qemu-devel] [PATCH] virtio-blk: Remove duplicate property definition

2012-11-16 Thread Stefan Hajnoczi
On Thu, Nov 15, 2012 at 01:12:14PM +1100, David Gibson wrote: For the virtio-blk device (via virtio-pci) the property config-wce is defined in two places. First, it's defined from the DEFINE_VIRTIO_BLK_FEATURES macro, second it's defined directly in virtio-pci, just two lines above the call

Re: [Qemu-devel] [PATCH v2 2/2] block: Avoid second open for format probing

2012-11-16 Thread Stefan Hajnoczi
On Thu, Nov 15, 2012 at 01:57:37PM +0100, Kevin Wolf wrote: This fixes problems that are caused by the additional open/close cycle of the existing format probing, for example related to qemu-nbd without -t option or file descriptor passing. Signed-off-by: Kevin Wolf kw...@redhat.com ---

Re: [Qemu-devel] [PATCH] trace: Remove info trace from documents

2012-11-16 Thread Stefan Hajnoczi
On Fri, Nov 16, 2012 at 03:10:49PM +0800, Liming Wang wrote: commit 88affa1c monitor: remove unused do_info_trace has removed info trace function from monitor, so remove it from documents. Signed-off-by: Liming Wang walimis...@gmail.com --- docs/tracing.txt |9 -

Re: [Qemu-devel] [PATCH V9 2/8] Buildsystem clean tests directory clearly

2012-11-16 Thread Wenchao Xia
于 2012-11-16 19:16, Paolo Bonzini 写道: Il 16/11/2012 12:01, Wenchao Xia ha scritto: ifneq ($(wildcard config-host.mak),) include $(SRC_PATH)/tests/Makefile endif to -include $(SRC_PATH)/tests/Makefile ? yes, but original author seems not include tests/Makefile in this condition on

Re: [Qemu-devel] [PATCH V9 2/8] Buildsystem clean tests directory clearly

2012-11-16 Thread Paolo Bonzini
Il 16/11/2012 13:40, Wenchao Xia ha scritto: I guess make distclean for second time would fail in this case, because $(SRC_PATH) is not set. I found a better way to do it without MAKEFILES: -ifneq ($(wildcard config-host.mak),) - include $(SRC_PATH)/tests/Makefile + include

Re: [Qemu-devel] TCP based PCIE request forwarding

2012-11-16 Thread lementec fabien
Hi, Thanks for your reply. Actually, I wanted to be independant of the QEMU event loop. Plus, some proprietary simulation environment provides a closed socket based interface to 'stimulate' the emulated device, at the PCIE level for instance. These environments are sometimes installed on cluster

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

2012-11-16 Thread Stefan Hajnoczi
The following changes since commit 6801038bc52d61f81ac8a25fbe392f1bad982887: target-mips: fix wrong microMIPS opcode encoding (2012-11-15 14:48:16 +0100) are available in the git repository at: git://github.com/stefanha/qemu.git tracing for you to fetch changes up to

[Qemu-devel] [PATCH 1/4] Avoid all systemtap reserved words

2012-11-16 Thread Stefan Hajnoczi
From: Daniel P. Berrange berra...@redhat.com Over time various systemtap reserved words have been blacklisted in the trace backend generator. The list is not complete though, so there is continued risk of problems in the future. Preempt such problems by specifying the full list of systemtap

[Qemu-devel] [PATCH 2/4] trace: allow disabling events in events file

2012-11-16 Thread Stefan Hajnoczi
From: Gerd Hoffmann kra...@redhat.com Disable trace events prefixed with a '-'. Useful to enable a group of tracepoints with exceptions, like this: usb_xhci_port_* -usb_xhci_port_read which will enable all xhci port tracepoints except reads. Signed-off-by: Gerd Hoffmann kra...@redhat.com

[Qemu-devel] [PATCH 4/4] trace: Remove info trace from documents

2012-11-16 Thread Stefan Hajnoczi
From: Liming Wang walimis...@gmail.com commit 88affa1c monitor: remove unused do_info_trace has removed info trace function from monitor, so remove it from documents. Signed-off-by: Liming Wang walimis...@gmail.com Signed-off-by: Stefan Hajnoczi stefa...@redhat.com --- docs/tracing.txt | 9

[Qemu-devel] [PATCH 3/4] trace: document '-' syntax for disabling events

2012-11-16 Thread Stefan Hajnoczi
Signed-off-by: Stefan Hajnoczi stefa...@redhat.com --- docs/tracing.txt | 4 1 file changed, 4 insertions(+) diff --git a/docs/tracing.txt b/docs/tracing.txt index c541133..7901409 100644 --- a/docs/tracing.txt +++ b/docs/tracing.txt @@ -139,6 +139,10 @@ having a common prefix in a batch.

Re: [Qemu-devel] usb: uhci: Look up queue by address, not token

2012-11-16 Thread Hans de Goede
Hi, On 11/15/2012 04:40 PM, Jan Kiszka wrote: Hi Hans, On 2012-11-15 16:19, Hans de Goede wrote: Hi Jan, I just saw your $subject patch in Gerd's usb-next tree, and I've a question about it. The token should be enough to uniquely identify a device + ep, and unless a guest uses multiple qhs

[Qemu-devel] [PATCH V2 0/4] Buildsystem fix and block func export

2012-11-16 Thread Wenchao Xia
Wenchao Xia (4): Buildsystem fix distclean error for pixman Buildsystem clean tests directory clearly Buildsystem move qapi generation to Makefile.objs block export function path_has_protocol Makefile | 30 +++--- Makefile.objs | 22

[Qemu-devel] [PATCH V2 1/4] Buildsystem fix distclean error for pixman

2012-11-16 Thread Wenchao Xia
Currently Makefile test if pixman have configure log, but it will return error if that file do not exist. This patch fix it. v2: print out the command. Signed-off-by: Wenchao Xia xiaw...@linux.vnet.ibm.com --- Makefile |2 +- 1 files changed, 1 insertions(+), 1 deletions(-) diff --git

[Qemu-devel] [PATCH V2 3/4] Buildsystem move qapi generation to Makefile.objs

2012-11-16 Thread Wenchao Xia
With this patch Makfile.objs control file auto generation, sub Makefiles could use file generation more easily. v2: rebase and better doc. Signed-off-by: Wenchao Xia xiaw...@linux.vnet.ibm.com --- Makefile | 22 -- Makefile.objs | 22 ++ 2

Re: [Qemu-devel] [PATCH v3 0/3] net: convert NetClientState to QOM

2012-11-16 Thread Stefan Hajnoczi
On Wed, Oct 24, 2012 at 9:07 AM, Stefan Hajnoczi stefa...@redhat.com wrote: This series converts NetClientState from a plain C struct to QOM. This means emulated NICs and netdevs now derive from the NetClientState base class. There should be no user-visible change. Converting the net

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

2012-11-16 Thread Stefan Hajnoczi
The following changes since commit 6801038bc52d61f81ac8a25fbe392f1bad982887: target-mips: fix wrong microMIPS opcode encoding (2012-11-15 14:48:16 +0100) 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] net: extract notify_link_status_changed() function

2012-11-16 Thread Stefan Hajnoczi
The code to invoke the NetClientInfo .link_status_changed() callback is duplicated in several places. Create a single notify_link_status_changed() function and avoid duplication. This is useful because later patches change net internals. By having a single function it is easier to make changes

[Qemu-devel] [PATCH 4/4] tap: reset vnet header size on open

2012-11-16 Thread Stefan Hajnoczi
From: Michael S. Tsirkin m...@redhat.com For tap, we currently assume the vnet header size is 10 (the default value) but that might not be the case if tap is persistent and has been used by qemu previously. To fix, set host header size in tap device on open. Signed-off-by: Michael S. Tsirkin

Re: [Qemu-devel] usb: uhci: Look up queue by address, not token

2012-11-16 Thread Gerd Hoffmann
On 11/16/12 12:24, Jan Kiszka wrote: On 2012-11-16 11:25, Gerd Hoffmann wrote: On 11/15/12 16:40, Jan Kiszka wrote: Hi Hans, On 2012-11-15 16:19, Hans de Goede wrote: Hi Jan, I just saw your $subject patch in Gerd's usb-next tree, and I've a question about it. The token should be enough

Re: [Qemu-devel] [Qemu-trivial] [PATCH V2] qemu-sockets: Fix parsing of the inet option 'to'.

2012-11-16 Thread Stefan Hajnoczi
On Fri, Nov 09, 2012 at 11:50:54AM +, Anthony PERARD wrote: Having a qemu command line argument like -vnc 127.0.0.1:0,to=99 is broken. This have been break with commit 879e45c72da1569e07fbbc6a1aa2a708ea796044. Signed-off-by: Anthony PERARD anthony.per...@citrix.com --- qemu-sockets.c

[Qemu-devel] [PULL for-1.3 0/2] vga bugfixes

2012-11-16 Thread Gerd Hoffmann
Hi, Two little fixes for the standard vga, sent to the list for review earlier this week. please pull, Gerd The following changes since commit 6801038bc52d61f81ac8a25fbe392f1bad982887: target-mips: fix wrong microMIPS opcode encoding (2012-11-15 14:48:16 +0100) are available in the git

[Qemu-devel] [PATCH 1/2] vga: fix bochs alignment issue

2012-11-16 Thread Gerd Hoffmann
The bochs dispi interface traditionally uses port 0x1ce as 16bit index register and port 0x1cf as 16bit data register. The later is unaligned, and probably for that reason the the data register was moved to 0x1d0 for non-x86 archs. This patch makes the data register available at 0x1d0 on x86

[Qemu-devel] [PATCH 2/2] vga: fix mmio vga register mapping

2012-11-16 Thread Gerd Hoffmann
--- hw/vga-pci.c |7 --- 1 files changed, 4 insertions(+), 3 deletions(-) diff --git a/hw/vga-pci.c b/hw/vga-pci.c index ec29cac..947e35c 100644 --- a/hw/vga-pci.c +++ b/hw/vga-pci.c @@ -84,9 +84,10 @@ static void pci_vga_ioport_write(void *ptr, hwaddr addr,

Re: [Qemu-devel] [PATCH] ehci: fix compile error with EHCI_DEBUG enabled

2012-11-16 Thread Stefan Hajnoczi
On Mon, Nov 12, 2012 at 09:49:19AM -0500, Gabriel L. Somlo wrote: This patch fixes a few debugging print statements whose arguments fell out of sync over time with changes being made to the active code base. Signed-off-by: Gabriel Somlo so...@cmu.edu --- On Mon, Nov 12, 2012 at 11:26:18AM

[Qemu-devel] [PATCH 05/14] ehci: Don't access packet after freeing it

2012-11-16 Thread Gerd Hoffmann
From: Hans de Goede hdego...@redhat.com ehci_state_writeback() will free the packet, so we should not access the packet after calling ehci_state_writeback(). Signed-off-by: Hans de Goede hdego...@redhat.com Signed-off-by: Gerd Hoffmann kra...@redhat.com --- hw/usb/hcd-ehci.c |9 + 1

Re: [Qemu-devel] [Qemu-trivial] [PATCH] sd: Send debug printfery to stderr not stdout

2012-11-16 Thread Stefan Hajnoczi
On Thu, Nov 15, 2012 at 04:32:53PM +1000, Peter Crosthwaite wrote: Some debug printfs for SD are coming up in stdout. Redirected them to stderr instead. Signed-off-by: Peter Crosthwaite peter.crosthwa...@xilinx.com --- hw/sd.c |4 ++-- 1 files changed, 2 insertions(+), 2 deletions(-)

[Qemu-devel] [PATCH 06/14] ehci: Fixup q-qtdaddr after cancelling an already completed packet

2012-11-16 Thread Gerd Hoffmann
From: Hans de Goede hdego...@redhat.com This avoids the q-qtdaddr == p-qtdaddr asserts we have triggering, when a queue contains multiple completed packages when we cancel the queue. I triggered this with windows7 + async interrupt endpoint handling (*) + not detecting circles in

[Qemu-devel] [PATCH 10/14] ehci: handle dma errors

2012-11-16 Thread Gerd Hoffmann
Starting with commit 1c380f9460522f32c8dd2577b2a53d518ec91c6d dma transfers can actually fail. This patch makes ehci keep track of the busmaster bit in pci config space, by setting/clearing the dma_context pointer. Attempts to dma without context will result in raising HSE (Host System Error)

Re: [Qemu-devel] [PATCH] ehci: fix compile error with EHCI_DEBUG enabled

2012-11-16 Thread Gerd Hoffmann
On 11/16/12 14:50, Stefan Hajnoczi wrote: On Mon, Nov 12, 2012 at 09:49:19AM -0500, Gabriel L. Somlo wrote: Turning EHCI_DEBUG on gave me compile errors. This is my best guess as to what the DPRINTF arguments *should* be, hope I guessed right :) In that case I'll let Gerd confirm whether or

Re: [Qemu-devel] [Qemu-trivial] [PATCH v2] target-mips: Clean up microMIPS32 major opcode

2012-11-16 Thread Stefan Hajnoczi
On Fri, Nov 16, 2012 at 10:05:52AM +0800, 陳韋任 (Wei-Ren Chen) wrote: Hi all, I check MIPS microMIPS manual [1], and found the major opcode might be wrong. I add a comment to explicitly indicate what manual I am refering to, and according that manual I remove microMIPS32 major opcodes 0x1f.

Re: [Qemu-devel] [Qemu-trivial] [PATCH] target-mips: Add comments on POOL32Axf encoding

2012-11-16 Thread Stefan Hajnoczi
On Fri, Nov 16, 2012 at 10:29:47AM +0800, 陳韋任 (Wei-Ren Chen) wrote: Current QEMU MIPS POOL32AXF encoding comes from microMIPS32 and microMIPS32 DSP. Add comment here to help reading. Please review, thanks. Regards, chenwj Signed-off-by: Chen Wei-Ren che...@iis.sinica.edu.tw ---

[Qemu-devel] [PATCH 03/14] usb-host: scan for usb devices when the vm starts

2012-11-16 Thread Gerd Hoffmann
Commit a844ed842d9a9d929645c09ae0f52f753d7a02e0 leads to usb-host detecting devices not right after qemu startup because the guest isn't running yet. Instead they are found on the first of the regular usb device poll runs. Which is too late for seabios to see them, so booting from usb sticks

[Qemu-devel] [PATCH 0/2] Trivial patches for 2 to 16 November 2012

2012-11-16 Thread Stefan Hajnoczi
The following changes since commit 6801038bc52d61f81ac8a25fbe392f1bad982887: target-mips: fix wrong microMIPS opcode encoding (2012-11-15 14:48:16 +0100) are available in the git repository at: git://github.com/stefanha/qemu.git trivial-patches for you to fetch changes up to

[Qemu-devel] [PATCH 1/2] qemu-sockets: Fix parsing of the inet option 'to'.

2012-11-16 Thread Stefan Hajnoczi
From: Anthony PERARD anthony.per...@citrix.com Having a qemu command line argument like -vnc 127.0.0.1:0,to=99 is broken. This have been break with commit 879e45c72da1569e07fbbc6a1aa2a708ea796044. Signed-off-by: Anthony PERARD anthony.per...@citrix.com Signed-off-by: Stefan Hajnoczi

[Qemu-devel] [PATCH 04/14] usb: host-linux: Ignore parsing errors of the device descriptors

2012-11-16 Thread Gerd Hoffmann
From: Jan Kiszka jan.kis...@siemens.com The Linux is more tolerant here as well: Just stop parsing the device descriptors when an error is detected but do not reset what was found so far. This allows to run buggy devices with partially invalid descriptors. Signed-off-by: Jan Kiszka

[Qemu-devel] [PATCH 2/2] sd: Send debug printfery to stderr not stdout

2012-11-16 Thread Stefan Hajnoczi
From: Peter Crosthwaite peter.crosthwa...@xilinx.com Some debug printfs for SD are coming up in stdout. Redirected them to stderr instead. Signed-off-by: Peter Crosthwaite peter.crosthwa...@xilinx.com Reviewed-by: Peter Maydell peter.mayd...@linaro.org Signed-off-by: Stefan Hajnoczi

[Qemu-devel] [PATCH V7 0/6] VMXNET3 paravirtual NIC device implementation

2012-11-16 Thread Dmitry Fleytman
NOTE: This is not a final patches for commit, they are just for the latest fixes review. This set of patches still misses test for the device. We didn't succeed to find any guide or sample for the king of tests required (packets transmission). If someone can provide

[Qemu-devel] [PATCH V7 1/6] Adding utility function net_checksum_add_cont() that allows checksum calculation of scattered data with odd chunk sizes

2012-11-16 Thread Dmitry Fleytman
Adding utility function net_raw_checksum() that calculates checksum of buffer given Signed-off-by: Dmitry Fleytman dmi...@daynix.com Signed-off-by: Yan Vugenfirer y...@daynix.com --- net/checksum.c | 13 +++-- net/checksum.h | 14 +- 2 files changed, 20 insertions(+), 7

[Qemu-devel] [PATCH V7 3/6] Adding common definitions for VMWARE devices

2012-11-16 Thread Dmitry Fleytman
Signed-off-by: Dmitry Fleytman dmi...@daynix.com Signed-off-by: Yan Vugenfirer y...@daynix.com --- hw/vmware_utils.h | 143 ++ 1 file changed, 143 insertions(+) create mode 100644 hw/vmware_utils.h diff --git a/hw/vmware_utils.h

[Qemu-devel] [PATCH V7 5/6] Adding packet abstraction for VMWARE network devices

2012-11-16 Thread Dmitry Fleytman
Signed-off-by: Dmitry Fleytman dmi...@daynix.com Signed-off-by: Yan Vugenfirer y...@daynix.com --- hw/vmxnet_pkt.c | 776 ++ hw/vmxnet_pkt.h | 311 ++ hw/vmxnet_utils.c | 6 +- hw/vmxnet_utils.h | 6 +- 4 files

[Qemu-devel] [PULL for-1.3 00/14] usb patch queue

2012-11-16 Thread Gerd Hoffmann
Hi, A bunch of usb bugfixes all over the place piled up in the usb patch queue. Here we go. please pull, Gerd The following changes since commit 6801038bc52d61f81ac8a25fbe392f1bad982887: target-mips: fix wrong microMIPS opcode encoding (2012-11-15 14:48:16 +0100) are available in the

[Qemu-devel] [PATCH 11/14] usb-redir: Only add actually in flight packets to the in flight queue

2012-11-16 Thread Gerd Hoffmann
From: Hans de Goede hdego...@redhat.com Packets which are queued up, but not yet handed over to the device, are *not* in flight. Signed-off-by: Hans de Goede hdego...@redhat.com Signed-off-by: Gerd Hoffmann kra...@redhat.com --- hw/usb/redirect.c |4 +++- 1 files changed, 3 insertions(+), 1

[Qemu-devel] [PATCH 14/14] usb-host: fix splitted transfers

2012-11-16 Thread Gerd Hoffmann
USBPacket-actual_length wasn't updated correctly for USBPackets splitted into multiple urbs. Fix it. Signed-off-by: Gerd Hoffmann kra...@redhat.com --- hw/usb/host-linux.c |7 +-- 1 files changed, 5 insertions(+), 2 deletions(-) diff --git a/hw/usb/host-linux.c b/hw/usb/host-linux.c

[Qemu-devel] [PATCH 13/14] usb-host: update tracing

2012-11-16 Thread Gerd Hoffmann
Now that we have separate status and length fields in USBPacket update the completion tracepoint to log both. Signed-off-by: Gerd Hoffmann kra...@redhat.com --- hw/usb/host-linux.c | 20 trace-events|2 +- 2 files changed, 13 insertions(+), 9 deletions(-) diff

[Qemu-devel] [PATCH 07/14] ehci: Better detection for qtd-s linked in circles

2012-11-16 Thread Gerd Hoffmann
From: Hans de Goede hdego...@redhat.com Windows links interrupt qtd-s in circles, which means that when interrupt endpoints return USB_RET_ASYNC, combined with the recent ehci: Retry to fill the queue while waiting for td completion patch, we keep adding the tds to the queue over and over again,

[Qemu-devel] [PATCH 01/14] fix live migration

2012-11-16 Thread Gerd Hoffmann
Commit 1c380f9460522f32c8dd2577b2a53d518ec91c6d breaks live migration. DMA stops working for ehci (and probably for any pci device) after restoring the guest because the bus master region never gets enabled. Add code doing that after loading the pci config space from vmstate. Cc: Avi Kivity

  1   2   >