Re: [Qemu-devel] [PATCH] fix bug of isa_bus irq

2012-03-12 Thread Wanpeng Li
On Sun, Mar 11, 2012 at 08:46:38AM +0100, Jan Kiszka wrote: On 2012-03-11 08:04, Wanpeng Li wrote: ISA bus only use IRQ 0~15, so don't need to give an array qemu_irq 0~23, just array qemu_irq i8259 is ok. Signed-off-by: Wanpeng Li l...@linux.vnet.ibm.com --- hw/pc_piix.c |3 ++- 1

Re: [Qemu-devel] [RFC][PATCH 05/16 v8] Add API to get memory mapping

2012-03-12 Thread HATAYAMA Daisuke
From: Jan Kiszka jan.kis...@siemens.com Subject: Re: [RFC][PATCH 05/16 v8] Add API to get memory mapping Date: Fri, 09 Mar 2012 14:24:41 +0100 On 2012-03-09 13:53, HATAYAMA Daisuke wrote: From: Jan Kiszka jan.kis...@siemens.com Subject: Re: [RFC][PATCH 05/16 v8] Add API to get memory mapping

Re: [Qemu-devel] How to trace all the guest OS instructions and the micro-ops

2012-03-12 Thread Chen Yufei
On Mon, Mar 12, 2012 at 5:43 AM, Mulyadi Santosa mulyadi.sant...@gmail.com wrote: Hi On Sun, Mar 11, 2012 at 10:12, Yue Chen ycyc...@gmail.com wrote: I am doing some research based on the QEMU. Does anyone know how to get (trace) all the instructions of the guest OS, and get all the

Re: [Qemu-devel] [RFC][PATCH 05/16 v8] Add API to get memory mapping

2012-03-12 Thread HATAYAMA Daisuke
From: HATAYAMA Daisuke d.hatay...@jp.fujitsu.com Subject: Re: [Qemu-devel] [RFC][PATCH 05/16 v8] Add API to get memory mapping Date: Mon, 12 Mar 2012 15:16:55 +0900 ( ) The assumption behind my idea is the host is running in a good condition but the quest in a bad condition. So we can use

[Qemu-devel] [PATCH v2 1/2] block: add the support to drain throttled requests

2012-03-12 Thread zwu . kernel
From: Zhi Yong Wu wu...@linux.vnet.ibm.com Signed-off-by: Zhi Yong Wu wu...@linux.vnet.ibm.com --- block.c | 21 + block_int.h |1 + 2 files changed, 22 insertions(+), 0 deletions(-) diff --git a/block.c b/block.c index 52ffe14..0825168 100644 --- a/block.c +++

[Qemu-devel] [PATCH v2 2/2] block: drain throttled requests for one block device

2012-03-12 Thread zwu . kernel
From: Zhi Yong Wu wu...@linux.vnet.ibm.com Signed-off-by: Zhi Yong Wu wu...@linux.vnet.ibm.com --- blockdev.c |4 ++-- hw/ide/macio.c |2 +- hw/ide/pci.c |3 ++- 3 files changed, 5 insertions(+), 4 deletions(-) diff --git a/blockdev.c b/blockdev.c index d78aa51..1bc4667 100644

Re: [Qemu-devel] How to trace all the guest OS instructions and the micro-ops

2012-03-12 Thread Yue Chen
Thanks a lot. So any approach to get the dynamic or static whole memory information of the guest OS ? Not the memory of each process. Sorry for the confusion. I do use version 1.0.1. I mention not in 0.9.1 because someone has already implemented the dynamic tracing in 0.9.1, but not in the

Re: [Qemu-devel] [PATCH] fix bug of isa_bus irq

2012-03-12 Thread Jan Kiszka
On 2012-03-12 07:08, Wanpeng Li wrote: On Sun, Mar 11, 2012 at 08:46:38AM +0100, Jan Kiszka wrote: On 2012-03-11 08:04, Wanpeng Li wrote: ISA bus only use IRQ 0~15, so don't need to give an array qemu_irq 0~23, just array qemu_irq i8259 is ok. Signed-off-by: Wanpeng Li

Re: [Qemu-devel] [PATCH v2 1/2] block: add the support to drain throttled requests

2012-03-12 Thread Paolo Bonzini
Il 12/03/2012 07:29, zwu.ker...@gmail.com ha scritto: From: Zhi Yong Wu wu...@linux.vnet.ibm.com Signed-off-by: Zhi Yong Wu wu...@linux.vnet.ibm.com --- block.c | 21 + block_int.h |1 + 2 files changed, 22 insertions(+), 0 deletions(-) diff --git

Re: [Qemu-devel] QAPI conversion status and async commands support

2012-03-12 Thread Alon Levy
On Wed, Mar 07, 2012 at 10:06:31PM +0200, Alon Levy wrote: On Wed, Mar 07, 2012 at 11:36:06AM -0600, Anthony Liguori wrote: On 03/07/2012 11:29 AM, Paolo Bonzini wrote: Il 07/03/2012 17:36, Luiz Capitulino ha scritto: Hi there, In the last few weeks we've had some proposals for new QMP

Re: [Qemu-devel] seamless migration with spice

2012-03-12 Thread Gerd Hoffmann
Hi, What about the second part? it's independant of the async issue. Isn't this a client problem? The client has this state, no? It is state of the client - server session. Today spice client creates a new session on migration, so there is simply no need to maintain any state. Drawback

Re: [Qemu-devel] [PATCH v2 2/3] make check: Add qemu-iotests subset

