From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1756437AbcEEIY3 (ORCPT ); Thu, 5 May 2016 04:24:29 -0400 Received: from mx2.suse.de ([195.135.220.15]:34631 "EHLO mx2.suse.de" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1755313AbcEEIY1 (ORCPT ); Thu, 5 May 2016 04:24:27 -0400 Date: Thu, 5 May 2016 10:24:23 +0200 From: Borislav Petkov To: Brooks Moses Cc: linux-kernel@vger.kernel.org, Ian Kasprzak , Paolo Bonzini Subject: Re: Is BIT() in arch/x86/include/uapi/asm/kvm.h defined? Where? Message-ID: <20160505082423.GB534@pd.tnic> References: MIME-Version: 1.0 Content-Type: text/plain; charset=utf-8 Content-Disposition: inline Content-Transfer-Encoding: 8bit In-Reply-To: User-Agent: Mutt/1.5.24 (2015-08-30) Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org On Wed, May 04, 2016 at 05:49:27PM -0700, Brooks Moses wrote: > When I run "make ARCH=x86 headers_install", and then write a simple C > file that #includes "asm/kvm.h" from the resulting tree, I get a > compiler error: the BIT() macro used on line 219 of that file is > undefined: The below patch should help... @Paulo: btw, any chance we can fix that "signifcant" typo :-) in KVM_CPUID_FLAG_SIGNIFCANT_INDEX or is it user-visible and cast in stone? --- From: Borislav Petkov Date: Thu, 5 May 2016 10:18:23 +0200 Subject: [PATCH] x86/kvm: Do not use BIT() in user-exported header Apparently, we're not exporting BIT() to userspace. Reported-by: Brooks Moses Signed-off-by: Borislav Petkov Cc: Paolo Bonzini --- arch/x86/include/uapi/asm/kvm.h | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) diff --git a/arch/x86/include/uapi/asm/kvm.h b/arch/x86/include/uapi/asm/kvm.h index cd54147cb365..739c0c594022 100644 --- a/arch/x86/include/uapi/asm/kvm.h +++ b/arch/x86/include/uapi/asm/kvm.h @@ -216,9 +216,9 @@ struct kvm_cpuid_entry2 { __u32 padding[3]; }; -#define KVM_CPUID_FLAG_SIGNIFCANT_INDEX BIT(0) -#define KVM_CPUID_FLAG_STATEFUL_FUNC BIT(1) -#define KVM_CPUID_FLAG_STATE_READ_NEXT BIT(2) +#define KVM_CPUID_FLAG_SIGNIFCANT_INDEX (1 << 0) +#define KVM_CPUID_FLAG_STATEFUL_FUNC (1 << 1) +#define KVM_CPUID_FLAG_STATE_READ_NEXT (1 << 2) /* for KVM_SET_CPUID2 */ struct kvm_cpuid2 { -- 2.7.3 SUSE Linux GmbH, GF: Felix Imendörffer, Jane Smithard, Graham Norton, HRB 21284 (AG Nürnberg) --