Re: RFC: New API for PPC for vcpu mmu access

2011-02-08 Thread Avi Kivity
On 02/07/2011 07:30 PM, Yoder Stuart-B08248 wrote: -Original Message- From: kvm-ppc-ow...@vger.kernel.org [mailto:kvm-ppc-ow...@vger.kernel.org] On Behalf Of Avi Kivity Sent: Monday, February 07, 2011 11:14 AM To: Alexander Graf Cc: Wood Scott-B07421; Yoder Stuart-B08248;

Re: [Qemu-devel] Re: [RFC: 0/2] patch for QEMU HPET periodic timer emulation to alleviate time drift

2011-02-08 Thread Avi Kivity
On 02/07/2011 09:30 PM, Anthony Liguori wrote: For the purposes of this, I think passing an opaque is better because the signature stays the same as the existing timer callback. That makes conversion a bit friendlier. I think it's better to avoid introducing stylistic changes with new

Re: PCI Passthrough, error: The driver 'pci-stub' is occupying your device 0000:08:06.2

2011-02-08 Thread James Neave
On Mon, Feb 7, 2011 at 1:26 PM, Daniel P. Berrange berra...@redhat.com wrote: On Sat, Feb 05, 2011 at 04:34:01PM +, James Neave wrote: Hi, I'm trying to pass a NOVA-T-500 TV Tuner card through to a gust VM. I'm getting the error The driver 'pci-stub' is occupying your device :08:06.2

Re: [PATCH] KVM: x86: Convert tsc_write_lock to raw_spinlock

2011-02-08 Thread Avi Kivity
On 02/07/2011 07:23 PM, Jan Kiszka wrote: On 2011-02-07 18:10, Avi Kivity wrote: On 02/07/2011 06:59 PM, Jan Kiszka wrote: (well, actually, cpufreq_notifier and kvm_arch_hardware_enable are already non preemptible, and the stats code should just go away?) The stats code is trivial to

Re: KVM call agenda for Feb 8

2011-02-08 Thread Stefan Hajnoczi
On Mon, Feb 7, 2011 at 10:40 PM, Chris Wright chr...@redhat.com wrote: Please send in any agenda items you are interested in covering. Automated builds and testing: maintainer trees, integrating KVM-Autotest, and QEMU tests we need but don't exist Stefan -- To unsubscribe from this list: send

[Bug 27052] Module KVM : unable to handle kernel NULL pointer dereference at

2011-02-08 Thread bugzilla-daemon
https://bugzilla.kernel.org/show_bug.cgi?id=27052 --- Comment #19 from prochazka prochazka.nico...@gmail.com 2011-02-08 09:46:04 --- An other log without patch #18 / #17 ( as #16 ) I try with #18 Feb 8 00:16:37 bergson18494 rmap_remove: gfn=0 index=511 level=1 Feb 8 00:16:37

Re: [PATCH] KVM: x86: Convert tsc_write_lock to raw_spinlock

