[PATCH 0/12] Get rid of kernel memory allocation

check. But for now, I think this is enough. series stat: Glauber Costa (12): start removing kernel memory functions remove tests of user memory from vl.c coalesce memory allocation remove KVM_CAP_USER_MEMORY reference from qemu-kvm.c remove KVM_CAP_USER_MEMORY from

[PATCH] remove tests of user memory from vl.c

From: Glauber Costa [EMAIL PROTECTED] kvm_qemu_create_context() will fail if we don't have userspace memory capability, so no need to check it here. Signed-off-by: Glauber Costa [EMAIL PROTECTED] --- qemu/vl.c | 18 +- 1 files changed, 5 insertions(+), 13 deletions(-) diff

[PATCH] coalesce memory allocation

From: Glauber Costa [EMAIL PROTECTED] Put all memory allocation under qemu_alloc_physram(). Protect the mem_area code with #ifdef USE_KVM. Although it is not really needed, it help us keep track of the fact that this is a kvm-only feature to the moment. Signed-off-by: Glauber Costa [EMAIL

[PATCH] remove KVM_CAP_USER_MEMORY reference from qemu-kvm.c

From: Glauber Costa [EMAIL PROTECTED] kvm_cpu_register_physical_memory() is its only user. Remove it. Signed-off-by: Glauber Costa [EMAIL PROTECTED] --- qemu/qemu-kvm.c | 52 +--- 1 files changed, 21 insertions(+), 31 deletions(-) diff --git

[PATCH] remove kernel memory code from hw/pc.c

From: Glauber Costa [EMAIL PROTECTED] Signed-off-by: Glauber Costa [EMAIL PROTECTED] --- qemu/hw/pc.c | 19 ++- 1 files changed, 2 insertions(+), 17 deletions(-) diff --git a/qemu/hw/pc.c b/qemu/hw/pc.c index 026e96d..8a50096 100644 --- a/qemu/hw/pc.c +++ b/qemu/hw/pc.c

Re: [PATCH] Fix up pxe boot

On Tue, Sep 02, 2008 at 06:20:55PM +0300, Avi Kivity wrote: Glauber Costa wrote: On Tue, Sep 2, 2008 at 5:39 AM, Avi Kivity [EMAIL PROTECTED] wrote: Glauber Costa wrote: diff --git a/target-i386/op_helper.c b/target-i386/op_helper.c index 0b5fdc0..433aa3f 100644 --- a/target-i386

Re: [PATCH] Fix up pxe boot

