Re: [Qemu-devel] [PATCH 4/7 V6] Header with various utility functions shared by VMWARE SCSI and network devices

2012-04-18 Thread Michael S. Tsirkin
On Tue, Apr 17, 2012 at 03:32:38PM +0300, Dmitry Fleytman wrote: From: Dmitry Fleytman dmi...@daynix.com Signed-off-by: Dmitry Fleytman dmi...@daynix.com Signed-off-by: Yan Vugenfirer y...@daynix.com I can easily see how different vmware devices would share some code. However: ---

Re: [Qemu-devel] [PATCH 0/7 V6] VMXNET3 paravirtual NIC device implementation

2012-04-18 Thread Gerhard Wiesinger
As already pretested this patch I got from Dmitry now I successfully tested it on Fedora 16 and Knoppix 6.7 (retested again successfully). netio performance on localhost (ok): Packet size 1k bytes: 33645 KByte/s Tx, 25279 KByte/s Rx. Packet size 2k bytes: 45884 KByte/s Tx, 24854 KByte/s

[Qemu-devel] [Bug 984476] Re: segmentaion error when DMAing

2012-04-18 Thread Stefan Weil
Was it TCP segmentaion Error? Then it is still there. Thanks for reporting. It will be fixed in latest QEMU. -- You received this bug notification because you are a member of qemu- devel-ml, which is subscribed to QEMU. https://bugs.launchpad.net/bugs/984476 Title: segmentaion error when

[Qemu-devel] [Bug 984476] Re: segmentaion error when DMAing

2012-04-18 Thread Stefan Weil
** Changed in: qemu Status: New = Confirmed -- You received this bug notification because you are a member of qemu- devel-ml, which is subscribed to QEMU. https://bugs.launchpad.net/bugs/984476 Title: segmentaion error when DMAing Status in QEMU: Confirmed Bug description: When

[Qemu-devel] [Bug 984516] Re: should use sdl-config for static build not pkg-config

2012-04-18 Thread Stefan Weil
pkg-config supports --static, and QEMU uses it. Please try whether pkg-config --libs --static sdl gives the correct flags with your distribution. If not, that distribution is buggy. -- You received this bug notification because you are a member of qemu- devel-ml, which is subscribed to

Re: [Qemu-devel] [PATCH v6 2/5] qerror: add five qerror strings

2012-04-18 Thread Orit Wasserman
On 04/17/2012 05:54 PM, Amos Kong wrote: Add five new qerror strings, they are about socket: QERR_SOCKET_CONNECT_IN_PROGRESS QERR_SOCKET_CONNECT_FAILED QERR_SOCKET_LISTEN_FAILED QERR_SOCKET_BIND_FAILED QERR_SOCKET_CREATE_FAILED Signed-off-by: Amos Kong ak...@redhat.com ---

Re: [Qemu-devel] [PATCH v6 1/5] sockets: change inet_connect() to support nonblock socket

2012-04-18 Thread Orit Wasserman
On 04/17/2012 05:54 PM, Amos Kong wrote: Add a bool argument to inet_connect() to assign if set socket to block/nonblock, and delete original argument 'socktype' that is unused. Retry to connect when following errors are got: -EINTR -EWOULDBLOCK (win32) Connect's successful for

Re: [Qemu-devel] [PATCH 15/18] qapi: implement support for variable argument list