2011-02-08 Thread Jan Kiszka
On 2011-02-08 10:15, Avi Kivity wrote: On 02/07/2011 07:23 PM, Jan Kiszka wrote: On 2011-02-07 18:10, Avi Kivity wrote: On 02/07/2011 06:59 PM, Jan Kiszka wrote: (well, actually, cpufreq_notifier and kvm_arch_hardware_enable are already non preemptible, and the stats code should just go

Re: [PATCH] KVM: x86: Convert tsc_write_lock to raw_spinlock

2011-02-08 Thread Avi Kivity
On 02/08/2011 11:55 AM, Jan Kiszka wrote: Tracepoints have marginally lower overhead when disabled, and somewhat higher overhead when enabled. A disadvantage of tracepoints is that it is harder to associate an event with a vm when that event is triggered by a workqueue, but I don't

Re: PCI Passthrough, error: The driver 'pci-stub' is occupying your device 0000:08:06.2

2011-02-08 Thread Kenni Lund
2011/2/7 Daniel P. Berrange berra...@redhat.com: On Sat, Feb 05, 2011 at 04:34:01PM +, James Neave wrote: Hi, I'm trying to pass a NOVA-T-500 TV Tuner card through to a gust VM. I'm getting the error The driver 'pci-stub' is occupying your device :08:06.2 This is a rather

Re: networking problems between vms

2011-02-08 Thread Avi Kivity
On 02/07/2011 01:27 PM, Volker Cordes wrote: Avi Kivityaviat redhat.com writes: On 02/07/2011 11:51 AM, Volker Cordes wrote: Hello, I'm running 4 virtual machines on one host, connected by a network bridge on the host (IP 192.168.0.1). All TAP devices get added to that

Re: PCI Passthrough, error: The driver 'pci-stub' is occupying your device 0000:08:06.2

2011-02-08 Thread James Neave
On Tue, Feb 8, 2011 at 9:59 AM, Kenni Lund ke...@kelu.dk wrote: 2011/2/7 Daniel P. Berrange berra...@redhat.com: On Sat, Feb 05, 2011 at 04:34:01PM +, James Neave wrote: Hi, I'm trying to pass a NOVA-T-500 TV Tuner card through to a gust VM. I'm getting the error The driver 'pci-stub' is

[Bug 27052] Module KVM : unable to handle kernel NULL pointer dereference at

2011-02-08 Thread bugzilla-daemon
https://bugzilla.kernel.org/show_bug.cgi?id=27052 --- Comment #20 from prochazka prochazka.nico...@gmail.com 2011-02-08 10:50:13 --- gfn=f001 idx=511 lvl=1 d=0 spte=0 sp-gfns[511] = f001 sp-gfns[511] = f001 sp-gfns[511] = f001 sp-gfns[510] =

[PATCH 04/19] qemu-char: export socket_set_nodelay().

2011-02-08 Thread Yoshiaki Tamura
Signed-off-by: Yoshiaki Tamura tamura.yoshi...@lab.ntt.co.jp --- qemu-char.c |2 +- qemu_socket.h |1 + 2 files changed, 2 insertions(+), 1 deletions(-) diff --git a/qemu-char.c b/qemu-char.c index ee4f4ca..7286aeb 100644 --- a/qemu-char.c +++ b/qemu-char.c @@ -2111,7 +2111,7 @@ static

[PATCH 11/19] ioport: insert event_tap_ioport() to ioport_write().

2011-02-08 Thread Yoshiaki Tamura
Record ioport event to replay it upon failover. Signed-off-by: Yoshiaki Tamura tamura.yoshi...@lab.ntt.co.jp --- ioport.c |2 ++ 1 files changed, 2 insertions(+), 0 deletions(-) diff --git a/ioport.c b/ioport.c index aa4188a..74aebf5 100644 --- a/ioport.c +++ b/ioport.c @@ -27,6 +27,7 @@

[PATCH 19/19] migration: add a parser to accept FT migration incoming mode.

2011-02-08 Thread Yoshiaki Tamura
The option looks like, -incoming kemari:protocol:address:port Signed-off-by: Yoshiaki Tamura tamura.yoshi...@lab.ntt.co.jp --- migration.c |6 ++ 1 files changed, 6 insertions(+), 0 deletions(-) diff --git a/migration.c b/migration.c index 9d2abff..4dc02a2 100644 --- a/migration.c +++

[PATCH 16/19] migration: introduce migrate_ft_trans_{put,get}_ready(), and modify migrate_fd_put_ready() when ft_mode is on.

2011-02-08 Thread Yoshiaki Tamura
Introduce migrate_ft_trans_put_ready() which kicks the FT transaction cycle. When ft_mode is on, migrate_fd_put_ready() would open ft_trans_file and turn on event_tap. To end or cancel FT transaction, ft_mode and event_tap is turned off. migrate_ft_trans_get_ready() is called to receive ack

[PATCH 10/19] Call init handler of event-tap at main() in vl.c.

2011-02-08 Thread Yoshiaki Tamura
Signed-off-by: Yoshiaki Tamura tamura.yoshi...@lab.ntt.co.jp --- vl.c |3 +++ 1 files changed, 3 insertions(+), 0 deletions(-) diff --git a/vl.c b/vl.c index 00155fb..f4d4abf 100644 --- a/vl.c +++ b/vl.c @@ -162,6 +162,7 @@ int main(int argc, char **argv) #include qemu-queue.h #include

[PATCH 15/19] savevm: introduce qemu_savevm_trans_{begin,commit}.

2011-02-08 Thread Yoshiaki Tamura
Introduce qemu_savevm_trans_{begin,commit} to send the memory and device info together, while avoiding cancelling memory state tracking. This patch also abstracts common code between qemu_savevm_state_{begin,iterate,commit}. Signed-off-by: Yoshiaki Tamura tamura.yoshi...@lab.ntt.co.jp ---

[PATCH 02/19] Introduce read() to FdMigrationState.

2011-02-08 Thread Yoshiaki Tamura
Currently FdMigrationState doesn't support read(), and this patch introduces it to get response from the other side. Signed-off-by: Yoshiaki Tamura tamura.yoshi...@lab.ntt.co.jp --- migration-tcp.c | 15 +++ migration.c | 13 + migration.h |3 +++ 3 files

[PATCH 08/19] savevm: introduce util functions to control ft_trans_file from savevm layer.

2011-02-08 Thread Yoshiaki Tamura
To utilize ft_trans_file function, savevm needs interfaces to be exported. Signed-off-by: Yoshiaki Tamura tamura.yoshi...@lab.ntt.co.jp --- hw/hw.h |5 ++ savevm.c | 149 ++ 2 files changed, 154 insertions(+), 0 deletions(-) diff

[PATCH 06/19] virtio: decrement last_avail_idx with inuse before saving.

2011-02-08 Thread Yoshiaki Tamura
For regular migration inuse == 0 always as requests are flushed before save. However, event-tap log when enabled introduces an extra queue for requests which is not being flushed, thus the last inuse requests are left in the event-tap queue. Move the last_avail_idx value sent to the remote back

[PATCH 18/19] Introduce -k option to enable FT migration mode (Kemari).

2011-02-08 Thread Yoshiaki Tamura
When -k option is set to migrate command, it will turn on ft_mode to start FT migration mode (Kemari). Signed-off-by: Yoshiaki Tamura tamura.yoshi...@lab.ntt.co.jp --- hmp-commands.hx |7 --- migration.c |4 qmp-commands.hx |7 --- 3 files changed, 12 insertions(+),

[PATCH 13/19] net: insert event-tap to qemu_send_packet() and qemu_sendv_packet_async().

2011-02-08 Thread Yoshiaki Tamura
event-tap function is called only when it is on. Signed-off-by: Yoshiaki Tamura tamura.yoshi...@lab.ntt.co.jp --- net.c |9 + 1 files changed, 9 insertions(+), 0 deletions(-) diff --git a/net.c b/net.c index 9ba5be2..1176124 100644 --- a/net.c +++ b/net.c @@ -36,6 +36,7 @@ #include

[PATCH 12/19] Insert event_tap_mmio() to cpu_physical_memory_rw() in exec.c.

2011-02-08 Thread Yoshiaki Tamura
Record mmio write event to replay it upon failover. Signed-off-by: Yoshiaki Tamura tamura.yoshi...@lab.ntt.co.jp --- exec.c |4 1 files changed, 4 insertions(+), 0 deletions(-) diff --git a/exec.c b/exec.c index e950df2..c81fd09 100644 --- a/exec.c +++ b/exec.c @@ -33,6 +33,7 @@

[PATCH 07/19] Introduce fault tolerant VM transaction QEMUFile and ft_mode.

2011-02-08 Thread Yoshiaki Tamura
This code implements VM transaction protocol. Like buffered_file, it sits between savevm and migration layer. With this architecture, VM transaction protocol is implemented mostly independent from other existing code. Signed-off-by: Yoshiaki Tamura tamura.yoshi...@lab.ntt.co.jp Signed-off-by:

[PATCH 05/19] vl.c: add deleted flag for deleting the handler.

2011-02-08 Thread Yoshiaki Tamura
Make deleting handlers robust against deletion of any elements in a handler by using a deleted flag like in file descriptors. Signed-off-by: Yoshiaki Tamura tamura.yoshi...@lab.ntt.co.jp --- vl.c | 13 + 1 files changed, 9 insertions(+), 4 deletions(-) diff --git a/vl.c b/vl.c

[PATCH 03/19] Introduce skip_header parameter to qemu_loadvm_state().

2011-02-08 Thread Yoshiaki Tamura
Introduce skip_header parameter to qemu_loadvm_state() so that it can be called iteratively without reading the header. Signed-off-by: Yoshiaki Tamura tamura.yoshi...@lab.ntt.co.jp --- migration.c |2 +- savevm.c| 24 +--- sysemu.h|2 +- 3 files changed, 15

[PATCH 09/19] Introduce event-tap.

2011-02-08 Thread Yoshiaki Tamura
event-tap controls when to start FT transaction, and provides proxy functions to called from net/block devices. While FT transaction, it queues up net/block requests, and flush them when the transaction gets completed. Signed-off-by: Yoshiaki Tamura tamura.yoshi...@lab.ntt.co.jp Signed-off-by:

[PATCH 14/19] block: insert event-tap to bdrv_aio_writev(), bdrv_aio_flush() and bdrv_flush().

2011-02-08 Thread Yoshiaki Tamura
event-tap function is called only when it is on, and requests were sent from device emulators. Signed-off-by: Yoshiaki Tamura tamura.yoshi...@lab.ntt.co.jp --- block.c | 15 +++ 1 files changed, 15 insertions(+), 0 deletions(-) diff --git a/block.c b/block.c index b476479..8ddce13

[PATCH 17/19] migration-tcp: modify tcp_accept_incoming_migration() to handle ft_mode, and add a hack not to close fd when ft_mode is enabled.

2011-02-08 Thread Yoshiaki Tamura
When ft_mode is set in the header, tcp_accept_incoming_migration() sets ft_trans_incoming() as a callback, and call qemu_file_get_notify() to receive FT transaction iteratively. We also need a hack no to close fd before moving to ft_transaction mode, so that we can reuse the fd for it.

[PATCH 01/19] Make QEMUFile buf expandable, and introduce qemu_realloc_buffer() and qemu_clear_buffer().

2011-02-08 Thread Yoshiaki Tamura
Currently buf size is fixed at 32KB. It would be useful if it could be flexible. Signed-off-by: Yoshiaki Tamura tamura.yoshi...@lab.ntt.co.jp --- hw/hw.h |2 ++ savevm.c | 20 +++- 2 files changed, 21 insertions(+), 1 deletions(-) diff --git a/hw/hw.h b/hw/hw.h index

Re: [Qemu-devel] [PATCH 22/23] x86: Fix MCA broadcast parameters for TCG case

2011-02-08 Thread Aurelien Jarno
On Fri, Feb 04, 2011 at 01:47:25PM -0200, Marcelo Tosatti wrote: From: Jan Kiszka jan.kis...@web.de When broadcasting MCEs, we need to set MCIP and RIPV in mcg_status like it is done for KVM. Use the symbolic constants at this chance. Signed-off-by: Jan Kiszka jan.kis...@siemens.com

Re: [Qemu-devel] [PATCH 22/23] x86: Fix MCA broadcast parameters for TCG case

2011-02-08 Thread Jan Kiszka
On 2011-02-08 12:39, Aurelien Jarno wrote: On Fri, Feb 04, 2011 at 01:47:25PM -0200, Marcelo Tosatti wrote: From: Jan Kiszka jan.kis...@web.de When broadcasting MCEs, we need to set MCIP and RIPV in mcg_status like it is done for KVM. Use the symbolic constants at this chance.

[PATCH 1/2] KVM: Convert kvm_lock to raw_spinlock

2011-02-08 Thread Jan Kiszka
Code under this lock requires non-preemptibility. Ensure this also over -rt by converting it to raw spinlock. Signed-off-by: Jan Kiszka jan.kis...@siemens.com --- arch/x86/include/asm/kvm_host.h |2 +- arch/x86/kvm/mmu.c |4 ++-- arch/x86/kvm/x86.c |4 ++--

[PATCH 2/2] KVM: Convert read-only users of vm_list to RCU

2011-02-08 Thread Jan Kiszka
Only for walking the list of VMs, we do not need to hold the preemption disabling kvm_lock. Convert stat services, the cpufreq callback and mmu_shrink to RCU. For the latter, special care is required to synchronize its list_move_tail with kvm_destroy_vm. Signed-off-by: Jan Kiszka

Re: [regression] KVM: hangs and irq timeout booting HURD unless -no-kvm-irqchip passed

2011-02-08 Thread Gleb Natapov
On Mon, Feb 07, 2011 at 07:40:55PM -0600, Jonathan Nieder wrote: Gleb Natapov wrote: Is this patch helps? diff --git a/arch/x86/kvm/i8259.c b/arch/x86/kvm/i8259.c index 3cece05..62b1dde 100644 --- a/arch/x86/kvm/i8259.c +++ b/arch/x86/kvm/i8259.c @@ -549,6 +549,9 @@ static void

Re: [PATCH 15/27] nVMX: Prepare vmcs02 from vmcs01 and vmcs12

2011-02-08 Thread Nadav Har'El
On Sun, Oct 17, 2010, Avi Kivity wrote about Re: [PATCH 15/27] nVMX: Prepare vmcs02 from vmcs01 and vmcs12: On 10/17/2010 12:11 PM, Nadav Har'El wrote: This patch contains code to prepare the VMCS which can be used to actually run the L2 guest, vmcs02. prepare_vmcs02 appropriately merges the

Re: [PATCH 18/19] Introduce -k option to enable FT migration mode (Kemari).

2011-02-08 Thread Paolo Bonzini
On 02/08/2011 12:01 PM, Yoshiaki Tamura wrote: When -k option is set to migrate command, it will turn on ft_mode to start FT migration mode (Kemari). This could also use a kemari: prefix. Paolo -- To unsubscribe from this list: send the line unsubscribe kvm in the body of a message to

[PATCH] kvm: x86: Fix building on IA32 machine

2011-02-08 Thread Arun Thomas
Signed-off-by: Arun Thomas arun.tho...@gmail.com --- target-i386/helper.c |2 +- 1 files changed, 1 insertions(+), 1 deletions(-) diff --git a/target-i386/helper.c b/target-i386/helper.c index 30c5d5e..f439be9 100644 --- a/target-i386/helper.c +++ b/target-i386/helper.c @@ -1150,7 +1150,7

Re: [PATCH 15/27] nVMX: Prepare vmcs02 from vmcs01 and vmcs12

2011-02-08 Thread Avi Kivity
On 02/08/2011 02:13 PM, Nadav Har'El wrote: But while doing this, I came across a question that I wonder if you can clarify for me: Among the other things it sets up, vmx_vcpu_setup() sets rdmsrl(MSR_IA32_SYSENTER_ESP, a); vmcs_writel(HOST_IA32_SYSENTER_ESP, a); /* 22.2.3 */

Re: [PATCH 15/27] nVMX: Prepare vmcs02 from vmcs01 and vmcs12

2011-02-08 Thread Avi Kivity
On 02/08/2011 02:13 PM, Nadav Har'El wrote: But while doing this, I came across a question that I wonder if you can clarify for me: Among the other things it sets up, vmx_vcpu_setup() sets rdmsrl(MSR_IA32_SYSENTER_ESP, a); vmcs_writel(HOST_IA32_SYSENTER_ESP, a); /* 22.2.3 */

Re: [PATCH 15/27] nVMX: Prepare vmcs02 from vmcs01 and vmcs12

2011-02-08 Thread Avi Kivity
On 02/08/2011 02:36 PM, Nadav Har'El wrote: On Tue, Feb 08, 2011, Avi Kivity wrote about Re: [PATCH 15/27] nVMX: Prepare vmcs02 from vmcs01 and vmcs12: So isn't the first setting, in vmx_vcpu_setup(), redundant? It is. It's just historical baggage - these lines were introduced about 20

Re: [PATCH] kvm: x86: Fix building on IA32 machine

2011-02-08 Thread Arun Thomas
I forgot to mention that this patch should fix these buildbot failures: http://buildbot.b1-systems.de/qemu-kvm/builders/disable_kvm_i386_out_of_tree http://buildbot.b1-systems.de/qemu-kvm/builders/disable_kvm_i386_debian_5_0 Best, Arun -- To unsubscribe from this list: send the line unsubscribe

Re: [PATCH 18/19] Introduce -k option to enable FT migration mode (Kemari).

2011-02-08 Thread Yoshiaki Tamura
2011/2/8 Paolo Bonzini pbonz...@redhat.com: On 02/08/2011 12:01 PM, Yoshiaki Tamura wrote: When -k option is set to migrate command, it will turn on ft_mode to start FT migration mode (Kemari). This could also use a kemari: prefix. Sorry, missed that comment. BTW, the help message would be

Re: [PATCH] kvm: x86: Fix building on IA32 machine

2011-02-08 Thread Jan Kiszka
On 2011-02-08 13:40, Arun Thomas wrote: I forgot to mention that this patch should fix these buildbot failures: http://buildbot.b1-systems.de/qemu-kvm/builders/disable_kvm_i386_out_of_tree http://buildbot.b1-systems.de/qemu-kvm/builders/disable_kvm_i386_debian_5_0 See

Migration of WinXP Guest - usb+network failure

2011-02-08 Thread Peter Lieven
Hi, is there any known issue when migrating a WinXP SP3 guest with qemu-kvm 0.13.0 or qemu-kvm-0.12.5? If I migrate such a guest with a Realtek rtl8139 Network Device and an USB Mouse Tablet after migration the USB Tablet doesn't work any more and network stalls. I have seen the mouse moving

Re: [PATCH 2/2] KVM: SVM: check for progress after IRET interception

2011-02-08 Thread Marcelo Tosatti
On Thu, Feb 03, 2011 at 05:07:33PM +0200, Avi Kivity wrote: On 02/03/2011 05:02 PM, Avi Kivity wrote: When we enable an NMI window, we ask for an IRET intercept, since the IRET re-enables NMIs. However, the IRET intercept happens before the instruction executes, while the NMI window

Re: [PATCH 2/2] KVM: SVM: check for progress after IRET interception

2011-02-08 Thread Avi Kivity
On 02/08/2011 03:49 PM, Marcelo Tosatti wrote: Fix by checking for forward progress by recording and comparing the IRET's rip. This is somewhat of a hack, since an unchaging rip does not mean that no forward progress has been made, but is the simplest fix for now. Looks good. So

Re: [regression] KVM: hangs and irq timeout booting HURD unless -no-kvm-irqchip passed

2011-02-08 Thread Marcelo Tosatti
On Tue, Feb 08, 2011 at 02:00:37PM +0200, Gleb Natapov wrote: On Mon, Feb 07, 2011 at 07:40:55PM -0600, Jonathan Nieder wrote: Gleb Natapov wrote: Is this patch helps? diff --git a/arch/x86/kvm/i8259.c b/arch/x86/kvm/i8259.c index 3cece05..62b1dde 100644 ---

Re: [regression] KVM: hangs and irq timeout booting HURD unless -no-kvm-irqchip passed

2011-02-08 Thread Gleb Natapov
On Tue, Feb 08, 2011 at 12:22:53PM -0200, Marcelo Tosatti wrote: On Tue, Feb 08, 2011 at 02:00:37PM +0200, Gleb Natapov wrote: On Mon, Feb 07, 2011 at 07:40:55PM -0600, Jonathan Nieder wrote: Gleb Natapov wrote: Is this patch helps? diff --git a/arch/x86/kvm/i8259.c

Re: [regression] KVM: hangs and irq timeout booting HURD unless -no-kvm-irqchip passed

2011-02-08 Thread Avi Kivity
On 02/08/2011 04:22 PM, Marcelo Tosatti wrote: I don't think the isr_ack logic is overly complex that it should be removed. For some cases it is still beneficial, see example case on commit e48258009d941, which is not handled by kick coalescing of kvm_vcpu_kick. On the other hand, I think it

Re: [regression] KVM: hangs and irq timeout booting HURD unless -no-kvm-irqchip passed

2011-02-08 Thread Gleb Natapov
On Tue, Feb 08, 2011 at 04:43:33PM +0200, Avi Kivity wrote: On 02/08/2011 04:22 PM, Marcelo Tosatti wrote: I don't think the isr_ack logic is overly complex that it should be removed. For some cases it is still beneficial, see example case on commit e48258009d941, which is not handled by kick

Re: [regression] KVM: hangs and irq timeout booting HURD unless -no-kvm-irqchip passed

2011-02-08 Thread Avi Kivity
On 02/08/2011 04:47 PM, Gleb Natapov wrote: On Tue, Feb 08, 2011 at 04:43:33PM +0200, Avi Kivity wrote: On 02/08/2011 04:22 PM, Marcelo Tosatti wrote: I don't think the isr_ack logic is overly complex that it should be removed. For some cases it is still beneficial, see example case on

KVM call minutes for Feb 8

2011-02-08 Thread Chris Wright
Automated builds and testing - found broken 32-bit - luiz suggested running against maintainer trees - daniel gollub offered to take on maintenance - integration with kvm-autotest? - lucas, daniel, stefan... - testing each git commit is probably overkill and too expensive - current autotest

Re: KVM call minutes for Feb 8

2011-02-08 Thread Stefan Hajnoczi
On Tue, Feb 8, 2011 at 3:55 PM, Chris Wright chr...@redhat.com wrote: Automated builds and testing - found broken 32-bit The broken build was found (and fixed?) before automated qemu.git builds. It's a good motivator though. Stefan -- To unsubscribe from this list: send the line unsubscribe

Re: [Qemu-devel] KVM call minutes for Feb 8

2011-02-08 Thread Anthony Liguori
On 02/08/2011 09:55 AM, Chris Wright wrote: Automated builds and testing - found broken 32-bit - luiz suggested running against maintainer trees - daniel gollub offered to take on maintenance - integration with kvm-autotest? - lucas, daniel, stefan... - testing each git commit is probably

Re: [regression] KVM: hangs and irq timeout booting HURD unless -no-kvm-irqchip passed

2011-02-08 Thread Marcelo Tosatti
On Tue, Feb 08, 2011 at 04:57:16PM +0200, Avi Kivity wrote: On 02/08/2011 04:47 PM, Gleb Natapov wrote: On Tue, Feb 08, 2011 at 04:43:33PM +0200, Avi Kivity wrote: On 02/08/2011 04:22 PM, Marcelo Tosatti wrote: I don't think the isr_ack logic is overly complex that it should be removed.

Re: [Qemu-devel] KVM call minutes for Feb 8

2011-02-08 Thread Markus Armbruster
Chris Wright chr...@redhat.com writes: [...] - qdev/vmstate both examples of partially completed work that need more attention As far as qdev's concerned, I can see two kinds of to-dos: * Further develop qdev so that more of the machine init code can becomes qdev declarations. Specific

Re: [Qemu-devel] KVM call minutes for Feb 8

2011-02-08 Thread Peter Maydell
On 8 February 2011 17:13, Markus Armbruster arm...@redhat.com wrote: As far as qdev's concerned, I can see two kinds of to-dos: * Further develop qdev so that more of the machine init code can becomes  qdev declarations.  Specific ideas welcome.  Patches even more, as  always. * Convert the

Re: [Qemu-devel] KVM call minutes for Feb 8

2011-02-08 Thread Aurelien Jarno
On Tue, Feb 08, 2011 at 06:13:53PM +0100, Markus Armbruster wrote: Chris Wright chr...@redhat.com writes: [...] - qdev/vmstate both examples of partially completed work that need more attention As far as qdev's concerned, I can see two kinds of to-dos: * Further develop qdev so

Re: [Qemu-devel] KVM call minutes for Feb 8

2011-02-08 Thread Alexander Graf
On 08.02.2011, at 18:13, Markus Armbruster arm...@redhat.com wrote: Chris Wright chr...@redhat.com writes: [...] - qdev/vmstate both examples of partially completed work that need more attention As far as qdev's concerned, I can see two kinds of to-dos: * Further develop qdev so

[PATCH] kvm: fix detection of BIOS disabling VMX

2011-02-08 Thread Joseph Cihula
This patch fixes the logic used to detect whether BIOS has disabled VMX. Signed-off-by: Joseph Cihula joseph.cih...@intel.com diff -uprN linux-2.6.38-rc3/arch/x86/kvm/vmx.c linux-2.6.38-rc3-patched/arch/x86/kvm/vmx.c --- linux-2.6.38-rc3/arch/x86/kvm/vmx.c 2011-01-31 19:05:49.0 -0800

Re: [regression] KVM: hangs and irq timeout booting HURD unless -no-kvm-irqchip passed

2011-02-08 Thread Jonathan Nieder
Gleb Natapov wrote: Thanks for testing. Can you test this one too please: i8259.c | 25 ++--- x86.c |4 2 files changed, 2 insertions(+), 27 deletions(-) Yes, it works, too. (Tested against v2.6.37.) -- To unsubscribe from this list: send the line

Re: [Qemu-devel] KVM call minutes for Feb 8

2011-02-08 Thread Anthony Liguori
On 02/08/2011 11:13 AM, Markus Armbruster wrote: Chris Wrightchr...@redhat.com writes: [...] - qdev/vmstate both examples of partially completed work that need more attention As far as qdev's concerned, I can see two kinds of to-dos: * Further develop qdev so that more of the

Re: [Qemu-devel] KVM call minutes for Feb 8

2011-02-08 Thread Anthony Liguori
On 02/08/2011 01:02 PM, Peter Maydell wrote: On 8 February 2011 17:13, Markus Armbrusterarm...@redhat.com wrote: As far as qdev's concerned, I can see two kinds of to-dos: * Further develop qdev so that more of the machine init code can becomes qdev declarations. Specific ideas

[PATCH] qemu-kvm: Update kvm_check_many_ioeventfds for qemu-kvm use

2011-02-08 Thread Jan Kiszka
From: Jan Kiszka jan.kis...@siemens.com qemu-kvm does not (yet) have CONFIG_IOTHREAD enabled. Without it, kvm_check_many_ioeventfds will always fail although there is no reason. Signed-off-by: Jan Kiszka jan.kis...@siemens.com --- kvm-all.c |2 +- 1 files changed, 1 insertions(+), 1

Re: [PATCH 3/3][RESEND] Provide control over unmapped pages (v4)

2011-02-08 Thread Andrew Morton
On Tue, 01 Feb 2011 22:25:45 +0530 Balbir Singh bal...@linux.vnet.ibm.com wrote: Changelog v4 1. Add max_unmapped_ratio and use that as the upper limit to check when to shrink the unmapped page cache (Christoph Lameter) Changelog v2 1. Use a config option to enable the code (Andrew

[PATCH] kvm/svm: fix DR interception handling on upcoming AMD CPUs

2011-02-08 Thread Andre Przywara
Somehow the code line advancing the RIP and checking for exceptions got dropped between the post on the ML and the commit. Add it again to let guests boot on upcoming AMD CPUs again. Reported-by: Joerg Roedel joerg.roe...@amd.com Signed-off-by: Andre Przywara andre.przyw...@amd.com ---

Re: [PATCH 07/15] kvm: Separate TCG from KVM cpu execution

2011-02-08 Thread Marcelo Tosatti
On Mon, Feb 07, 2011 at 12:19:18PM +0100, Jan Kiszka wrote: Mixing up TCG bits with KVM already led to problems around eflags emulation on x86. Moreover, quite some code that TCG requires on cpu enty/exit is useless for KVM. So dispatch between tcg_cpu_exec and kvm_cpu_exec as early as

Re: [PATCH 02/15] Refactor cpu_has_work/any_cpu_has_work in cpus.c

2011-02-08 Thread Marcelo Tosatti
On Mon, Feb 07, 2011 at 12:19:13PM +0100, Jan Kiszka wrote: Avoid duplicate use of the function name cpu_has_work, it's confusing. Refactor cpu_has_work to cpu_is_idle and do the same with any_cpu_has_work. Signed-off-by: Jan Kiszka jan.kis...@siemens.com --- cpus.c | 43

Re: [PATCH 04/15] Improve vm_stop reason declarations

2011-02-08 Thread Marcelo Tosatti
On Mon, Feb 07, 2011 at 12:19:15PM +0100, Jan Kiszka wrote: index d6556c9..3397566 100644 --- a/gdbstub.c +++ b/gdbstub.c @@ -2194,14 +2194,14 @@ static void gdb_vm_state_change(void *opaque, int running, int reason) const char *type; int ret; -if (running || (reason !=

Re: Network performance with small packets

2011-02-08 Thread Rusty Russell
On Wed, 2 Feb 2011 03:12:22 pm Michael S. Tsirkin wrote: On Wed, Feb 02, 2011 at 10:09:18AM +0530, Krishna Kumar2 wrote: Michael S. Tsirkin m...@redhat.com 02/02/2011 03:11 AM On Tue, Feb 01, 2011 at 01:28:45PM -0800, Shirley Ma wrote: On Tue, 2011-02-01 at 23:21 +0200, Michael S.

Re: Network performance with small packets

2011-02-08 Thread Michael S. Tsirkin
On Wed, Feb 09, 2011 at 11:07:20AM +1030, Rusty Russell wrote: On Wed, 2 Feb 2011 03:12:22 pm Michael S. Tsirkin wrote: On Wed, Feb 02, 2011 at 10:09:18AM +0530, Krishna Kumar2 wrote: Michael S. Tsirkin m...@redhat.com 02/02/2011 03:11 AM On Tue, Feb 01, 2011 at 01:28:45PM -0800,

Re: Network performance with small packets

2011-02-08 Thread Rusty Russell
On Wed, 9 Feb 2011 11:23:45 am Michael S. Tsirkin wrote: On Wed, Feb 09, 2011 at 11:07:20AM +1030, Rusty Russell wrote: On Wed, 2 Feb 2011 03:12:22 pm Michael S. Tsirkin wrote: On Wed, Feb 02, 2011 at 10:09:18AM +0530, Krishna Kumar2 wrote: Michael S. Tsirkin m...@redhat.com 02/02/2011

Re: Network performance with small packets

2011-02-08 Thread Michael S. Tsirkin
On Wed, Feb 09, 2011 at 12:09:35PM +1030, Rusty Russell wrote: On Wed, 9 Feb 2011 11:23:45 am Michael S. Tsirkin wrote: On Wed, Feb 09, 2011 at 11:07:20AM +1030, Rusty Russell wrote: On Wed, 2 Feb 2011 03:12:22 pm Michael S. Tsirkin wrote: On Wed, Feb 02, 2011 at 10:09:18AM +0530,

[PATCH uq/master -v2 1/2] Add qemu_ram_remap

2011-02-08 Thread Huang Ying
qemu_ram_remap() unmaps the specified RAM pages, then re-maps these pages again. This is used by KVM HWPoison support to clear HWPoisoned page tables across guest rebooting, so that a new page may be allocated later to recover the memory error. Signed-off-by: Huang Ying ying.hu...@intel.com ---

[PATCH uq/master -v2 2/2] KVM, MCE, unpoison memory address across reboot

2011-02-08 Thread Huang Ying
In Linux kernel HWPoison processing implementation, the virtual address in processes mapping the error physical memory page is marked as HWPoison. So that, the further accessing to the virtual address will kill corresponding processes with SIGBUS. If the error physical memory page is used by a

Re: [PATCH V6 1/4] nmi: convert cpu_index to cpu-index

2011-02-08 Thread Lai Jiangshan
On 02/01/2011 09:29 PM, Luiz Capitulino wrote: On Thu, 27 Jan 2011 16:20:27 +0800 Lai Jiangshan la...@cn.fujitsu.com wrote: cpu-index which uses hyphen is better name. Signed-off-by: Lai Jiangshan la...@cn.fujitsu.com It looks ok from a quick pass, but I can't apply it on current

Re: Network performance with small packets

2011-02-08 Thread Stefan Hajnoczi
On Wed, Feb 9, 2011 at 1:55 AM, Michael S. Tsirkin m...@redhat.com wrote: On Wed, Feb 09, 2011 at 12:09:35PM +1030, Rusty Russell wrote: On Wed, 9 Feb 2011 11:23:45 am Michael S. Tsirkin wrote: On Wed, Feb 09, 2011 at 11:07:20AM +1030, Rusty Russell wrote: On Wed, 2 Feb 2011 03:12:22 pm

Re: [PATCH 07/15] kvm: Separate TCG from KVM cpu execution

2011-02-08 Thread Jan Kiszka
On 2011-02-09 00:39, Marcelo Tosatti wrote: On Mon, Feb 07, 2011 at 12:19:18PM +0100, Jan Kiszka wrote: Mixing up TCG bits with KVM already led to problems around eflags emulation on x86. Moreover, quite some code that TCG requires on cpu enty/exit is useless for KVM. So dispatch between

Re: RFC: New API for PPC for vcpu mmu access

2011-02-08 Thread Avi Kivity
On 02/07/2011 07:30 PM, Yoder Stuart-B08248 wrote: -Original Message- From: kvm-ppc-ow...@vger.kernel.org [mailto:kvm-ppc-ow...@vger.kernel.org] On Behalf Of Avi Kivity Sent: Monday, February 07, 2011 11:14 AM To: Alexander Graf Cc: Wood Scott-B07421; Yoder Stuart-B08248;