On Tue, Sep 2, 2008 at 5:39 AM, Avi Kivity [EMAIL PROTECTED] wrote: Glauber Costa wrote: diff --git a/target-i386/op_helper.c b/target-i386/op_helper.c index 0b5fdc0..433aa3f 100644 --- a/target-i386/op_helper.c +++ b/target-i386/op_helper.c @@ -600,7 +600,7 @@ do {\ #define PUSHL(ssp, sp

Re: [PATCH] Fix up pxe boot

On Tue, Sep 2, 2008 at 12:20 PM, Avi Kivity [EMAIL PROTECTED] wrote: Glauber Costa wrote: On Tue, Sep 2, 2008 at 5:39 AM, Avi Kivity [EMAIL PROTECTED] wrote: Glauber Costa wrote: diff --git a/target-i386/op_helper.c b/target-i386/op_helper.c index 0b5fdc0..433aa3f 100644 --- a/target

Re: serial/ rs232 and qemu /kvm?

-serial option Marc Weber -- To unsubscribe from this list: send the line unsubscribe kvm in the body of a message to [EMAIL PROTECTED] More majordomo info at http://vger.kernel.org/majordomo-info.html -- Glauber Costa. Free as in Freedom http://glommer.net The less confident you

Re: Un-googlable

-- To unsubscribe from this list: send the line unsubscribe kvm in the body of a message to [EMAIL PROTECTED] More majordomo info at http://vger.kernel.org/majordomo-info.html -- Glauber Costa. Free as in Freedom http://glommer.net The less confident you are, the more serious you have to act

Re: [PATCH 5/9] substitute is_allocated_mem with more general is_containing_region

On Wed, Aug 13, 2008 at 8:41 AM, Avi Kivity [EMAIL PROTECTED] wrote: Glauber Costa wrote: Signed-off-by: Glauber Costa [EMAIL PROTECTED] --- libkvm/libkvm.c | 34 +- libkvm/libkvm.h |2 +- qemu/qemu-kvm.c |2 +- 3 files changed, 23 insertions

[RFC 0/9] Memory registration rework

Hi folks, The following series contain a proposal for our memory registration framework. This is by no means complete, and rather, a first step only. This first step, btw, has the goal of taking the kvm-specific memory registration functions from all over the code, so we can make the merging

[PATCH 8/9] coalesce mmio regions without an explicit call

commit. Signed-off-by: Glauber Costa [EMAIL PROTECTED] --- libkvm/libkvm.c |4 qemu/qemu-kvm.c | 13 +++-- 2 files changed, 7 insertions(+), 10 deletions(-) diff --git a/libkvm/libkvm.c b/libkvm/libkvm.c index d62cb2a..c563bb6 100644 --- a/libkvm/libkvm.c +++ b/libkvm/libkvm.c

[PATCH 3/9] do not use mem_hole anymore.

Signed-off-by: Glauber Costa [EMAIL PROTECTED] --- libkvm/libkvm.c | 78 --- qemu/qemu-kvm.c | 11 --- 2 files changed, 6 insertions(+), 83 deletions(-) diff --git a/libkvm/libkvm.c b/libkvm/libkvm.c index 0bacb43..eb85445 100644

[PATCH 9/9] remove explicit calls to kvm_qemu_register_coalesced_mmio

It is now done automatically for IO regions inside kvm_cpu_register_physical_memory(). --- qemu/hw/cirrus_vga.c |2 -- qemu/hw/e1000.c | 12 qemu/hw/pci.c|3 --- qemu/hw/vga.c|4 4 files changed, 0 insertions(+), 21 deletions(-) diff --git

[PATCH 6/9] move kvm_cpu_register_memory_area into qemu's

are mmio regions too. This is to be bisection friendly. Direct calls are to be removed in a later commit. Signed-off-by: Glauber Costa [EMAIL PROTECTED] --- libkvm/libkvm.c | 84 -- libkvm/libkvm.h |6 qemu/exec.c |3 ++ qemu/qemu

[PATCH 5/9] substitute is_allocated_mem with more general is_containing_region

Signed-off-by: Glauber Costa [EMAIL PROTECTED] --- libkvm/libkvm.c | 34 +- libkvm/libkvm.h |2 +- qemu/qemu-kvm.c |2 +- 3 files changed, 23 insertions(+), 15 deletions(-) diff --git a/libkvm/libkvm.c b/libkvm/libkvm.c index 33f00b7..c885dee 100644

Re: Merging adjacent memory slots

a linear search, for ex), instead of doing this slot merging. Anyway, you, or someone else, may have a different argument on this, but that's my position so far. -- Glauber Costa. Free as in Freedom http://glommer.net The less confident you are, the more serious you have to act

Re: Merging adjacent memory slots

!!! -- Glauber Costa. Free as in Freedom http://glommer.net The less confident you are, the more serious you have to act. -- To unsubscribe from this list: send the line unsubscribe kvm in the body of a message to [EMAIL PROTECTED] More majordomo info at http://vger.kernel.org/majordomo-info.html

[PATCH 1/2] factor out cpu_khz to common code

KVM intends to use paravirt code to calibrate khz. Xen current code will do just fine. So as a first step, factor out code to pvclock.c. Signed-off-by: Glauber Costa [EMAIL PROTECTED] --- arch/x86/kernel/pvclock.c | 12 arch/x86/xen/time.c | 11 ++- include/asm-x86

Re: [PATCH 0/2] Paravirt loops per jiffy calculation

of a message to [EMAIL PROTECTED] More majordomo info at http://vger.kernel.org/majordomo-info.html -- Glauber Costa. Free as in Freedom http://glommer.net The less confident you are, the more serious you have to act. -- To unsubscribe from this list: send the line unsubscribe kvm in the body

[PATCH 1/2] factor out cpu_khz to common code

KVM intends to use paravirt code to calibrate khz. Xen current code will do just fine. So as a first step, factor out code to pvclock.c. Signed-off-by: Glauber Costa [EMAIL PROTECTED] --- arch/x86/kernel/pvclock.c | 12 arch/x86/xen/time.c | 11 ++- include/asm-x86

[PATCH 2/2] use paravirt function to calculate cpu khz

host to guest, and have it preset. Signed-off-by: Glauber Costa [EMAIL PROTECTED] --- arch/x86/kernel/kvmclock.c | 30 ++ 1 files changed, 30 insertions(+), 0 deletions(-) diff --git a/arch/x86/kernel/kvmclock.c b/arch/x86/kernel/kvmclock.c index d02def0..84db1bb

Re: KVM status and dynamic mem/cpu question.

majordomo info at http://vger.kernel.org/majordomo-info.html -- Glauber Costa. Free as in Freedom http://glommer.net The less confident you are, the more serious you have to act. -- To unsubscribe from this list: send the line unsubscribe kvm in the body of a message to [EMAIL PROTECTED] More

Re: [PATCH] mask out clflush

clflush()ing mmio addresses? Strange as these are not normally cached. It seems so, Glauber mentioned that the address was an MMIO address. yes. It is address 0xc8821000, apparently part of a pci controller initialization. Regards, Anthony Liguori -- Glauber Costa. Free as in Freedom