2012-03-12 Thread Kevin Wolf
Am 09.03.2012 20:31, schrieb Anthony Liguori: On 03/09/2012 06:46 AM, Kevin Wolf wrote: Run the 'quick' group from qemu-iotests during 'make check'. Signed-off-by: Kevin Wolfkw...@redhat.com --- tests/Makefile |5 + tests/qemu-iotests-quick.sh | 17

[Qemu-devel] [PATCH 1/4] add MIPS DSP helpers define

2012-03-12 Thread Jia Liu
This patch is the helper define of MIPS ASE DSP. Signed-off-by: Jia Liu pro...@gmail.com --- target-mips/helper.h | 152 ++ 1 files changed, 152 insertions(+), 0 deletions(-) diff --git a/target-mips/helper.h b/target-mips/helper.h index

[Qemu-devel] [PATCH 0/4] MIPS ASE DSP Support for Qemu

2012-03-12 Thread Jia Liu
Hi all This is the MIPS ASE DSP Support for Qemu. Jia Liu (4): add MIPS DSP helpers define add MIPS DSP helpers implement add MIPS DSP translation add MIPS DSP testcase target-mips/helper.h | 152 + target-mips/op_helper.c| 3936

[Qemu-devel] [PATCH 3/4] add MIPS DSP translation

2012-03-12 Thread Jia Liu
This patch is the translation of MIPS ASE DSP. Signed-off-by: Jia Liu pro...@gmail.com --- target-mips/translate.c | 1114 +-- 1 files changed, 1088 insertions(+), 26 deletions(-) diff --git a/target-mips/translate.c b/target-mips/translate.c index

Re: [Qemu-devel] [Spice-devel] seamless migration with spice

2012-03-12 Thread Hans de Goede
Hi, On 03/11/2012 02:16 PM, Yonit Halperin wrote: Hi, We would like to implement seamless migration for Spice, i.e., keeping the currently opened spice client session valid after migration. Today, the spice client establishes the connection to the destination before migration starts, and

Re: [Qemu-devel] QAPI conversion status and async commands support

2012-03-12 Thread Wen Congyang
At 03/08/2012 02:12 AM, Luiz Capitulino Wrote: On Wed, 07 Mar 2012 11:36:06 -0600 Anthony Liguori anth...@codemonkey.ws wrote: On 03/07/2012 11:29 AM, Paolo Bonzini wrote: Il 07/03/2012 17:36, Luiz Capitulino ha scritto: Hi there, In the last few weeks we've had some proposals for new QMP

Re: [Qemu-devel] [PATCH v2 1/2] block: add the support to drain throttled requests

2012-03-12 Thread Zhi Yong Wu
On Mon, Mar 12, 2012 at 3:27 PM, Paolo Bonzini pbonz...@redhat.com wrote: Il 12/03/2012 07:29, zwu.ker...@gmail.com ha scritto: From: Zhi Yong Wu wu...@linux.vnet.ibm.com Signed-off-by: Zhi Yong Wu wu...@linux.vnet.ibm.com ---  block.c     |   21 +  block_int.h |    1 +

Re: [Qemu-devel] [PATCH v2 1/2] block: add the support to drain throttled requests

2012-03-12 Thread Paolo Bonzini
Il 12/03/2012 09:42, Zhi Yong Wu ha scritto: This doesn't work, qemu_aio_flush can start new I/O. Do you mean that it will start next I/O via the current request's cb? Yes. Paolo

Re: [Qemu-devel] [Spice-devel] seamless migration with spice

2012-03-12 Thread Hans de Goede
Hi, On 03/12/2012 08:57 AM, Gerd Hoffmann wrote: Hi, What about the second part? it's independant of the async issue. Isn't this a client problem? The client has this state, no? It is state of the client- server session. Today spice client creates a new session on migration, so

Re: [Qemu-devel] [PATCH v1 01/12] net: Add a hub net client

