[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH 4/7 v2] KVM regsync: Add register bitmap paramet
From: |
Blue Swirl |
Subject: |
Re: [Qemu-devel] [PATCH 4/7 v2] KVM regsync: Add register bitmap parameter to do_kvm_cpu_synchronize_state |
Date: |
Sat, 12 Jan 2013 11:08:14 +0000 |
On Thu, Jan 10, 2013 at 3:29 PM, Jason J. Herne <address@hidden> wrote:
> From: "Jason J. Herne" <address@hidden>
>
> do_kvm_cpu_synchronize_state is called via run_on_cpu, so we can only pass
> a single argument. Create SyncStateArgs struct for this purpose and add
> register bitmap data member to it.
>
> Signed-off-by: Jason J. Herne <address@hidden>
> Reviewed-by: Christian Borntraeger <address@hidden>
> ---
> include/sysemu/kvm.h | 6 ++++++
> kvm-all.c | 27 +++++++++++++++++----------
> 2 files changed, 23 insertions(+), 10 deletions(-)
>
> diff --git a/include/sysemu/kvm.h b/include/sysemu/kvm.h
> index e0738ba..193d1f4 100644
> --- a/include/sysemu/kvm.h
> +++ b/include/sysemu/kvm.h
> @@ -223,6 +223,12 @@ int kvm_check_extension(KVMState *s, unsigned int
> extension);
>
> uint32_t kvm_arch_get_supported_cpuid(KVMState *env, uint32_t function,
> uint32_t index, int reg);
> +
> +struct kvm_cpu_syncstate_args {
KVMCPUSyncStateArgs, don't forget the typedef.
> + CPUState *cpu;
> + int regmap;
> +};
> +
> void kvm_cpu_synchronize_state(CPUArchState *env);
> void kvm_cpu_synchronize_post_reset(CPUArchState *env);
> void kvm_cpu_synchronize_post_init(CPUArchState *env);
> diff --git a/kvm-all.c b/kvm-all.c
> index 1aa61bb..77ab72a 100644
> --- a/kvm-all.c
> +++ b/kvm-all.c
> @@ -231,7 +231,7 @@ int kvm_init_vcpu(CPUArchState *env)
>
> cpu->kvm_fd = ret;
> cpu->kvm_state = s;
> - cpu->kvm_vcpu_dirty = true;
> + cpu->kvm_vcpu_dirty = KVM_REGSYNC_FULL_STATE;
>
> mmap_size = kvm_ioctl(s, KVM_GET_VCPU_MMAP_SIZE, 0);
> if (mmap_size < 0) {
> @@ -1491,20 +1491,27 @@ void kvm_flush_coalesced_mmio_buffer(void)
>
> static void do_kvm_cpu_synchronize_state(void *arg)
> {
> - CPUState *cpu = arg;
> + struct kvm_cpu_syncstate_args *args = arg;
>
> - if (!cpu->kvm_vcpu_dirty) {
> - kvm_arch_get_registers(cpu, KVM_REGSYNC_FULL_STATE);
> - cpu->kvm_vcpu_dirty = true;
> + /* Do not sync regs that are already dirty */
> + int regs_to_get = args->regmap & ~args->cpu->kvm_vcpu_dirty;
> +
> + if (regs_to_get) {
> + kvm_arch_get_registers(args->cpu, regs_to_get);
> + args->cpu->kvm_vcpu_dirty |= regs_to_get;
> }
> }
>
> void kvm_cpu_synchronize_state(CPUArchState *env)
> {
> CPUState *cpu = ENV_GET_CPU(env);
> + struct kvm_cpu_syncstate_args args;
> +
> + args.cpu = cpu;
> + args.regmap = KVM_REGSYNC_FULL_STATE;
>
> - if (!cpu->kvm_vcpu_dirty) {
> - run_on_cpu(cpu, do_kvm_cpu_synchronize_state, cpu);
> + if (args.regmap & ~cpu->kvm_vcpu_dirty) {
> + run_on_cpu(cpu, do_kvm_cpu_synchronize_state, &args);
> }
> }
>
> @@ -1513,7 +1520,7 @@ void kvm_cpu_synchronize_post_reset(CPUArchState *env)
> CPUState *cpu = ENV_GET_CPU(env);
>
> kvm_arch_put_registers(cpu, KVM_REGSYNC_RESET_STATE);
> - cpu->kvm_vcpu_dirty = false;
> + cpu->kvm_vcpu_dirty &= ~KVM_REGSYNC_RESET_STATE;
> }
>
> void kvm_cpu_synchronize_post_init(CPUArchState *env)
> @@ -1521,7 +1528,7 @@ void kvm_cpu_synchronize_post_init(CPUArchState *env)
> CPUState *cpu = ENV_GET_CPU(env);
>
> kvm_arch_put_registers(cpu, KVM_REGSYNC_FULL_STATE);
> - cpu->kvm_vcpu_dirty = false;
> + cpu->kvm_vcpu_dirty &= ~KVM_REGSYNC_FULL_STATE;
> }
>
> int kvm_cpu_exec(CPUArchState *env)
> @@ -1540,7 +1547,7 @@ int kvm_cpu_exec(CPUArchState *env)
> do {
> if (cpu->kvm_vcpu_dirty) {
> kvm_arch_put_registers(cpu, KVM_REGSYNC_RUNTIME_STATE);
> - cpu->kvm_vcpu_dirty = false;
> + cpu->kvm_vcpu_dirty &= ~KVM_REGSYNC_RUNTIME_STATE;
> }
>
> kvm_arch_pre_run(cpu, run);
> --
> 1.7.9.5
>
>
- [Qemu-devel] [PATCH 4/7 v2] KVM regsync: Add register bitmap parameter to do_kvm_cpu_synchronize_state, Jason J. Herne, 2013/01/10
- Re: [Qemu-devel] [PATCH 4/7 v2] KVM regsync: Add register bitmap parameter to do_kvm_cpu_synchronize_state,
Blue Swirl <=
- Re: [Qemu-devel] [PATCH 4/7 v2] KVM regsync: Add register bitmap parameter to do_kvm_cpu_synchronize_state, Bhushan Bharat-R65777, 2013/01/14
- Re: [Qemu-devel] [PATCH 4/7 v2] KVM regsync: Add register bitmap parameter to do_kvm_cpu_synchronize_state, Marcelo Tosatti, 2013/01/16
- Re: [Qemu-devel] [PATCH 4/7 v2] KVM regsync: Add register bitmap parameter to do_kvm_cpu_synchronize_state, Bhushan Bharat-R65777, 2013/01/16
- Re: [Qemu-devel] [PATCH 4/7 v2] KVM regsync: Add register bitmap parameter to do_kvm_cpu_synchronize_state, Marcelo Tosatti, 2013/01/16
- Re: [Qemu-devel] [PATCH 4/7 v2] KVM regsync: Add register bitmap parameter to do_kvm_cpu_synchronize_state, Alexander Graf, 2013/01/24
- Re: [Qemu-devel] [PATCH 4/7 v2] KVM regsync: Add register bitmap parameter to do_kvm_cpu_synchronize_state, Scott Wood, 2013/01/24
- Re: [Qemu-devel] [PATCH 4/7 v2] KVM regsync: Add register bitmap parameter to do_kvm_cpu_synchronize_state, Peter Maydell, 2013/01/24
- Re: [Qemu-devel] [PATCH 4/7 v2] KVM regsync: Add register bitmap parameter to do_kvm_cpu_synchronize_state, Alexander Graf, 2013/01/24
- Re: [Qemu-devel] [PATCH 4/7 v2] KVM regsync: Add register bitmap parameter to do_kvm_cpu_synchronize_state, Scott Wood, 2013/01/24
- Re: [Qemu-devel] [PATCH 4/7 v2] KVM regsync: Add register bitmap parameter to do_kvm_cpu_synchronize_state, Alexander Graf, 2013/01/24