Commit 6b397158 authored Nov 20, 2014 by Radim Krčmář Committed by Paolo Bonzini Nov 21, 2014

kvm: remove IA64 ioctls



KVM ia64 is no longer present so new applications shouldn't use them.
The main problem is that they most likely didn't work even before,
because of a conflict in the #defines:

  #define KVM_SET_GUEST_DEBUG       _IOW(KVMIO,  0x9b, struct kvm_guest_debug)
  #define KVM_IA64_VCPU_SET_STACK   _IOW(KVMIO,  0x9b, void *)

The argument to KVM_SET_GUEST_DEBUG is:

  struct kvm_guest_debug {
  	__u32 control;
  	__u32 pad;
  	struct kvm_guest_debug_arch arch;
  };

  struct kvm_guest_debug_arch {
  };

meaning that sizeof(struct kvm_guest_debug) == sizeof(void *) == 8
and KVM_SET_GUEST_DEBUG == KVM_IA64_VCPU_SET_STACK.

KVM_SET_GUEST_DEBUG is handled in virt/kvm/kvm_main.c before even calling
kvm_arch_vcpu_ioctl (which would have handled KVM_IA64_VCPU_SET_STACK),
so KVM_IA64_VCPU_SET_STACK would just return -EINVAL.

Signed-off-by: Radim Krčmář <rkrcmar@redhat.com>
Signed-off-by: Paolo Bonzini <pbonzini@redhat.com>

parent 3bf58e9a

include/uapi/linux/kvm.h

+0 −3

Original line number	Diff line number	Diff line
		@@ -1099,9 +1099,6 @@ struct kvm_s390_ucas_mapping {
		#define KVM_X86_SETUP_MCE _IOW(KVMIO, 0x9c, __u64)
		#define KVM_X86_GET_MCE_CAP_SUPPORTED _IOR(KVMIO, 0x9d, __u64)
		#define KVM_X86_SET_MCE _IOW(KVMIO, 0x9e, struct kvm_x86_mce)
		/* IA64 stack access */
		#define KVM_IA64_VCPU_GET_STACK _IOR(KVMIO, 0x9a, void *)
		#define KVM_IA64_VCPU_SET_STACK _IOW(KVMIO, 0x9b, void *)
		/* Available with KVM_CAP_VCPU_EVENTS */
		#define KVM_GET_VCPU_EVENTS _IOR(KVMIO, 0x9f, struct kvm_vcpu_events)
		#define KVM_SET_VCPU_EVENTS _IOW(KVMIO, 0xa0, struct kvm_vcpu_events)

Admin message