2012-03-12 Thread Zhi Yong Wu
On Fri, Mar 9, 2012 at 6:33 PM, Paolo Bonzini pbonz...@redhat.com wrote: Il 09/03/2012 10:00, zwu.ker...@gmail.com ha scritto: +        /* TODO use qemu_send_packet() or need to call *_deliver_* directly? */ +        /* TODO ignore return value? */ +        qemu_send_packet(port-nc, buf,

Re: [Qemu-devel] [PATCH] virtio-serial-bus: use correct lengths in control_out() message

2012-03-12 Thread Amit Shah
On (Sun) 11 Mar 2012 [17:52:59], Michael Tokarev wrote: In case of more than one control message, the code will use size of the largest message so far for all subsequent messages, instead of using size of current one. Fix it. Makes sense. How did you detect this? Any reproducible test-case?

Re: [Qemu-devel] [PATCH 0/2 v3] kvm: notify host when guest panicked

2012-03-12 Thread Wen Congyang
At 03/09/2012 09:21 AM, Wen Congyang Wrote: At 03/08/2012 07:13 PM, Avi Kivity Wrote: On 03/08/2012 09:57 AM, Wen Congyang wrote: We can know the guest is paniced when the guest runs on xen. But we do not have such feature on kvm. Another purpose of this feature is: management app(for

[Qemu-devel] qemu in domain of xen

2012-03-12 Thread 李安伦
hi there guys recently, i install a qemu in the domian U of xen , the OS of the domain U is debian squeeze . i want to make use of the qemu to install a WINDOWS XP in the domain U, but when i create it , the interface is staying in the processing of formatting the disk. the system of the domain U

[Qemu-devel] tuning parameters qed

2012-03-12 Thread PANKAJ RAWAT
Hi all i have a kvm machine and i use qed format as the disk image format I want to improve the I/O performance of my machine can anyone tell other then cluster_size and cache what are the tunining parameters for qed so that i can improve the performance of my VM ? -- *Pankaj Rawat*

Re: [Qemu-devel] [PATCH v1 01/12] net: Add a hub net client

2012-03-12 Thread Paolo Bonzini
Il 12/03/2012 09:59, Zhi Yong Wu ha scritto: However, then I noticed that qemu_can_send_packet is not called very much, and I do not understand why qemu_net_queue_send and qemu_net_queue_send_iov do not call qemu_can_send_packet before calling deliver/deliver_iov. If they did, hubs could

Re: [Qemu-devel] [PATCH] virtio-serial-bus: use correct lengths in control_out() message

2012-03-12 Thread Michael Tokarev
On 12.03.2012 12:59, Amit Shah wrote: On (Sun) 11 Mar 2012 [17:52:59], Michael Tokarev wrote: In case of more than one control message, the code will use size of the largest message so far for all subsequent messages, instead of using size of current one. Fix it. Makes sense. How did you

Re: [Qemu-devel] Support for Nested Paging

2012-03-12 Thread Stefan Hajnoczi
On Sat, Mar 10, 2012 at 04:44:27PM -0500, Ankur Agrawal wrote: I am a graduate student at Stony Brook University and am working on design and implementation of hypervisors for OSCAR lab ( http://oscar.cs.stonybrook.edu/). Currently I am working on implementing emulation of Nested Page Tables

Re: [Qemu-devel] Support for Nested Paging

2012-03-12 Thread Stefan Hajnoczi
On Sun, Mar 04, 2012 at 01:28:04AM +0800, 陳韋任 wrote: Also I am trying to understand the QEMU source with an objective of participating in the Google Summer of Code and contributing to QEMU. I have tried tracing through the code but seems this link

Re: [Qemu-devel] [PATCH 1/2] Support @documentencoding in scripts/texi2pod.pl

2012-03-12 Thread Stefan Hajnoczi
On Sun, Mar 11, 2012 at 01:56:41PM +0400, Michael Tokarev wrote: Ping? It's been more than a month since this patch has been posted. Maybe it is a good candidate for -trivial queue? One thing I don't understand is where you actually use @documentencoding? I don't see it in you 2 patches.

[Qemu-devel] [PATCH 1/2] trace-events: Rename 'next' argument

2012-03-12 Thread Kevin Wolf
'next' is a systemtap keyword, so it's a bad idea to use it as an argument name. Signed-off-by: Kevin Wolf kw...@redhat.com --- trace-events |2 +- 1 files changed, 1 insertions(+), 1 deletions(-) diff --git a/trace-events b/trace-events index 606d903..2a96353 100644 --- a/trace-events +++

[Qemu-devel] [PATCH 2/2] tracetool: Forbid argument name 'next'

2012-03-12 Thread Kevin Wolf
It has happened more than once that patches that look perfectly sane and work with simpletrace broke systemtap because they use 'next' as an argument name for a tracing function. However, 'next' is a keyword for systemtap, so we shouldn't use it. Signed-off-by: Kevin Wolf kw...@redhat.com ---

Re: [Qemu-devel] [RFC PATCH 14/17] block: support FALLOC_FL_PUNCH_HOLE trimming

2012-03-12 Thread Paolo Bonzini
Il 09/03/2012 21:36, Richard Laager ha scritto: fallocate() only supports files. In my patch, I was fstat()ing the fd just after opening it and caching an is_device boolean. Then, when doing discards, if !is_device, call fallocate(), else (i.e. for devices) do the following (note: untested

[Qemu-devel] [PATCH] acpi: beginnings of piix acpi interface doc

2012-03-12 Thread Michael S. Tsirkin
Before we start tweaking and enhancing hardware, I think it makes sense to document what we currently have, to make sure we stay compatible. This documents the hotplug interface for piix. Stubs for cpu hotplug, PM. Signed-off-by: Michael S. Tsirkin m...@redhat.com --- docs/acpi.txt | 32

Re: [Qemu-devel] [PATCH RFC v4 44/44] qom: Introduce CPU class

2012-03-12 Thread Igor Mammedov
On 03/10/2012 03:28 AM, Andreas Färber wrote: Reintroduce CPUState as QOM object: It's abstract and derived directly from TYPE_OBJECT for compatibility with the user emulators. The identifier CPUState avoids conflicts between CPU() and the struct. Introduce $(qom-twice-y) to build it separately

Re: [Qemu-devel] KVM call agenda for tuesday 31

2012-03-12 Thread Igor Mitsyanko
On 02/21/2012 07:33 PM, Peter Maydell wrote: Short summary: * switch wp groups to bitfield rather than int array * convert sd.c to use memory_region_init_ram() to allocate the wp groups (being careful to use memory_region_set_dirty() when we touch them) * we don't need variable-length

Re: [Qemu-devel] [Spice-devel] seamless migration with spice

2012-03-12 Thread Gerd Hoffmann
Hi, The problem with (b) is, that iirc the way b was implemented in the past was still the big blob approach, but then pass the blob through the client, which means an evil client could modify it, causing all sorts of interesting behavior inside spice-server. Since we're re-implementing

Re: [Qemu-devel] tuning parameters qed

2012-03-12 Thread Stefan Hajnoczi
On Mon, Mar 12, 2012 at 9:12 AM, PANKAJ RAWAT pankajr...@gmail.com wrote: Hi all i have a kvm machine and i use qed format as the disk image format I want to improve the I/O performance of my machine can anyone tell other then cluster_size and cache what are the tunining parameters for qed so

Re: [Qemu-devel] [Spice-devel] seamless migration with spice

2012-03-12 Thread Alon Levy
On Mon, Mar 12, 2012 at 10:46:44AM +0100, Gerd Hoffmann wrote: Hi, The problem with (b) is, that iirc the way b was implemented in the past was still the big blob approach, but then pass the blob through the client, which means an evil client could modify it, causing all sorts of

Re: [Qemu-devel] change the default value of timeout

2012-03-12 Thread Stefan Hajnoczi
On Mon, Mar 12, 2012 at 3:17 AM, Zhang, Yang Z yang.z.zh...@intel.com wrote:        Currently, if not using nonblocking mode, the default timeout of select() in main_loop_wait is 1000ms. There has no problem if you run few VMs. But when running more VMs like 32 or 64, then the problem is

Re: [Qemu-devel] [PATCH 1/2] Support @documentencoding in scripts/texi2pod.pl

2012-03-12 Thread Michael Tokarev
On 12.03.2012 12:21, Stefan Hajnoczi wrote: On Sun, Mar 11, 2012 at 01:56:41PM +0400, Michael Tokarev wrote: Ping? It's been more than a month since this patch has been posted. Maybe it is a good candidate for -trivial queue? One thing I don't understand is where you actually use

Re: [Qemu-devel] [PATCH] Tracing documentation changes

2012-03-12 Thread Stefan Hajnoczi
On Sun, Mar 11, 2012 at 4:13 PM, Peter Teoh htmldevelo...@gmail.com wrote: I was trying out the tracing feature of QEMU after checking out the git tree at git://git.qemu.org/qemu.git, and managed to generate some traces, but the following are the changes needed to the documentation, in order

Re: [Qemu-devel] [PATCH v2 1/5] qxl: switch qxl.c to trace-events

2012-03-12 Thread Gerd Hoffmann
On 03/11/12 20:26, Alon Levy wrote: dprint is still used for qxl_init_common one time prints. I think we shouldn't simply convert the dprintf's into trace-points. We should look at each dprintf and check whenever it makes sense at all, whenever it makes sense at that place before converting it

Re: [Qemu-devel] [PATCH] ide: Adds model=s option, allowing the user to override the default disk model name QEMU HARDDISK

2012-03-12 Thread Stefan Hajnoczi
On Sat, Mar 10, 2012 at 7:56 PM, Floris Bos b...@je-eigen-domein.nl wrote: @@ -1885,6 +1885,22 @@ int ide_init_drive(IDEState *s, BlockDriverState *bs, IDEDriveKind kind,         snprintf(s-drive_serial_str, sizeof(s-drive_serial_str),                  QM%05d, s-drive_serial);     } +    if

Re: [Qemu-devel] [PATCH] maintainers: Add docs/tracing.txt to Tracing

2012-03-12 Thread Stefan Hajnoczi
On Sat, Mar 10, 2012 at 12:37 PM, Andreas Färber afaer...@suse.de wrote: The topic of whether and by whom docs/tracing.txt is maintained was brought up. It currently does not have an official maintainer. Add it to the tracing section so that Stefan gets cc'ed on patches. Signed-off-by:

Re: [Qemu-devel] [Spice-devel] seamless migration with spice

2012-03-12 Thread Gerd Hoffmann
Hi, As for certain other data, such as (but not limited to) partially parsed agent messages, these should be send through the regular vmstate methods IMHO. That isn't easy to handle via vmstate, at least as soon as this goes beyond a fixed number of fields aka 'migrate over this struct

Re: [Qemu-devel] [PATCH 0/2 v3] kvm: notify host when guest panicked

2012-03-12 Thread Avi Kivity
On 03/09/2012 03:21 AM, Wen Congyang wrote: Changes from v2 to v3: 1. correct spelling Changes from v1 to v2: 1. split up host and guest-side changes 2. introduce new request flag to avoid changing return values. I see no Documentation/ changes. What shoude be writen into

Re: [Qemu-devel] [PATCH 0/2 v3] kvm: notify host when guest panicked

2012-03-12 Thread Avi Kivity
On 03/12/2012 11:04 AM, Wen Congyang wrote: Do you have any other comments about this patch? Not really, but I'm not 100% convinced the patch is worthwhile. It's likely to only be used by Linux, which has kexec facilities, and you can put talk to management via virtio-serial and describe the

Re: [Qemu-devel] [PATCH 1/2] Support @documentencoding in scripts/texi2pod.pl

2012-03-12 Thread Stefan Hajnoczi
On Mon, Mar 12, 2012 at 10:14 AM, Michael Tokarev m...@tls.msk.ru wrote: On 12.03.2012 12:21, Stefan Hajnoczi wrote: On Sun, Mar 11, 2012 at 01:56:41PM +0400, Michael Tokarev wrote: Ping?  It's been more than a month since this patch has been posted. Maybe it is a good candidate for -trivial

Re: [Qemu-devel] [PATCH v4 1/3] kvmclock: Always register type

2012-03-12 Thread Avi Kivity
On 03/10/2012 03:35 AM, Andreas Färber wrote: Am 05.03.2012 10:23, schrieb Avi Kivity: On 03/04/2012 10:32 PM, Andreas Färber wrote: Currently, the kvmclock type is only registered when kvm_enabled(). This breaks when moving type registration to before command line parsing (so that QOM

[Qemu-devel] Re : Re : Regression: more 0.12 regression (SeaBIOS related?)

2012-03-12 Thread Alain Ribière
On Wed, Mar 07, 2012 at 06:31:31AM -0800, Alain Ribière wrote: I ran qemu 1.0.1 and the latest SeaBIOS (from the git) with the following options : qemu-system-i386 -L git/bios -fda disk.img -no-fd-bootchk -boot a -m 16 Here is the log :

Re: [Qemu-devel] [RFC PATCH 07/17] block: make high level discard operation always zero

2012-03-12 Thread Avi Kivity
On 03/09/2012 06:42 PM, Kevin Wolf wrote: While (3) can be worked around, the only way around the other two, unfortunately, is support in the formats and protocols. We will still provide device options to opt out of this, but with raw and qed covered (+ qcow2 without backing file, and

Re: [Qemu-devel] [RFC PATCH 07/17] block: make high level discard operation always zero

2012-03-12 Thread Kevin Wolf
Am 12.03.2012 11:42, schrieb Avi Kivity: On 03/09/2012 06:42 PM, Kevin Wolf wrote: While (3) can be worked around, the only way around the other two, unfortunately, is support in the formats and protocols. We will still provide device options to opt out of this, but with raw and qed covered

Re: [Qemu-devel] [PATCH 2/2] tracetool: Forbid argument name 'next'

2012-03-12 Thread Stefan Hajnoczi
On Mon, Mar 12, 2012 at 9:34 AM, Kevin Wolf kw...@redhat.com wrote: It has happened more than once that patches that look perfectly sane and work with simpletrace broke systemtap because they use 'next' as an argument name for a tracing function. However, 'next' is a keyword for systemtap, so

Re: [Qemu-devel] [PATCH 2/2] tracetool: Forbid argument name 'next'

2012-03-12 Thread Kevin Wolf
Am 12.03.2012 12:01, schrieb Stefan Hajnoczi: On Mon, Mar 12, 2012 at 9:34 AM, Kevin Wolf kw...@redhat.com wrote: It has happened more than once that patches that look perfectly sane and work with simpletrace broke systemtap because they use 'next' as an argument name for a tracing function.

Re: [Qemu-devel] [PATCH] virtio-serial-bus: use correct lengths in control_out() message

2012-03-12 Thread Amit Shah
On (Mon) 12 Mar 2012 [13:22:22], Michael Tokarev wrote: On 12.03.2012 12:59, Amit Shah wrote: On (Sun) 11 Mar 2012 [17:52:59], Michael Tokarev wrote: In case of more than one control message, the code will use size of the largest message so far for all subsequent messages, instead of

Re: [Qemu-devel] [PATCH] acpi: beginnings of piix acpi interface doc

2012-03-12 Thread Gleb Natapov
On Mon, Mar 12, 2012 at 11:35:29AM +0200, Michael S. Tsirkin wrote: Before we start tweaking and enhancing hardware, I think it makes sense to document what we currently have, to make sure we stay compatible. This documents the hotplug interface for piix. Stubs for cpu hotplug, PM. We

Re: [Qemu-devel] [Spice-devel] seamless migration with spice

2012-03-12 Thread Hans de Goede
Hi, On 03/12/2012 10:46 AM, Gerd Hoffmann wrote: Hi, The problem with (b) is, that iirc the way b was implemented in the past was still the big blob approach, but then pass the blob through the client, which means an evil client could modify it, causing all sorts of interesting behavior

Re: [Qemu-devel] [Spice-devel] seamless migration with spice

2012-03-12 Thread Alon Levy
On Mon, Mar 12, 2012 at 11:26:50AM +0100, Gerd Hoffmann wrote: Hi, As for certain other data, such as (but not limited to) partially parsed agent messages, these should be send through the regular vmstate methods IMHO. That isn't easy to handle via vmstate, at least as soon as

Re: [Qemu-devel] [PATCH] ide: Adds model=s option, allowing the user to override the default disk model name QEMU HARDDISK

2012-03-12 Thread Floris Bos / Maxnet
On 03/12/2012 11:26 AM, Stefan Hajnoczi wrote: On Sat, Mar 10, 2012 at 7:56 PM, Floris Bosb...@je-eigen-domein.nl wrote: @@ -1885,6 +1885,22 @@ int ide_init_drive(IDEState *s, BlockDriverState *bs, IDEDriveKind kind, snprintf(s-drive_serial_str, sizeof(s-drive_serial_str),

Re: [Qemu-devel] Add GSoC project ideas to the wiki!

2012-03-12 Thread Kevin Wolf
Am 24.02.2012 10:19, schrieb Stefan Hajnoczi: This is a reminder that QEMU will apply for Google Summer of Code 2012 and we need project ideas and mentors. Libvirt and kvm.ko projects are also welcome! http://wiki.qemu.org/Google_Summer_of_Code_2012 Please add yourself to the wiki now if

Re: [Qemu-devel] [Spice-devel] seamless migration with spice

2012-03-12 Thread Gerd Hoffmann
On 03/12/12 12:29, Alon Levy wrote: On Mon, Mar 12, 2012 at 11:26:50AM +0100, Gerd Hoffmann wrote: Hi, Migrate this struct n times for me. I think for the agent case this isn't needed. Or is every client allowed to speak to the agent in case of multiple clients connected? I somehow

Re: [Qemu-devel] [PATCH] ide: Adds model=s option, allowing the user to override the default disk model name QEMU HARDDISK

2012-03-12 Thread Kevin Wolf
Am 12.03.2012 12:30, schrieb Floris Bos / Maxnet: On 03/12/2012 11:26 AM, Stefan Hajnoczi wrote: On Sat, Mar 10, 2012 at 7:56 PM, Floris Bosb...@je-eigen-domein.nl wrote: @@ -1885,6 +1885,22 @@ int ide_init_drive(IDEState *s, BlockDriverState *bs, IDEDriveKind kind,

Re: [Qemu-devel] [PATCH] qemu-io: add option to enable tracing

2012-03-12 Thread Stefan Hajnoczi
On Wed, Dec 21, 2011 at 3:35 PM, Stefan Hajnoczi stefa...@linux.vnet.ibm.com wrote: Ping? Do you want to take this through the block tree? It can be useful to enable QEMU tracing when trying out block layer interfaces via qemu-io.  Tracing can be enabled using the new -t FILE option where

Re: [Qemu-devel] [PATCH v2 1/5] qxl: switch qxl.c to trace-events

2012-03-12 Thread Alon Levy
On Mon, Mar 12, 2012 at 11:20:55AM +0100, Gerd Hoffmann wrote: On 03/11/12 20:26, Alon Levy wrote: dprint is still used for qxl_init_common one time prints. I think we shouldn't simply convert the dprintf's into trace-points. We should look at each dprintf and check whenever it makes

Re: [Qemu-devel] [PATCH] virtio-serial-bus: use correct lengths in control_out() message

2012-03-12 Thread Michael Tokarev
On 12.03.2012 15:06, Amit Shah wrote: On (Mon) 12 Mar 2012 [13:22:22], Michael Tokarev wrote: On 12.03.2012 12:59, Amit Shah wrote: On (Sun) 11 Mar 2012 [17:52:59], Michael Tokarev wrote: In case of more than one control message, the code will use size of the largest message so far for all

Re: [Qemu-devel] [PATCH] qemu-io: add option to enable tracing

2012-03-12 Thread Kevin Wolf
Am 12.03.2012 12:39, schrieb Stefan Hajnoczi: On Wed, Dec 21, 2011 at 3:35 PM, Stefan Hajnoczi stefa...@linux.vnet.ibm.com wrote: Ping? Do you want to take this through the block tree? Sorry, I completely missed this one. It can be useful to enable QEMU tracing when trying out block layer

Re: [Qemu-devel] [PATCH] ide: Adds model=s option, allowing the user to override the default disk model name QEMU HARDDISK

2012-03-12 Thread Kevin Wolf
Am 10.03.2012 20:56, schrieb Floris Bos: Some Linux distributions use the /dev/disk/by-id/scsi-SATA_name-of-disk-model_serial addressing scheme when refering to partitions in /etc/fstab and elsewhere. This causes problems when starting a disk image taken from an existing physical server

Re: [Qemu-devel] [PATCH 2/2] memory: print aliased IO ranges in info mtree

2012-03-12 Thread Avi Kivity
On 03/11/2012 01:00 PM, Blue Swirl wrote: Print also I/O ports behind bridges and other aliases. Signed-off-by: Blue Swirl blauwir...@gmail.com --- memory.c | 13 + 1 files changed, 13 insertions(+), 0 deletions(-) diff --git a/memory.c b/memory.c index 4c3dc49..0201392

Re: [Qemu-devel] [RFC PATCH 07/17] block: make high level discard operation always zero

2012-03-12 Thread Avi Kivity
On 03/12/2012 01:04 PM, Kevin Wolf wrote: qcow2 can't handle clusters that are referenced twice from the same L1 table. This would require a reverse lookup to adjust the QCOW_O_COPIED flags in the L2 tables containing the other references. Don't follow, sorry. What adjustment are you

[Qemu-devel] [PATCH v2] ide: Adds model=s option, allowing the user to override the default disk model name QEMU HARDDISK

2012-03-12 Thread Floris Bos
Some Linux distributions use the /dev/disk/by-id/scsi-SATA_name-of-disk-model_serial addressing scheme when refering to partitions in /etc/fstab and elsewhere. This causes problems when starting a disk image taken from an existing physical server under qemu, because when running under qemu

Re: [Qemu-devel] [PATCH] kvm: add flightrecorder script

2012-03-12 Thread Avi Kivity
On 03/09/2012 04:13 PM, Stefan Hajnoczi wrote: The kvm kernel module includes a number of trace events which can be useful when debugging system behavior. Even on production systems these trace events can be used to observe guest behavior and identify the source of problems. The

Re: [Qemu-devel] [PATCH RESEND v5 3/6] Set runstate to INMIGRATE earlier

2012-03-12 Thread Stefano Stabellini
On Wed, 7 Mar 2012, Luiz Capitulino wrote: On Tue, 28 Feb 2012 15:51:32 + Stefano Stabellini stefano.stabell...@eu.citrix.com wrote: Set runstate to RUN_STATE_INMIGRATE as soon as we can on resume. Signed-off-by: Stefano Stabellini stefano.stabell...@eu.citrix.com Would be nice

Re: [Qemu-devel] [PATCH] ide: Adds model=s option, allowing the user to override the default disk model name QEMU HARDDISK

2012-03-12 Thread Floris Bos / Maxnet
On 03/12/2012 12:57 PM, Kevin Wolf wrote: Am 10.03.2012 20:56, schrieb Floris Bos: Some Linux distributions use the /dev/disk/by-id/scsi-SATA_name-of-disk-model_serial addressing scheme when refering to partitions in /etc/fstab and elsewhere. This causes problems when starting a disk image

Re: [Qemu-devel] [PATCH v1 01/12] net: Add a hub net client

2012-03-12 Thread Zhi Yong Wu
On Mon, Mar 12, 2012 at 5:12 PM, Paolo Bonzini pbonz...@redhat.com wrote: Il 12/03/2012 09:59, Zhi Yong Wu ha scritto: However, then I noticed that qemu_can_send_packet is not called very much, and I do not understand why qemu_net_queue_send and qemu_net_queue_send_iov do not call

Re: [Qemu-devel] [PATCH 2/2] tracetool: Forbid argument name 'next'

2012-03-12 Thread Lluís Vilanova
Stefan Hajnoczi writes: On Mon, Mar 12, 2012 at 9:34 AM, Kevin Wolf kw...@redhat.com wrote: It has happened more than once that patches that look perfectly sane and work with simpletrace broke systemtap because they use 'next' as an argument name for a tracing function. However, 'next' is a

Re: [Qemu-devel] [Spice-devel] seamless migration with spice

2012-03-12 Thread Gerd Hoffmann
Hi, Is there a complete list of the session state we need to save? There is still code in spice-server for the old seamless migration, someone could go through that and use that as an initial list of session state we need to save. That doesn't help much as it is _way_ too old. Predates

Re: [Qemu-devel] [RFC PATCH 06/17] block: use bdrv_{co, aio}_discard for write_zeroes operations

2012-03-12 Thread Paolo Bonzini
Il 10/03/2012 19:02, Richard Laager ha scritto: I propose adding the following behaviors in any event: * If a QEMU block device reports a discard_granularity 0, it must be equal to 2^n (n = 0), or QEMU's block core will change it to 0. (Non-power-of-two granularities are

Re: [Qemu-devel] [Spice-devel] seamless migration with spice

2012-03-12 Thread Alon Levy
On Mon, Mar 12, 2012 at 12:34:42PM +0100, Gerd Hoffmann wrote: On 03/12/12 12:29, Alon Levy wrote: On Mon, Mar 12, 2012 at 11:26:50AM +0100, Gerd Hoffmann wrote: Hi, Migrate this struct n times for me. I think for the agent case this isn't needed. Or is every client allowed to

[Qemu-devel] [PATCH 7/7] vga: add trace event for ppm_save

2012-03-12 Thread Stefan Hajnoczi
From: Alon Levy al...@redhat.com Signed-off-by: Alon Levy al...@redhat.com Signed-off-by: Stefan Hajnoczi stefa...@linux.vnet.ibm.com --- hw/vga.c |2 ++ trace-events |3 +++ 2 files changed, 5 insertions(+), 0 deletions(-) diff --git a/hw/vga.c b/hw/vga.c index 5994f43..6dc98f6

[Qemu-devel] [PATCH 3/7] trace: make trace_thread_create() use its function arg

2012-03-12 Thread Stefan Hajnoczi
From: Jun Koi junkoi2...@gmail.com This patch makes trace_thread_create() to use its function arg to initialize thread. The other choice is to make this a function to use void arg, but i prefer this way. Signed-off-by: Jun Koi junkoi2...@gmail.com Signed-off-by: Stefan Hajnoczi

Re: [Qemu-devel] [Spice-devel] seamless migration with spice

2012-03-12 Thread David Jaša
Hans de Goede píše v Po 12. 03. 2012 v 09:51 +0100: Hi, On 03/12/2012 08:57 AM, Gerd Hoffmann wrote: Hi, What about the second part? it's independant of the async issue. Isn't this a client problem? The client has this state, no? It is state of the client- server session.

[Qemu-devel] KVM call agenda for Tuesday 13

2012-03-12 Thread Juan Quintela
Hi Please send in any agenda items you are interested in covering. Cheers, Juan.

Re: [Qemu-devel] [PATCH RESEND v5 1/6] cirrus_vga: do not reset videoram

2012-03-12 Thread Avi Kivity
On 02/28/2012 05:51 PM, Stefano Stabellini wrote: There is no need to set the videoram to 0xff in cirrus_reset, because it is the BIOS' job. Reviewed-by: Avi Kivity a...@redhat.com -- error compiling committee.c: too many arguments to function

[Qemu-devel] [PATCH 4/7] docs: correct ./configure line in tracing.txt

2012-03-12 Thread Stefan Hajnoczi
From: Jun Koi junkoi2...@gmail.com This patch corrects the configure's trace option in docs/tracing.txt. Signed-off-by: Jun Koi junkoi2...@gmail.com Signed-off-by: Stefan Hajnoczi stefa...@linux.vnet.ibm.com --- docs/tracing.txt |2 +- 1 files changed, 1 insertions(+), 1 deletions(-) diff

Re: [Qemu-devel] [Spice-devel] seamless migration with spice

2012-03-12 Thread Yonit Halperin
On 03/12/2012 11:46 AM, Gerd Hoffmann wrote: Hi, The problem with (b) is, that iirc the way b was implemented in the past was still the big blob approach, but then pass the blob through the client, which means an evil client could modify it, causing all sorts of interesting behavior inside

[Qemu-devel] [PATCH 5/7] maintainers: Add docs/tracing.txt to Tracing

2012-03-12 Thread Stefan Hajnoczi
From: Andreas Färber afaer...@suse.de The topic of whether and by whom docs/tracing.txt is maintained was brought up. It currently does not have an official maintainer. Add it to the tracing section so that Stefan gets cc'ed on patches. Signed-off-by: Andreas Färber afaer...@suse.de Acked-by:

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

2012-03-12 Thread Stefan Hajnoczi
First round of tracing patches. I should have sent these out a long time ago. Once these are out of the way I can review and handle bigger patches from Lluís and Harsh. The following changes since commit a348f108842fb928563865c9918642900cd0d477: Add missing const attributes for

[Qemu-devel] [PATCH 6/7] console: add some trace events

2012-03-12 Thread Stefan Hajnoczi
From: Alon Levy al...@redhat.com Signed-off-by: Alon Levy al...@redhat.com Signed-off-by: Stefan Hajnoczi stefa...@linux.vnet.ibm.com --- console.h|3 +++ trace-events |4 2 files changed, 7 insertions(+), 0 deletions(-) diff --git a/console.h b/console.h index a95b581..4334db5

[Qemu-devel] [PATCH 2/7] tracetool: Omit useless QEMU_*_ENABLED() check

2012-03-12 Thread Stefan Hajnoczi
SystemTap provides a semaphore that can optionally be tested before executing a trace event. The purpose of this mechanism is to skip expensive tracing code when the trace event is disabled. For example, some applications may have trace events that format or convert strings for trace events.

Re: [Qemu-devel] [libvirt] Modern CPU models cannot be used with libvirt

2012-03-12 Thread Eduardo Habkost
On Sun, Mar 11, 2012 at 09:12:58AM -0500, Anthony Liguori wrote: On 03/11/2012 08:27 AM, Gleb Natapov wrote: On Sat, Mar 10, 2012 at 12:24:47PM -0600, Anthony Liguori wrote: Let's step back here. Why are you writing these patches? It's probably not because you have a desire to say -cpu

Re: [Qemu-devel] [PATCH 2/4] block: image fragmentation statistics for qed

2012-03-12 Thread Stefan Hajnoczi
On Wed, Mar 7, 2012 at 9:22 AM, Dong Xu Wang wdon...@linux.vnet.ibm.com wrote: +    for (i = 0; i table_noffsets; i++) { +        l2_offset = s-l1_table-offsets[i]; +        if (l2_offset == 0) { if (qed_offset_is_unalloc_cluster(l2_offset)) { +            continue; +        } +        

Re: [Qemu-devel] [RFC PATCH 06/17] block: use bdrv_{co, aio}_discard for write_zeroes operations

2012-03-12 Thread Kevin Wolf
Am 12.03.2012 13:27, schrieb Paolo Bonzini: Il 10/03/2012 19:02, Richard Laager ha scritto: I propose adding the following behaviors in any event: * If a QEMU block device reports a discard_granularity 0, it must be equal to 2^n (n = 0), or QEMU's block core will change

Re: [Qemu-devel] [libvirt] Modern CPU models cannot be used with libvirt

2012-03-12 Thread Daniel P. Berrange
On Mon, Mar 12, 2012 at 09:52:27AM -0300, Eduardo Habkost wrote: On Sun, Mar 11, 2012 at 09:12:58AM -0500, Anthony Liguori wrote: On 03/11/2012 08:27 AM, Gleb Natapov wrote: On Sat, Mar 10, 2012 at 12:24:47PM -0600, Anthony Liguori wrote: Let's step back here. Why are you writing these

Re: [Qemu-devel] [PATCH 1/4] block: add image fragmentation statistics to qemu-img

2012-03-12 Thread Stefan Hajnoczi
On Wed, Mar 7, 2012 at 9:22 AM, Dong Xu Wang wdon...@linux.vnet.ibm.com wrote: From: Dong Xu Wang wdon...@linux.vnet.ibm.com Discussion can be found at: http://patchwork.ozlabs.org/patch/128730/ This patch add image fragmentation statistics while using qemu-img info. Signed-off-by: Dong Xu

Re: [Qemu-devel] [libvirt] Modern CPU models cannot be used with libvirt

2012-03-12 Thread Eduardo Habkost
On Sun, Mar 11, 2012 at 10:41:32AM -0500, Anthony Liguori wrote: On 03/11/2012 10:12 AM, Gleb Natapov wrote: On Sun, Mar 11, 2012 at 09:16:49AM -0500, Anthony Liguori wrote: If libvirt assumes anything about what kvm actually supports it is working only by sheer luck. Well the simple answer

Re: [Qemu-devel] [PATCH 1/4] block: add image fragmentation statistics to qemu-img

2012-03-12 Thread Kevin Wolf
Am 12.03.2012 14:07, schrieb Stefan Hajnoczi: On Wed, Mar 7, 2012 at 9:22 AM, Dong Xu Wang wdon...@linux.vnet.ibm.com wrote: From: Dong Xu Wang wdon...@linux.vnet.ibm.com Discussion can be found at: http://patchwork.ozlabs.org/patch/128730/ This patch add image fragmentation statistics

[Qemu-devel] [PATCH/RFC 2/7] Allow a qemu_fopen_socket() to be opened for writing

2012-03-12 Thread Marc-André Lureau
--- migration-tcp.c |2 +- migration-unix.c |2 +- qemu-file.h |2 +- savevm.c | 36 4 files changed, 35 insertions(+), 7 deletions(-) diff --git a/migration-tcp.c b/migration-tcp.c index 35a5781..f567898 100644 ---

Re: [Qemu-devel] [libvirt] Modern CPU models cannot be used with libvirt

2012-03-12 Thread Gleb Natapov
On Mon, Mar 12, 2012 at 01:04:19PM +, Daniel P. Berrange wrote: On Mon, Mar 12, 2012 at 09:52:27AM -0300, Eduardo Habkost wrote: On Sun, Mar 11, 2012 at 09:12:58AM -0500, Anthony Liguori wrote: On 03/11/2012 08:27 AM, Gleb Natapov wrote: On Sat, Mar 10, 2012 at 12:24:47PM -0600,

Re: [Qemu-devel] [PATCH] ide: Adds model=s option, allowing the user to override the default disk model name QEMU HARDDISK

2012-03-12 Thread Kevin Wolf
Am 12.03.2012 13:09, schrieb Floris Bos / Maxnet: On 03/12/2012 12:57 PM, Kevin Wolf wrote: Am 10.03.2012 20:56, schrieb Floris Bos: Some Linux distributions use the /dev/disk/by-id/scsi-SATA_name-of-disk-model_serial addressing scheme when refering to partitions in /etc/fstab and elsewhere.

  1   2   3   4   >