Re: [PATCH] mask out clflush

a virtual address, not a physical address IIUC. Ah, of course. How's that one ? -- error compiling committee.c: too many arguments to function -- Glauber Costa. Free as in Freedom http://glommer.net The less confident you are, the more serious you have to act. 0001-properly-decode

[PATCH] mask out clflush

clflush is a non-privileged instruction that flushes the cacheline given by its parameter, in terms of linear address. As it is non-privileged, it is quite tricky, because a guest doing clflush will actually be trying to flush a host kernel address. Signed-off-by: Glauber Costa [EMAIL PROTECTED

Re: [PATCH] mask out clflush

On Tue, Jul 8, 2008 at 4:34 PM, Anthony Liguori [EMAIL PROTECTED] wrote: Glauber Costa wrote: clflush is a non-privileged instruction that flushes the cacheline given by its parameter, in terms of linear address. As it is non-privileged, it is quite tricky, because a guest doing clflush

Re: [Qemu-devel] Re: QEMU Accel - git tree

On Mon, Jul 7, 2008 at 2:12 PM, Jamie Lokier [EMAIL PROTECTED] wrote: Gerd Hoffmann wrote: Glauber Costa wrote: Talking to hollis last week, he pointed out that a better solution would be to make all of them to use the same kernel interface. It's a valid approach, but I believe an user

Re: kvm guest loops_per_jiffy miscalibration under host load

On Sun, Jul 6, 2008 at 10:56 PM, Anthony Liguori [EMAIL PROTECTED] wrote: Glauber Costa wrote: Marcelo Tosatti wrote: Hello, I have been discussing with Glauber and Gerd the problem where KVM guests miscalibrate loops_per_jiffy if there's sufficient load on the host

Re: kvm guest loops_per_jiffy miscalibration under host load

Glauber Costa wrote: On Mon, Jul 7, 2008 at 4:21 PM, Anthony Liguori [EMAIL PROTECTED] wrote: Marcelo Tosatti wrote: On Mon, Jul 07, 2008 at 03:27:16PM -0300, Glauber Costa wrote: I agree. A paravirt solution solves the problem. Please, look at the patch I've attached. It does __delay

QEMU Accel - git tree

you enjoy. -- Glauber Costa. Free as in Freedom http://glommer.net The less confident you are, the more serious you have to act. -- To unsubscribe from this list: send the line unsubscribe kvm in the body of a message to [EMAIL PROTECTED] More majordomo info at http://vger.kernel.org/majordomo

Re: QEMU Accel - git tree

On Fri, Jul 4, 2008 at 6:38 PM, Carlo Marcelo Arenas Belon [EMAIL PROTECTED] wrote: On Fri, Jul 04, 2008 at 05:40:03PM -0300, Glauber Costa wrote: Hey Folks I'm publishing a git repository for the QEMU Accel patches. It lives at git://git.kernel.org/pub/scm/virt/qemu/glommer/qemu-accel.git

Re: [Bug] Real mode guests never wake up after an HLT instruction

it also happens with -no-kvm-irqchip, that does not exercise this path. I looked into it today, will look further, but my current impression is that hlt is working fine. It might be the case that the problem lies in any kind of interrupt injection that were supposed to wake it up. -- Glauber Costa

[PATCH 01/20] split kqemu_init into two

we separate kqemu_init() into a part that depends on env, and other that does not. The later can be initialized earlier --- exec.c |5 - kqemu.c | 10 +++--- target-i386/helper.c |2 +- 3 files changed, 12 insertions(+), 5 deletions(-) diff --git

[PATCH 02/20] introduce QEMUAccel and fill it with interrupt specific driver

This patch introduces QEMUAccel, a placeholder for function pointers that aims at helping qemu to abstract accelerators such as kqemu and kvm (actually, the 'accelerator' name was proposed by avi kivity, since he loves referring to kvm that way). To begin with, the accelerator is given the

[PATCH 10/20] wrap modify_page through accel calls

--- accel.h|8 exec-all.h |2 +- exec.c | 27 --- kqemu.c| 26 +++--- 4 files changed, 36 insertions(+), 27 deletions(-) diff --git a/accel.h b/accel.h index 7cef043..5395e05 100644 --- a/accel.h +++ b/accel.h @@ -6,6

[PATCH 18/20] accel_trace_io

kqemu keeps trace of the last io done. Do it through an accel_wrapper. Signed-off-by: Glauber Costa [EMAIL PROTECTED] --- accel.h | 16 +++- kqemu.c | 18 ++ vl.c| 30 ++ 3 files changed, 39 insertions(+), 25 deletions(-) diff

[PATCH 19/20] get_env accel wrapper

Allow the current accelerator to provide it's own, customized address of the CPUState structure used for the env variable. Signed-off-by: Glauber Costa [EMAIL PROTECTED] --- accel.h |8 target-i386/helper.c |2 +- 2 files changed, 9 insertions(+), 1 deletions

[PATCH 15/20] shift for masks.

kvm uses, but pretty general --- target-i386/cpu.h |7 +-- 1 files changed, 5 insertions(+), 2 deletions(-) diff --git a/target-i386/cpu.h b/target-i386/cpu.h index d32f9ee..5798cf3 100644 --- a/target-i386/cpu.h +++ b/target-i386/cpu.h @@ -178,8 +178,11 @@ #define HF2_NMI_MASK

[PATCH 12/20] build list of available accelerators

instead of hardcoding kqemu_start() in exec.c, which would require such a hack for all available accelerators, semantics of register_qemu_accel() is changed a little bit. It only builds a list of available accelerators. The last one registered is the first tried. This is a temporary solution,

[PATCH 08/20] move disabling code to kqemu.c instead of vl.c

this is for the case in which we run more than one cpu --- kqemu.c |3 ++- vl.c|4 2 files changed, 2 insertions(+), 5 deletions(-) diff --git a/kqemu.c b/kqemu.c index bcbe3cc..bfb7339 100644 --- a/kqemu.c +++ b/kqemu.c @@ -156,6 +156,7 @@ static void

[PATCH 04/20] wrap cache flushing functions into accel drivers

--- accel.h | 13 + exec.c | 15 +++ kqemu.c | 15 +-- 3 files changed, 29 insertions(+), 14 deletions(-) diff --git a/accel.h b/accel.h index da66cd5..8344b45 100644 --- a/accel.h +++ b/accel.h @@ -1,6 +1,8 @@ typedef struct QEMUAccel { void

[PATCH 13/20] decorate application name

Signed-off-by: Glauber Costa [EMAIL PROTECTED] --- accel.h | 10 ++ sdl.c |7 +-- vl.c| 12 vnc.c |6 -- 4 files changed, 31 insertions(+), 4 deletions(-) diff --git a/accel.h b/accel.h index 99f4742..9755a9b 100644 --- a/accel.h +++ b/accel.h

[PATCH 09/20] set_notdirty goes through accel wrapper

--- accel.h|7 +++ exec-all.h |2 +- exec.c | 18 +++--- kqemu.c| 23 +++ 4 files changed, 26 insertions(+), 24 deletions(-) diff --git a/accel.h b/accel.h index f80b15f..7cef043 100644 --- a/accel.h +++ b/accel.h @@ -5,6 +5,7 @@

[PATCH 05/20] turn info kqemu into generic info accelerator

--- accel.h |8 kqemu.c | 24 monitor.c | 36 +--- 3 files changed, 45 insertions(+), 23 deletions(-) diff --git a/accel.h b/accel.h index 8344b45..f6d53ea 100644 --- a/accel.h +++ b/accel.h @@ -3,6 +3,7 @@ typedef

[PATCH 03/20] init env made accel driver

--- accel.h |8 kqemu.c |1 + target-i386/helper.c |6 +++--- 3 files changed, 12 insertions(+), 3 deletions(-) diff --git a/accel.h b/accel.h index e16ca69..da66cd5 100644 --- a/accel.h +++ b/accel.h @@ -1,5 +1,6 @@ typedef struct QEMUAccel {

Re: [PATCH] Do not calculate linear rip in emulation failure report

On Fri, Jun 13, 2008 at 11:14 PM, Avi Kivity [EMAIL PROTECTED] wrote: Glauber Costa wrote: I've changed it to use printk_ratelimit(). I've tested this option here before sending out the patch, since it would address all issues. But in error cases, it still seemed to generate too many

[PATCH 1/5] remove REGWPTR

Blue Swirl points out that it is not used anymore. Signed-off-by: Glauber Costa [EMAIL PROTECTED] --- cpu-exec.c | 13 - target-sparc/exec.h |4 2 files changed, 0 insertions(+), 17 deletions(-) diff --git a/cpu-exec.c b/cpu-exec.c index b660954..ea0e5b1 100644

[PATCH 3/5] cpu_info_ip

This commit makes the code for info_cpus simpler in monitor.c Signed-off-by: Glauber Costa [EMAIL PROTECTED] --- exec-all.h|2 ++ monitor.c | 17 + target-alpha/helper.c |2 ++ target-arm/helper.c |2 ++ target-cris/helper.c |2

[PATCH 2/5] simplify cpu_exec

This is a first attempt to simplify cpu_exec(): it has simply too many ifdefs, which is not a very good practice at all. Following some work I've already posted in the past, I'm moving the target-b ifdefs to target-xxx/helper.c, encapsuled into relevant functions. Signed-off-by: Glauber Costa

<    5   6   7   8   9   10