2012-04-18 Thread Paolo Bonzini
Il 17/04/2012 22:42, Luiz Capitulino ha scritto: On Tue, 17 Apr 2012 22:26:55 +0200 Paolo Bonzini pbonz...@redhat.com wrote: Il 17/04/2012 21:36, Luiz Capitulino ha scritto: +switch(qobject_type(obj)) { +case QTYPE_QSTRING: +qstring_append(arglist,

Re: [Qemu-devel] [PATCH v6 3/5] sockets: use error class to pass connect error

2012-04-18 Thread Orit Wasserman
On 04/17/2012 05:54 PM, Amos Kong wrote: Add a new argument in inet_connect()/inet_connect_opts() to pass back connect error. Change nbd, vnc to use new interface. Signed-off-by: Amos Kong ak...@redhat.com --- nbd.c |2 +- qemu-char.c|2 +- qemu-sockets.c | 13

[Qemu-devel] [PATCH 00/12 v12] introducing a new, dedicated guest memory dump mechanism

2012-04-18 Thread Wen Congyang
Hi, all 'virsh dump' can not work when host pci device is used by guest. We have discussed this issue here: http://lists.nongnu.org/archive/html/qemu-devel/2011-10/msg00736.html The last version is here: http://lists.nongnu.org/archive/html/qemu-devel/2012-03/msg04743.html We have determined to

Re: [Qemu-devel] [PATCH v6 5/5] use inet_listen()/inet_connect() to support ipv6 migration

2012-04-18 Thread Orit Wasserman
On 04/17/2012 05:54 PM, Amos Kong wrote: Use help functions in qemu-socket.c for tcp migration, which already support ipv6 addresses. Currently errp will be set to UNDEFINED_ERROR when migration fails, qemu would output migration failed: ..., and current user can see a message(An undefined

[Qemu-devel] [PATCH 01/12 v12] Add API to create memory mapping list

2012-04-18 Thread Wen Congyang
The memory mapping list stores virtual address and physical address mapping. The virtual address and physical address are contiguous in the mapping. The folloing patch will use this information to create PT_LOAD in the vmcore. Signed-off-by: Wen Congyang we...@cn.fujitsu.com --- Makefile.target

[Qemu-devel] [PATCH 02/12 v12] Add API to check whether a physical address is I/O address

2012-04-18 Thread Wen Congyang
This API will be used in the following patch. Signed-off-by: Wen Congyang we...@cn.fujitsu.com --- cpu-common.h |2 ++ exec.c |9 + 2 files changed, 11 insertions(+), 0 deletions(-) diff --git a/cpu-common.h b/cpu-common.h index dca5175..fcd50dc 100644 --- a/cpu-common.h

[Qemu-devel] [PATCH 03/12 v12] implement cpu_get_memory_mapping()

2012-04-18 Thread Wen Congyang
Walk cpu's page table and collect all virtual address and physical address mapping. Then, add these mapping into memory mapping list. If the guest does not use paging, it will do nothing. Note: the I/O memory will be skipped. Signed-off-by: Wen Congyang we...@cn.fujitsu.com --- Makefile.target

[Qemu-devel] [PATCH 04/12 v12] Add API to check whether paging mode is enabled

2012-04-18 Thread Wen Congyang
This API will be used in the following patch. Signed-off-by: Wen Congyang we...@cn.fujitsu.com --- cpu-all.h |6 ++ target-i386/arch_memory_mapping.c |7 ++- 2 files changed, 12 insertions(+), 1 deletions(-) diff --git a/cpu-all.h b/cpu-all.h index

[Qemu-devel] [PATCH 05/12 v12] Add API to get memory mapping

2012-04-18 Thread Wen Congyang
Add API to get all virtual address and physical address mapping. If the guest doesn't use paging, the virtual address is equal to the phyical address. The virtual address and physical address mapping is for gdb's user, and it does not include the memory that is not referenced by the page table. So

[Qemu-devel] [PATCH 06/12 v12] Add API to get memory mapping without do paging

2012-04-18 Thread Wen Congyang
crash does not need the virtual address and physical address mapping, and the mapping does not include the memory that is not referenced by the page table. crash does not use the virtual address, so we can create the mapping for all physical memory(virtual address is always 0). This patch provides

[Qemu-devel] [PATCH 07/12 v12] target-i386: Add API to write elf notes to core file

2012-04-18 Thread Wen Congyang
The core file contains register's value. These APIs write registers to core file, and them will be called in the following patch. Signed-off-by: Wen Congyang we...@cn.fujitsu.com --- Makefile.target |1 + configure |4 + cpu-all.h | 23 +

[Qemu-devel] [PATCH 08/12 v12] target-i386: Add API to write cpu status to core file

2012-04-18 Thread Wen Congyang
The core file has register's value. But it does not include all registers value. Store the cpu status into QEMU note, and the user can get more information from vmcore. If you change QEMUCPUState, please count up QEMUCPUSTATE_VERSION. Signed-off-by: Wen Congyang we...@cn.fujitsu.com ---

[Qemu-devel] [PATCH 09/12 v12] target-i386: add API to get dump info

2012-04-18 Thread Wen Congyang
Dump info contains: endian, class and architecture. The next patch will use these information to create vmcore. Note: on x86 box, the class is ELFCLASS64 if the memory is larger than 4G. Signed-off-by: Wen Congyang we...@cn.fujitsu.com --- cpu-all.h |7 +++ dump.h

[Qemu-devel] [PATCH 10/12 v12] make gdb_id() generally avialable and rename it to cpu_index()

2012-04-18 Thread Wen Congyang
The following patch also needs this API, so make it generally avialable. The function gdb_id() will not be used in gdbstub.c now, so its name is not suitable, and rename it to cpu_index() Signed-off-by: Wen Congyang we...@cn.fujitsu.com --- gdbstub.c | 19 +-- gdbstub.h |9

[Qemu-devel] [PATCH 11/12 v12] QError: Introduce new error for the dump-guest-memory command

2012-04-18 Thread Wen Congyang
The new error is QERR_PIPE_OR_SOCKET_FD, which is going to be used by the QAPI dump-guest-memory command. Signed-off-by: Wen Congyang we...@cn.fujitsu.com --- qerror.c |4 qerror.h |3 +++ 2 files changed, 7 insertions(+), 0 deletions(-) diff --git a/qerror.c b/qerror.c index

Re: [Qemu-devel] xen build failure

2012-04-18 Thread Alexander Graf
On 17.04.2012, at 18:53, Stefano Stabellini wrote: On Tue, 17 Apr 2012, Alon Levy wrote: I have xen-devel-4.1.2-14.fc17.x86_64 and am getting the following build error: CCx86_64-softmmu/xen-all.o /home/alon/src/spice_upstream/qemu/xen-all.c: In function ‘xen_hvm_inject_msi’:

[Qemu-devel] [PATCH 12/12 v12] introduce a new monitor command 'dump-guest-memory' to dump guest's memory

2012-04-18 Thread Wen Congyang
The command's usage: dump [-p] protocol [begin] [length] The supported protocol can be file or fd: 1. file: the protocol starts with file:, and the following string is the file's path. 2. fd: the protocol starts with fd:, and the following string is the fd's name. Note: 1. If you want

Re: [Qemu-devel] [PATCH] block: Fix spelling in comment (ineffcient - inefficient)

2012-04-18 Thread Kevin Wolf
Am 17.04.2012 19:41, schrieb Stefan Weil: Signed-off-by: Stefan Weil s...@weilnetz.de --- block/cow.c |2 +- 1 files changed, 1 insertions(+), 1 deletions(-) diff --git a/block/cow.c b/block/cow.c index 8d3c9f8..a5a00eb 100644 --- a/block/cow.c +++ b/block/cow.c @@ -103,7 +103,7 @@

[Qemu-devel] [PATCH 1/2 v8] add-cow file format

2012-04-18 Thread Dong Xu Wang
From: Dong Xu Wang wdon...@linux.vnet.ibm.com Provide a new file format: add-cow. The usage can be found in add-cow.txt of this patch. CC: Marcelo Tosatti mtosa...@redhat.com CC: Kevin Wolf kw...@redhat.com CC: Stefan Hajnoczi stefa...@linux.vnet.ibm.com Signed-off-by: Dong Xu Wang

[Qemu-devel] [PATCH 2/2 v8] add-cow: support snapshot_blkdev

2012-04-18 Thread Dong Xu Wang
From: Dong Xu Wang wdon...@linux.vnet.ibm.com Raw file format does not support snapshot_blkdev command, we can use add-cow to do this. CC: Marcelo Tosatti mtosa...@redhat.com CC: Kevin Wolf kw...@redhat.com CC: Stefan Hajnoczi stefa...@linux.vnet.ibm.com Signed-off-by: Dong Xu Wang

Re: [Qemu-devel] [PATCH 2/3] qtest: enable qtest for most targets

2012-04-18 Thread Peter Maydell
On 17 April 2012 22:33, Anthony Liguori aligu...@us.ibm.com wrote: Kernel loading is a hack.  I'll go out on a limb and say that most non-x86 boards are doing it completely wrong.  Messing around with CPU state has no business in machine init.  It creates horrible dependencies about RAM

[Qemu-devel] [RFC PATCH 0/2] Standard SD host controller model

2012-04-18 Thread Igor Mitsyanko
First patch introduces standard SD host controller model. This is accumulated version of my previous patch I sent a while ago and a recent SDHCI patch by Peter A. G. Crosthwaite. Second patch introduces Exynos4210-specific SDHCI built on top of standard SDHCI model. Igor Mitsyanko (2): hw:

[Qemu-devel] [RFC PATCH 2/2] target-arm: introduce Exynos4210 SD host controller model

2012-04-18 Thread Igor Mitsyanko
Exynos4210 SD/MMC host controller is based on SD association standart host controller ver. 2.00 Signed-off-by: Igor Mitsyanko i.mitsya...@samsung.com --- Makefile.target |1 + hw/exynos4210.c | 20 +++ hw/exynos4210_sdhci.c | 438

Re: [Qemu-devel] [PATCH 12/12 v12] introduce a new monitor command 'dump-guest-memory' to dump guest's memory

2012-04-18 Thread Daniel P. Berrange
On Wed, Apr 18, 2012 at 03:30:04PM +0800, Wen Congyang wrote: The command's usage: dump [-p] protocol [begin] [length] The supported protocol can be file or fd: 1. file: the protocol starts with file:, and the following string is the file's path. 2. fd: the protocol starts with fd:,

[Qemu-devel] [PATCH] qxl-render: fix broken vnc+spice since commit f934493

2012-04-18 Thread Alon Levy
Notify any listeners such as vnc that the displaysurface has been changed, otherwise they will segfault when first accessing the freed old displaysurface data. Signed-off-by: Alon Levy al...@redhat.com --- hw/qxl-render.c |1 + 1 file changed, 1 insertion(+) diff --git a/hw/qxl-render.c

[Qemu-devel] [PATCH v2] qxl-render: fix broken vnc+spice since commit f934493

2012-04-18 Thread Alon Levy
Notify any listeners such as vnc that the displaysurface has been changed, otherwise they will segfault when first accessing the freed old displaysurface data. Signed-off-by: Alon Levy al...@redhat.com --- hw/qxl-render.c |1 + 1 file changed, 1 insertion(+) diff --git a/hw/qxl-render.c

Re: [Qemu-devel] [PATCH 12/12 v12] introduce a new monitor command 'dump-guest-memory' to dump guest's memory

2012-04-18 Thread Wen Congyang
At 04/18/2012 05:23 PM, Daniel P. Berrange Wrote: On Wed, Apr 18, 2012 at 03:30:04PM +0800, Wen Congyang wrote: The command's usage: dump [-p] protocol [begin] [length] The supported protocol can be file or fd: 1. file: the protocol starts with file:, and the following string is the

Re: [Qemu-devel] [PATCH 12/12 v12] introduce a new monitor command 'dump-guest-memory' to dump guest's memory

2012-04-18 Thread Wen Congyang
At 04/18/2012 05:23 PM, Daniel P. Berrange Wrote: On Wed, Apr 18, 2012 at 03:30:04PM +0800, Wen Congyang wrote: The command's usage: dump [-p] protocol [begin] [length] The supported protocol can be file or fd: 1. file: the protocol starts with file:, and the following string is the

Re: [Qemu-devel] [PATCH 12/12 v12] introduce a new monitor command 'dump-guest-memory' to dump guest's memory

2012-04-18 Thread Daniel P. Berrange
On Wed, Apr 18, 2012 at 05:44:36PM +0800, Wen Congyang wrote: At 04/18/2012 05:23 PM, Daniel P. Berrange Wrote: On Wed, Apr 18, 2012 at 03:30:04PM +0800, Wen Congyang wrote: The command's usage: dump [-p] protocol [begin] [length] The supported protocol can be file or fd: 1. file:

Re: [Qemu-devel] [PATCH RFC 2/6] target-i386: add cpu-model property to x86_cpu

2012-04-18 Thread Andreas Färber
Am 17.04.2012 01:36, schrieb Igor Mammedov: From: Igor Mammedov niall...@gmail.com Signed-off-by: Igor Mammedov niall...@gmail.com --- cpu-defs.h |2 +- hw/pc.c | 10 -- target-i386/cpu.c| 33 +

Re: [Qemu-devel] [PATCH v2] qxl-render: fix broken vnc+spice since commit f934493

2012-04-18 Thread Gerd Hoffmann
On 04/18/12 11:27, Alon Levy wrote: Notify any listeners such as vnc that the displaysurface has been changed, otherwise they will segfault when first accessing the freed old displaysurface data. Patch added to spice patch queue. thanks, Gerd

[Qemu-devel] [PATCH 2/3] qxl: set default values of vram*_size_mb to -1

2012-04-18 Thread Gerd Hoffmann
From: Alon Levy al...@redhat.com The addition of those values caused a regression where not specifying any value for the vram bar size would result in a 4096 _byte_ surface area. This is ok for the windows driver but causes the X driver to be unusable. Also, it's a regression. This patch returns

[Qemu-devel] [PATCH 3/3] qxl-render: fix broken vnc+spice since commit f934493

2012-04-18 Thread Gerd Hoffmann
From: Alon Levy al...@redhat.com Notify any listeners such as vnc that the displaysurface has been changed, otherwise they will segfault when first accessing the freed old displaysurface data. Signed-off-by: Alon Levy al...@redhat.com Signed-off-by: Gerd Hoffmann kra...@redhat.com ---

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

2012-04-18 Thread Gerd Hoffmann
Hi, Three little fixes accumulated over the last few weeks. please pull, Gerd The following changes since commit 158fd3ce98afd21f2e2639600f6414ea703a9121: qemu-timer.c: Remove 250us timeouts (2012-04-16 12:56:48 -0500) are available in the git repository at:

Re: [Qemu-devel] [PATCH 0/3] switch to seavgabios

2012-04-18 Thread Gerd Hoffmann
Hi, Second, the display panning via vesa pmi was broken in qemu for three years(!) and nobody noticed. The linux kernel's vesafb can use the vesa pmi, it is disabled by default though due to bioses tending to be buggy. I'm not aware of other users. = Is this actually used by anyone?

[Qemu-devel] [PATCH] vhost-net: adjust vhost_net.[c|h] - vhost-net.[c|h]

2012-04-18 Thread zwu . kernel
From: Zhi Yong Wu wu...@linux.vnet.ibm.com Keep the consistent file naming style with other files Signed-off-by: Zhi Yong Wu wu...@linux.vnet.ibm.com --- Makefile.target |2 +- hw/vhost-net.c | 250 +++ hw/vhost-net.h | 20 +

Re: [Qemu-devel] [PATCH v6 1/5] sockets: change inet_connect() to support nonblock socket

2012-04-18 Thread Amos Kong
On 18/04/12 14:52, Orit Wasserman wrote: On 04/17/2012 05:54 PM, Amos Kong wrote: Add a bool argument to inet_connect() to assign if set socket to block/nonblock, and delete original argument 'socktype' that is unused. Retry to connect when following errors are got: -EINTR -EWOULDBLOCK

[Qemu-devel] [PATCH 1/3] trace-events: remove unused qxl_vga_ioport_while_not_in_vga_mode

2012-04-18 Thread Gerd Hoffmann
From: Alon Levy al...@redhat.com The resulting stp file fails to load because of an unresolvable probe. Signed-off-by: Alon Levy al...@redhat.com Signed-off-by: Gerd Hoffmann kra...@redhat.com --- trace-events |1 - 1 files changed, 0 insertions(+), 1 deletions(-) diff --git a/trace-events

[Qemu-devel] [PATCH] qxl: don't assert on guest create_guest_primary

2012-04-18 Thread Alon Levy
initiate the implicit destroy ourselves. Signed-off-by: Alon Levy al...@redhat.com --- hw/qxl.c | 10 +- 1 file changed, 9 insertions(+), 1 deletion(-) diff --git a/hw/qxl.c b/hw/qxl.c index 6776a7f..1ab3348 100644 --- a/hw/qxl.c +++ b/hw/qxl.c @@ -1163,7 +1163,15 @@ static void

Re: [Qemu-devel] [PATCH 0/3] switch to seavgabios

2012-04-18 Thread Michael Tokarev
On 18.04.2012 14:31, Gerd Hoffmann wrote: Hi, Second, the display panning via vesa pmi was broken in qemu for three years(!) and nobody noticed. The linux kernel's vesafb can use the vesa pmi, it is disabled by default though due to bioses tending to be buggy. I'm not aware of other

Re: [Qemu-devel] xen build failure

2012-04-18 Thread Stefan Hajnoczi
On Wed, Apr 18, 2012 at 09:26:44AM +0200, Alexander Graf wrote: On 17.04.2012, at 18:53, Stefano Stabellini wrote: On Tue, 17 Apr 2012, Alon Levy wrote: I have xen-devel-4.1.2-14.fc17.x86_64 and am getting the following build error: CCx86_64-softmmu/xen-all.o

Re: [Qemu-devel] [PATCH v6 2/5] qerror: add five qerror strings

2012-04-18 Thread Paolo Bonzini
Il 18/04/2012 08:51, Orit Wasserman ha scritto: On 04/17/2012 05:54 PM, Amos Kong wrote: Add five new qerror strings, they are about socket: QERR_SOCKET_CONNECT_IN_PROGRESS QERR_SOCKET_CONNECT_FAILED QERR_SOCKET_LISTEN_FAILED QERR_SOCKET_BIND_FAILED QERR_SOCKET_CREATE_FAILED

Re: [Qemu-devel] [RFC PATCH 1/3] iommu: Introduce iommu_group

2012-04-18 Thread David Gibson
On Mon, Apr 02, 2012 at 03:14:40PM -0600, Alex Williamson wrote: IOMMUs often do not have visibility of individual devices in the system. Due to IOMMU design, bus topology, or device quirks, we can often only identify groups of devices. Examples include Intel VT-d AMD-Vi which often have

Re: [Qemu-devel] [PATCH v6 2/5] qerror: add five qerror strings

2012-04-18 Thread Amos Kong
On 18/04/12 19:25, Paolo Bonzini wrote: Il 18/04/2012 08:51, Orit Wasserman ha scritto: On 04/17/2012 05:54 PM, Amos Kong wrote: Add five new qerror strings, they are about socket: QERR_SOCKET_CONNECT_IN_PROGRESS QERR_SOCKET_CONNECT_FAILED QERR_SOCKET_LISTEN_FAILED

Re: [Qemu-devel] [PATCH v6 2/5] qerror: add five qerror strings

2012-04-18 Thread Orit Wasserman
On 04/18/2012 02:25 PM, Paolo Bonzini wrote: Il 18/04/2012 08:51, Orit Wasserman ha scritto: On 04/17/2012 05:54 PM, Amos Kong wrote: Add five new qerror strings, they are about socket: QERR_SOCKET_CONNECT_IN_PROGRESS QERR_SOCKET_CONNECT_FAILED QERR_SOCKET_LISTEN_FAILED

Re: [Qemu-devel] [RFC PATCH 0/2] Standard SD host controller model

2012-04-18 Thread Peter Maydell
On 18 April 2012 09:43, Igor Mitsyanko i.mitsya...@samsung.com wrote: First patch introduces standard SD host controller model. This is accumulated version of my previous patch I sent a while ago and a recent SDHCI patch by Peter A. G. Crosthwaite. Second patch introduces Exynos4210-specific

Re: [Qemu-devel] [RFC PATCH 2/3] iommu: Create basic group infrastructure and update AMD-Vi Intel VT-d

2012-04-18 Thread David Gibson
On Mon, Apr 02, 2012 at 03:14:46PM -0600, Alex Williamson wrote: IOMMU groups define the minimum granularity of the IOMMU. We therefore create groups using a dma_dev which is the effective requestor ID for the entire group. Additional devices can be added to groups based on system topology,

Re: [Qemu-devel] [PATCH 2/3] qtest: enable qtest for most targets

2012-04-18 Thread Anthony Liguori
On 04/18/2012 03:36 AM, Peter Maydell wrote: On 17 April 2012 22:33, Anthony Liguorialigu...@us.ibm.com wrote: Kernel loading is a hack. I'll go out on a limb and say that most non-x86 boards are doing it completely wrong. Messing around with CPU state has no business in machine init. It

Re: [Qemu-devel] [RFC PATCH 0/2] Standard SD host controller model

2012-04-18 Thread Igor Mitsyanko
Yes, I've sent it as RFC for exactly that reason, to discuss it with Peter and anyone else who wants to participate, see if he'll agree or not. On 04/18/2012 04:04 PM, Peter Maydell wrote: On 18 April 2012 09:43, Igor Mitsyankoi.mitsya...@samsung.com wrote: First patch introduces standard SD

Re: [Qemu-devel] [PATCH 2/3] qtest: enable qtest for most targets

2012-04-18 Thread Andreas Färber
Am 18.04.2012 14:09, schrieb Anthony Liguori: On 04/18/2012 03:36 AM, Peter Maydell wrote: On 17 April 2012 22:33, Anthony Liguorialigu...@us.ibm.com wrote: Kernel loading is a hack. I'll go out on a limb and say that most non-x86 boards are doing it completely wrong. Messing around with

[Qemu-devel] [PATCH] qemu-io: Add command line switch for cache mode

2012-04-18 Thread Kevin Wolf
To be used as in 'qemu-io -t writeback test.img' Signed-off-by: Kevin Wolf kw...@redhat.com --- qemu-io.c | 10 +- 1 files changed, 9 insertions(+), 1 deletions(-) diff --git a/qemu-io.c b/qemu-io.c index e6fcd77..3095a22 100644 --- a/qemu-io.c +++ b/qemu-io.c @@ -1784,6 +1784,7 @@

[Qemu-devel] SD card subsystem synchronous I/O

2012-04-18 Thread Stefan Hajnoczi
Recently there have been new SD card emulation patches so I want to raise the issue of synchronous I/O while there is focus on the SD subsystem. Maybe some of the people who are improving the SD subsystem will be able to help. sd_blk_read() and sd_blk_write() use the synchronous block I/O

Re: [Qemu-devel] [PATCH v2 0/2] ide: convert pio code path to asynchronous I/O

2012-04-18 Thread Richard Davies
Stefan Hajnoczi wrote: Chris and Richard: Please test this to confirm that it fixes the hang you reported. ... Ping? We never explicitly said, but yes v2 does fix the hang for us, like v1 did. We are certainly +1 for this going into qemu 1.1. Thanks, Richard.

[Qemu-devel] [PATCH 00/12 v13] introducing a new, dedicated guest memory dump mechanism

2012-04-18 Thread Wen Congyang
Hi, all 'virsh dump' can not work when host pci device is used by guest. We have discussed this issue here: http://lists.nongnu.org/archive/html/qemu-devel/2011-10/msg00736.html The last version is here: http://lists.nongnu.org/archive/html/qemu-devel/2012-04/msg02440.html We have determined to

Re: [Qemu-devel] [PATCH RFC 2/6] target-i386: add cpu-model property to x86_cpu

2012-04-18 Thread Igor Mammedov
On 04/18/2012 12:03 PM, Andreas Färber wrote: Am 17.04.2012 01:36, schrieb Igor Mammedov: From: Igor Mammedovniall...@gmail.com Signed-off-by: Igor Mammedovniall...@gmail.com --- cpu-defs.h |2 +- hw/pc.c | 10 -- target-i386/cpu.c| 33

[Qemu-devel] [PATCH 01/12 v13] Add API to create memory mapping list

2012-04-18 Thread Wen Congyang
The memory mapping list stores virtual address and physical address mapping. The virtual address and physical address are contiguous in the mapping. The folloing patch will use this information to create PT_LOAD in the vmcore. Signed-off-by: Wen Congyang we...@cn.fujitsu.com --- Makefile.target

[Qemu-devel] [PATCH 02/12 v13] Add API to check whether a physical address is I/O address

2012-04-18 Thread Wen Congyang
This API will be used in the following patch. Signed-off-by: Wen Congyang we...@cn.fujitsu.com --- cpu-common.h |2 ++ exec.c |9 + 2 files changed, 11 insertions(+), 0 deletions(-) diff --git a/cpu-common.h b/cpu-common.h index dca5175..fcd50dc 100644 --- a/cpu-common.h

[Qemu-devel] [PATCH 03/12 v13] implement cpu_get_memory_mapping()

2012-04-18 Thread Wen Congyang
Walk cpu's page table and collect all virtual address and physical address mapping. Then, add these mapping into memory mapping list. If the guest does not use paging, it will do nothing. Note: the I/O memory will be skipped. Signed-off-by: Wen Congyang we...@cn.fujitsu.com --- Makefile.target

[Qemu-devel] [PATCH 04/12 v13] Add API to check whether paging mode is enabled

2012-04-18 Thread Wen Congyang
This API will be used in the following patch. Signed-off-by: Wen Congyang we...@cn.fujitsu.com --- cpu-all.h |6 ++ target-i386/arch_memory_mapping.c |7 ++- 2 files changed, 12 insertions(+), 1 deletions(-) diff --git a/cpu-all.h b/cpu-all.h index

[Qemu-devel] [PATCH 05/12 v13] Add API to get memory mapping

2012-04-18 Thread Wen Congyang
Add API to get all virtual address and physical address mapping. If the guest doesn't use paging, the virtual address is equal to the phyical address. The virtual address and physical address mapping is for gdb's user, and it does not include the memory that is not referenced by the page table. So

[Qemu-devel] [PATCH 06/12 v13] Add API to get memory mapping without do paging

2012-04-18 Thread Wen Congyang
crash does not need the virtual address and physical address mapping, and the mapping does not include the memory that is not referenced by the page table. crash does not use the virtual address, so we can create the mapping for all physical memory(virtual address is always 0). This patch provides

Re: [Qemu-devel] [PATCH 15/18] qapi: implement support for variable argument list

2012-04-18 Thread Luiz Capitulino
On Wed, 18 Apr 2012 08:57:19 +0200 Paolo Bonzini pbonz...@redhat.com wrote: Il 17/04/2012 22:42, Luiz Capitulino ha scritto: On Tue, 17 Apr 2012 22:26:55 +0200 Paolo Bonzini pbonz...@redhat.com wrote: Il 17/04/2012 21:36, Luiz Capitulino ha scritto: +

[Qemu-devel] [PATCH 07/12 v13] target-i386: Add API to write elf notes to core file

2012-04-18 Thread Wen Congyang
The core file contains register's value. These APIs write registers to core file, and them will be called in the following patch. Signed-off-by: Wen Congyang we...@cn.fujitsu.com --- Makefile.target |1 + configure |4 + cpu-all.h | 22 +

[Qemu-devel] [PATCH 08/12 v13] target-i386: Add API to write cpu status to core file

2012-04-18 Thread Wen Congyang
The core file has register's value. But it does not include all registers value. Store the cpu status into QEMU note, and the user can get more information from vmcore. If you change QEMUCPUState, please count up QEMUCPUSTATE_VERSION. Signed-off-by: Wen Congyang we...@cn.fujitsu.com ---

[Qemu-devel] [PATCH 09/12 v13] target-i386: add API to get dump info

2012-04-18 Thread Wen Congyang
Dump info contains: endian, class and architecture. The next patch will use these information to create vmcore. Note: on x86 box, the class is ELFCLASS64 if the memory is larger than 4G. Signed-off-by: Wen Congyang we...@cn.fujitsu.com --- cpu-all.h |7 +++ dump.h

[Qemu-devel] [PATCH 10/12 v13] target-i386: Add API to get note's size

2012-04-18 Thread Wen Congyang
We should know where the note and memory is stored before writing them to vmcore. If we know this, we can avoid using lseek() when creating vmcore. Signed-off-by: Wen Congyang we...@cn.fujitsu.com --- cpu-all.h |6 ++ target-i386/arch_dump.c | 33

[Qemu-devel] [PATCH 11/12 v13] make gdb_id() generally avialable and rename it to cpu_index()

2012-04-18 Thread Wen Congyang
The following patch also needs this API, so make it generally avialable. The function gdb_id() will not be used in gdbstub.c now, so its name is not suitable, and rename it to cpu_index() Signed-off-by: Wen Congyang we...@cn.fujitsu.com --- gdbstub.c | 19 +-- gdbstub.h |9

[Qemu-devel] [PATCH 12/12 v13] introduce a new monitor command 'dump-guest-memory' to dump guest's memory

2012-04-18 Thread Wen Congyang
The command's usage: dump [-p] protocol [begin] [length] The supported protocol can be file or fd: 1. file: the protocol starts with file:, and the following string is the file's path. 2. fd: the protocol starts with fd:, and the following string is the fd's name. Note: 1. If you want

Re: [Qemu-devel] [PATCH 00/12 v13] introducing a new, dedicated guest memory dump mechanism

2012-04-18 Thread Wen Congyang
At 04/18/2012 08:49 PM, Wen Congyang Wrote: Hi, all 'virsh dump' can not work when host pci device is used by guest. We have discussed this issue here: http://lists.nongnu.org/archive/html/qemu-devel/2011-10/msg00736.html The last version is here:

Re: [Qemu-devel] block-stream/drive-mirror and default bandwidth

2012-04-18 Thread Stefan Hajnoczi
On Tue, Apr 17, 2012 at 1:14 PM, Eric Blake ebl...@redhat.com wrote: On 04/17/2012 03:20 AM, Stefan Hajnoczi wrote: I think it's cleanest to support block-job-set-speed even when no job is running.  The speed will be used as the default value when a job is started.  This poses the question of

Re: [Qemu-devel] [PATCH 1/2] [trivial] Generic elimination of auto-generated files

2012-04-18 Thread Stefan Hajnoczi
On Sat, Apr 14, 2012 at 12:19:03AM +0200, Lluís Vilanova wrote: - rm -f $(GENERATED_HEADERS) - rm -f $(GENERATED_SOURCES) + rm -f $(foreach f,$(GENERATED_HEADERS),$(f) $(f)-timestamp */$(f) */$(f)-timestamp) + rm -f $(foreach f,$(GENERATED_SOURCES),$(f) $(f)-timestamp */$(f)

Re: [Qemu-devel] [PATCH 0/3] switch to seavgabios

2012-04-18 Thread Gerd Hoffmann
Hi, [ adding seabios list to Cc:, topic is the missing vesa 2.0 protected mode interface in seavgabios ] Pointer? I'd like to have a test case which breaks with the new vgabios. We talked with malc briefly on irc yesterday, and this is what he gave me:

Re: [Qemu-devel] [PATCH 2/2] [trivial] Simpler handling of tracetool-generated files in makefiles

2012-04-18 Thread Stefan Hajnoczi
On Sat, Apr 14, 2012 at 12:19:08AM +0200, Lluís Vilanova wrote: Adds 'tracetool-gen' to generate files with tracetool into a temporary file, and 'tracetool-ci' to commit the generation from the temporaty file into the actual destination file if there were any changes in the produced file.

[Qemu-devel] [PATCH 0/3] block: more fixes to job cancellation

2012-04-18 Thread Paolo Bonzini
Kevin noted how the part of stream_run() that is after the for loop runs with s-common.busy = false. At least bdrv_change_backing_file() can yield, possibly other functions, too. The race window is really small but it's there. The patch takes a more flexible implementation of

[Qemu-devel] [PATCH 3/3] block: mark streaming job busy at the end

2012-04-18 Thread Paolo Bonzini
Kevin noted how the part of stream_run() that is after the for loop runs with s-common.busy = false. At least bdrv_change_backing_file() can yield, possibly other functions, too. The race window is really small but it's there. Thanks to the new, more flexible implementation of

[Qemu-devel] [PATCH 1/3] block: allow interrupting a co_sleep_ns

2012-04-18 Thread Paolo Bonzini
In the next patch we want to reenter the coroutine from block_job_cancel_sync and cancel the timer. Signed-off-by: Paolo Bonzini pbonz...@redhat.com --- qemu-coroutine-sleep.c |3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) diff --git a/qemu-coroutine-sleep.c b/qemu-coroutine-sleep.c

[Qemu-devel] [PATCH 2/3] block: wait for job callback in block_job_cancel_sync

2012-04-18 Thread Paolo Bonzini
The limitation on not having I/O after cancellation cannot really be held. Even streaming has a very small race window where you could cancel a job and have it report completion. If this window is hit, bdrv_change_backing_file() will yield and possibly cause accesses to dangling pointers etc.

Re: [Qemu-devel] [PATCH] qemu-io: Add command line switch for cache mode

2012-04-18 Thread Paolo Bonzini
Il 18/04/2012 14:35, Kevin Wolf ha scritto: To be used as in 'qemu-io -t writeback test.img' Signed-off-by: Kevin Wolf kw...@redhat.com --- qemu-io.c | 10 +- 1 files changed, 9 insertions(+), 1 deletions(-) diff --git a/qemu-io.c b/qemu-io.c index e6fcd77..3095a22 100644

Re: [Qemu-devel] [PATCH v2 0/2] ide: convert pio code path to asynchronous I/O

2012-04-18 Thread Paolo Bonzini
Il 29/03/2012 11:31, Stefan Hajnoczi ha scritto: IDE PIO mode is currently implemented using synchronous I/O functions. There's no need to do this because the IDE interface is actually designed with polling and interrupts in mind - we can do asynchronous I/O and let the guest know when

Re: [Qemu-devel] [PATCH 15/18] qapi: implement support for variable argument list

2012-04-18 Thread Anthony Liguori
On 04/17/2012 03:26 PM, Paolo Bonzini wrote: Il 17/04/2012 21:36, Luiz Capitulino ha scritto: +switch(qobject_type(obj)) { +case QTYPE_QSTRING: +qstring_append(arglist, + qstring_get_str(qobject_to_qstring(obj))); +

Re: [Qemu-devel] [PATCH 15/18] qapi: implement support for variable argument list

2012-04-18 Thread Paolo Bonzini
Il 18/04/2012 14:51, Luiz Capitulino ha scritto: It seems much easier to use no_gen and qemu_opts_from_qdict... Then cmd_netdev_add can be netdev_add/del is expected to be a stable interface, so we can't use no_gen. You can have hmp_netdev_add and the no_gen qmp_netdev_add

Re: [Qemu-devel] [PATCH 1/2] [trivial] Generic elimination of auto-generated files

2012-04-18 Thread Lluís Vilanova
Stefan Hajnoczi writes: On Sat, Apr 14, 2012 at 12:19:03AM +0200, Lluís Vilanova wrote: -rm -f $(GENERATED_HEADERS) -rm -f $(GENERATED_SOURCES) +rm -f $(foreach f,$(GENERATED_HEADERS),$(f) $(f)-timestamp */$(f) */$(f)-timestamp) +rm -f $(foreach f,$(GENERATED_SOURCES),$(f)

Re: [Qemu-devel] [PATCH v2 0/2] ide: convert pio code path to asynchronous I/O

2012-04-18 Thread Zhi Yong Wu
On Thu, Mar 29, 2012 at 5:31 PM, Stefan Hajnoczi stefa...@linux.vnet.ibm.com wrote: IDE PIO mode is currently implemented using synchronous I/O functions.   There's no need to do this because the IDE interface is actually designed with polling and interrupts in mind - we can do asynchronous

Re: [Qemu-devel] [PATCH v2 0/2] ide: convert pio code path to asynchronous I/O

2012-04-18 Thread Zhi Yong Wu
On Thu, Mar 29, 2012 at 5:31 PM, Stefan Hajnoczi stefa...@linux.vnet.ibm.com wrote: IDE PIO mode is currently implemented using synchronous I/O functions.   There's no need to do this because the IDE interface is actually designed with polling and interrupts in mind - we can do asynchronous

Re: [Qemu-devel] [PATCH 2/2] [trivial] Simpler handling of tracetool-generated files in makefiles

2012-04-18 Thread Lluís Vilanova
Stefan Hajnoczi writes: On Sat, Apr 14, 2012 at 12:19:08AM +0200, Lluís Vilanova wrote: trace.h-timestamp: $(SRC_PATH)/trace-events $(BUILD_DIR)/config-host.mak -$(call quiet-command,$(PYTHON) $(SRC_PATH)/scripts/tracetool.py --format=h --backend=$(TRACE_BACKEND) $ $@, GEN trace.h)

Re: [Qemu-devel] xen build failure

2012-04-18 Thread Stefano Stabellini
On Wed, 18 Apr 2012, Stefan Hajnoczi wrote: On Wed, Apr 18, 2012 at 09:26:44AM +0200, Alexander Graf wrote: On 17.04.2012, at 18:53, Stefano Stabellini wrote: On Tue, 17 Apr 2012, Alon Levy wrote: I have xen-devel-4.1.2-14.fc17.x86_64 and am getting the following build error:

[Qemu-devel] [PATCH 5/9] tracetool: Add support for the 'simple' backend

2012-04-18 Thread Stefan Hajnoczi
From: Lluís Vilanova vilan...@ac.upc.edu Signed-off-by: Lluís Vilanova vilan...@ac.upc.edu Signed-off-by: Stefan Hajnoczi stefa...@linux.vnet.ibm.com --- scripts/tracetool/backend/simple.py | 55 +++ 1 file changed, 55 insertions(+) create mode 100644

[Qemu-devel] [PATCH 9/9] tracetool: handle DTrace keywords 'in', 'next', 'self'

2012-04-18 Thread Stefan Hajnoczi
Language keywords cannot be used as argument names. The DTrace backend appends an underscore to the argument name in order to make the argument name legal. This patch adds 'in', 'next', and 'self' keywords to dtrace.py. Also drop the unnecessary argument name lstrip() call. The

[Qemu-devel] [PATCH 8/9] tracetool: Add MAINTAINERS info

2012-04-18 Thread Stefan Hajnoczi
From: Lluís Vilanova vilan...@ac.upc.edu Update the MAINTAINERS file to reflect the new Python tracetool code. [Commit description written by Stefan Hajnoczi] Signed-off-by: Lluís Vilanova vilan...@ac.upc.edu Signed-off-by: Stefan Hajnoczi stefa...@linux.vnet.ibm.com --- MAINTAINERS |2 ++

[Qemu-devel] [PATCH 6/9] tracetool: Add support for the 'ust' backend

2012-04-18 Thread Stefan Hajnoczi
From: Lluís Vilanova vilan...@ac.upc.edu Signed-off-by: Lluís Vilanova vilan...@ac.upc.edu Signed-off-by: Stefan Hajnoczi stefa...@linux.vnet.ibm.com --- scripts/tracetool/backend/ust.py | 90 ++ 1 file changed, 90 insertions(+) create mode 100644

[Qemu-devel] [PATCH 2/9] tracetool: Add module for the 'c' format

2012-04-18 Thread Stefan Hajnoczi
From: Lluís Vilanova vilan...@ac.upc.edu Signed-off-by: Lluís Vilanova vilan...@ac.upc.edu Signed-off-by: Stefan Hajnoczi stefa...@linux.vnet.ibm.com --- scripts/tracetool/format/c.py | 20 1 file changed, 20 insertions(+) create mode 100644 scripts/tracetool/format/c.py

[Qemu-devel] [PATCH 1/9] tracetool: Rewrite infrastructure as python modules

2012-04-18 Thread Stefan Hajnoczi
From: Lluís Vilanova vilan...@ac.upc.edu The tracetool script is written in shell and has hit several portability problems due to shell quirks or external tools across host platforms. Additionally the amount of string processing and lack of real data structures makes it tough to implement code

[Qemu-devel] [PATCH 7/9] tracetool: Add support for the 'dtrace' backend

2012-04-18 Thread Stefan Hajnoczi
From: Lluís Vilanova vilan...@ac.upc.edu Signed-off-by: Lluís Vilanova vilan...@ac.upc.edu Signed-off-by: Stefan Hajnoczi stefa...@linux.vnet.ibm.com --- scripts/tracetool.py| 32 +++- scripts/tracetool/__init__.py | 11 +++- scripts/tracetool/backend/dtrace.py

  1   2   3   >