Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1757125AbaKTNiR (ORCPT ); Thu, 20 Nov 2014 08:38:17 -0500 Received: from mx1.redhat.com ([209.132.183.28]:40643 "EHLO mx1.redhat.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1751419AbaKTNiP (ORCPT ); Thu, 20 Nov 2014 08:38:15 -0500 Date: Thu, 20 Nov 2014 14:38:10 +0100 From: Radim =?utf-8?B?S3LEjW3DocWZ?= To: Paolo Bonzini Cc: linux-kernel@vger.kernel.org, kvm@vger.kernel.org, linux-ia64@vger.kernel.org, Tony Luck , Fenghua Yu Subject: Re: [PATCH] KVM: ia64: remove Message-ID: <20141120133809.GA30792@potion.brq.redhat.com> References: <1416431143-31843-1-git-send-email-pbonzini@redhat.com> MIME-Version: 1.0 Content-Type: text/plain; charset=utf-8 Content-Disposition: inline Content-Transfer-Encoding: 8bit In-Reply-To: <1416431143-31843-1-git-send-email-pbonzini@redhat.com> Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org 2014-11-19 22:05+0100, Paolo Bonzini: > KVM for ia64 has been marked as broken not just once, but twice even, > and the last patch from the maintainer is now roughly 5 years old. > Time for it to rest in piece. > > Signed-off-by: Paolo Bonzini > --- Nice, if only every diffstat was like that! I propose another removal. (The reasoning below wasn't confirmed with ia64 compiler. I'd remove the ioctls even if they worked.) ---8<--- KVM: remove buggy ia64 specific ioctls 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 we have misused ioctl #define KVM_SET_GUEST_DEBUG _IOW(KVMIO, 0x9b, struct kvm_guest_debug) #define KVM_IA64_VCPU_SET_STACK _IOW(KVMIO, 0x9b, void *) as struct kvm_guest_debug { __u32 control; __u32 pad; struct kvm_guest_debug_arch arch; }; and struct kvm_guest_debug_arch { }; mean that sizeof(struct kvm_guest_debug) == sizeof(void *) == 8 thus KVM_SET_GUEST_DEBUG == KVM_IA64_VCPU_SET_STACK and KVM_SET_GUEST_DEBUG is handled before KVM_IA64_VCPU_SET_STACK. Signed-off-by: Radim Krčmář --- include/uapi/linux/kvm.h | 3 --- 1 file changed, 3 deletions(-) diff --git a/include/uapi/linux/kvm.h b/include/uapi/linux/kvm.h index 6d59e5b..a37fd12 100644 --- a/include/uapi/linux/kvm.h +++ b/include/uapi/linux/kvm.h @@ -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) -- 2.1.0 -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/