Re: [PATCH] Rename KVM_UPSTREAM to OBSOLETE_KVM_IMPL

[Jan Kiszka <jan.kiszka@web.de>]

[-- Attachment #1: Type: text/plain, Size: 6065 bytes --]

Am 13.09.2010 19:54, Avi Kivity wrote:
> The symbol KVM_UPSTREAM is used to mark sections of code that are part of
> the upstream kvm implemetation that is not used in qemu-kvm.  However the
> name becomes ambiguous if qemu-kvm is merged upstream.

I doubt this is describing all cases correctly as well. Some changes
should rather happen the other way around (e.g. you surely don't want to
obsolete x86 kvm_arch_put/get_registers in favor of
kvm_arch_load/save_regs, do you?).

Jan

> 
> Rename the symbol to avoid confusion.
> 
> Signed-off-by: Avi Kivity <avi@redhat.com>
> ---
>  cpus.c            |    2 +-
>  kvm-all.c         |   16 ++++++++--------
>  kvm.h             |    6 +++---
>  target-i386/kvm.c |   10 +++++-----
>  vl.c              |    4 ++--
>  5 files changed, 19 insertions(+), 19 deletions(-)
> 
> diff --git a/cpus.c b/cpus.c
> index c545a62..99c04d1 100644
> --- a/cpus.c
> +++ b/cpus.c
> @@ -299,7 +299,7 @@ void qemu_notify_event(void)
>      }
>  }
>  
> -#if defined(KVM_UPSTREAM) || !defined(CONFIG_KVM)
> +#if defined(OBSOLETE_KVM_IMPL) || !defined(CONFIG_KVM)
>  void qemu_mutex_lock_iothread(void) {}
>  void qemu_mutex_unlock_iothread(void) {}
>  #endif
> diff --git a/kvm-all.c b/kvm-all.c
> index 4ff75c4..d4b0861 100644
> --- a/kvm-all.c
> +++ b/kvm-all.c
> @@ -41,7 +41,7 @@
>      do { } while (0)
>  #endif
>  
> -#ifdef KVM_UPSTREAM
> +#ifdef OBSOLETE_KVM_IMPL
>  
>  typedef struct KVMSlot
>  {
> @@ -156,7 +156,7 @@ static int kvm_set_user_memory_region(KVMState *s, KVMSlot *slot)
>      return kvm_vm_ioctl(s, KVM_SET_USER_MEMORY_REGION, &mem);
>  }
>  
> -#ifdef KVM_UPSTREAM
> +#ifdef OBSOLETE_KVM_IMPL
>  static void kvm_reset_vcpu(void *opaque)
>  {
>      CPUState *env = opaque;
> @@ -176,7 +176,7 @@ int kvm_pit_in_kernel(void)
>  }
>  
>  
> -#ifdef KVM_UPSTREAM
> +#ifdef OBSOLETE_KVM_IMPL
>  int kvm_init_vcpu(CPUState *env)
>  {
>      KVMState *s = kvm_state;
> @@ -594,7 +594,7 @@ void kvm_cpu_register_phys_memory_client(void)
>      cpu_register_phys_memory_client(&kvm_cpu_phys_memory_client);
>  }
>  
> -#ifdef KVM_UPSTREAM
> +#ifdef OBSOLETE_KVM_IMPL
>  
>  int kvm_init(int smp_cpus)
>  {
> @@ -816,7 +816,7 @@ void kvm_flush_coalesced_mmio_buffer(void)
>  #endif
>  }
>  
> -#ifdef KVM_UPSTREAM
> +#ifdef OBSOLETE_KVM_IMPL
>  
>  static void do_kvm_cpu_synchronize_state(void *_env)
>  {
> @@ -1038,7 +1038,7 @@ int kvm_has_debugregs(void)
>      return kvm_state->debugregs;
>  }
>  
> -#ifdef KVM_UPSTREAM
> +#ifdef OBSOLETE_KVM_IMPL
>  int kvm_has_xsave(void)
>  {
>      return kvm_state->xsave;
> @@ -1069,10 +1069,10 @@ void kvm_setup_guest_memory(void *start, size_t size)
>  }
>  
>  #ifdef KVM_CAP_SET_GUEST_DEBUG
> -#ifndef KVM_UPSTREAM
> +#ifndef OBSOLETE_KVM_IMPL
>  #define run_on_cpu on_vcpu
>  static void on_vcpu(CPUState *env, void (*func)(void *data), void *data);
> -#endif /* !KVM_UPSTREAM */
> +#endif /* !OBSOLETE_KVM_IMPL */
>  
>  struct kvm_sw_breakpoint *kvm_find_sw_breakpoint(CPUState *env,
>                                                   target_ulong pc)
> diff --git a/kvm.h b/kvm.h
> index d321fce..56236ae 100644
> --- a/kvm.h
> +++ b/kvm.h
> @@ -31,13 +31,13 @@ extern int kvm_allowed;
>  #define kvm_enabled() (0)
>  #endif
>  
> -#ifdef KVM_UPSTREAM
> +#ifdef OBSOLETE_KVM_IMPL
>  struct kvm_run;
>  
>  /* external API */
>  
>  int kvm_init(int smp_cpus);
> -#endif /* KVM_UPSTREAM */
> +#endif /* OBSOLETE_KVM_IMPL */
>  
>  int kvm_has_sync_mmu(void);
>  int kvm_has_vcpu_events(void);
> @@ -96,7 +96,7 @@ int kvm_arch_handle_exit(CPUState *env, struct kvm_run *run);
>  
>  int kvm_arch_pre_run(CPUState *env, struct kvm_run *run);
>  
> -#ifdef KVM_UPSTREAM
> +#ifdef OBSOLETE_KVM_IMPL
>  int kvm_arch_process_irqchip_events(CPUState *env);
>  #endif
>  
> diff --git a/target-i386/kvm.c b/target-i386/kvm.c
> index b00e80d..f4fc063 100644
> --- a/target-i386/kvm.c
> +++ b/target-i386/kvm.c
> @@ -188,7 +188,7 @@ int kvm_arch_init_vcpu(CPUState *env)
>          return r;
>      }
>  
> -#ifdef KVM_UPSTREAM
> +#ifdef OBSOLETE_KVM_IMPL
>  
>      env->mp_state = KVM_MP_STATE_RUNNABLE;
>  
> @@ -304,7 +304,7 @@ void kvm_arch_reset_vcpu(CPUState *env)
>          env->mp_state = KVM_MP_STATE_RUNNABLE;
>      }
>  }
> -#ifdef KVM_UPSTREAM
> +#ifdef OBSOLETE_KVM_IMPL
>  
>  static int kvm_has_msr_star(CPUState *env)
>  {
> @@ -644,7 +644,7 @@ static void kvm_msr_entry_set(struct kvm_msr_entry *entry,
>      entry->data = value;
>  }
>  
> -#ifdef KVM_UPSTREAM
> +#ifdef OBSOLETE_KVM_IMPL
>  static int kvm_put_msrs(CPUState *env, int level)
>  {
>      struct {
> @@ -1104,7 +1104,7 @@ static int kvm_get_debugregs(CPUState *env)
>      return 0;
>  }
>  
> -#ifdef KVM_UPSTREAM
> +#ifdef OBSOLETE_KVM_IMPL
>  int kvm_arch_put_registers(CPUState *env, int level)
>  {
>      int ret;
> @@ -1242,7 +1242,7 @@ int kvm_arch_post_run(CPUState *env, struct kvm_run *run)
>      return 0;
>  }
>  
> -#ifdef KVM_UPSTREAM
> +#ifdef OBSOLETE_KVM_IMPL
>  
>  int kvm_arch_process_irqchip_events(CPUState *env)
>  {
> diff --git a/vl.c b/vl.c
> index 22a3616..378a176 100644
> --- a/vl.c
> +++ b/vl.c
> @@ -2466,7 +2466,7 @@ int main(int argc, char **argv, char **envp)
>              case QEMU_OPTION_smbios:
>                  do_smbios_option(optarg);
>                  break;
> -#ifdef KVM_UPSTREAM
> +#ifdef OBSOLETE_KVM_IMPL
>              case QEMU_OPTION_enable_kvm:
>                  kvm_allowed = 1;
>  #endif
> @@ -2803,7 +2803,7 @@ int main(int argc, char **argv, char **envp)
>      if (kvm_allowed) {
>          int ret = kvm_init(smp_cpus);
>          if (ret < 0) {
> -#if defined(KVM_UPSTREAM) || defined(CONFIG_NO_CPU_EMULATION)
> +#if defined(OBSOLETE_KVM_IMPL) || defined(CONFIG_NO_CPU_EMULATION)
>              if (!kvm_available()) {
>                  printf("KVM not supported for this target\n");
>              } else {


[-- Attachment #2: OpenPGP digital signature --]
[-- Type: application/pgp-signature, Size: 259 bytes --]