Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1751714AbaKKTMr (ORCPT ); Tue, 11 Nov 2014 14:12:47 -0500 Received: from mail.skyhub.de ([78.46.96.112]:34347 "EHLO mail.skyhub.de" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1751421AbaKKTMq (ORCPT ); Tue, 11 Nov 2014 14:12:46 -0500 Date: Tue, 11 Nov 2014 20:12:39 +0100 From: Borislav Petkov To: Ross Zwisler Cc: linux-kernel@vger.kernel.org, H Peter Anvin , Ingo Molnar , Thomas Gleixner , David Airlie , dri-devel@lists.freedesktop.org, x86@kernel.org Subject: Re: [PATCH 3/6] x86: Add support for the clwb instruction Message-ID: <20141111191239.GC31523@pd.tnic> References: <1415731396-19364-1-git-send-email-ross.zwisler@linux.intel.com> <1415731396-19364-4-git-send-email-ross.zwisler@linux.intel.com> MIME-Version: 1.0 Content-Type: text/plain; charset=utf-8 Content-Disposition: inline In-Reply-To: <1415731396-19364-4-git-send-email-ross.zwisler@linux.intel.com> User-Agent: Mutt/1.5.23 (2014-03-12) Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org On Tue, Nov 11, 2014 at 11:43:13AM -0700, Ross Zwisler wrote: > Add support for the new clwb instruction. This instruction was > announced in the document "Intel Architecture Instruction Set Extensions > Programming Reference" with reference number 319433-022. > > https://software.intel.com/sites/default/files/managed/0d/53/319433-022.pdf > > Here are some things of note: > > - As with the clflushopt patches before this, I'm assuming that the addressing > mode generated by the original clflush instruction will match the new > clflush instruction with the 0x66 prefix for clflushopt, and for the > xsaveopt instruction with the 0x66 prefix for clwb. For all the test cases > that I've come up with and for the new clwb code generated by this patch > series, this has proven to be true on my test machine. > > - According to the SDM, xsaveopt has a form where it has a REX.W prefix. I > believe that this prefix will not be generated by gcc in x86_64 kernel code. > Based on this, I don't believe I need to account for this extra prefix when > dealing with the assembly language created for clwb. Please correct me if > I'm wrong. > > Signed-off-by: Ross Zwisler > Cc: H Peter Anvin > Cc: Ingo Molnar > Cc: Thomas Gleixner > Cc: David Airlie > Cc: dri-devel@lists.freedesktop.org > Cc: x86@kernel.org > --- > arch/x86/include/asm/cpufeature.h | 1 + > arch/x86/include/asm/special_insns.h | 10 ++++++++++ > 2 files changed, 11 insertions(+) > > diff --git a/arch/x86/include/asm/cpufeature.h b/arch/x86/include/asm/cpufeature.h > index b3e6b89..fbbed34 100644 > --- a/arch/x86/include/asm/cpufeature.h > +++ b/arch/x86/include/asm/cpufeature.h > @@ -227,6 +227,7 @@ > #define X86_FEATURE_SMAP ( 9*32+20) /* Supervisor Mode Access Prevention */ > #define X86_FEATURE_PCOMMIT ( 9*32+22) /* PCOMMIT instruction */ > #define X86_FEATURE_CLFLUSHOPT ( 9*32+23) /* CLFLUSHOPT instruction */ > +#define X86_FEATURE_CLWB ( 9*32+24) /* CLWB instruction */ > #define X86_FEATURE_AVX512PF ( 9*32+26) /* AVX-512 Prefetch */ > #define X86_FEATURE_AVX512ER ( 9*32+27) /* AVX-512 Exponential and Reciprocal */ > #define X86_FEATURE_AVX512CD ( 9*32+28) /* AVX-512 Conflict Detection */ > diff --git a/arch/x86/include/asm/special_insns.h b/arch/x86/include/asm/special_insns.h > index 1709a2e..a328460 100644 > --- a/arch/x86/include/asm/special_insns.h > +++ b/arch/x86/include/asm/special_insns.h > @@ -199,6 +199,16 @@ static inline void clflushopt(volatile void *__p) > "+m" (*(volatile char __force *)__p)); > } > > +static inline void clwb(volatile void *__p) > +{ > + alternative_io_2(".byte " __stringify(NOP_DS_PREFIX) "; clflush %P0", Any particular reason for using 0x3e as a prefix to have the insns be the same size or is it simply because CLFLUSH can stomach it? :-) > + ".byte 0x66; clflush %P0", > + X86_FEATURE_CLFLUSHOPT, > + ".byte 0x66; xsaveopt %P0", Huh, XSAVEOPT?!? Shouldn't that be CLWB?? > + X86_FEATURE_CLWB, > + "+m" (*(volatile char __force *)__p)); > +} > + > static inline void pcommit(void) > { > alternative(ASM_NOP4, ".byte 0x66, 0x0f, 0xae, 0xf8", > -- > 1.9.3 > > -- > 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/ > -- Regards/Gruss, Boris. Sent from a fat crate under my desk. Formatting is fine. -- -- 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/