[Qemu-devel] [PATCH] savevm: fix corruption in vmstate_subsection_load().

2010-12-14 Thread Yoshiaki Tamura
Although it's rare to happen in live migration, when the head of a byte stream contains 0x05 which is the marker of subsection, the loader gets corrupted because vmstate_subsection_load() continues even the device doesn't require it. This patch adds a checker whether subsection is needed, and

[Qemu-devel] Re: [PATCH v4 2/2] RAM API: Make use of it for x86 PC

2010-12-14 Thread Avi Kivity
On 12/13/2010 11:24 PM, Alex Williamson wrote: Register the actual VM RAM using the new API @@ -913,14 +913,11 @@ void pc_memory_init(ram_addr_t ram_size, /* allocate RAM */ ram_addr = qemu_ram_alloc(NULL, pc.ram, below_4g_mem_size +

Re: [Qemu-devel] [PATCH] disable sigcld handling before calling pclose()

2010-12-14 Thread Wen Congyang
At 2010-12-09 11:41, Wen Congyang Write: When I use the command 'virsh save' to save the domain state, I receive the following error message: operation failed: Migration unexpectedly failed. I debug the qemu by adding some printf(), and find the function pclose() returns -1. I use strace

Re: [Qemu-devel] Can any one help me?

2010-12-14 Thread Artyom Tarasenko
2010/12/14 欧阳晓华 toadouy...@gmail.com: I use qemu-0.13.0, and I want to emulate SPARC system, I did these: 1.qemu-img create solaris.img 10G 2.qemu-system-sparc -m 256 -hda solaris.img -boot d -cdrom sol-9-905-sparc.iso 3.qemu reported Unhandled Exception 0X0007, and then Stopping

Re: [Qemu-devel] Re: [PATCH] PCI: Bus number from the bridge, not the device

2010-12-14 Thread Michael S. Tsirkin
On Mon, Dec 13, 2010 at 10:04:24PM -0700, Alex Williamson wrote: On Tue, 2010-12-14 at 06:57 +0200, Michael S. Tsirkin wrote: On Mon, Dec 13, 2010 at 09:49:21PM -0700, Alex Williamson wrote: On Tue, 2010-12-14 at 06:46 +0200, Michael S. Tsirkin wrote: On Mon, Dec 13, 2010 at 01:04:23PM

[Qemu-devel] Re: [PATCH] rtl8139: IO memory is not part of vmstate

2010-12-14 Thread Michael S. Tsirkin
On Mon, Dec 13, 2010 at 10:00:48PM -0700, Alex Williamson wrote: On Tue, 2010-12-14 at 06:43 +0200, Michael S. Tsirkin wrote: On Mon, Dec 13, 2010 at 12:15:08PM -0700, Alex Williamson wrote: On Mon, 2010-12-13 at 21:06 +0200, Michael S. Tsirkin wrote: On Mon, Dec 13, 2010 at 11:59:16AM

[Qemu-devel] Re: KVM call agenda for Dec 14

2010-12-14 Thread Chris Wright
* Chris Wright (chr...@redhat.com) wrote: Please send in any agenda items you are interested in covering. No agenda, today's call is cancelled.

Re: [Qemu-devel] KVM call agenda for Dec 14

2010-12-14 Thread Jes Sorensen
On 12/14/10 01:12, Chris Wright wrote: Please send in any agenda items you are interested in covering. thanks, -chris Chris, Any chance you could fix your cronjob to send out the CFA a day earlier? 15 hrs before is a bit short notice. Cheers, Jes

[Qemu-devel] Re: [PATCH v4 2/2] RAM API: Make use of it for x86 PC

2010-12-14 Thread Alex Williamson
On Tue, 2010-12-14 at 11:18 +0200, Avi Kivity wrote: On 12/13/2010 11:24 PM, Alex Williamson wrote: Register the actual VM RAM using the new API @@ -913,14 +913,11 @@ void pc_memory_init(ram_addr_t ram_size, /* allocate RAM */ ram_addr = qemu_ram_alloc(NULL, pc.ram,

[Qemu-devel] Re: [PATCH v4 2/2] RAM API: Make use of it for x86 PC

2010-12-14 Thread Anthony Liguori
On 12/14/2010 09:16 AM, Alex Williamson wrote: On Tue, 2010-12-14 at 11:18 +0200, Avi Kivity wrote: On 12/13/2010 11:24 PM, Alex Williamson wrote: Register the actual VM RAM using the new API @@ -913,14 +913,11 @@ void pc_memory_init(ram_addr_t ram_size, /* allocate RAM */

Re: [Qemu-devel] [PATCH 2/3] QMP: Drop dead code

2010-12-14 Thread Markus Armbruster
Luiz Capitulino lcapitul...@redhat.com writes: The first if/else clause in handler_audit() makes no sense for two reasons: 1. this function is now called only by QMP code, so testing if it's a QMP call makes no sense anymore 2. the else clause first asserts that there's no error

Re: [Qemu-devel] [PATCH 0/3] QMP: small fixes

2010-12-14 Thread Markus Armbruster
Luiz Capitulino lcapitul...@redhat.com writes: Please, check individual patches for details. ACK series

[Qemu-devel] [PATCH 12/11] ide: honor ncq for atapi

2010-12-14 Thread Alexander Graf
ATAPI also can do ncq, so let's expose the capability. This patch makes CD-ROM support work on Windows 7 for me. Signed-off-by: Alexander Graf ag...@suse.de --- hw/ide/core.c |6 ++ 1 files changed, 6 insertions(+), 0 deletions(-) diff --git a/hw/ide/core.c b/hw/ide/core.c index

Re: [Qemu-devel] KVM call agenda for Dec 14

2010-12-14 Thread Chris Wright
* Jes Sorensen (jes.soren...@redhat.com) wrote: Any chance you could fix your cronjob to send out the CFA a day earlier? 15 hrs before is a bit short notice. Sure.

[Qemu-devel] Re: [PATCH] rtl8139: IO memory is not part of vmstate

2010-12-14 Thread Alex Williamson
On Tue, 2010-12-14 at 14:32 +0200, Michael S. Tsirkin wrote: On Mon, Dec 13, 2010 at 10:00:48PM -0700, Alex Williamson wrote: On Tue, 2010-12-14 at 06:43 +0200, Michael S. Tsirkin wrote: On Mon, Dec 13, 2010 at 12:15:08PM -0700, Alex Williamson wrote: On Mon, 2010-12-13 at 21:06 +0200,

[Qemu-devel] Re: [PATCH] rtl8139: IO memory is not part of vmstate

2010-12-14 Thread Paolo Bonzini
On 12/14/2010 04:41 PM, Alex Williamson wrote: Maybe in this particular case the advantage is minimal. But it seems easier to stick to a rule of no more version bumps than argue about each case. Do we have such a rule? If we have a subsection who's needed function is return 1, I think

[Qemu-devel] Re: [PATCH 03/11] ide: Split out BMDMA code from ATA core

2010-12-14 Thread Kevin Wolf
Am 14.12.2010 01:34, schrieb Alexander Graf: The ATA core is currently heavily intertwined with BMDMA code. Let's loosen that a bit, so we can happily replace the DMA backend with different implementations. Signed-off-by: Alexander Graf ag...@suse.de Something in this patch makes qemu abort

[Qemu-devel] Re: [PATCH 04/11] ide: move transfer_start after variable modification

2010-12-14 Thread Kevin Wolf
Am 14.12.2010 01:34, schrieb Alexander Graf: We hook into transfer_start and immediately call the end function for ahci. This means that everything needs to be in place for the end function when we start the transfer, so let's move the function down to where all state is in place.

[Qemu-devel] Re: [PATCH 00/11] AHCI emulation support v9

2010-12-14 Thread Kevin Wolf
Am 14.12.2010 01:34, schrieb Alexander Graf: This patch adds support for AHCI emulation. I have tested and verified it works in Linux, OpenBSD, Windows Vista and Windows 7. This AHCI emulation supports NCQ, so multiple read or write requests can be outstanding at the same time. The code is

Re: [Qemu-devel] [PATCH] Convert fprintf() to error_request(): virtio-9p

2010-12-14 Thread Markus Armbruster
Stefan Hajnoczi stefa...@gmail.com writes: On Wed, Dec 1, 2010 at 6:01 PM, Venkateswararao Jujjuri (JV) jv...@linux.vnet.ibm.com wrote: On 12/1/2010 1:35 AM, Stefan Hajnoczi wrote: On Tue, Nov 30, 2010 at 9:52 AM, Sanchit Garg sancg...@linux.vnet.ibm.com wrote: @@ -3707,19 +3708,19 @@

Re: [Qemu-devel] [PATCH] RFC: delay pci_update_mappings for 64-bit BARs

2010-12-14 Thread Cam Macdonell
On Mon, Dec 13, 2010 at 8:00 PM, Isaku Yamahata yamah...@valinux.co.jp wrote: On Mon, Dec 13, 2010 at 03:43:44PM -0700, Cam Macdonell wrote: Do not call pci_update_mappings on the lower 32-bits of a 64-bit bar.  Wait for the upper 32 or else Qemu will try to map on just the lower 32 which is

Re: [Qemu-devel] Re: [PATCH] PCI: Bus number from the bridge, not the device

2010-12-14 Thread Alex Williamson
On Tue, 2010-12-14 at 14:26 +0200, Michael S. Tsirkin wrote: On Mon, Dec 13, 2010 at 10:04:24PM -0700, Alex Williamson wrote: I've only ever seen config[PCI_SECONDARY_BUS] be non-zero for an assigned device, so I'm pretty sure we're not going to hurt migration, but the code is clearly

Re: [Qemu-devel] Re: [PATCHv8 00/16] boot order specification

2010-12-14 Thread Andreas Färber
Am 12.12.2010 um 00:22 schrieb Benjamin Herrenschmidt: On Sat, 2010-12-11 at 18:06 +0200, Gleb Natapov wrote: http://playground.sun.com/pub/p1275/bindings/pci/pci2_1.pdf has table on page 10 that defines how pci class code should be translated into OF name. This is what my patch is using.

Re: [Qemu-devel] TCG flow vs dyngen

2010-12-14 Thread Stefano Bonifazi
On 12/11/2010 03:44 PM, Blue Swirl wrote: On Sat, Dec 11, 2010 at 2:32 PM, Stefano Bonifazi stefboombas...@gmail.com wrote: Where does the execution of host binary take place in the previous list of events? Between point 5) and 6) ? After 6) ? In what QEMU source code file/function does the

Re: [Qemu-devel] Re: [PATCHv8 00/16] boot order specification

2010-12-14 Thread Benjamin Herrenschmidt
The only working system emulation we have are Macs (G3 beige, G4, G5), so we can't just ignore Apple. Alex even made me stick to their odd 0x41 rtas-version property. ;) Hah :-) Nothing ever used RTAS on these... afaik, it didn't even work properly. No, but that may be OpenBIOS' fault.

Re: [Qemu-devel] Re: [PATCHv8 00/16] boot order specification

2010-12-14 Thread Alexander Graf
On 14.12.2010, at 21:31, Benjamin Herrenschmidt wrote: The only working system emulation we have are Macs (G3 beige, G4, G5), so we can't just ignore Apple. Alex even made me stick to their odd 0x41 rtas-version property. ;) Hah :-) Nothing ever used RTAS on these... afaik, it didn't

[Qemu-devel] Re: [PATCH 03/11] ide: Split out BMDMA code from ATA core

2010-12-14 Thread Alexander Graf
On 14.12.2010, at 17:20, Kevin Wolf wrote: Am 14.12.2010 01:34, schrieb Alexander Graf: The ATA core is currently heavily intertwined with BMDMA code. Let's loosen that a bit, so we can happily replace the DMA backend with different implementations. Signed-off-by: Alexander Graf

[Qemu-devel] [PATCH 03/12] ide: Split out BMDMA code from ATA core

2010-12-14 Thread Alexander Graf
The ATA core is currently heavily intertwined with BMDMA code. Let's loosen that a bit, so we can happily replace the DMA backend with different implementations. Signed-off-by: Alexander Graf ag...@suse.de --- v7 - v8: - rewrite as DMA ops v8 - v9: - fold in: split out irq setting -

[Qemu-devel] [PATCH 04/12] ide: move transfer_start after variable modification

2010-12-14 Thread Alexander Graf
We hook into transfer_start and immediately call the end function for ahci. This means that everything needs to be in place for the end function when we start the transfer, so let's move the function down to where all state is in place. Signed-off-by: Alexander Graf ag...@suse.de --- v9 - v10:

[Qemu-devel] Re: [PATCH 00/11] AHCI emulation support v9

2010-12-14 Thread Alexander Graf
On 14.12.2010, at 17:28, Kevin Wolf wrote: Am 14.12.2010 01:34, schrieb Alexander Graf: This patch adds support for AHCI emulation. I have tested and verified it works in Linux, OpenBSD, Windows Vista and Windows 7. This AHCI emulation supports NCQ, so multiple read or write requests can

[Qemu-devel] [Qestion] What status of memory stats feature

2010-12-14 Thread Ken'ichi Ohmichi
Hi, I tried to get the memory stats by using virDomainMemoryStats() of libvirt, but it could not do it because of the following patch: [PATCH 03/23] disable guest-provided stats on info balloon command 2010/10/01 from Luiz Capitulino

[Qemu-devel] Sometimes qem-kvm hang at kvm_put_vcpu_events when virsh restore. Both version 0.12.5 / 0.13.0

2010-12-14 Thread changlimin
BTW windows 2008 guest start failed at qem-kvm 0.13.0, but 0.12.5 start success. Best regards

Re: [Qemu-devel] Sometimes qem-kvm hang at kvm_put_vcpu_events when virsh restore. Both version 0.12.5 / 0.13.0

2010-12-14 Thread Mulyadi Santosa
2010/12/15 changlimin changli...@h3c.com: BTW windows 2008 guest start failed at qem-kvm 0.13.0, but 0.12.5 start success. I am not qemu/kvm expert, but I think if you could provide either qemu instruction trace (using -d) or traces inside the guest OS itself (windows live debugger maybe?), it